On Thu, Mar 17, 2011 at 9:07 AM, Kevin Wolf wrote:
> Am 16.03.2011 18:00, schrieb Stefan Hajnoczi:
>> On Wed, Mar 16, 2011 at 2:08 PM, Christoph Hellwig wrote:
>>> On Wed, Mar 16, 2011 at 09:42:37AM +, Stefan Hajnoczi wrote:
- writethrough = ((flags & BDRV_O_CACHE_MASK) == 0);
+
Am 16.03.2011 18:00, schrieb Stefan Hajnoczi:
> On Wed, Mar 16, 2011 at 2:08 PM, Christoph Hellwig wrote:
>> On Wed, Mar 16, 2011 at 09:42:37AM +, Stefan Hajnoczi wrote:
>>> -writethrough = ((flags & BDRV_O_CACHE_MASK) == 0);
>>> +writethrough = ((flags & (BDRV_O_CACHE_WB | BDRV_O_NO_F
On Wed, Mar 16, 2011 at 2:08 PM, Christoph Hellwig wrote:
> On Wed, Mar 16, 2011 at 09:42:37AM +, Stefan Hajnoczi wrote:
>> - writethrough = ((flags & BDRV_O_CACHE_MASK) == 0);
>> + writethrough = ((flags & (BDRV_O_CACHE_WB | BDRV_O_NO_FLUSH)) == 0);
>
> or rather
>
> writethrough
On Wed, Mar 16, 2011 at 09:42:37AM +, Stefan Hajnoczi wrote:
> -writethrough = ((flags & BDRV_O_CACHE_MASK) == 0);
> +writethrough = ((flags & (BDRV_O_CACHE_WB | BDRV_O_NO_FLUSH)) == 0);
or rather
writethrough = ((flags & (BDRV_O_CACHE_WB) != );
but yes, this code had sneaked
Am 16.03.2011 10:42, schrieb Stefan Hajnoczi:
> On Tue, Mar 15, 2011 at 2:11 PM, Christoph Hellwig wrote:
>> Change BDRV_O_NOCACHE to only imply bypassing the host OS file cache,
>> but no writeback semantics. All existing callers are changed to also
>> specify BDRV_O_CACHE_WB to give them writeb
On Tue, Mar 15, 2011 at 2:11 PM, Christoph Hellwig wrote:
> Change BDRV_O_NOCACHE to only imply bypassing the host OS file cache,
> but no writeback semantics. All existing callers are changed to also
> specify BDRV_O_CACHE_WB to give them writeback semantics.
>
> Signed-off-by: Christoph Hellwig
