| Message ID | 1629877513-23501-2-git-send-email-huangguangbin2@huawei.com |
|---|---|
| State | New |
| Headers | show |
| Series | [V2,ethtool-next,1/2] update UAPI header copies | expand |
On Wed, Aug 25, 2021 at 03:45:12PM +0800, Guangbin Huang wrote: > Update to kernel commit 5b4ecc3d4c4a. > > Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com> > --- > uapi/linux/ethtool.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/uapi/linux/ethtool.h b/uapi/linux/ethtool.h > index c6ec1111ffa3..bd1f09b23cf5 100644 > --- a/uapi/linux/ethtool.h > +++ b/uapi/linux/ethtool.h > @@ -637,6 +637,8 @@ enum ethtool_link_ext_substate_link_logical_mismatch { > enum ethtool_link_ext_substate_bad_signal_integrity { > ETHTOOL_LINK_EXT_SUBSTATE_BSI_LARGE_NUMBER_OF_PHYSICAL_ERRORS = 1, > ETHTOOL_LINK_EXT_SUBSTATE_BSI_UNSUPPORTED_RATE, > + ETHTOOL_LINK_EXT_SUBSTATE_BSI_SERDES_REFERENCE_CLOCK_LOST, > + ETHTOOL_LINK_EXT_SUBSTATE_BSI_SERDES_ALOS, > }; > > /* More information in addition to ETHTOOL_LINK_EXT_STATE_CABLE_ISSUE. */ > -- > 2.8.1 > I replaced this commit with a full update of uapi headers. The point is that if we keep cherry picking only specific changes in the headers, it will become harder and harder to check if something is missing or if we diverged from kernel. This is why an update of uapi headers should always update all of them to the state of the same kernel commit (usually current master or net-next tree). I added the link to ethtool-import-uapi script to devel documentation on the ethtool web page make things easier. Michal
On 2021/8/26 17:45, Michal Kubecek wrote: > On Wed, Aug 25, 2021 at 03:45:12PM +0800, Guangbin Huang wrote: >> Update to kernel commit 5b4ecc3d4c4a. >> >> Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com> >> --- >> uapi/linux/ethtool.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/uapi/linux/ethtool.h b/uapi/linux/ethtool.h >> index c6ec1111ffa3..bd1f09b23cf5 100644 >> --- a/uapi/linux/ethtool.h >> +++ b/uapi/linux/ethtool.h >> @@ -637,6 +637,8 @@ enum ethtool_link_ext_substate_link_logical_mismatch { >> enum ethtool_link_ext_substate_bad_signal_integrity { >> ETHTOOL_LINK_EXT_SUBSTATE_BSI_LARGE_NUMBER_OF_PHYSICAL_ERRORS = 1, >> ETHTOOL_LINK_EXT_SUBSTATE_BSI_UNSUPPORTED_RATE, >> + ETHTOOL_LINK_EXT_SUBSTATE_BSI_SERDES_REFERENCE_CLOCK_LOST, >> + ETHTOOL_LINK_EXT_SUBSTATE_BSI_SERDES_ALOS, >> }; >> >> /* More information in addition to ETHTOOL_LINK_EXT_STATE_CABLE_ISSUE. */ >> -- >> 2.8.1 >> > > I replaced this commit with a full update of uapi headers. The point is > that if we keep cherry picking only specific changes in the headers, it > will become harder and harder to check if something is missing or if we > diverged from kernel. This is why an update of uapi headers should > always update all of them to the state of the same kernel commit > (usually current master or net-next tree). > > I added the link to ethtool-import-uapi script to devel documentation on > the ethtool web page make things easier. > > Michal > Ok, thanks!
diff --git a/uapi/linux/ethtool.h b/uapi/linux/ethtool.h index c6ec1111ffa3..bd1f09b23cf5 100644 --- a/uapi/linux/ethtool.h +++ b/uapi/linux/ethtool.h @@ -637,6 +637,8 @@ enum ethtool_link_ext_substate_link_logical_mismatch { enum ethtool_link_ext_substate_bad_signal_integrity { ETHTOOL_LINK_EXT_SUBSTATE_BSI_LARGE_NUMBER_OF_PHYSICAL_ERRORS = 1, ETHTOOL_LINK_EXT_SUBSTATE_BSI_UNSUPPORTED_RATE, + ETHTOOL_LINK_EXT_SUBSTATE_BSI_SERDES_REFERENCE_CLOCK_LOST, + ETHTOOL_LINK_EXT_SUBSTATE_BSI_SERDES_ALOS, }; /* More information in addition to ETHTOOL_LINK_EXT_STATE_CABLE_ISSUE. */
Update to kernel commit 5b4ecc3d4c4a. Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com> --- uapi/linux/ethtool.h | 2 ++ 1 file changed, 2 insertions(+)