On 30.11.2020 11:31, Oleksandr Tyshchenko wrote: > --- a/xen/include/xen/ioreq.h > +++ b/xen/include/xen/ioreq.h > @@ -21,6 +21,13 @@ > > #include <xen/sched.h> > > +static inline bool ioreq_needs_completion(const ioreq_t *ioreq) > +{ > + return ioreq->state == STATE_IOREQ_READY && > + !ioreq->data_is_ptr && > + (ioreq->type != IOREQ_TYPE_PIO || ioreq->dir != IOREQ_WRITE); > +} > + > #define HANDLE_BUFIOREQ(s) \ > ((s)->bufioreq_handling != HVM_IOREQSRV_BUFIOREQ_OFF)
Personally I would have suggested to keep the #define first, but I see you've already got Paul's R-b. Applicable parts Acked-by: Jan Beulich <jbeul...@suse.com> Jan