[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opene
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd |
Date: |
Mon, 18 Jun 2012 10:10:56 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
Am 15.06.2012 22:00, schrieb Eric Blake:
> On 06/15/2012 01:19 PM, Corey Bryant wrote:
>
>>>> There are some flags that I don't think we'll be able to change. For
>>>> example: O_RDONLY, O_WRONLY, O_RDWR. I assume libvirt would open all
>>>> files O_RDWR.
>>>
>>> I think we need to check all of them and fail qemu_open() if they don't
>>> match. Those that qemu can change, should be just changed, of course.
>>>
>>
>> Ok. I remember a scenario where QEMU opens a file read-only (perhaps to
>> check headers and determine the file format) before re-opening it
>> read-write. Perhaps this is only when format= isn't specified with
>> -drive. I'm thinking we may need to change flags to read-write where
>> they used to be read-only, in some circumstances.
>
> In those situations, libvirt would pass fd with O_RDWR, and qemu_open()
> would be fine requesting O_RDONLY the first time (subset is okay), and
> O_RDWR the second time. Where you have to error out is where libvirt
> passes O_RDONLY but qemu wants O_RDWR, and so forth.
Let's try it with requiring an exact match first. If you pass the
format, I think the probing is completely avoided indeed, and having
read-only images really opened O_RDONLY protects against stupid mistakes.
Or if we really need to open the file for probing, maybe we could add a
flag that relaxes the check and that isn't used in the real bdrv_open().
Kevin
- Re: [Qemu-devel] [PATCH v3 5/5] block: Prevent /dev/fd/X filename from being detected as floppy, (continued)
- [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Corey Bryant, 2012/06/14
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Eric Blake, 2012/06/15
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Kevin Wolf, 2012/06/15
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Corey Bryant, 2012/06/15
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Eric Blake, 2012/06/15
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Corey Bryant, 2012/06/15
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd,
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd, Corey Bryant, 2012/06/19
Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Eric Blake, 2012/06/19
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Kevin Wolf, 2012/06/19
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Eric Blake, 2012/06/19
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Kevin Wolf, 2012/06/20
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Daniel P. Berrange, 2012/06/20
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Eric Blake, 2012/06/20
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Corey Bryant, 2012/06/20
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Eric Blake, 2012/06/20
- Re: [Qemu-devel] [PATCH v3 0/5] file descriptor passing using pass-fd, Corey Bryant, 2012/06/20