mbox series

[net-next,v2,0/1] hyperv: Link queues to NAPIs

Message ID 20240930172709.57417-1-jdamato@fastly.com (mailing list archive)
Headers show
Series hyperv: Link queues to NAPIs | expand

Message

Joe Damato Sept. 30, 2024, 5:27 p.m. UTC
Greetings:

Welcome to v2.

This was previously an RFC [1], see changelog below.

I've only compile tested this series; I don't have the software for testing
this so I am hoping some one from Microsoft can review and test this
following the instructions below :)

This change allows users to query the mapping of queues to NAPIs using
the netdev-genl interface.

Once this patch has been applied, this can be tested using the cli included
in the kernel tree like this:

$ ./tools/net/ynl/cli.py --spec Documentation/netlink/specs/netdev.yaml \
                         --dump queue-get --json='{"ifindex": 2}'

Substituting the ifindex above for the correct ifindex on your system
(which is, presumably, a hyper-V VM).

A sample of expected output would look like:

[{'id': 0, 'ifindex': 2, 'napi-id': 145, 'type': 'rx'},
 {'id': 0, 'ifindex': 2, 'napi-id': 145, 'type': 'tx'}]

Which shows a mapping of queue ID (0) to NAPI ID (145) for both RX and TX
queues. Having this mapping is extremely useful for user apps for a variety
of use cases, including epoll-based busy poll which relies on the NAPI ID.

It would be really great to add support for this API to hyper-V so that
applications (including CI and automated testing facilities) could make use
of this API in VMs.

Sorry, I don't know much at all about hyper-V, but please let me know if
there is anything I can do to help.

Thanks,
Joe

[1]: https://lore.kernel.org/netdev/20240924234851.42348-1-jdamato@fastly.com/

v2:
  - Added Haiyang Zhang's Reviewed-by
  - Added Shradha Gupta's Tested-by
  - Commit message updated to include test output from Shradha
  - Wrapped lines to 80 characters, as suggested by Simon Horman (no
    functional changes introduced)

Joe Damato (1):
  hv_netvsc: Link queues to NAPIs

 drivers/net/hyperv/netvsc.c       | 13 ++++++++++++-
 drivers/net/hyperv/rndis_filter.c |  9 +++++++--
 2 files changed, 19 insertions(+), 3 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Oct. 6, 2024, 3:30 p.m. UTC | #1
Hello:

This patch was applied to netdev/net-next.git (main)
by David S. Miller <davem@davemloft.net>:

On Mon, 30 Sep 2024 17:27:08 +0000 you wrote:
> Greetings:
> 
> Welcome to v2.
> 
> This was previously an RFC [1], see changelog below.
> 
> I've only compile tested this series; I don't have the software for testing
> this so I am hoping some one from Microsoft can review and test this
> following the instructions below :)
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/1] hv_netvsc: Link queues to NAPIs
    https://git.kernel.org/netdev/net-next/c/8b641b5e4c78

You are awesome, thank you!