Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-25 Thread Vladimir Sementsov-Ogievskiy
25.11.2019 18:46, Kevin Wolf wrote: > Am 25.11.2019 um 11:08 hat Vladimir Sementsov-Ogievskiy geschrieben: >> Ping? >> >> Hi! Why so silent? Postpone this to 5.0? This is fixing the same >> problem with block commit, like Kevin's series, just commit not to mid >> but to base.. > > To be honest, I

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-25 Thread Kevin Wolf
Am 25.11.2019 um 11:08 hat Vladimir Sementsov-Ogievskiy geschrieben: > Ping? > > Hi! Why so silent? Postpone this to 5.0? This is fixing the same > problem with block commit, like Kevin's series, just commit not to mid > but to base.. To be honest, I think by now we've found so many problems

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-25 Thread Vladimir Sementsov-Ogievskiy
Ping? Hi! Why so silent? Postpone this to 5.0? This is fixing the same problem with block commit, like Kevin's series, just commit not to mid but to base.. 16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I wanted to understand, what is the real difference between >

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-20 Thread Vladimir Sementsov-Ogievskiy
20.11.2019 16:30, Kevin Wolf wrote: > Am 20.11.2019 um 13:04 hat Vladimir Sementsov-Ogievskiy geschrieben: >> 20.11.2019 14:44, Kevin Wolf wrote: >>> Am 20.11.2019 um 11:20 hat Vladimir Sementsov-Ogievskiy geschrieben: 16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: > Hi all! >

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-20 Thread Vladimir Sementsov-Ogievskiy
20.11.2019 15:04, Vladimir Sementsov-Ogievskiy wrote: > 20.11.2019 14:44, Kevin Wolf wrote: >> Am 20.11.2019 um 11:20 hat Vladimir Sementsov-Ogievskiy geschrieben: >>> 16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: Hi all! I wanted to understand, what is the real difference

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-20 Thread Kevin Wolf
Am 20.11.2019 um 13:04 hat Vladimir Sementsov-Ogievskiy geschrieben: > 20.11.2019 14:44, Kevin Wolf wrote: > > Am 20.11.2019 um 11:20 hat Vladimir Sementsov-Ogievskiy geschrieben: > >> 16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: > >>> Hi all! > >>> > >>> I wanted to understand, what is

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-20 Thread Vladimir Sementsov-Ogievskiy
20.11.2019 14:44, Kevin Wolf wrote: > Am 20.11.2019 um 11:20 hat Vladimir Sementsov-Ogievskiy geschrieben: >> 16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: >>> Hi all! >>> >>> I wanted to understand, what is the real difference between >>> bdrv_block_status_above >>> and

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-20 Thread Kevin Wolf
Am 20.11.2019 um 11:20 hat Vladimir Sementsov-Ogievskiy geschrieben: > 16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: > > Hi all! > > > > I wanted to understand, what is the real difference between > > bdrv_block_status_above > > and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-20 Thread Vladimir Sementsov-Ogievskiy
16.11.2019 19:34, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I wanted to understand, what is the real difference between > bdrv_block_status_above > and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work through > bdrv_block_status_above.. > > And I found the problem:

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 19:58, Stefan Hajnoczi wrote: > On Sat, Nov 16, 2019 at 07:34:06PM +0300, Vladimir Sementsov-Ogievskiy wrote: >> Hi all! >> >> I wanted to understand, what is the real difference between >> bdrv_block_status_above >> and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Stefan Hajnoczi
On Sat, Nov 16, 2019 at 07:34:06PM +0300, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I wanted to understand, what is the real difference between > bdrv_block_status_above > and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work through > bdrv_block_status_above.. > > And I

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Kevin Wolf
Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: > Hi all! > > I wanted to understand, what is the real difference between > bdrv_block_status_above > and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work through > bdrv_block_status_above.. > > And I found

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Kevin Wolf
Am 19.11.2019 um 13:17 hat Vladimir Sementsov-Ogievskiy geschrieben: > 19.11.2019 15:05, Kevin Wolf wrote: > > Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: > >> Hi all! > >> > >> I wanted to understand, what is the real difference between > >> bdrv_block_status_above and

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Kevin Wolf
Am 19.11.2019 um 13:30 hat Vladimir Sementsov-Ogievskiy geschrieben: > 19.11.2019 15:20, Max Reitz wrote: > > On 19.11.19 13:02, Denis V. Lunev wrote: > >> On 11/19/19 1:22 PM, Max Reitz wrote: > >>> On 16.11.19 17:34, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I wanted to

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 15:34, Vladimir Sementsov-Ogievskiy wrote: > 19.11.2019 15:32, Vladimir Sementsov-Ogievskiy wrote: >> 19.11.2019 15:17, Vladimir Sementsov-Ogievskiy wrote: >>> 19.11.2019 15:05, Kevin Wolf wrote: Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: > Hi all!

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 15:32, Vladimir Sementsov-Ogievskiy wrote: > 19.11.2019 15:17, Vladimir Sementsov-Ogievskiy wrote: >> 19.11.2019 15:05, Kevin Wolf wrote: >>> Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: Hi all! I wanted to understand, what is the real difference

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 15:17, Vladimir Sementsov-Ogievskiy wrote: > 19.11.2019 15:05, Kevin Wolf wrote: >> Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: >>> Hi all! >>> >>> I wanted to understand, what is the real difference between >>> bdrv_block_status_above and

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 15:20, Max Reitz wrote: > On 19.11.19 13:02, Denis V. Lunev wrote: >> On 11/19/19 1:22 PM, Max Reitz wrote: >>> On 16.11.19 17:34, Vladimir Sementsov-Ogievskiy wrote: Hi all! I wanted to understand, what is the real difference between bdrv_block_status_above

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Max Reitz
On 19.11.19 13:02, Denis V. Lunev wrote: > On 11/19/19 1:22 PM, Max Reitz wrote: >> On 16.11.19 17:34, Vladimir Sementsov-Ogievskiy wrote: >>> Hi all! >>> >>> I wanted to understand, what is the real difference between >>> bdrv_block_status_above >>> and bdrv_is_allocated_above, IMHO

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 15:05, Kevin Wolf wrote: > Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: >> Hi all! >> >> I wanted to understand, what is the real difference between >> bdrv_block_status_above and bdrv_is_allocated_above, IMHO >> bdrv_is_allocated_above should work through

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Vladimir Sementsov-Ogievskiy
19.11.2019 15:02, Denis Lunev wrote: > On 11/19/19 1:22 PM, Max Reitz wrote: >> On 16.11.19 17:34, Vladimir Sementsov-Ogievskiy wrote: >>> Hi all! >>> >>> I wanted to understand, what is the real difference between >>> bdrv_block_status_above >>> and bdrv_is_allocated_above, IMHO

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Kevin Wolf
Am 16.11.2019 um 17:34 hat Vladimir Sementsov-Ogievskiy geschrieben: > Hi all! > > I wanted to understand, what is the real difference between > bdrv_block_status_above and bdrv_is_allocated_above, IMHO > bdrv_is_allocated_above should work through bdrv_block_status_above.. > > And I found the

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Denis V. Lunev
On 11/19/19 1:22 PM, Max Reitz wrote: > On 16.11.19 17:34, Vladimir Sementsov-Ogievskiy wrote: >> Hi all! >> >> I wanted to understand, what is the real difference between >> bdrv_block_status_above >> and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work through >>

Re: [PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-19 Thread Max Reitz
On 16.11.19 17:34, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > I wanted to understand, what is the real difference between > bdrv_block_status_above > and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work through > bdrv_block_status_above.. > > And I found the problem:

[PATCH 0/4] fix & merge block_status_above and is_allocated_above

2019-11-16 Thread Vladimir Sementsov-Ogievskiy
Hi all! I wanted to understand, what is the real difference between bdrv_block_status_above and bdrv_is_allocated_above, IMHO bdrv_is_allocated_above should work through bdrv_block_status_above.. And I found the problem: bdrv_is_allocated_above considers space after EOF as UNALLOCATED for