Thread (23 messages) flat view 23 messages, 8 authors, 2017-12-22

Re: [RFC PATCH] virtio_net: Extend virtio to use VF datapath when available

From: "Samudrala, Sridhar" <sridhar.samudrala@intel.com>
Date: 2017-12-19 19:42:33


On 12/19/2017 10:41 AM, Stephen Hemminger wrote:
On Tue, 19 Dec 2017 13:21:17 -0500 (EST)
David Miller [off-list ref] wrote:
quoted
From: Stephen Hemminger <stephen@networkplumber.org>
Date: Tue, 19 Dec 2017 09:55:48 -0800
quoted
could be 10ms, just enough to let udev do its renaming
Please, move to some kind of notification or event based handling of
this problem.

No delay is safe, what if userspace gets swapped out or whatever
else might make userspace stall unexpectedly?
The plan is to remove the delay and do the naming in the kernel.
This was suggested by Lennart since udev is only doing naming policy
because kernel names were not repeatable.

This makes the VF show up as "ethN_vf" on Hyper-V which is user friendly.

Patch is pending.
Do we really need to delay the setup until the name is changed?
Can't we call dev_set_mtu() and dev_open() until dev_change_name() is done?

Thanks
Sridhar
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help