linux-kernel - Re: [PATCH 0/5][v2] Simplify devpts code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090307181733.GB30380@us.ibm.com>
Date:	Sat, 7 Mar 2009 10:17:33 -0800
From:	Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com>
To:	Christoph Hellwig <hch@....de>, Andrew Morton <akpm@...l.org>,
	Alan Cox <alan@...rguk.ukuu.org.uk>
Cc:	serue@...ibm.com, hpa@...or.com, sukadev@...ibm.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/5][v2] Simplify devpts code


Pls ignore this mail. I have resent patch 1/5 with correct subject line.



Sukadev Bhattiprolu [sukadev@...ux.vnet.ibm.com] wrote:
| From: Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com>
| Subject: [v3][PATCH 1/5] Unroll essentials of do_remount_sb() into devpts
| 
| On remount, devpts fs only needs to parse the mount options. Users cannot
| directly create/dirty files in /dev/pts so the MS_RDONLY flag and
| shrinking the dcache does not really apply to devpts.
| 
| So effectively on remount, devpts only parses the mount options and updates
| these options in its super block. As such, we could replace do_remount_sb()
| call with a direct parse_mount_options().
| 
| Doing so enables subsequent patches to avoid parsing the mount options twice
| and simplify the code.
| 
| Signed-off-by: Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com>
| Acked-by: Serge Hallyn <serue@...ibm.com>
| ---
|  fs/devpts/inode.c |    7 ++++++-
|  1 files changed, 6 insertions(+), 1 deletions(-)
| 
| diff --git a/fs/devpts/inode.c b/fs/devpts/inode.c
| index ad186b4..de15e73 100644
| --- a/fs/devpts/inode.c
| +++ b/fs/devpts/inode.c
| @@ -442,6 +442,8 @@ static int get_init_pts_sb(struct file_system_type *fs_type, int flags,
|  		void *data, struct vfsmount *mnt)
|  {
|  	struct super_block *s;
| +	struct pts_mount_opts *opts;
| +	struct pts_fs_info *fsi;
|  	int error;
| 
|  	s = sget(fs_type, compare_init_pts_sb, set_anon_super, NULL);
| @@ -458,7 +460,10 @@ static int get_init_pts_sb(struct file_system_type *fs_type, int flags,
|  		}
|  		s->s_flags |= MS_ACTIVE;
|  	}
| -	do_remount_sb(s, flags, data, 0);
| +	fsi = DEVPTS_SB(s);
| +	opts = &fsi->mount_opts;
| +	parse_mount_options(data, PARSE_REMOUNT, opts);
| +
|  	simple_set_mnt(mnt, s);
|  	return 0;
|  }
| -- 
| 1.5.2.5
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/