在 2021/9/30 1:15, Andrew Lunn 写道:
On Wed, Sep 29, 2021 at 11:50:47PM +0800, Jian Shen wrote:
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.
What you should of done is converted just one MAC driver. That gives us enough we can review the basic idea, etc, and not need to delete 130 nearly identical patches.
Andrew.
OK, I will refine it, make it more reviewable, thanks!
shenjian