WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Fix blkback/blktap sysfs read bug.

To: Joe Jin <joe.jin@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Fix blkback/blktap sysfs read bug.
From: Daniel Stodden <daniel.stodden@xxxxxxxxxx>
Date: Thu, 21 Jan 2010 10:01:44 -0800
Cc: "greg.marsden@xxxxxxxxxx" <greg.marsden@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "deepak.patel@xxxxxxxxxx" <deepak.patel@xxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Delivery-date: Thu, 21 Jan 2010 10:02:03 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20100121074928.GA31296@xxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20100119141338.GA22249@xxxxxxxxxxxxxxxxxxxxxxx> <4B55E9E7020000780002AC17@xxxxxxxxxxxxxxxxxx> <20100120020605.GA25697@xxxxxxxxxxxxxxxxxxxxxxx> <4B56C2DB020000780002AE45@xxxxxxxxxxxxxxxxxx> <20100120105136.GA6801@xxxxxxxxxxxxxxxxxxxxxxx> <4B56F1B6020000780002AEFA@xxxxxxxxxxxxxxxxxx> <20100120114518.GA10851@xxxxxxxxxxxxxxxxxxxxxxx> <1264040197.12544.3679.camel@xxxxxxxxxxxxxxxxxxxxxxx> <20100121031307.GA29727@xxxxxxxxxxxxxxxxxxxxxxx> <1264058809.6898.181.camel@xxxxxxxxxxxxxxxxxxx> <20100121074928.GA31296@xxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Thu, 2010-01-21 at 02:49 -0500, Joe Jin wrote:

> > The reason is in sysfs_deactivate(), which will sync callers against any
> > remaining thread in .show()
> > - show() hangs on the lock
> > - the lock holder in sysfs_remove_group(), 
> >   waiting for show() to complete.
> > 
> > Pardon me -- I'm not entirely sure where/how these patches are currently
> > submitted and merged. I suppose yours are only for linux-2.6.18.hg, not
> > e.g. pvops? Then sorry for any confusion.
> > 
> 
> Daniel, 
> 
> Thanks a lot of your comments, it really help for me, yes my patch based 
> linux-2.6.18.hg branch. As Jan have pointed out in previous email, it should 
> be sysfs's issue, looked like later kernel sysfs have fixed the issue?

Yes. Later sysfs won't need the locking. Ordering sysfs removal before
backend release will be sufficient.

Daniel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel