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

[Xen-devel] [PATCH 00 of 10] blkfront pvops updates, v2

To: Xen <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] [PATCH 00 of 10] blkfront pvops updates, v2
From: Daniel Stodden <daniel.stodden@xxxxxxxxxx>
Date: Fri, 30 Apr 2010 22:01:14 -0000
Cc: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Delivery-date: Fri, 30 Apr 2010 15:04:02 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mercurial-patchbomb/1.4.3
Updates since the last time:

 * Make sure everything applies against xen/frontend. That fortunately
   made no difference.

 * Two more loopholes:

    - The path freeing info in xenbus_remove took a more carefully
      sync with an unfortunate bdops->release. This is because the
      locking can't just nest. Fixed by resyncing the info pointer
      through disk->private_data.

    - Similar effect during bdev open. This also needs to anticipate
      the not-so-unlikely case where not only the gendisk but the
      entire info struct was deleted just before entry. Again, syncing
      on disk->private_data once holding bd_mutex does the trick.

    I chose to just merge those in.

 * To be honest, that dev_warn the last patch added was broken -- doh.

 * Found the switch [again] for HG to strip those headers -- cheers.

Feedback certainly welcome. I am aware this thing looks manic, but at
it stands, we recently happend to come across exactly that family of
races in XCP again, so I really want these paths fixed.

Thanks,
Daniel

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