lkml.org 
[lkml]   [2004]   [Jun]   [21]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateMon, 21 Jun 2004 10:01:29 +0200
FromArjan van de Ven <>
SubjectRe: [PATCH 1/4]Diskdump Update
On Mon, Jun 21, 2004 at 04:59:52PM +0900, Takao Indoh wrote:
> Hi,
> 
> Now I am fixing diskdump according to comments by you and Christoph.
> 
> On Fri, 11 Jun 2004 13:50:45 +0200, Arjan van de Ven wrote:
> 
> >> +#ifdef CONFIG_PROC_FS
> >> +static int proc_ioctl(struct inode *inode, struct file *file, unsigned 
> >> int cmd, unsigned long param)
> >
> >
> >ehhh this looks evil
> 
> Do you mean I should use not ioctl but the following style?
> 
> echo "add /dev/hda1" > /proc/diskdump
> echo "delete /dev/hda1" > /proc/diskdump

well no since /dev/hda is pointless; major/minor pairs maybe.
But why in /proc???? it sounds like a sysfs job to me, where you probably
want to represent a dump relationship with a symlink, and use "rm" to remove
an entry..
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2005-03-22 14:04    [from the cache]
©2003-2008