Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757372AbcCCTcH (ORCPT ); Thu, 3 Mar 2016 14:32:07 -0500 Received: from mail-lb0-f179.google.com ([209.85.217.179]:32982 "EHLO mail-lb0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755537AbcCCTcD (ORCPT ); Thu, 3 Mar 2016 14:32:03 -0500 MIME-Version: 1.0 In-Reply-To: <20160303171846.GC20826@kroah.com> References: <20160303012530.GA23521@kroah.com> <20160303171846.GC20826@kroah.com> Date: Thu, 3 Mar 2016 11:32:00 -0800 Message-ID: Subject: Re: allocate an official device major number for virtio device? From: Jin Qian To: Greg Kroah-Hartman Cc: "Ning, Yu" , "linux-kernel@vger.kernel.org" , Jeff Sharkey , David Turner , "pprabhu@google.com" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2113 Lines: 63 On Thu, Mar 3, 2016 at 9:18 AM, Greg Kroah-Hartman wrote: > A: No. > Q: Should I include quotations after my reply? > > http://daringfireball.net/2007/07/on_top > > On Thu, Mar 03, 2016 at 03:52:20AM +0000, Ning, Yu wrote: >> Well, virtio_blk does use dynamic major number allocation, but the >> allocated block major just happens to fall in the "experimental" range >> (240-254)... > > That all depends on what else is registered in the system at the moment. > >> In more detail: >> >> virtio_blk calls register_blkdev() with major = 0 in init() (drivers/block/virtio_blk.c:872): >> >> major = register_blkdev(0, "virtblk"); >> >> This line has been there since day one. And register_blkdev() implements dynamic major allocation pretty straightforwardly: >> >> /* temporary */ >> if (major == 0) { >> for (index = ARRAY_SIZE(major_names)-1; index > 0; index--) { >> if (major_names[index] == NULL) >> break; >> } >> >> So it goes from index = 254 to 1 and picks the first unused. >> Apparently, there's a good chance that the allocated major is between >> 240-254 (although lower numbers are also possible, theoretically). >> Indeed, we always get 253 for virtio_blk with the x86_64 Android >> emulator kernel. >> >> But "dynamic" means we can't rely on checking major == 253 to detect >> virtio_blk. > > Nor should you, why would you care? > >> That's why we are doing a fnmatch() using pattern >> /sys/devices/*/block/vd* instead. Is that the recommended approach? > > Yes, or just look at the device node that is already created in /dev/ > for you automatially by devtmpfs. Doesn't that work as expected today? > > I still don't understand the issue you are having here at all, sorry. Maybe Jeff can comment in more detail about the issue and which method is preferred? Just to summarize, we have a few ways to detect the device 1. match /sys/devices/... 2. match /dev/... 3. use uevent rule to detect virtio device creation 4. check a fixed major# Thanks, jin > > greg k-h