<br><tt><font size=2>libvir-list-bounces@redhat.com wrote on 08/13/2010
04:44:38 PM:<br>
<br>
<br>
> Incomplete subject line?  It used to be<br>
>  nwfilter: extend nwfilter reload support</font></tt>
<br>
<br><tt><font size=2>Yes. Due to a race condition :-)</font></tt>
<br><tt><font size=2><br>
> <br>
> On 08/13/2010 02:29 PM, Stefan Berger wrote:<br>
> >  v3:<br>
> >     - Fixed an indentation problem<br>
> >     - added bool parameter to function terminating
the IP address<br>
> > learner threads to determine whether future threads may still
run<br>
> > (needed in case of driver reload) or all must terminate (need
in case of<br>
> > libvirtd termination)<br>
> > <br>
> <br>
> > +    conn = virConnectOpen("qemu:///system");<br>
> > +<br>
> > +    if (conn) {<br>
> > +        /* shut down all threads -- qemud
for example will restart them */<br>
> > +        virNWFilterLearnThreadsTerminate(true);<br>
> > +<br>
> > +        nwfilterDriverLock(driverState);<br>
> > +        virNWFilterPoolLoadAllConfigs(conn,<br>
> > + &driverState->pools,<br>
> <br>
> Indentation problem still looks like it is here.</font></tt>
<br>
<br><tt><font size=2>The indentation problem doesn't exist in my local
patch. Could this be another Thunderbird formatting issue?</font></tt>
<br><tt><font size=2><br>
> <br>
> > @@ -1912,7 +1926,11 @@ qemudStartup(int privileged) {<br>
> >                  
               1, NULL, NULL) <
0)<br>
> >          goto error;<br>
> > <br>
> > -    qemuReconnectDomains(qemu_driver);<br>
> > +    conn = virConnectOpen(qemu_driver->privileged
?<br>
> > +                  
       "qemu:///system" :<br>
> > +                  
       "qemu:///session");<br>
> <br>
> How come this one is conditional, but the other one was hard-coded
to<br>
> system?</font></tt>
<br>
<br><tt><font size=2>The same type of check exists several times in this
file of the qemu driver. I ended up hard-coding it in the nwfilter driver
case above since I don't have a flag in the nwfilter driver for whether
it's privileged or not and I don't know what the difference would be.</font></tt>
<br>
<br><tt><font size=2>Stefan</font></tt>