Re: [PATCH V9 16/24] LoongArch: Add misc common routines

From: Palmer Dabbelt
Date: Sat Apr 30 2022 - 09:22:45 EST


On Sat, 30 Apr 2022 03:41:59 PDT (-0700), Arnd Bergmann wrote:
On Sat, Apr 30, 2022 at 12:00 PM Huacai Chen <chenhuacai@xxxxxxxxx> wrote:

On Sat, Apr 30, 2022 at 5:50 PM Arnd Bergmann <arnd@xxxxxxxx> wrote:
>
> On Sat, Apr 30, 2022 at 11:05 AM Huacai Chen <chenhuacai@xxxxxxxxxxx> wrote:
>
> > +unsigned long __xchg_small(volatile void *ptr, unsigned long val, unsigned int size)
> > +{
> > + u32 old32, mask, temp;
> > + volatile u32 *ptr32;
> > + unsigned int shift;
> > +
> > + /* Check that ptr is naturally aligned */
>
> As discussed, please remove this function and all the references to it.

It seems that "generic ticket spinlock" hasn't been merged in 5.18?

No, but we can merge it together with the loongarch architecture for 5.19.

I suggested you coordinate with Guo Ren and Palmer Dabbelt about how
to best merge it. The latest version was pasted two weeks ago [1], and
it sounds like there are only minor issues to work out and that I can merge
v4 into the asm-generic tree before merging the loongarch code in the
same place.

Arnd

[1] https://lore.kernel.org/lkml/20220414220214.24556-1-palmer@xxxxxxxxxxxx/

I can just send another version, IIRC it was just that discussion about the memory barrier and there's already prototype code so it shouldn't be too bad. I was hoping to do it sooner, sorry.