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

Re: [Xen-devel] [PATCH V2 1/2] x86/hvm: Add check when register io handler

Hi Paul,

On 05/16/2016 03:01 AM, Paul Durrant wrote:
-----Original Message-----
>Sent: 13 May 2016 20:37
>To:xen-devel@xxxxxxxxxxxxx; George Dunlap;jbeulich@xxxxxxxx
>Cc: Paul Durrant; Suravee Suthikulpanit; Suravee Suthikulpanit
>Subject: [PATCH V2 1/2] x86/hvm: Add check when register io handler
>From: Suravee Suthikulpanit<Suravee.Suthikulpanit@xxxxxxx>
>At the time of registering HVM I/O handler, the HVM domain might
>not have been initialized,
I/O handler registration is internal to Xen so any caller that attempt to 
register before domain initialization should be removed and replaced with one 
that does it at the right time.

Ok. I'll just remove that call for now.

>which means the hvm_domain.io_handler
>would be NULL. In the hvm_next_io_handler(), this should be checked
>before returning and referencing the array. Also, the io_handler_count
>should only be incremented on success.
>So, this patch adds error handling in hvm_next_io_handler.
This isn't necessary. An ASSERT would be preferable so that buggy callers can 
be easily caught.

Ok, I'll update the patch to ASSERT() and send it out. Although, just want to make sure that you think it should really be doing assert and not warning + handling error? It seems quite aggressive to crash the hypervisor simply because some io handler are not properly call.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.