qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [PATCH v4 1/1] block: improve error handling in raw_ope


From: Max Reitz
Subject: Re: [Qemu-block] [PATCH v4 1/1] block: improve error handling in raw_open
Date: Fri, 23 Sep 2016 18:13:27 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0

On 23.09.2016 18:04, Max Reitz wrote:
> On 08.09.2016 15:59, Halil Pasic wrote:
>> Make raw_open for POSIX more consistent in handling errors by setting
>> the error object also when qemu_open fails. The error object was set
>> generally set in case of errors, but I guess this case was overlooked.
>> Do the same for win32.
>>
>> Signed-off-by: Halil Pasic <address@hidden>
>> Reviewed-by: Sascha Silbe <address@hidden>
>> Tested-by: Marc Hartmayer <address@hidden> (POSIX only)
>>
>> ---
>>
>> Stumbled upon this (POSIX) while testing VMs with too many SCSI disks in
>> respect to my nofile limit. When open hits the nofile limit while trying
>> to hotplug yet another SCSI disk via libvirt we end up with no adequate
>> error message (one stating too many files). Sadly this patch in not
>> sufficient to fix this problem because drive_new (/qemu/blockdev.c)
>> handles errors using error_report_err which is documented as not to be
>> used in QMP context. Do not have a patch for that, because I'm unsure
>> whats the best way to deal with it. My guess right now is to make sure
>> we propagate errors at least until reaching code which is called  only
>> QMP in context and handle communicating the error to the requester of
>> the operation there. Any suggestions or ideas?
>>
>> The win32 part was not tested, and the sole reason I touched it is
>> to not introduce unnecessary divergence.
>>
>> v3 -> v4:
>> * rebased on current master
>> v2 -> v3:
>> * first save errno then error_setg_errno
>> v1 -> v2:
>> * fixed win32 by the correct error_setg_*
>> * use the original errno consequently
>> ---
>>  block/raw-posix.c | 1 +
>>  block/raw-win32.c | 1 +
>>  2 files changed, 2 insertions(+)
> 
> Thank you, I've applied this patch to my block branch:
> 
> https://github.com/XanClic/qemu/commits/block

I'm afraid I might have been too hasty. This changes the output of a
number of qemu-iotests test cases (at least 51, 69, 82, 85, 111), so
their reference output should be amended in this patch, too.

Therefore, I've dropped this patch from my queue again.

Max

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]