|   | 
      | 
  
  
      | 
      | 
  
 
     | 
    | 
  
  
     | 
    | 
  
  
    |   | 
      | 
  
  
    | 
         
xen-devel
[Xen-devel] [PATCH 00 of 10] blkfront pvops updates, v2
 
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
 |   
 
| <Prev in Thread] | 
Current Thread | 
[Next in Thread>
 |  
- [Xen-devel] [PATCH 00 of 10] blkfront pvops updates, v2,
Daniel Stodden <=
- [Xen-devel] [PATCH 03 of 10] blkfront: Fix gendisk leak, Daniel Stodden
 
- [Xen-devel] [PATCH 01 of 10] xenbus: Make xenbus_switch_state	transactional (again), Daniel Stodden
 
- [Xen-devel] [PATCH 02 of 10] blkfront: Fix backtrace in del_gendisk, Daniel Stodden
 
- [Xen-devel] [PATCH 06 of 10] blkfront: Fix blkfront backend switch	race (bdev open), Daniel Stodden
 
- [Xen-devel] [PATCH 04 of 10] blkfront: Clean up vbd release, Daniel Stodden
 
- [Xen-devel] [PATCH 05 of 10] blkfront: Lock blkfront_info when	closing, Daniel Stodden
 
- [Xen-devel] [PATCH 09 of 10] blkfront: Remove obsolete info->users, Daniel Stodden
 
- [Xen-devel] [PATCH 07 of 10] blkfront: Fix blkfront backend switch	race (bdev release), Daniel Stodden
 
- [Xen-devel] [PATCH 08 of 10] blkfront: Lock blockfront_info during	xbdev removal, Daniel Stodden
 
- [Xen-devel] [PATCH 10 of 10] blkfront: Klog the unclean release path, Daniel Stodden
 
 
 
 
 |  
  
 | 
    | 
  
  
    |   | 
    |