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

2016-05-16 Thread Paul Durrant
> -Original Message- > From: Suravee Suthikulpanit [mailto:suravee.suthikulpa...@amd.com] > Sent: 16 May 2016 17:03 > To: Paul Durrant; xen-devel@lists.xen.org; George Dunlap; > jbeul...@suse.com > Subject: Re: [PATCH V2 1/2] x86/hvm: Add check when register io handler > > Hi Paul, > > On

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

2016-05-16 Thread Suravee Suthikulpanit
Hi Paul, On 05/16/2016 03:01 AM, Paul Durrant wrote: -Original Message- >From:suravee.suthikulpa...@amd.com >[mailto:suravee.suthikulpa...@amd.com] >Sent: 13 May 2016 20:37 >To:xen-devel@lists.xen.org; George Dunlap;jbeul...@suse.com >Cc: Paul Durrant; Suravee Suthikulpanit; Suravee Suth

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

2016-05-16 Thread Paul Durrant
> -Original Message- > From: suravee.suthikulpa...@amd.com > [mailto:suravee.suthikulpa...@amd.com] > Sent: 13 May 2016 20:37 > To: xen-devel@lists.xen.org; George Dunlap; jbeul...@suse.com > Cc: Paul Durrant; Suravee Suthikulpanit; Suravee Suthikulpanit > Subject: [PATCH V2 1/2] x86/hvm: A

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

2016-05-13 Thread suravee.suthikulpanit
From: Suravee Suthikulpanit At the time of registering HVM I/O handler, the HVM domain might not have been initialized, 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