RE: [PATCH 1/2] Fix undefined behaviour

[Paul Durrant]

> -----Original Message-----
> From: Artur Puzio <address@hidden>
> Sent: 28 April 2020 10:41
> To: address@hidden; 'Grzegorz Uriasz' <address@hidden>; address@hidden
> Cc: address@hidden; address@hidden; address@hidden; 'Stefano
> Stabellini' <address@hidden>; 'Anthony Perard' <address@hidden>; xen-
> address@hidden
> Subject: Re: [PATCH 1/2] Fix undefined behaviour
> 
> On 28.04.2020 10:10, Paul Durrant wrote:
> >> -----Original Message-----
> >> From: Grzegorz Uriasz <address@hidden>
> >> Sent: 28 April 2020 07:29
> >> To: address@hidden
> >> Cc: Grzegorz Uriasz <address@hidden>; address@hidden; address@hidden;
> >> address@hidden; address@hidden; Stefano Stabellini <address@hidden>;
> Anthony
> >> Perard <address@hidden>; Paul Durrant <address@hidden>; address@hidden
> >> Subject: [PATCH 1/2] Fix undefined behaviour
> >>
> >> Signed-off-by: Grzegorz Uriasz <address@hidden>
> > I think we need more of a commit comment for both this and patch #2 to 
> > explain why you are making
> the changes.
> >
> >   Paul
> 
> I agree Grzegorz should improve the commit messages. In the mean time
> see email with subject "[PATCH 0/2] Fix QEMU crashes when passing IGD to
> a guest VM under XEN", it contains quite detailed explanation for both
> "Fix undefined behaviour" and "Improve legacy vbios handling" patches.
> 

Ok. Can you please make sure maintainers are cc-ed on patch #0 too.

  Paul