|   | 
      | 
  
  
      | 
      | 
  
 
     | 
    | 
  
  
     | 
    | 
  
  
    |   | 
      | 
  
  
    | 
         
xen-devel
Re: [Xen-devel] [PATCH] ioemu : add flush in ide_unplug_harddisks()
 
Stefano Stabellini writes ("Re: [Xen-devel] [PATCH] ioemu : add flush in 
ide_unplug_harddisks()"):
> Tomonari Horikoshi wrote:
> > I think the "flush" is necessary. 
> > When "ioemu ide device" changes into "vbd" by "ide_unplug_harddisks()". 
> 
> I don't think that calling bdrv_aio_flush is a good idea since it is
> asyncronous and right after the call we are going to close the device
> without waiting for completion.
Yes, bdrv_aio_flush is wrong I think.
This seems related to the discussion we had in January.  As I wrote
in
 http://lists.xensource.com/archives/html/xen-devel/2009-01/msg00165.html
it seems to me that the guest should know about be able to flush
for itself ?  After all we implement the IDE flush cache command.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 |   
 
 | 
    | 
  
  
    |   | 
    |