[PATCH 0/2] fscontext: do not consume log entries for -EMSGSIZE case

Aleksa Sarai posted 2 patches 2 months ago
fs/fsopen.c                                    |  22 ++--
tools/testing/selftests/filesystems/.gitignore |   1 +
tools/testing/selftests/filesystems/Makefile   |   2 +-
tools/testing/selftests/filesystems/fclog.c    | 137 +++++++++++++++++++++++++
4 files changed, 153 insertions(+), 9 deletions(-)
[PATCH 0/2] fscontext: do not consume log entries for -EMSGSIZE case
Posted by Aleksa Sarai 2 months ago
Userspace generally expects APIs that return EMSGSIZE to allow for them
to adjust their buffer size and retry the operation. However, the
fscontext log would previously clear the message even in the EMSGSIZE
case.

Given that it is very cheap for us to check whether the buffer is too
small before we remove the message from the ring buffer, let's just do
that instead.

Fixes: 007ec26cdc9f ("vfs: Implement logging through fs_context")
Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
---
Aleksa Sarai (2):
      fscontext: do not consume log entries for -EMSGSIZE case
      selftests/filesystems: add basic fscontext log tests

 fs/fsopen.c                                    |  22 ++--
 tools/testing/selftests/filesystems/.gitignore |   1 +
 tools/testing/selftests/filesystems/Makefile   |   2 +-
 tools/testing/selftests/filesystems/fclog.c    | 137 +++++++++++++++++++++++++
 4 files changed, 153 insertions(+), 9 deletions(-)
---
base-commit: 66639db858112bf6b0f76677f7517643d586e575
change-id: 20250806-fscontext-log-cleanups-50f0143674ae

Best regards,
-- 
Aleksa Sarai <cyphar@cyphar.com>