From: Max Reitz <mreitz@redhat.com>
To: Peter Lieven <pl@kamp.de>, Eric Blake <eblake@redhat.com>,
qemu-block@nongnu.org
Cc: kwolf@redhat.com, qemu-devel@nongnu.org
Subject: Re: [PATCH] qemu-img: add seek and -n option to dd command
Date: Fri, 5 Feb 2021 10:16:12 +0100 [thread overview]
Message-ID: <50cb6864-b0a9-c1dc-2cba-9a35a2970ba2@redhat.com> (raw)
In-Reply-To: <70e0dbc0-4770-a409-945e-c31e0d93081b@kamp.de>
On 05.02.21 09:47, Peter Lieven wrote:
> Am 05.02.21 um 09:18 schrieb Max Reitz:
>> On 04.02.21 21:09, Peter Lieven wrote:
>>> Am 02.02.21 um 16:51 schrieb Eric Blake:
>>>> On 1/28/21 8:07 AM, Peter Lieven wrote:
>>>>> Signed-off-by: Peter Lieven <pl@kamp.de>
>>>> Your commit message says 'what', but not 'why'. Generally, the one-line
>>>> 'what' works well as the subject line, but you want the commit body to
>>>> give an argument why your patch should be applied, rather than blank.
>>>>
>>>> Here's the last time we tried to improve qemu-img dd:
>>>> https://lists.gnu.org/archive/html/qemu-devel/2018-08/msg02618.html
>>>
>>>
>>> I was not aware of that story. My use case is that I want to be
>>>
>>> able to "patch" an image that Qemu is able to handle by overwriting
>>>
>>> certain sectors. And I especially do not want to "mount" that image
>>>
>>> via qemu-nbd because I might not trust it. I totally want to avoid that the host
>>>
>>> system tries to analyse that image in terms of scanning the bootsector, partprobe,
>>>
>>> lvm etc. pp.
>>
>> qemu will have FUSE exporting as of 6.0 (didn’t quite make it into 5.2), so you can do something like this:
>>
>> $ qemu-storage-daemon \
>> --blockdev node-name=export,driver=qcow2,\
>> file.driver=file,file.filename=image.qcow2 \
>> --export fuse,id=fuse,node-name=export,mountpoint=image.qcow2
>>
>> This exports the image on image.qcow2 (i.e., on itself) and so by accessing the image file you then get raw access to its contents (so you can use system tools like dd).
>>
>> Doesn’t require root rights, and shouldn’t make the kernel scan anything, because it’s exported as just a regular file.
>
>
> Okay, but that is still more housekeeping than just invoking a single command.
Yes, but I personally see this as much better than copying all of dd’s
functionality into qemu-img.
My personal complaint is only that it’s a pain in the ass to invoke QSD
this way. It would be nice to have a script that does the same via
$ qemu-blk-fuse-export image.qcow2
Would probably be trivial to write, but well, first we gotta do it, and
have justification to keep it as part of qemu...
And if that’s still too much housekeeping, we could even write a qemu-dd
script that scans all file arguments for non-raw images, launches a QSD
instance to present them as raw, and then invokes dd.
All much simpler and much more feature-complete than to add more dd
functionality to qemu-img.
> Would it be an option to extend qemu-io to write data at a certain offset which it reads from STDIN?
You mean read the data from stdin? Isn’t that what “write -s <file>” does?
Max
next prev parent reply other threads:[~2021-02-05 9:18 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-28 14:07 [PATCH] qemu-img: add seek and -n option to dd command Peter Lieven
2021-02-02 10:20 ` David Edmondson
2021-02-02 15:51 ` Eric Blake
2021-02-04 20:09 ` Peter Lieven
2021-02-04 20:44 ` Eric Blake
2021-02-05 10:43 ` Richard W.M. Jones
2021-02-05 8:18 ` Max Reitz
2021-02-05 8:47 ` Peter Lieven
2021-02-05 9:16 ` Max Reitz [this message]
2021-02-05 10:06 ` Max Reitz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50cb6864-b0a9-c1dc-2cba-9a35a2970ba2@redhat.com \
--to=mreitz@redhat.com \
--cc=eblake@redhat.com \
--cc=kwolf@redhat.com \
--cc=pl@kamp.de \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.