I think it should be ok. If we do hit any performance issue, we'll revisit this.

Thanks,
Palash

-----Original Message-----
From: Arnd Bergmann [mailto:a...@arndb.de] 
Sent: Wednesday, January 26, 2011 5:45 AM
To: Mauro Carvalho Chehab
Cc: Greg KH; linux-ker...@vger.kernel.org; Andrew Hendry; Andrew Morton; Arjan 
van de Ven; Arnaldo Carvalho de Melo; autofs@linux.kernel.org; Chris Wilson; 
Dave Airlie; dri-de...@lists.freedesktop.org; Evgeniy Dushistov; Frederic 
Weisbecker; H. Peter Anvin; Ian Kent; Ingo Molnar; Jeff Layton; 
linux-c...@vger.kernel.org; linux-fsde...@vger.kernel.org; 
linux-...@vger.kernel.org; Mikulas Patocka; net...@vger.kernel.org; Nick 
Bowler; Palash Bandyopadhyay; Ross Cohen; Russell King; Stuart Swales; Takahiro 
Hirofuchi; Thomas Gleixner
Subject: Re: [RFC 00/20] Proposal for remaining BKL users

On Wednesday 26 January 2011, Mauro Carvalho Chehab wrote:
> I guess you're meaning cx25821, right? 

Right, sorry for the typo.
 
> Palash should take a look on it and review. This is a device that allows
> 12 simultaneous streams, so, I suspect that he'll need to do some
> changes at the locking schema, to avoid performance bottlenecks.

I would be surprised if there was any measurable performance change.
The BKL was used only in the open() function to iterate the device
list, and that code is nowhere on a critical path, as far as I can tell.

        Arnd

Conexant E-mail Firewall (Conexant.Com) made the following annotations
---------------------------------------------------------------------
********************** Legal Disclaimer **************************** 

"This email may contain confidential and privileged material for the sole use 
of the intended recipient. Any unauthorized review, use or distribution by 
others is strictly prohibited. If you have received the message in error, 
please advise the sender by reply email and delete the message. Thank you." 

********************************************************************** 

---------------------------------------------------------------------


_______________________________________________
autofs mailing list
autofs@linux.kernel.org
http://linux.kernel.org/mailman/listinfo/autofs

Reply via email to