From mboxrd@z Thu Jan 1 00:00:00 1970 From: Glauber de Oliveira Costa Subject: Re: [PATCH] Unmatched decrementing of net device reference count Date: Thu, 21 Dec 2006 11:02:20 -0200 Message-ID: <20061221130220.GA20308@redhat.com> References: <20061221084932.GA4471@gondor.apana.org.au> <20061221124024.GA6043@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20061221124024.GA6043@gondor.apana.org.au> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Herbert Xu Cc: xen-devel@lists.xensource.com, Keir Fraser List-Id: xen-devel@lists.xenproject.org On Thu, Dec 21, 2006 at 11:40:24PM +1100, Herbert Xu wrote: > On Thu, Dec 21, 2006 at 11:03:50AM +0000, Keir Fraser wrote: > > On 21/12/06 08:49, "Herbert Xu" wrote: > > > > > Thanks for the explanation. I understand the problem now. However, > > > I think your patch isn't adequate because the closing of the backend > > > no longer shuts down the transmitter in the frontend. > > > > Changeset 13100:e99ba0c6c is what I checked in to fix this issue. I think it > > goes far enough, but please do take a look and check. > > Yes it's OK for now because the frontend doesn't hold any resources > from the backend. So even if the frontend tries to xmit after the > backend starts closing, it won't cause any problems. It indeed looks sane, and passes my tests. Thanks! -- Glauber de Oliveira Costa Red Hat Inc. "Free as in Freedom"