<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">Is this still up-to-date? &nbsp;I know that we've had a lot of discussion and<br>a couple of patch versions related to netdevs this week.<br></blockquote></div></div></div></blockquote><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><br></blockquote><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">Yup, this is still up-to-date. I've looked over the netdev code committed in the last week and did a sanity test to make sure everything still works.&nbsp;</blockquote><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><br></blockquote><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><br></blockquote><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">&nbsp;have to say that I don't understand the previous paragraph of the<br>commit message. &nbsp;Maybe you could spell it out more precisely? &nbsp;Also, the<br>netdev portion of the commit seems to have little to do with the rest,<br>so I'd be inclined to make it a separate preparatory commit.</blockquote></div></div></div></blockquote><br></blockquote><br></div><div class="gmail_quote">After re-reading my commit, I totally agree. I'll resubmit the patch split into 2 with a more clear description.</div><div class="gmail_quote"><br></div></div></div></div><div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: medium; font-variant: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><b>Ryan Wilson</b><br><i style="font-weight: normal; ">Member of Technical Staff</i><br><a href="mailto:wryan@vmware.com">wryan@vmware.com</a><br>3401 Hillview Avenue, Palo Alto, CA<br>650.427.1511 Office<br>916.588.7783 Mobile</div>
</div>
<br><div><div>On May 9, 2014, at 9:59 AM, Alex Wang &lt;<a href="mailto:alexw@nicira.com">alexw@nicira.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
<div dir="ltr">Hey Ben,<div><br></div><div>I think this one is up-to-date. &nbsp;I'm reviewing it.</div><div><br></div><div>Your comments make sense. &nbsp;I think Ryan will adjust it.</div><div><br></div><div>Also, I'd also like you or Ethan to review it.</div>

<div><br></div><div>Thanks,</div><div>Alex Wang,<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 9, 2014 at 8:29 AM, Ben Pfaff <span dir="ltr">&lt;<a href="mailto:blp@nicira.com" target="_blank">blp@nicira.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">Is this still up-to-date? &nbsp;I know that we've had a lot of discussion and<br>
a couple of patch versions related to netdevs this week.<br>
<br>
More below.<br>
<div class=""><br>
On Tue, May 06, 2014 at 09:54:12AM -0700, Ryan Wilson wrote:<br>
&gt; Before, a global read-write lock protected the ofproto-dpif / ofproto-dpif-xlate<br>
&gt; interface. Handler and revalidator threads had to wait while configuration was<br>
&gt; being changed. This patch implements RCU locking which allows handlers and<br>
&gt; revalidators to operate while configuration is being updated.<br>
&gt;<br>
&gt; This patch also frees netdev with ovsrcu_postpone. This is because if RCU<br>
&gt; reader threads take a ref to the netdev, the netdev cannot be deleted and<br>
&gt; re-created with different types.<br>
<br>
</div>I have to say that I don't understand the previous paragraph of the<br>
commit message. &nbsp;Maybe you could spell it out more precisely? &nbsp;Also, the<br>
netdev portion of the commit seems to have little to do with the rest,<br>
so I'd be inclined to make it a separate preparatory commit.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
dev mailing list<br>
<a href="mailto:dev@openvswitch.org">dev@openvswitch.org</a><br>
<a href="https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailman/listinfo/dev&amp;k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&amp;r=TfBS78Vw3dzttvXidhbffg%3D%3D%0A&amp;m=RAKnIIXHCmI3s2%2BLAePA95RJxiS15u0HNdtodh5T548%3D%0A&amp;s=499132edfb3bd837e33675f3a94e4530141494d95d9355ebb695e623b2a5e689" target="_blank">http://openvswitch.org/mailman/listinfo/dev</a><br>
</div></div></blockquote></div><br></div></div></div>
</blockquote></div><br></body></html>