> -----Original Message-----
> From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf
> Of Jan Beulich
> Sent: 09 April 2018 14:44
> To: Paul Durrant <paul.durr...@citrix.com>
> Cc: Juergen Gross <jgr...@suse.com>; Andrew Cooper
> <andrew.coop...@citrix.com>; Kevin Tian <kevin.t...@intel.com>; Jun
> Nakajima <jun.nakaj...@intel.com>; xen-devel <xen-
> de...@lists.xenproject.org>
> Subject: Re: [Xen-devel] [PATCH 2/2] x86/HVM: alter completion-needed
> checking
> 
> >>> On 09.04.18 at 15:33, <paul.durr...@citrix.com> wrote:
> >>  -----Original Message-----
> >> From: Jan Beulich [mailto:jbeul...@suse.com]
> >> Sent: 09 April 2018 14:24
> >> To: xen-devel <xen-devel@lists.xenproject.org>
> >> Cc: Andrew Cooper <andrew.coop...@citrix.com>; Paul Durrant
> >> <paul.durr...@citrix.com>; Jun Nakajima <jun.nakaj...@intel.com>;
> Kevin
> >> Tian <kevin.t...@intel.com>; Juergen Gross <jgr...@suse.com>
> >> Subject: [PATCH 2/2] x86/HVM: alter completion-needed checking
> >>
> >> The function only looks at the ioreq_t, so pass it a pointer to just
> >> that. Also use it in hvmemul_do_io().
> >>
> >> Suggested-by: Paul Durrant <paul.durr...@citrix.com>
> >> Signed-off-by: Jan Beulich <jbeul...@suse.com>
> >> ---
> >> RFC: While this avoids some open coding, generated code looks to be
> >>      worse for that particular case. I'm therefore not certain that we
> >>      want this change (or perhaps just the function name/signature
> >>      change portion).
> >>
> >
> > FAOD my reason for suggesting it was such that exactly the same test
> > implementation is used in all cases to decide whether I/O completion is
> > needed.
> 
> So does that mean you think the change is worthwhile? If so, do
> you have any comments, or are you willing to ack it (despite it
> not really being a 4.11 candidate at this point in time)?
> 

Yes, I think the change is worthwhile as it makes the code clearer (to me at 
least) so you can consider it

Reviewed-by: Paul Durrant <paul.durr...@citrix.com>

> Jan
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xenproject.org
> https://lists.xenproject.org/mailman/listinfo/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to