[ovs-dev] [PATCH] rhel-systemd: start vswitchd after udev

Ben Pfaff blp at ovn.org
Thu May 18 21:51:17 UTC 2017


On Thu, May 18, 2017 at 05:11:03PM -0400, Aaron Conole wrote:
> Ben Pfaff <blp at ovn.org> writes:
> 
> > On Sat, May 13, 2017 at 11:08:13AM -0400, Aaron Conole wrote:
> >> Aaron Conole writes:
> >> > It's possible to race with the udev service, such that dpdk ports are
> >> > not finished being bound until after ovs-vswitchd has been started.
> >> > This means that attempts to use the port will fail.  While it is
> >> > possible to work around this for some NICs using port hotplug, not all
> >> > port types are supported (for instance vfio), and it requires manual
> >> > intervention.
> >> >
> >> > Fixes: 36af136b690c ("rhel-systemd: Delay shutting down the services")
> >> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1397299
> >> > Suggested-by: Flavio Leitner <fbl at sysclose.org>
> >> > Signed-off-by: Aaron Conole <aconole at redhat.com>
> >> > ---
> >> >  rhel/usr_lib_systemd_system_ovs-vswitchd.service | 2 +-
> >> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >> 
> >> Ping?  Should I rebase / resend this?  I'm not sure if it still applies
> >> to the tree.
> >
> > It still applies fine but I'm not comfortable reviewing it myself.  Is
> > Flavio the right one to review it or should someone else?
> 
> Flavio had actually suggested it.  I'm CC'ing the tester.  Maybe we can
> at least get a Tested-by tag.

I didn't realize there was a tester.  I guess that I should have looked
at the bugzilla URL.

If Maxim tested it successfully, we don't really need him to give us the
Tested-by, we can add it ourselves.  (But now that we've asked, we might
as well get an answer.)

Thanks,

Ben.


More information about the dev mailing list