On Sun, 2020-07-12 at 08:12 -0700, Tom Rix wrote:
On 7/12/20 6:09 AM, H. Peter Anvin wrote:
On 2020-07-12 05:59, trix@redhat.com wrote:
From: Tom Rix trix@redhat.com
[]
So change the type in sizeof to 'unsigned int'
You must be kidding.
If you want to change it, change it to sizeof(bd->dbuf) instead, but this flag is at least in my opinion a total joke. For sizeof(int) != sizeof(unsigned int) is beyond bizarre, no matter how stupid the platform.
Using the actual type is more correct that using a type of the same size.
Sure.
But this hardly matters as this same type conversion from signed to unsigned or the other way round is _everywhere_ in the kernel.
And especially the cc of stable is unnecessary.