RE: [PATCH 1/2] Fix undefined behaviour

[Paul Durrant]

> -----Original Message-----
> From: Paul Durrant <address@hidden>
> Sent: 28 April 2020 13:33
> To: 'Artur Puzio' <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
> 
> > -----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.
> 

Actually they are, sorry. My MUA is playing tricks on me.

  Paul