Re: Patch "net: move ethtool-related netdev state into its own struct" has been added to the 6.10-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 05, 2024 at 01:45:18PM +0100, Edward Cree wrote:
On 05/08/2024 13:19, Sasha Levin wrote:
This is a note to let you know that I've just added the patch titled

    net: move ethtool-related netdev state into its own struct

to the 6.10-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     net-move-ethtool-related-netdev-state-into-its-own-s.patch
and it can be found in the queue-6.10 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.

This, and the series it's from, are absolutely not -stable material.
The commits do not fix any existing bugs, they are in support of new
features (netlink dumping of RSS contexts), and are a fairly large
and complex set of changes, which have not even stabilised yet — we
have already found issues both within the set and exposed by it in
other code, which are being fixed for 6.11.

commit e331e73ff4c5c89a7f51a465ae40a7ad9fcd7a28
Author: Edward Cree <ecree.xilinx@xxxxxxxxx>
Date:   Thu Jun 27 16:33:46 2024 +0100

    net: move ethtool-related netdev state into its own struct

    [ Upstream commit 3ebbd9f6de7ec6d538639ebb657246f629ace81e ]

    net_dev->ethtool is a pointer to new struct ethtool_netdev_state, which
     currently contains only the wol_enabled field.

    Suggested-by: Jakub Kicinski <kuba@xxxxxxxxxx>
    Signed-off-by: Edward Cree <ecree.xilinx@xxxxxxxxx>
    Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@xxxxxxxxx>
    Link: https://patch.msgid.link/293a562278371de7534ed1eb17531838ca090633.1719502239.git.ecree.xilinx@xxxxxxxxx
    Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx>
    Stable-dep-of: 7195f0ef7f5b ("ethtool: fix setting key and resetting indir at once")
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

As far as I can tell, 7195f0ef7f5b should backport fairly cleanly
to 6.10 with only simple textual fuzz.
It should not be necessary to backport the "ethtool: track custom
RSS contexts in the core" series to support this.

The above NAK also applies to the backports of:
    net-ethtool-attach-an-xarray-of-custom-rss-contexts-.patch
    net-ethtool-record-custom-rss-contexts-in-the-xarray.patch
    net-ethtool-add-a-mutex-protecting-rss-contexts.patch
which were notified at the same time.

Makes sense! I've modified the backport of 7195f0ef7f5b ("ethtool: fix
setting key and resetting indir at once") as proposed.

--
Thanks,
Sasha




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux