Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Mon, 5 Aug 2002 18:09:22 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Mon, 5 Aug 2002 18:09:22 -0400 Received: from vindaloo.ras.ucalgary.ca ([136.159.55.21]:28317 "EHLO vindaloo.ras.ucalgary.ca") by vger.kernel.org with ESMTP id ; Mon, 5 Aug 2002 18:09:21 -0400 Date: Mon, 5 Aug 2002 16:12:53 -0600 Message-Id: <200208052212.g75MCrN17655@vindaloo.ras.ucalgary.ca> From: Richard Gooch To: Roman Zippel Cc: linux-kernel@vger.kernel.org Subject: Re: [BK PATCH] devfs cleanups for 2.5.29 In-Reply-To: References: <200207310032.g6V0WmW12258@vindaloo.ras.ucalgary.ca> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 965 Lines: 38 Roman Zippel writes: > Hi, > > On Tue, 30 Jul 2002, Richard Gooch wrote: > > > > Are you sure it's save in devfs_open() too? > > > > Yes. RTFS. > > I'm trying - without getting headache. Take a valium. > In the "devfs=only" case, where is the module count incremented, when a > block device is opened? The module count is incremented when the device is opened, irrespective of whether it's a character or block device, or even a "regular" file. > > > Even if it's save/fixed, it's still code duplication. > > > > No. I leverage fops_get(), a common function. > > Which is also insufficiently protected. Incorrect. Regards, Richard.... Permanent: rgooch@atnf.csiro.au Current: rgooch@ras.ucalgary.ca - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/