Re: [PATCH v3 07/38] mount: attach mappings to mounts

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

To: Christian Brauner <christian.brauner@xxxxxxxxxx>
Subject: Re: [PATCH v3 07/38] mount: attach mappings to mounts
From: Christoph Hellwig <hch@xxxxxx>
Date: Tue, 1 Dec 2020 11:50:25 +0100
Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, John Johansen <john.johansen@xxxxxxxxxxxxx>, James Morris <jmorris@xxxxxxxxx>, Mimi Zohar <zohar@xxxxxxxxxxxxx>, Dmitry Kasatkin <dmitry.kasatkin@xxxxxxxxx>, Stephen Smalley <stephen.smalley.work@xxxxxxxxx>, Casey Schaufler <casey@xxxxxxxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, Andreas Dilger <adilger.kernel@xxxxxxxxx>, OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>, Geoffrey Thomas <geofft@xxxxxxxxxxxxx>, Mrunal Patel <mpatel@xxxxxxxxxx>, Josh Triplett <josh@xxxxxxxxxxxxxxxx>, Andy Lutomirski <luto@xxxxxxxxxx>, Theodore Tso <tytso@xxxxxxx>, Alban Crequy <alban@xxxxxxxxxx>, Tycho Andersen <tycho@xxxxxxxx>, David Howells <dhowells@xxxxxxxxxx>, James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>, Seth Forshee <seth.forshee@xxxxxxxxxxxxx>, Stéphane Graber <stgraber@xxxxxxxxxx>, Aleksa Sarai <cyphar@xxxxxxxxxx>, Lennart Poettering <lennart@xxxxxxxxxxxxxx>, "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>, smbarber@xxxxxxxxxxxx, Phil Estes <estesp@xxxxxxxxx>, Serge Hallyn <serge@xxxxxxxxxx>, Kees Cook <keescook@xxxxxxxxxxxx>, Todd Kjos <tkjos@xxxxxxxxxx>, Paul Moore <paul@xxxxxxxxxxxxxx>, Jonathan Corbet <corbet@xxxxxxx>, containers@xxxxxxxxxxxxxxxxxxxxxxxxxx, fstests@xxxxxxxxxxxxxxx, linux-security-module@xxxxxxxxxxxxxxx, linux-api@xxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, linux-integrity@xxxxxxxxxxxxxxx, selinux@xxxxxxxxxxxxxxx, Christoph Hellwig <hch@xxxxxx>
In-reply-to: <20201128213527.2669807-8-christian.brauner@ubuntu.com>
References: <20201128213527.2669807-1-christian.brauner@ubuntu.com> <20201128213527.2669807-8-christian.brauner@ubuntu.com>
User-agent: Mutt/1.5.17 (2007-11-01)

The READ_ONCE still looks suspect as it generally needs to be paired
with a WRITE_ONCE.  The rest looks sane to me.

Follow-Ups:
- Re: [PATCH v3 07/38] mount: attach mappings to mounts
  - From: Christian Brauner
- Re: [PATCH v3 07/38] mount: attach mappings to mounts
  - From: Tycho Andersen

References:
- [PATCH v3 00/38] idmapped mounts
  - From: Christian Brauner
- [PATCH v3 07/38] mount: attach mappings to mounts
  - From: Christian Brauner

Prev by Date: Re: [PATCH v3 06/38] fs: add id translation helpers
Next by Date: Re: [PATCH v3 10/38] inode: add idmapped mount aware init and permission helpers
Previous by thread: [PATCH v3 07/38] mount: attach mappings to mounts
Next by thread: Re: [PATCH v3 07/38] mount: attach mappings to mounts
Index(es):
- Date
- Thread

[Index of Archives] [Reiser Filesystem Development] [Ceph FS] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite National Park] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Device Mapper] [Linux Media]