Re: [PATCH] locking/arch: Move qrwlock.h include after qspinlock.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <longman@xxxxxxxxxx>

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 there.

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.

Cheers,
Longman




[Index of Archives]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux