Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Dec 2008 11:55:58 -0800
From:      Andrew Thompson <thompsa@FreeBSD.org>
To:        Hans Petter Selasky <hselasky@c2i.net>
Cc:        Ed Schouten <ed@80386.nl>, freebsd-usb@freebsd.org
Subject:   Re: Giant on serial devices in USB2?
Message-ID:  <20081223195558.GL89448@citylink.fud.org.nz>
In-Reply-To: <200812231956.21666.hselasky@c2i.net>
References:  <20081223175911.GZ1176@hoeg.nl> <200812231956.21666.hselasky@c2i.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 23, 2008 at 07:56:21PM +0100, Hans Petter Selasky wrote:
> On Tuesday 23 December 2008, Ed Schouten wrote:
> > Hello HPS, others,
> >
> > I was just running a grep on the USB2 code and saw the following:
> >
> > $ grep -r Giant sys/dev/usb2/serial | wc -l
> >       70
> >
> > WHat's the exact reason USB2 still uses Giant on TTY related drivers?
> > Couldn't it just use the per-TTY mutex? If not, maybe it should create a
> > custom system-wide mutex, instead of using Giant?
> >
> > Yours,
> 
> Hi,
> 
> I don't have all those USB devices at hand, so I simply cannot test if the 
> driver still works without Giant.
> 
> BTW: It should be trivial to remove Giant from the drivers. I think Andrew 
> Thompson has already done some drivers Giant-free.

Its just a mechanical change, it works fine for the couple of driver I
have converted.

Andrew



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20081223195558.GL89448>