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

2022-12-06 Thread Per Bilse (3P)
On 28/11/2022 08:21, Jan Beulich wrote: > In addition I think ignoring failure (and, as said by Julien, only because > of no bufioreq being registered) is (kind of implicitly) valid only for > buffered requests. Hence I'm unconvinced of the need of a new boolean > function parameter. Instead I

Re: [PATCH] drivers/xen/hypervisor: Expose VM SIF flags to userspace

2022-11-29 Thread Per Bilse (3P)
On 29/11/2022 16:41, Andrew Cooper wrote: > As for the actual flags exposed, it would be very beneficial not to copy > the exist proc interface.  It would be better to expose a subdir that > had files containing booleans, because that also gives userspace an easy > way to figure out if the

Re: [PATCH] drivers/xen/hypervisor: Expose VM SIF flags to userspace

2022-11-29 Thread Per Bilse (3P)
On 29/11/2022 16:29, Boris Ostrovsky wrote: > Why not simply show unprocessed xen_start_flags as a hex value? Providing a text representation follows what is currently the case: [root@lcy2-dt128 /proc/xen]# cat capabilities control_d [root@lcy2-dt128 /proc/xen]# The migrated form would yield:

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

2022-11-28 Thread Per Bilse (3P)
On 28/11/2022 08:21, Jan Beulich wrote: > On 26.11.2022 23:19, Julien Grall wrote: >> >> The commit message is quite vague, so it is hard to know what you are >> trying to solve exactly. AFAIU, there are two reasons for >> ioreq_broadcast to fails: >>1) The IOREQ server didn't register the