Page MenuHomeFreeBSD

hyperv: netvsc: VRSS step1: multiple channel support
ClosedPublic

Authored by honzhan_microsoft.com on Mar 1 2016, 3:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 22 2024, 7:05 PM
Unknown Object (File)
Feb 21 2024, 10:02 AM
Unknown Object (File)
Dec 20 2023, 6:10 AM
Unknown Object (File)
Nov 13 2023, 12:21 PM
Unknown Object (File)
Nov 7 2023, 4:29 PM
Unknown Object (File)
Oct 25 2023, 8:28 AM
Unknown Object (File)
Sep 16 2023, 1:45 AM
Unknown Object (File)
Aug 10 2023, 7:30 AM

Details

Summary

Implement multiple channel for netvsc.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2716
Build 2736: arc lint + arc unit

Event Timeline

honzhan_microsoft.com retitled this revision from to hyperv: netvsc: VRSS step1: multiple channel support.
honzhan_microsoft.com updated this object.

This is the first step for VRSS: multiple channel support. It passes build & simple test, and need more test.

hv_nv_on_receive_completion() bits will be committed shortly.

callback_buf bits will be committed shortly.

I have to change the hyperv/hn to redo the # of channel configuration bits. That should only have small impact on this patch.

hyperv/hn # of ring configuration is committed. I am moving on merging this patch.

sys/dev/hyperv/include/hyperv.h
125

doesn't sound right. if this is generic for all driver, why not just rename priv1/2 to the rxr/txr? if it is not generic, why you put into hyperv.h?

sys/dev/hyperv/vmbus/hv_channel_mgmt.c
145

seems this buf is not used by vmbus driver. why not push it down to netvsc?

Generally speaking, only the RNDIS, and NetVSP bits protocol bits were taken from this submission, that's also why I didn't mention this review in the commit message to avoid confusion.

If you want to take a look at vRSS, just checkout the latest code.

sys/dev/hyperv/include/hyperv.h
125

This part is deleted in the committed code.

sys/dev/hyperv/vmbus/hv_channel_mgmt.c
145

This one is also deleted in the committed code.

sepherosa_gmail.com edited edge metadata.

It had been committed for quite some time.

This revision is now accepted and ready to land.Sep 13 2016, 8:57 AM