Re: WTF: patch "[PATCH] selftests/landlock: Format with clang-format" was seriously submitted to be applied to the 5.18-stable tree?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Greg,

The 21 Landlock commits merged for 5.19-rc1 and tagged with Cc stable@ should indeed be backported up to 5.15 . The first commits are pure cosmetic changes but they need to be backported to avoid backport conflicts (for this series and future backports). They help maintain this subsystem, including to backport future changes. The following changes up to commit 8ba0005ff418 ("landlock: Fix same-layer rule unions") are required to fix some edge case issues (i.e. syscall argument ordering checks and same-layer rule unions). New tests are added to check that everything work as expected for these backportable changes, and to make it possible for more test environments to run. I successfully tested the backport of all these commits to 5.15 . Please backport them to all stable branches.

Here is the full list of the commits to backport (already marked with Cc: stable@xxxxxxxxxxxxxxx):

8ba0005ff418 landlock: Fix same-layer rule unions
2cd7cd6eed88 landlock: Create find_rule() from unmask_layers()
75c542d6c6cc landlock: Reduce the maximum number of layers to 16
5f2ff33e1084 landlock: Define access_mask_t to enforce a consistent access mask size 6533d0c3a86e selftests/landlock: Test landlock_create_ruleset(2) argument check ordering
eba39ca4b155 landlock: Change landlock_restrict_self(2) check ordering
589172e5636c landlock: Change landlock_add_rule(2) argument check ordering
d1788ad99087 selftests/landlock: Add tests for O_PATH
6a1bdd4a0bfc selftests/landlock: Fully test file rename with "remove" access
d18955d094d0 selftests/landlock: Extend access right tests to directories
c56b3bf566da selftests/landlock: Add tests for unknown access rights
291865bd7e8b selftests/landlock: Extend tests for minimal valid attribute size
87129ef13603 selftests/landlock: Make tests build with old libc
a13e248ff90e landlock: Fix landlock_add_rule(2) documentation
81709f3dccac samples/landlock: Format with clang-format
9805a722db07 samples/landlock: Add clang-format exceptions
371183fa578a selftests/landlock: Format with clang-format
135464f9d29c selftests/landlock: Normalize array assignment
4598d9abf421 selftests/landlock: Add clang-format exceptions
06a1c40a09a8 landlock: Format with clang-format
6cc2df8e3a39 landlock: Add clang-format exceptions

Regards,
 Mickaël


On 06/06/2022 09:45, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
The patch below was submitted to be applied to the 5.18-stable tree.

I fail to see how this patch meets the stable kernel rules as found at
Documentation/process/stable-kernel-rules.rst.

I could be totally wrong, and if so, please respond to
<stable@xxxxxxxxxxxxxxx> and let me know why this patch should be
applied.  Otherwise, it is now dropped from my patch queues, never to be
seen again.

thanks,

greg k-h

------------------ original commit in Linus's tree ------------------

From 371183fa578a4cf56b3ae12e54b7f01a4249add1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micka=C3=ABl=20Sala=C3=BCn?= <mic@xxxxxxxxxxx>
Date: Fri, 6 May 2022 18:05:11 +0200
Subject: [PATCH] selftests/landlock: Format with clang-format
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Let's follow a consistent and documented coding style.  Everything may
not be to our liking but it is better than tacit knowledge.  Moreover,
this will help maintain style consistency between different developers.

This contains only whitespace changes.

Automatically formatted with:
clang-format-14 -i tools/testing/selftests/landlock/*.[ch]

Link: https://lore.kernel.org/r/20220506160513.523257-6-mic@xxxxxxxxxxx
Cc: stable@xxxxxxxxxxxxxxx
[mic: Update style according to
https://lore.kernel.org/r/02494cb8-2aa5-1769-f28d-d7206f284e5a@xxxxxxxxxxx]
Signed-off-by: Mickaël Salaün <mic@xxxxxxxxxxx>



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux