Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755332AbYK3Nof (ORCPT ); Sun, 30 Nov 2008 08:44:35 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751580AbYK3No0 (ORCPT ); Sun, 30 Nov 2008 08:44:26 -0500 Received: from ey-out-2122.google.com ([74.125.78.26]:8127 "EHLO ey-out-2122.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751471AbYK3No0 (ORCPT ); Sun, 30 Nov 2008 08:44:26 -0500 Message-ID: Date: Sun, 30 Nov 2008 14:44:24 +0100 From: "Kay Sievers" To: "Al Viro" Subject: Re: [PATCH] fix pktcdvd breakage from commit e105b8bfc769b0545b6f0f395179d1e43cbee822 Cc: "Linus Torvalds" , gregkh@suse.de, petero2@telia.com, linux-kernel@vger.kernel.org In-Reply-To: <20081130134048.GA28946@ZenIV.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20081130121909.GV28946@ZenIV.linux.org.uk> <20081130132118.GY28946@ZenIV.linux.org.uk> <20081130133229.GZ28946@ZenIV.linux.org.uk> <20081130134048.GA28946@ZenIV.linux.org.uk> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1618 Lines: 36 On Sun, Nov 30, 2008 at 14:40, Al Viro wrote: > On Sun, Nov 30, 2008 at 01:32:29PM +0000, Al Viro wrote: >> On Sun, Nov 30, 2008 at 02:25:43PM +0100, Kay Sievers wrote: >> >> > Pktcdvd creates char _and_ block device nodes at the same time, while >> > the char nodes are not allocated, but created and conflict in >> > /sys/dev/ with properly allocated ones from other subsystems. Your >> > patch just papers over this bug. >> >> Where the hell is it creating any char device nodes? Show me. >> And note that device_create() is not creating *any* device nodes, >> char or block. It creates a directory in some place in sysfs, >> associated with struct device (which has arseloads of uses unrelated >> to block *or* char devices) and, if dev_t argument is non-zero, >> dumps its value into one of the attributes. >> >> These days it might also dump a symlink into /sys/dev/*. >> >> Which part of the above constitutes a creation of char device node? > > Wait a minute... Are you saying that something in userland ends up > seeing that sucker, noticing .../dev and proceeding to do mknod? Sure, it does mkod, as the kernel says it should. But that does not cause any real problem. These device nodes are not used for anything. Opening the created char pktcdvd device node in userspace would in many cases just open a random usb device. :) Kay -- 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/