[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] raw-posix: Detect CDROM via ioctl
From: |
Cole Robinson |
Subject: |
Re: [Qemu-devel] [PATCH v3] raw-posix: Detect CDROM via ioctl |
Date: |
Thu, 14 Jan 2010 11:09:53 -0500 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20090922 Fedora/3.0-3.9.b4.fc12 Thunderbird/3.0b4 |
On 01/14/2010 05:45 AM, malc wrote:
> On Wed, 13 Jan 2010, Cole Robinson wrote:
>
>> On 01/13/2010 07:11 PM, malc wrote:
>>> On Wed, 13 Jan 2010, Cole Robinson wrote:
>>>
>>>> Current CDROM detection is hardcoded based on source file name.
>>>> Make this smarter by attempting a CDROM specific ioctl.
>>>>
>>>> This makes '-cdrom /dev/sr0' succeed with no media present.
>>>>
>>>> v2:
>>>> Give ioctl check higher priority than filename check,
>>>>
>>>> v3:
>>>> Actually initialize 'prio' variable
>>>> Check for ioctl success rather than absence of specific failure
>>>
>>> Does it even compile on BSDs, Darwin etc?
>>>
>>
>> The changed functions are all under #ifdef __linux__, so I assume its fine.
>> Haven't tested though.
>
> Fine if so, it's not immediatelly aparent from the patches nor from
> the commit messages though.
>
True. I'll resend with a better commit message.
Thanks,
Cole