<div dir="ltr">Thanks. Will send out V3 soon. I dropped the list by mistake. <br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 11, 2014 at 1:37 PM, Ben Pfaff <span dir="ltr"><<a href="mailto:blp@nicira.com" target="_blank">blp@nicira.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I guess you (or I?) dropped the list by mistake, so I'm adding it back.<br>
<br>
Definitely. With very few exceptions, we basically don't block anywhere<br>
in vswitchd. If you do, it can delay everything else happening in that<br>
thread.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Feb 11, 2014 at 01:31:29PM -0800, Andy Zhou wrote:<br>
> Do you mean that we should consider set_config() as part of main loop<br>
> although it is only called a few times for configuration?<br>
><br>
><br>
> On Tue, Feb 11, 2014 at 1:27 PM, Ben Pfaff <<a href="mailto:blp@nicira.com">blp@nicira.com</a>> wrote:<br>
><br>
> > On Tue, Feb 11, 2014 at 01:15:01PM -0800, Andy Zhou wrote:<br>
> > > The netdev-dummy unit test ran into the reconnect condition on Jarno's<br>
> > > machine. With his test environment, we found and fixed the bugs in<br>
> > > handling reconnect.<br>
> > ><br>
> > > Co-authored-by: Jarno Rajahalme <<a href="mailto:jrajahalme@nicira.com">jrajahalme@nicira.com</a>><br>
> > > Signed-off-by: Jarno Rajahalme <<a href="mailto:jrajahalme@nicira.com">jrajahalme@nicira.com</a>><br>
> > > Signed-off-by: Andy Zhou <<a href="mailto:azhou@nicira.com">azhou@nicira.com</a>><br>
> ><br>
> > This still calls stream_open_block().<br>
> ><br>
</div></div></blockquote></div><br></div>