Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-08-09 Thread Max Reitz
On 09.08.2016 20:39, Reda Sallahi wrote: > Hi Max, > Thanks for the review! > > On 8/8/16, Max Reitz wrote: >> On 25.07.2016 07:58, Reda Sallahi wrote: >>> +if (in->bsz == 0) { >>> +error_report("invalid number: '%s'", arg); >> >> It's not an invalid number, it's just an invalid block

Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-08-09 Thread Reda Sallahi
Hi Max, Thanks for the review! On 8/8/16, Max Reitz wrote: > On 25.07.2016 07:58, Reda Sallahi wrote: >> +if (in->bsz == 0) { >> +error_report("invalid number: '%s'", arg); > > It's not an invalid number, it's just an invalid block size. In my > understanding, those are two different

Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-08-09 Thread Kevin Wolf
Am 25.07.2016 um 07:58 hat Reda Sallahi geschrieben: > This patch adds a basic dd subcommand analogous to dd(1) to qemu-img. > > For the start, this implements the bs, if, of and count options and requires > both if and of to be specified (no stdin/stdout if not specified) and doesn't > support tt

Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-08-08 Thread Max Reitz
On 25.07.2016 07:58, Reda Sallahi wrote: > This patch adds a basic dd subcommand analogous to dd(1) to qemu-img. > > For the start, this implements the bs, if, of and count options and requires > both if and of to be specified (no stdin/stdout if not specified) and doesn't > support tty, pipes, et

Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-08-08 Thread Stefan Hajnoczi
On Mon, Jul 25, 2016 at 6:58 AM, Reda Sallahi wrote: > This patch adds a basic dd subcommand analogous to dd(1) to qemu-img. Ping? Reda has additional patches on top of this one. It would be good to get this into the block-next branch so his later work can be merged. > For the start, this impl

Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-07-28 Thread Stefan Hajnoczi
On Mon, Jul 25, 2016 at 6:58 AM, Reda Sallahi wrote: > This patch adds a basic dd subcommand analogous to dd(1) to qemu-img. > > For the start, this implements the bs, if, of and count options and requires > both if and of to be specified (no stdin/stdout if not specified) and doesn't > support tt

Re: [Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-07-27 Thread Fam Zheng
On Mon, 07/25 07:58, Reda Sallahi wrote: > This patch adds a basic dd subcommand analogous to dd(1) to qemu-img. > > For the start, this implements the bs, if, of and count options and requires > both if and of to be specified (no stdin/stdout if not specified) and doesn't > support tty, pipes, et

[Qemu-block] [PATCH v6] qemu-img: add the 'dd' subcommand

2016-07-24 Thread Reda Sallahi
This patch adds a basic dd subcommand analogous to dd(1) to qemu-img. For the start, this implements the bs, if, of and count options and requires both if and of to be specified (no stdin/stdout if not specified) and doesn't support tty, pipes, etc. The image format must be specified with -O for