Re: tmpfs on contemporary -current: panic: locked against myself

From: Xin LI <delphij_at_delphij.net>
Date: Mon, 12 Nov 2007 11:02:53 -0800
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dag-Erling Smørgrav wrote:
> Kostik Belousov <kostikbel_at_gmail.com> writes:
>> The following patch fixed it for me.
>>
>> diff --git a/sys/fs/tmpfs/tmpfs_vfsops.c b/sys/fs/tmpfs/tmpfs_vfsops.c
>> index 4a3cdec..55e9891 100644
>> --- a/sys/fs/tmpfs/tmpfs_vfsops.c
>> +++ b/sys/fs/tmpfs/tmpfs_vfsops.c
>> _at__at_ -201,7 +201,7 _at__at_ tmpfs_mount(struct mount *mp, struct thread *td)
>>  	/* Root node attributes. */
>>  	uid_t	root_uid;
>>  	gid_t	root_gid;
>> -	mode_t	root_mode;
>> +	int	root_mode;
>>  
>>  	struct vattr	va;
>>  
> 
> Alternatively:
> 
> _at__at_ -231,7 +231,7 _at__at_
>             vfs_scanopt(mp->mnt_optnew, "uid", "%d", &root_uid) != 1)
>                 root_uid = va.va_uid;
>         if (mp->mnt_cred->cr_ruid != 0 ||
> -           vfs_scanopt(mp->mnt_optnew, "mode", "%o", &root_mode) != 1)
> +           vfs_scanopt(mp->mnt_optnew, "mode", "%ho", &root_mode) != 1)
>                 root_mode = va.va_mode;
>         if(vfs_scanopt(mp->mnt_optnew, "inodes", "%d", &nodes_max) != 1)
>                 nodes_max = 0;

Oh...  My fault, thanks for pointing out.  I have just checked it into
- -HEAD.

> Incidentally, later edits to the tmpfs code have introduced a number of
> style(9) violations, one of which is visible in my patch.

Could you please give me a copy of the patch? :)

Cheers,
- --
Xin LI <delphij_at_delphij.net>	http://www.delphij.net/
FreeBSD - The Power to Serve!
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (FreeBSD)

iD8DBQFHOKNdhcUczkLqiksRAhZ5AJ9MKCYlDT+NGGESAHf2BE0mSm5FeACgqHO3
Nk6Lk2Bt8L3z78oza08d9Xw=
=TfYA
-----END PGP SIGNATURE-----
Received on Mon Nov 12 2007 - 18:11:43 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:22 UTC