Re: [Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-17 Thread Max Reitz
On 2018-08-16 09:15, Kevin Wolf wrote: > Am 16.08.2018 um 04:49 hat Max Reitz geschrieben: >> On 2018-08-16 04:39, Eric Blake wrote: >>> If convert were more powerful, I'd be fine dropping 'qemu-img dd' after >>> a proper deprecation period. >> >> Technically it has those features already, with

Re: [Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-16 Thread Kevin Wolf
Am 16.08.2018 um 04:49 hat Max Reitz geschrieben: > On 2018-08-16 04:39, Eric Blake wrote: > > If convert were more powerful, I'd be fine dropping 'qemu-img dd' after > > a proper deprecation period. > > Technically it has those features already, with the raw block driver's > offset and size

Re: [Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-15 Thread Eric Blake
On 08/15/2018 09:49 PM, Max Reitz wrote: In my opinion, we do not want feature parity with dd.  What we do want is feature parity with convert. Well, convert is lacking a way to specify a subset of one file to move to a (possibly different) subset of the other.  I'm fine if we want to enhance

Re: [Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-15 Thread Max Reitz
On 2018-08-16 04:39, Eric Blake wrote: > On 08/15/2018 09:20 PM, Max Reitz wrote: >> On 2018-08-15 04:56, Eric Blake wrote: >>> For feature parity with dd, we want to be able to specify >>> the offset within the output file, just as we can specify >>> the offset for the input (in particular, this

Re: [Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-15 Thread Eric Blake
On 08/15/2018 09:20 PM, Max Reitz wrote: On 2018-08-15 04:56, Eric Blake wrote: For feature parity with dd, we want to be able to specify the offset within the output file, just as we can specify the offset for the input (in particular, this makes copying a subset range of guest-visible bytes

Re: [Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-15 Thread Max Reitz
On 2018-08-15 04:56, Eric Blake wrote: > For feature parity with dd, we want to be able to specify > the offset within the output file, just as we can specify > the offset for the input (in particular, this makes copying > a subset range of guest-visible bytes from one file to > another much

[Qemu-block] [PATCH 2/2] qemu-img: Add dd seek= option

2018-08-14 Thread Eric Blake
For feature parity with dd, we want to be able to specify the offset within the output file, just as we can specify the offset for the input (in particular, this makes copying a subset range of guest-visible bytes from one file to another much easier). The code style for 'qemu-img dd' was pretty