[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [linux-lvm] LVM with devfs questions and patches



Benoit Gaussen writes:
> I saw the changes in the CVS tree except for one thing that I forgot to tell
> about in my previous mail. 
> 
> There is a mistake in the devfs registration of /dev/lvm, the major and
> minor numbers are swapped :
> 
> Just an extract of the patch :
> 
>  lvm_devfs_handle = devfs_register(
> -		0 , "lvm", 0, 0, LVM_CHAR_MAJOR,
> +		0 , "lvm", 0, LVM_CHAR_MAJOR, 0,
>  		S_IFCHR | S_IRUSR | S_IWUSR | S_IRGRP,
>  		&lvm_chr_fops, NULL);

I noticed your change, and it is also in the devfs patch that I submitted.

Cheers, Andreas
-- 
Andreas Dilger  \ "If a man ate a pound of pasta and a pound of antipasto,
                 \  would they cancel out, leaving him still hungry?"
http://www-mddsp.enel.ucalgary.ca/People/adilger/               -- Dogbert


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]