From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752325AbZJKVvL (ORCPT ); Sun, 11 Oct 2009 17:51:11 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752055AbZJKVvI (ORCPT ); Sun, 11 Oct 2009 17:51:08 -0400 Received: from mail-fx0-f227.google.com ([209.85.220.227]:53477 "EHLO mail-fx0-f227.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752059AbZJKVvH (ORCPT ); Sun, 11 Oct 2009 17:51:07 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to :mime-version:content-type:content-transfer-encoding:message-id; b=BrvMAXhxUyAIVs62lsha97Ni9VMlk595bjE23BwO8X++beYnLqsNeAhTwx0W0T0GJN 5pFdk1Z+mYTbPtBibtWQw31IvR6hxVFEpHnRE9N25jC9jp7zhvqT/Ewdj3Z5sAQoS+BF TB/qSly81aeuWp6fYadzxviM83Sxzdz6WdiwI= From: Arnd Bergmann To: Frederic Weisbecker Subject: Re: [patch 11/28] nvram: Drop the bkl from nvram_llseek() Date: Sun, 11 Oct 2009 23:50:24 +0200 User-Agent: KMail/1.12.1 (Linux/2.6.31-11-generic; KDE/4.3.1; x86_64; ; ) Cc: Arnd Bergmann , Thomas Gleixner , LKML , Andrew Morton , Ingo Molnar , Peter Zijlstra , Vincent Sanders , John Kacur , Jonathan Corbet , Christoph Hellwig , "Sven-Thorsten Dietrich" , Alessio Igor Bogani , Benjamin Herrenschmidt , Greg KH References: <20091010153314.827301943@linutronix.de> <20091011210809.GC5486@nowhere> <20091011214053.GD5486@nowhere> In-Reply-To: <20091011214053.GD5486@nowhere> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200910112350.24870.arnd@arndb.de> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sunday 11 October 2009, Frederic Weisbecker wrote: > Now I'm adding the ioctl() sites too: > > git-grep "\.ioctl *=" | grep -P "^\S+\.c" | wc -l > 452 > > Hehe :) Not all of them, fortunately. There are various *_operations structures that have a .ioctl pointer. While there are a lot of struct file_operations with a locked .ioctl operation, stuff like block_device_operations does not hold the BKL in .ioctl but in .locked_ioctl. Arnd <><