[PATCH] virfile: Report error when changing pipe size fails

Claudio Fontana cfontana at suse.de
Mon Mar 28 13:24:36 UTC 2022


On 3/28/22 2:24 PM, Claudio Fontana wrote:
> On 3/28/22 2:22 PM, Daniel P. Berrangé wrote:
>> On Mon, Mar 28, 2022 at 01:46:08PM +0200, Michal Privoznik wrote:
>>> When changing the size of pipe that virFileWrapperFdNew() creates
>>> we start at 1MiB and if that fails because it's above the system
>>> wide limit we get EPERM and continue with half of the size.
>>>
>>> However, we might get another error in which case we should
>>> report proper system error and return failure from
>>> virFileWrapperFdNew().
>>>
>>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>>> ---
>>>  src/util/virfile.c | 18 +++++++++++-------
>>>  1 file changed, 11 insertions(+), 7 deletions(-)
>>
>> Reviewed-by: Daniel P. Berrangé <berrange at redhat.com>
>>
>>
>> With regards,
>> Daniel
>>
> 
> testing it right now, thanks!
> 

Tested-by: Claudio Fontana <cfontana at suse.de>



More information about the libvir-list mailing list