Marc-André Lureau <[email protected]> writes:

> Hi
>
> On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster <[email protected]> wrote:
>> Yes, the chardev is commonly useless after we read a bad version from
>> it, but destroying it is inappropriate anyway: the user created it, so
>> the user should be able to hold on to it as long as he likes.  We
>> don't destroy it on other errors.  Screwed up in commit 5105b1d.
>>
>> Stop reading instead.
>>
>> Also note QEMU's behavior in ivshmem-spec.txt.
>>
>> Signed-off-by: Markus Armbruster <[email protected]>
>
> Interestingly enough, it seems both smartcard "passthru" and usb
> redirect do chr_delete(). It would a nice follow up to standardize
> this there too.

Wasn't aware of them.  I'll see what I can do.

> Reviewed-by: Marc-André Lureau <[email protected]>

Thanks!

Reply via email to