Re: [PATCH 4.14 00/31] 4.14.1-stable review

From: Ben Hutchings
Date: Tue Nov 21 2017 - 14:07:29 EST


On Tue, 2017-11-21 at 18:09 +0100, Greg Kroah-Hartman wrote:
> On Tue, Nov 21, 2017 at 04:46:18PM +0000, Ben Hutchings wrote:
> > On Tue, 2017-11-21 at 17:35 +0100, Greg Kroah-Hartman wrote:
> > > On Tue, Nov 21, 2017 at 03:26:10PM +0000, Ben Hutchings wrote:
> >
> > [...]
> > > > Not all 32-bit configurations can provide cmpxchg64().ÂÂi40e's use of
> > > > cmpxchg64() appears to be fixed by:
> > > >
> > > > b74f571f59a8 i40e/i40evf: organize and re-number feature flags
> > > > b48be9978e4b i40e: fix flags declaration
> > >
> > > So without those patches, are any specific arches/configs broken for
> > > 4.14?
> >
> > 32-bit parisc is.
>
> Ok, but that's a horrid hack on the i40e driver, it just happens to move
> the bitfield to a 32bit variable.ÂÂCan't we just provide a "real"
> cmpxchg64() for 32-bit parisc?

No. There is a generic implementation of cmpxchg64() but it is only
suitable for non-SMP configurations.

> Or mark this driver as not buildable on that platform?

I suggested adding a 'depends 64BIT' but got no response.

Ben.

> I doubt anyone is doing 40gb networking on parisc :)
>
> thanks,
>
> greg k-h
>
--
Ben Hutchings
Software Developer, Codethink Ltd.