From: Randy Dunlap <rdunlap@infradead.org>
To: David Howells <dhowells@redhat.com>
Cc: mszeredi@redhat.com, viro@zeniv.linux.org.uk,
linux-nfs@vger.kernel.org, jlayton@redhat.com,
linux-kernel@vger.kernel.org,
linux-security-module@vger.kernel.org,
linux-fsdevel@vger.kernel.org
Subject: Re: Updated docs
Date: Wed, 14 Jun 2017 18:53:21 -0700 [thread overview]
Message-ID: <ea73b244-bb11-f95e-78ff-d6abeda50f16@infradead.org> (raw)
In-Reply-To: <11943.1497481089@warthog.procyon.org.uk>
Thanks for the update. I have 2 more things for you to consider.
a. In multiple places, there is something like "return 0 when no error
and return a negative error code otherwise."
I would say "or return a negative error code otherwise."
b. (below)
> ===========================
> FILESYSTEM CONTEXT SECURITY
> ===========================
>
> The filesystem context contains a security pointer that the LSMs can use for
> building up a security context for the superblock to be mounted. There are a
> number of operations used by the new mount code for this purpose:
>
> (*) int security_fs_context_alloc(struct fs_context *fc,
> struct super_block *src_sb);
>
> Called to initialise fc->security (which is preset to NULL) and allocate
> any resources needed. It should return 0 on success and a negative error
> code on failure.
>
> src_sb is non-NULL in the case of a remount (FS_CONTEXT_FOR_REMOUNT) in
> which case it indicates the superblock to be remounted or in the case of a
> submount (FS_CONTEXT_FOR_SUBMOUNT) in which case it indicates the parent
> superblock.
>
-ETOOMANYCASES
Maybe:
For a remount (FS_CONTEXT_FOR_REMOUNT), src_sb is non-NULL and indicates
the superblock to be remounted. For a submount (FS_CONTEXT_FOR_SUBMOUNT),
src_sb is also non-NULL and represents the parent superblock.
cheers.
--
~Randy
next prev parent reply other threads:[~2017-06-15 1:53 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-14 15:15 [RFC][PATCH 00/27] VFS: Introduce filesystem context [ver #5] David Howells
2017-06-14 15:15 ` [PATCH 01/27] Provide a function to create a NUL-terminated string from unterminated data " David Howells
2017-06-14 15:15 ` [PATCH 02/27] VFS: Clean up whitespace in fs/namespace.c and fs/super.c " David Howells
2017-06-14 15:15 ` [PATCH 03/27] VFS: Make get_mnt_ns() return the namespace " David Howells
2017-06-15 9:09 ` Al Viro
2017-06-14 15:15 ` [PATCH 04/27] VFS: Make get_filesystem() return the affected filesystem " David Howells
2017-06-14 15:16 ` [PATCH 05/27] VFS: Provide empty name qstr " David Howells
2017-06-14 15:16 ` [PATCH 06/27] Provide supplementary error message facility " David Howells
2017-08-18 3:09 ` Kim Phillips
2017-06-14 15:16 ` [PATCH 07/27] VFS: Differentiate mount flags (MS_*) from internal superblock flags " David Howells
2017-06-15 9:39 ` Al Viro
2017-06-16 9:06 ` Christoph Hellwig
2017-06-16 14:53 ` David Howells
2017-06-16 15:49 ` Christoph Hellwig
2017-06-16 15:54 ` David Howells
2017-06-14 15:16 ` [PATCH 08/27] VFS: Introduce the structs and doc for a filesystem context " David Howells
2017-06-14 18:02 ` Randy Dunlap
2017-06-14 20:03 ` Casey Schaufler
2017-06-14 20:42 ` David Howells
2017-06-14 20:53 ` Casey Schaufler
2017-06-17 9:57 ` Theodore Ts'o
2017-06-17 14:18 ` David Howells
2017-06-17 14:56 ` Jeff Layton
2017-06-17 15:11 ` Randy Dunlap
2017-06-19 7:47 ` David Howells
2017-06-14 22:58 ` Updated docs David Howells
2017-06-15 1:53 ` Randy Dunlap [this message]
2017-06-14 15:16 ` [PATCH 09/27] VFS: Add LSM hooks for filesystem context [ver #5] David Howells
2017-06-14 15:16 ` [PATCH 10/27] VFS: Implement a filesystem superblock creation/configuration " David Howells
2017-06-14 15:17 ` [PATCH 11/27] VFS: Remove unused code after filesystem context changes " David Howells
2017-06-14 15:17 ` [PATCH 12/27] VFS: Implement fsopen() to prepare for a mount " David Howells
2017-06-14 15:17 ` [PATCH 13/27] VFS: Implement fsmount() to effect a pre-configured " David Howells
2017-06-14 15:17 ` [PATCH 14/27] VFS: Add a sample program for fsopen/fsmount " David Howells
2017-06-14 15:17 ` [PATCH 15/27] procfs: Move proc_fill_super() to fs/proc/root.c " David Howells
2017-06-14 15:17 ` [PATCH 16/27] proc: Add fs_context support to procfs " David Howells
2017-06-15 10:14 ` Al Viro
2017-06-14 15:17 ` [PATCH 17/27] NFS: Move mount parameterisation bits into their own file " David Howells
2017-06-14 15:18 ` [PATCH 18/27] NFS: Constify mount argument match tables " David Howells
2017-06-14 15:18 ` [PATCH 19/27] NFS: Rename struct nfs_parsed_mount_data to struct nfs_fs_context " David Howells
2017-06-14 15:18 ` [PATCH 20/27] NFS: Split nfs_parse_mount_options() " David Howells
2017-06-14 15:18 ` [PATCH 21/27] NFS: Deindent nfs_fs_context_parse_option() " David Howells
2017-06-14 15:18 ` [PATCH 22/27] NFS: Add a small buffer in nfs_fs_context to avoid string dup " David Howells
2017-06-14 15:18 ` [PATCH 23/27] NFS: Do some tidying of the parsing code " David Howells
2017-06-14 15:18 ` [PATCH 24/27] NFS: Add fs_context support. " David Howells
2017-06-15 15:28 ` Anna Schumaker
2017-06-14 15:19 ` [PATCH 25/27] ipc: Convert mqueue fs to fs_context " David Howells
2017-06-15 10:07 ` Al Viro
2017-06-15 14:47 ` David Howells
2017-06-14 15:19 ` [PATCH 26/27] cpuset: Use " David Howells
2017-06-14 15:19 ` [PATCH 27/27] kernfs, sysfs, cgroup: Support " David Howells
2017-06-14 17:54 ` Tejun Heo
2017-06-23 15:29 ` David Howells
2017-06-14 22:31 ` [PATCH 27/27] ... and the intel_rdt driver David Howells
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ea73b244-bb11-f95e-78ff-d6abeda50f16@infradead.org \
--to=rdunlap@infradead.org \
--cc=dhowells@redhat.com \
--cc=jlayton@redhat.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=linux-security-module@vger.kernel.org \
--cc=mszeredi@redhat.com \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).