From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750830AbWAEAVe (ORCPT ); Wed, 4 Jan 2006 19:21:34 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750832AbWAEAVe (ORCPT ); Wed, 4 Jan 2006 19:21:34 -0500 Received: from gprs189-60.eurotel.cz ([160.218.189.60]:63896 "EHLO amd.ucw.cz") by vger.kernel.org with ESMTP id S1750830AbWAEAVd (ORCPT ); Wed, 4 Jan 2006 19:21:33 -0500 Date: Thu, 5 Jan 2006 01:18:37 +0100 From: Pavel Machek To: Greg KH Cc: "Rafael J. Wysocki" , Linux PM , LKML Subject: Re: [linux-pm] [RFC/RFT][PATCH -mm 2/5] swsusp: userland interface (rev. 2) Message-ID: <20060105001837.GA1751@elf.ucw.cz> References: <200601042340.42118.rjw@sisk.pl> <200601042351.58667.rjw@sisk.pl> <20060104234918.GA15983@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060104234918.GA15983@kroah.com> X-Warning: Reading this can be dangerous to your mental health. User-Agent: Mutt/1.5.9i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Hi! > > +static int __init snapshot_dev_init(void) > > +{ > > + int error; > > + > > + error = alloc_chrdev_region(&interface.devno, 0, 1, interface.name); > > + if (error) > > + return error; > > + cdev_init(&interface.cdev, &snapshot_fops); > > + interface.cdev.ops = &snapshot_fops; > > + error = cdev_add(&interface.cdev, interface.devno, 1); > > + if (error) > > + goto Unregister; > > + error = sysfs_create_file(&power_subsys.kset.kobj, &snapshot_attr.attr); > > Heh, that's a neat hack, register a sysfs file that contains the > major:minor (there is a function that will print that the correct way, > if you really want to do that), in sysfs. It's better to just register > a misc character device with the name "snapshot", and then udev will > create your userspace node with the proper major:minor all automatically > for you. > > Unless you want to turn these into syscalls :) Well, I think we simply want to get static major/minor allocated for this device. It really uses read/write, IIRC, so no, I do not think we want to make it a syscall. Pavel -- Thanks, Sharp!