qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v3 2/4] char: Add ability to provide a callback


From: Paul Brook
Subject: Re: [Qemu-devel] [PATCH v3 2/4] char: Add ability to provide a callback when write won't return -EAGAIN
Date: Tue, 20 Apr 2010 13:28:48 +0100
User-agent: KMail/1.12.4 (Linux/2.6.33-2-amd64; KDE/4.3.4; x86_64; ; )

> I sent out this series as a "feeler" to see if the approach was
> acceptable.
> 
> Paul didn't reply to my reply addressing his concern, so I take that as
> he's OK with the approach as well :-)

I'd probably exposed this as an asyncronous write rather than nonblocking 
operation. However both have their issues and I guess for character devices 
your approach makes sense (c.f. block devices where we want concurrent 
transfers).

It would be useful to have a debugging mode where the chardev layer 
deliberately returns spurious EAGAIN and short writes. Otherwise you've got a 
lot of very poorly tested device fallback code. I have low confidence in 
getting this right first time :-)

Paul




reply via email to

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