Re: [PATCH v3 02/12] xen: fix len type for guest copy functions

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

On 4/20/26 5:44 PM, Jan Beulich wrote:
> On 10.04.2026 17:54, Oleksii Kurochko wrote:
> > Widen the len argument of copy_to_guest_phys_flush_dcache() and the
> > copy_to_guest_phys_cb typedef from unsigned int to unsigned long, as
> > the function can be used to copy large blobs such as the initrd which
> > may exceed 4GB. Update the internal copy_guest() len argument to
> > unsigned long accordingly.
> >
> > Change the type for local variable size in copy_guest() to avoid
> > compilation error because of type mismatch.
> >
> > raw_* wrappers above copy_guest() keep returning unsigned long to
> > avoid type narrowing; it is not an issue for raw_*'s len argument
> > to remain 'unsigned int' since the assignment to copy_guest()'s wider
> > unsigned long parameter is safe and there is no raw_* users who
> > are using a value bigger than what can fit into 'unsigned int'.
> >
> > Reported-by: Jan Beulich <jbeulich@xxxxxxxx>
> > Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
> Actually: You say "fix" in the subject. How about sorting out a correct
> Fixes: tag then?
I will add then before Reported-by:

Fixes: 2986481b3d9e6 ("xen/arm: guest_copy: Extend the prototype to pass 
the vCPU")
Fixes: 5302bd490bea7 ("xen/arm: Introduce copy_to_guest_phys_flush_dcache")
Fixes: d07b7369aa65b ("xen/common: dom0less: introduce common 
domain-build.c")