Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 13.08.19 17:03, Kevin Wolf wrote: > Am 13.08.2019 um 16:53 hat Max Reitz geschrieben: >> On 13.08.19 16:46, Vladimir Sementsov-Ogievskiy wrote: >>> 13.08.2019 17:31, Max Reitz wrote: On 13.08.19 13:51, Kevin Wolf wrote: [...] > Hm... This is a mess. :-) Just out

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

Am 13.08.2019 um 16:53 hat Max Reitz geschrieben: > On 13.08.19 16:46, Vladimir Sementsov-Ogievskiy wrote: > > 13.08.2019 17:31, Max Reitz wrote: > >> On 13.08.19 13:51, Kevin Wolf wrote: > >> > >> [...] > >> > >>> Hm... This is a mess. :-) > >> > >> Just out of curiosity: Why? > >> > >> Aren’t the

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 13.08.19 16:46, Vladimir Sementsov-Ogievskiy wrote: > 13.08.2019 17:31, Max Reitz wrote: >> On 13.08.19 13:51, Kevin Wolf wrote: >> >> [...] >> >>> Hm... This is a mess. :-) >> >> Just out of curiosity: Why? >> >> Aren’t there only two things we really need from the block_status >> infrastructur

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 8/13/19 9:31 AM, Max Reitz wrote: > On 13.08.19 13:51, Kevin Wolf wrote: > > [...] > >> Hm... This is a mess. :-) > > Just out of curiosity: Why? > > Aren’t there only two things we really need from the block_status > infrastructure? > > (1) Whether something is allocated in the given layer

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

13.08.2019 17:31, Max Reitz wrote: > On 13.08.19 13:51, Kevin Wolf wrote: > > [...] > >> Hm... This is a mess. :-) > > Just out of curiosity: Why? > > Aren’t there only two things we really need from the block_status > infrastructure? > > (1) Whether something is allocated in the given layer o

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 13.08.19 11:34, Kevin Wolf wrote: > Am 12.08.2019 um 21:46 hat Max Reitz geschrieben: >> On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: >>> Hi all! >>> >>> I'm not sure, is it a bug or a feature, but using qcow2 under raw is >>> broken. It should be either fixed like I propose (by Max's

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 13.08.19 13:51, Kevin Wolf wrote: [...] > Hm... This is a mess. :-) Just out of curiosity: Why? Aren’t there only two things we really need from the block_status infrastructure? (1) Whether something is allocated in the given layer of the backing chain, (2) Whether we know that a given ran

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

13.08.2019 14:51, Kevin Wolf wrote: > Am 13.08.2019 um 13:14 hat Vladimir Sementsov-Ogievskiy geschrieben: >> 13.08.2019 12:33, Vladimir Sementsov-Ogievskiy wrote: >>> 13.08.2019 12:01, Vladimir Sementsov-Ogievskiy wrote: 13.08.2019 11:39, Vladimir Sementsov-Ogievskiy wrote: > 12.08.2019 2

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

Am 13.08.2019 um 13:14 hat Vladimir Sementsov-Ogievskiy geschrieben: > 13.08.2019 12:33, Vladimir Sementsov-Ogievskiy wrote: > > 13.08.2019 12:01, Vladimir Sementsov-Ogievskiy wrote: > >> 13.08.2019 11:39, Vladimir Sementsov-Ogievskiy wrote: > >>> 12.08.2019 22:50, Max Reitz wrote: > On 12.08.

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

13.08.2019 12:33, Vladimir Sementsov-Ogievskiy wrote: > 13.08.2019 12:01, Vladimir Sementsov-Ogievskiy wrote: >> 13.08.2019 11:39, Vladimir Sementsov-Ogievskiy wrote: >>> 12.08.2019 22:50, Max Reitz wrote: On 12.08.19 21:46, Max Reitz wrote: > On 12.08.19 20:11, Vladimir Sementsov-Ogievski

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

13.08.2019 12:01, Vladimir Sementsov-Ogievskiy wrote: > 13.08.2019 11:39, Vladimir Sementsov-Ogievskiy wrote: >> 12.08.2019 22:50, Max Reitz wrote: >>> On 12.08.19 21:46, Max Reitz wrote: On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I'm not sure, is it a bug

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

Am 12.08.2019 um 21:46 hat Max Reitz geschrieben: > On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: > > Hi all! > > > > I'm not sure, is it a bug or a feature, but using qcow2 under raw is > > broken. It should be either fixed like I propose (by Max's suggestion) > > or somehow forbidden (j

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

13.08.2019 11:39, Vladimir Sementsov-Ogievskiy wrote: > 12.08.2019 22:50, Max Reitz wrote: >> On 12.08.19 21:46, Max Reitz wrote: >>> On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: Hi all! I'm not sure, is it a bug or a feature, but using qcow2 under raw is broken. It sh

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

12.08.2019 22:50, Max Reitz wrote: > On 12.08.19 21:46, Max Reitz wrote: >> On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: >>> Hi all! >>> >>> I'm not sure, is it a bug or a feature, but using qcow2 under raw is >>> broken. It should be either fixed like I propose (by Max's suggestion) >>>

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 12.08.19 21:46, Max Reitz wrote: > On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: >> Hi all! >> >> I'm not sure, is it a bug or a feature, but using qcow2 under raw is >> broken. It should be either fixed like I propose (by Max's suggestion) >> or somehow forbidden (just forbid backing-f

Re: [Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

On 12.08.19 20:11, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I'm not sure, is it a bug or a feature, but using qcow2 under raw is > broken. It should be either fixed like I propose (by Max's suggestion) > or somehow forbidden (just forbid backing-file supporting node to be > file child of

[Qemu-devel] [PATCH 0/2] deal with BDRV_BLOCK_RAW

Hi all! I'm not sure, is it a bug or a feature, but using qcow2 under raw is broken. It should be either fixed like I propose (by Max's suggestion) or somehow forbidden (just forbid backing-file supporting node to be file child of raw-format node). Vladimir Sementsov-Ogievskiy (2): block/raw-fo