Re: [Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-06 Thread Don Slutz
On 02/06/15 02:35, Jan Beulich wrote: On 05.02.15 at 20:02, dsl...@verizon.com wrote: On 02/03/15 09:58, Jan Beulich wrote: On 02.02.15 at 16:22, dsl...@verizon.com wrote: Ok, will be working on a much better commit message. Do you want the new commit message copied here (in the summary of

Re: [Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-05 Thread Jan Beulich
On 05.02.15 at 20:02, dsl...@verizon.com wrote: On 02/03/15 09:58, Jan Beulich wrote: On 02.02.15 at 16:22, dsl...@verizon.com wrote: Ok, will be working on a much better commit message. Do you want the new commit message copied here (in the summary of the changes), or just that fact that

Re: [Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-05 Thread Don Slutz
On 02/03/15 09:58, Jan Beulich wrote: On 02.02.15 at 16:22, dsl...@verizon.com wrote: Jan Beulich: The change on what to do when hvm_send_assist_req() fails is bad. That is correct. Made hvm_has_dm() do full checking so that the extra VMEXIT and VMENTRY can be skipped.

Re: [Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-03 Thread Jan Beulich
On 02.02.15 at 16:22, dsl...@verizon.com wrote: Jan Beulich: The change on what to do when hvm_send_assist_req() fails is bad. That is correct. Made hvm_has_dm() do full checking so that the extra VMEXIT and VMENTRY can be skipped. hvm_complete_assist_req() be a

Re: [Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-03 Thread Paul Durrant
-Original Message- From: Don Slutz [mailto:dsl...@verizon.com] Sent: 02 February 2015 15:22 To: xen-devel@lists.xen.org; Jan Beulich; Paul Durrant Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Keir (Xen.org); Stefano Stabellini; Wei Liu; Don Slutz Subject: [PATCH v2

[Xen-devel] [PATCH v2 3/3] hvm_has_dm: Do a full check for backing DM

2015-02-02 Thread Don Slutz
This saves a VMENTRY and a VMEXIT since we not longer retry the ioport read on backing DM not handling a given ioreq. To only call on hvm_select_ioreq_server() once in this code path, return s. Also switch to hvm_send_assist_req_to_ioreq_server(). Signed-off-by: Don Slutz dsl...@verizon.com ---