Re: [PATCH v2] ioreq_broadcast(): accept partial broadcast success

2022-12-07 Thread Jan Beulich
(Cc to xen-devel@ re-added; I don't see why it was dropped) On 07.12.2022 11:25, Per Bilse (3P) wrote: > On 07/12/2022 09:05, Jan Beulich wrote: >> On 06.12.2022 18:52, Per Bilse wrote: >>> +if ( !s->enabled || (buffered && >>> +s->bufioreq_handling ==

Re: [PATCH v2] ioreq_broadcast(): accept partial broadcast success

2022-12-07 Thread Jan Beulich
On 06.12.2022 18:52, Per Bilse wrote: > --- a/xen/common/ioreq.c > +++ b/xen/common/ioreq.c > @@ -1317,7 +1317,8 @@ unsigned int ioreq_broadcast(ioreq_t *p, bool buffered) > > FOR_EACH_IOREQ_SERVER(d, id, s) > { > -if ( !s->enabled ) > +if ( !s->enabled || (buffered &&

Re: [PATCH v2] ioreq_broadcast(): accept partial broadcast success

2022-12-06 Thread Paul Durrant
On 06/12/2022 17:52, Per Bilse wrote: Avoid incorrectly triggering an error when a broadcast buffered ioreq is not handled by all registered clients, as long as the failure is strictly because the client doesn't handle buffered ioreqs. Signed-off-by: Per Bilse --- v2: Complete rethink with

[PATCH v2] ioreq_broadcast(): accept partial broadcast success

2022-12-06 Thread Per Bilse
Avoid incorrectly triggering an error when a broadcast buffered ioreq is not handled by all registered clients, as long as the failure is strictly because the client doesn't handle buffered ioreqs. Signed-off-by: Per Bilse --- v2: Complete rethink with better information. A lot of simplicity was