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)
Sat, May 6, 6:38 PM
Unknown Object (File)
Fri, May 5, 6:13 AM
Unknown Object (File)
Mar 6 2023, 4:02 AM
Unknown Object (File)
Feb 18 2023, 7:45 AM
Unknown Object (File)
Feb 14 2023, 2:01 AM
Unknown Object (File)
Dec 15 2022, 12:21 PM
Unknown Object (File)
Mar 31 2017, 7:49 AM
Unknown Object (File)
Dec 31 2016, 4:12 PM

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