[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH 10/54] vvfat: Use opened node as backing file

From: Kevin Wolf
Subject: Re: [Qemu-block] [PATCH 10/54] vvfat: Use opened node as backing file
Date: Thu, 23 Feb 2017 13:25:35 +0100
User-agent: Mutt/1.5.21 (2010-09-15)

Am 23.02.2017 um 12:49 hat Fam Zheng geschrieben:
> On Tue, 02/21 15:58, Kevin Wolf wrote:
> > -    backing = bdrv_new();
> > +    backing = bdrv_new_open_driver(&vvfat_write_target, NULL, 
> > +                                   &error_abort);
> > +    *(void**) backing->opaque = s;
> Could be simply "backing->opaque = s".

That's not semantically the same.

Or if you just mean that vvfat shouldn't be using an implicit
BdrvVVFATState with just one pointer element rather than just directly
assigning the pointer to bs->opaque, that might have been an option, but
vvfat isn't written that way and I don't intend to change it at least in
this specific patch. (If I were to change it, I would probably just use
an explicit BdrvVVFATState, though.)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]