On (Tue) Jan 05 2010 [10:27:19], Gerd Hoffmann wrote:
>   Hi,
>
>> +/* Guest wants to notify us of some event */
>> +static void handle_control_message(VirtIOSerial *vser, void *buf)
>> +{
>
>> +    cpkt->event = le16_to_cpu(cpkt->event);
>> +    cpkt->value = le16_to_cpu(cpkt->value);
>
> Does this modify guest memory?

Hm, I don't want to. I'll use a shadow struct.

>
>> +        DEFINE_PROP_UINT8("is_console", VirtConsole, port.is_console, 1),
>
> virtser_bus_dev_print() prints this.  Properties are printed by "info  
> qtree" anyway, so you can drop that from virtser_bus_dev_print ...

I had to make is_console a property for other reasons; I'll remove it
from here.

                Amit


Reply via email to