Re: [Xen-devel] [PATCH 01/14] x86/ticketlock: clean up types and accessors

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

>  static inline int __ticket_spin_is_locked(arch_spinlock_t *lock)
>  {
> -     int tmp = ACCESS_ONCE(lock->slock);
> +     struct __raw_tickets tmp = ACCESS_ONCE(lock->tickets);
>  
> -     return !!(((tmp >> TICKET_SHIFT) ^ tmp) & ((1 << TICKET_SHIFT) - 1));
> +     return !!(tmp.tail ^ tmp.head);

Does it make sense to mask it here it here with TICKET_MASK as it was done 
before?

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel