Re: RFC: small syscons and kbd patch

On Tue, Dec 2, 2008 at 5:01 PM, Maksim Yevmenkin
<maksim.yevmenkin_at_gmail.com> wrote:
> Dear Hackers,
>
> can someone please review the attached small patch for syscons and
> kbd? it should be a no-op mostly. the patch basically does
>
> 1) removes bogus layering in syscons, i.e. basically removes sccngetch();
> 2) implements advisory lock for kbd (based on atomic(9));
> 3) implements new POLLED flag for kbd;
>
> this is a part of a plan to fix keyboard access races in syscons.
>
> thanks,
> max

Max,
     Why are you double and triple negating on this line?

+	return (atomic_cmpset_acq_int(&kbd->kb_locked, !!!on, !!on));

-Garrett