Re: [PATCH] enic: add ethtool get_channel support

From: Simon Horman
Date: Wed Jun 19 2024 - 13:59:28 EST


On Tue, Jun 18, 2024 at 09:01:46AM -0700, Jon Kohler wrote:
> Add .get_channel to enic_ethtool_ops to enable basic ethtool -l
> support to get the current channel configuration.

This is nice :)

> Note that the driver does not support dynamically changing queue
> configuration, so .set_channel is intentionally unused. Instead, users
> should use Cisco's hardware management tools (UCSM/IMC) to modify
> virtual interface card configuration out of band.

That is a shame :(

> Signed-off-by: Jon Kohler <jon@xxxxxxxxxxx>

Sad face aside, this looks good to me.

Reviewed-by: Simon Horman <horms@xxxxxxxxxx>

...