Re: [PATCH v6.1 17/27] af_unix: Skip GC if no cycle exists.

[Sasha Levin <sashal@kernel.org>]

[ Sasha's backport helper bot ]

Hi,

Summary of potential issues:
ℹ️ This is part 17/27 of a series
⚠️ Found follow-up fixes in mainline

The upstream commit SHA1 provided is correct: 77e5593aebba823bcbcf2c4b58b07efcd63933b8

WARNING: Author mismatch between patch and upstream commit:
Backport author: Lee Jones<lee@kernel.org>
Commit author: Kuniyuki Iwashima<kuniyu@amazon.com>

Status in newer kernel trees:
6.14.y | Present (exact SHA1)
6.12.y | Present (exact SHA1)
6.6.y | Not found

Found fixes commits:
1af2dface5d2 af_unix: Don't access successor in unix_del_edges() during GC.

Note: The patch differs from the upstream commit:
---
1:  77e5593aebba8 ! 1:  15f6ca2c3df2b af_unix: Skip GC if no cycle exists.
    @@ Metadata
      ## Commit message ##
         af_unix: Skip GC if no cycle exists.
     
    +    [ Upstream commit 77e5593aebba823bcbcf2c4b58b07efcd63933b8 ]
    +
         We do not need to run GC if there is no possible cyclic reference.
         We use unix_graph_maybe_cyclic to decide if we should run GC.
     
    @@ Commit message
         Acked-by: Paolo Abeni <pabeni@redhat.com>
         Link: https://lore.kernel.org/r/20240325202425.60930-11-kuniyu@amazon.com
         Signed-off-by: Jakub Kicinski <kuba@kernel.org>
    +    (cherry picked from commit 77e5593aebba823bcbcf2c4b58b07efcd63933b8)
    +    Signed-off-by: Lee Jones <lee@kernel.org>
     
      ## net/unix/garbage.c ##
     @@ net/unix/garbage.c: static struct unix_vertex *unix_edge_successor(struct unix_edge *edge)
---

NOTE: These results are for this patch alone. Full series testing will be
performed when all parts are received.

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-6.6.y        |  Success    |  Success   |