From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Stodden Subject: [PATCH 00 of 10] blkfront pvops updates, v2 Date: Fri, 30 Apr 2010 22:01:14 -0000 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Xen Cc: Jeremy Fitzhardinge List-Id: xen-devel@lists.xenproject.org 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