Thread (181 messages) 181 messages, 6 authors, 2022-01-27

Re: [RFCv2 net-next 000/167] net: extend the netdev_features_t

From: Leon Romanovsky <leon@kernel.org>
Date: 2022-01-27 09:50:38

On Thu, Jan 27, 2022 at 05:40:02PM +0800, shenjian (K) wrote:

在 2022/1/27 17:16, Leon Romanovsky 写道:
quoted
On Wed, Sep 29, 2021 at 11:50:47PM +0800, Jian Shen wrote:
quoted
For the prototype of netdev_features_t is u64, and the number
of netdevice feature bits is 64 now. So there is no space to
introduce new feature bit.

This patchset try to solve it by change the prototype of
netdev_features_t from u64 to bitmap. With this change,
it's necessary to introduce a set of bitmap operation helpers
for netdev features. Meanwhile, the functions which use
netdev_features_t as return value are also need to be changed,
return the result as an output parameter.

With above changes, it will affect hundreds of files, and all the
nic drivers. To make it easy to be reviewed, split the changes
to 167 patches to 5 parts.

patch 1~22: convert the prototype which use netdev_features_t
as return value
patch 24: introduce fake helpers for bitmap operation
patch 25~165: use netdev_feature_xxx helpers
patch 166: use macro __DECLARE_NETDEV_FEATURE_MASK to replace
netdev_feature_t declaration.
patch 167: change the type of netdev_features_t to bitmap,
and rewrite the bitmap helpers.

Sorry to send a so huge patchset, I wanna to get more suggestions
to finish this work, to make it much more reviewable and feasible.

The former discussing for the changes, see [1]
[1]. https://www.spinics.net/lists/netdev/msg753528.html
------------------------------------------------

Is anyone actively working on this task?

Thanks
.
Hi Leon,

I have sent RFCv4  [1] three months ago, and according Andrew' suggestion,
I'm trying to
continue this work with semantic-patches, and waiting for more comments
for the scheme.
But I'm not familiar with it, and  busy with some other work recently, so it
got delayed.

Sorry for this. I will speed up it.
Thanks
[1] https://lore.kernel.org/netdev/YYvKyruLcemj6J+i@lunn.ch/T/ (local)

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