[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v8 21/21] nbd: assert that Error** is not NULL in nbd_iter_ch
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [PATCH v8 21/21] nbd: assert that Error** is not NULL in nbd_iter_channel_error |
Date: |
Fri, 6 Dec 2019 18:19:36 +0000 |
06.12.2019 18:58, Markus Armbruster wrote:
> Eric Blake <address@hidden> writes:
>
>> On 12/5/19 11:46 AM, Vladimir Sementsov-Ogievskiy wrote:
>>> The local_err parameter is not here to return information about
>>> nbd_iter_channel_error failure. Instead it's assumed to be filled when
>>> passed to the function. This is already stressed by its name
>>> (local_err, instead of classic errp). Stress it additionally by
>>> assertion.
>>>
>>> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
>>> ---
>>> block/nbd.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>
>> Our timing resulted in crossed mails - I was replying to v7 when this
>> landed, and my reply there is still relevant (namely, a better commit
>> message is needed, but the code gets my R-b with that change).
>
> If v8 turns out to be fine except for commit message tweaks, I'll gladly
> to these in my tree. Just tell me what to do in a reply to this
> message.
>
Yes, this would be great, thanks!
The only thing is your suggestion on patch 21, but it may be applied in
separate (and it's actually a separate thing)
--
Best regards,
Vladimir
[PATCH v8 10/21] block/snapshot: rename Error ** parameter to more common errp, Vladimir Sementsov-Ogievskiy, 2019/12/05