[lkml]   [2021]   [Feb]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] locking/arch: Move qrwlock.h include after qspinlock.h
On 2/10/21 1:28 PM, Paolo Bonzini wrote:
> On 10/02/21 15:45, Waiman Long wrote:
>> The queued rwlock code has a dependency on the current spinlock
>> implementation (likely to be qspinlock), but not vice versa. Including
>> qrwlock.h before qspinlock.h can be problematic when expanding qrwlock
>> functionality.
>> If both qspinlock.h and qrwlock.h are to be included, the qrwlock.h
>> include should always be after qspinlock.h. Update the current set of
>> asm/spinlock.h files to enforce that.
>> Signed-off-by: Waiman Long <>
> arch/sparc/include/asm/spinlock_64.h is missing.  Also, the include in
> kernel/locking/qrwlock.c is not necessary (it may be there for
> aesthetic reasons, but it complicates thing in this case).

Sorry for missing arch/sparc/include/asm/spinlock_64.h. I was just
focusing on asm/spinlock.h and not aware that there are other variants

It is true that the asm/qrwlock.h include in qrwlock.c is not really
necessary. I can't recall why it was there.

> I'll send a v2 that is based on the kvm/next tree.
> Paolo
Thanks for taking care of that.


 \ /
  Last update: 2021-02-10 19:57    [W:0.982 / U:1.876 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site