Thread (17 messages) 17 messages, 4 authors, 2024-07-02

Re: [PATCH net-next 1/5] netdevice: convert private flags > BIT(31) to bitfields

From: Alexander Lobakin <aleksander.lobakin@intel.com>
Date: 2024-06-28 10:37:44
Also in: lkml

From: Jakub Kicinski <kuba@kernel.org>
Date: Thu, 27 Jun 2024 12:55:41 -0700
On Thu, 27 Jun 2024 11:50:40 +0200 Alexander Lobakin wrote:
quoted
quoted
I don't think we should group them indiscriminately. Better to add the
asserts flag by flag. Neither of the flags you're breaking out in this
patch are used on the fast path.

Or is the problem that CACHELINE_ASSERT_GROUP_MEMBER doesn't work on
bitfields?  
It generates sizeof(bitfield) which the compilers don't like and don't
want to compile ._.
Mm. Okay, I have no better ideas then.

Do consider moving the cold flags next to wol_enabled, tho?
Hmm, sounds good!

Thanks,
Olek


 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help