Re: [PATCH net-next] ethtool: ioctl: account for sopass diff in set_wol

[Justin Chen]

On 6/12/23 3:28 PM, Andrew Lunn wrote:
> On Mon, Jun 12, 2023 at 02:37:00PM -0700, Justin Chen wrote:
> > sopass won't be set if wolopt doesn't change. This means the following
> > will fail to set the correct sopass.
> > ethtool -s eth0 wol s sopass 11:22:33:44:55:66
> > ethtool -s eth0 wol s sopass 22:44:55:66:77:88
> >
> > Make sure we call into the driver layer set_wol if sopass is different.
> >
> > Fixes: 55b24334c0f2 ("ethtool: ioctl: improve error checking for set_wol")
> > Signed-off-by: Justin Chen <justin.chen@xxxxxxxxxxxx>
> > ---
> >
> > Note: Tagged "Fixes" patch has not hit rc yet.
> >
> >   net/ethtool/ioctl.c | 3 ++-
>
> Hi Justin
>
> Does the netlink version get this correct?

Yup netlink gets this correct

> And just for my own curiosity, why are you so interested in the ioctl
> version, which is deprecated and not used by modern versions of
> ethtool?
No particular reason. I was working on another driver and was planning 
to remove the set_wol error checking in the driver layer when I realized 
netlink set_wol was already doing it. Then on further inspection I 
noticed the ioctl version doesn't do the same thing.

Justin

> 	Andrew

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature