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, wrote: >> On 02/03/15 09:58, Jan Beulich wrote: >> On 02.02.15 at 16:22, 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

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, wrote: > On 02/03/15 09:58, Jan Beulich wrote: > On 02.02.15 at 16:22, 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 there is a new commit mess

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, 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_

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, 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 separate function

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: [PATC

[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 --- v2: Paul Durrant: