[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20220329182245.0127ccbc@kernel.org>
Date: Tue, 29 Mar 2022 18:22:45 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: jmaloy@...hat.com
Cc: netdev@...r.kernel.org, davem@...emloft.net,
tipc-discussion@...ts.sourceforge.net,
tung.q.nguyen@...tech.com.au, hoang.h.le@...tech.com.au,
tuong.t.lien@...tech.com.au, maloy@...jonn.com, xinl@...hat.com,
ying.xue@...driver.com, parthasarathy.bhuvaragan@...il.com
Subject: Re: [net-next] tipc: clarify meaning of 'inactive' field in struct
tipc_subscription
On Tue, 29 Mar 2022 13:32:18 -0400 jmaloy@...hat.com wrote:
> From: Jon Maloy <jmaloy@...hat.com>
>
> struct tipc_subscription has a boolean field 'inactive' which purpose
> is not immediately obvious. When the subscription timer expires we are
> still in interrupt context, and cannot easily just delete the
> subscription. We therefore delay that action until the expiration
> event has reached the work queue context where it is being sent to the
> user. However, in the meantime other events may occur, which must be
> suppressed to avoid any unexpected behavior.
>
> We now clarify this with renaming the field and adding a comment.
>
> Signed-off-by: Jon Maloy <jmaloy@...hat.com>
# Form letter - net-next is closed
We have already sent the networking pull request for 5.18
and therefore net-next is closed for new drivers, features,
code refactoring and optimizations. We are currently accepting
bug fixes only.
Please repost when net-next reopens after 5.18-rc1 is cut.
RFC patches sent for review only are obviously welcome at any time.
Powered by blists - more mailing lists