linux-kernel - Re: [PATCH 08/18] lirc driver for the Soundgraph IMON IR Receivers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 10 Sep 2008 23:22:22 -0400
From:	Jarod Wilson <jwilson@...hat.com>
To:	Janne Grunau <j@...nau.net>
Cc:	Jonathan Corbet <corbet@....net>, linux-kernel@...r.kernel.org,
	Christoph Bartelmus <lirc@...telmus.de>
Subject: Re: [PATCH 08/18] lirc driver for the Soundgraph IMON IR Receivers

On Wednesday 10 September 2008 17:23:51 Janne Grunau wrote:
> On Wednesday 10 September 2008 23:02:29 Jonathan Corbet wrote:
> > > +#define SUCCESS		0
> > > +#define	TRUE		1
> > > +#define FALSE		0
> >
> > (See my grumble in previous reviews...:)
> >
> > > +#define LOCK_CONTEXT	mutex_lock(&context->lock)
> > > +#define UNLOCK_CONTEXT	mutex_unlock(&context->lock)
> >
> > Here too.
> >
> > > +/* to prevent races between open() and disconnect() */
> > > +static DECLARE_MUTEX(disconnect_sem);
> >
> > This should be a real mutex, I think.
>
> All three already resolved. I think it would wise to stop the review of
> this patch set until we repost or review based on Jarod's git
> repository (http://git.wilsonet.com/linux-2.6-lirc.git/).

Yeah, I'd say for any driver that hasn't yet been reviewed, if anyone wants to 
continue reviewing, it might be best to poke the version in that git tree, as 
a number of issues common across multiple drivers have already (hopefully) 
been whacked. We'll definitely put together an updated patch set in the near 
future, but at the moment, we're still working through all the bits that have 
received feedback so far.

> Thanks for the reviews so far.

Indeed, very much appreciated!

-- 
Jarod Wilson
jarod@...hat.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/