On Mon, May 29, 2017 at 7:30 AM, Karthik Sundaravel <[email protected]> wrote: > Hi > > Tested-by: [email protected] > I've tested the patch. Please see https://bugzilla.redhat. > com/show_bug.cgi?id=1397299#c17.
Thanks for testing and reporting back! Sorry for the slow review. I've applied this to master and branch-2.7. > > Regards > Karthik S > > On Fri, May 19, 2017 at 3:21 AM, Ben Pfaff <[email protected]> wrote: > >> On Thu, May 18, 2017 at 05:11:03PM -0400, Aaron Conole wrote: >> > Ben Pfaff <[email protected]> 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 <[email protected]> >> > >> > Signed-off-by: Aaron Conole <[email protected]> >> > Tested-by: [email protected] > >> > >> > --- >> > >> > 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. >> _______________________________________________ >> dev mailing list >> [email protected] >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev -- Russell Bryant _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
