Re: [Linux-kernel-mentees][PATCH v2] packet: Fix undefined behavior in bit shift

From: Jiunn Chang
Date: Thu Jun 27 2019 - 12:57:31 EST


On Thu, Jun 27, 2019 at 09:22:53AM -0700, David Miller wrote:
> From: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>
> Date: Wed, 26 Jun 2019 21:32:52 -0600
>
> > On 6/26/19 9:25 PM, Jiunn Chang wrote:
> >> Shifting signed 32-bit value by 31 bits is undefined. Changing most
> >> significant bit to unsigned.
> >> Changes included in v2:
> >> - use subsystem specific subject lines
> >> - CC required mailing lists
> >>
> >
> > These version change lines don't belong in the change log.
>
> For networking changes I actually like the change lines to be in the
> commit log. So please don't stray people this way, thanks.

Hello David,

Would you like me to send v3 with the change log in the patch description?

I would be happy to do that.

THX,

Jiunn