Thread (22 messages) flat view 22 messages, 3 authors, 2012-09-19

Re: [RFC 1/2] virtio_console: Add support for DMA memory allocation

From: "Michael S. Tsirkin" <mst@redhat.com>
Date: 2012-09-04 18:54:26
Also in: lkml

Possibly related (same subject, not in this thread)

On Tue, Sep 04, 2012 at 06:58:47PM +0200, Sjur Brændeland wrote:
Hi Michael,
quoted
Exactly. Though if we just fail load it will be much less code.

Generally, using a feature bit for this is a bit of a problem though:
normally driver is expected to be able to simply ignore
a feature bit. In this case driver is required to
do something so a feature bit is not a good fit.
I am not sure what the right thing to do is.
I see - so in order to avoid the binding between driver and device
there are two options I guess. Either make virtio_dev_match() or
virtcons_probe() fail. Neither of them seems like the obvious choice.

Maybe adding a check for VIRTIO_CONSOLE_F_DMA_MEM match
between device and driver in virtcons_probe() is the lesser evil?

Regards,
Sjur
A simplest thing to do is change dev id. rusty?

-- 
MST
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help