Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1443688
| From | "W. Trevor King" <wking@tremily.us> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 3/5] nsfs: add ioctl to get an owning user namespace for ns file descriptor |
| Date | 2016-07-14 20:50 +0200 |
| Message-ID | <rUTRU-4Q7-25@gated-at.bofh.it> (permalink) |
| References | <rUTyx-4Je-5@gated-at.bofh.it> <rUTyx-4Je-3@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
[Multipart message — attachments visible in raw view] - view raw
On Thu, Jul 14, 2016 at 11:20:17AM -0700, Andrey Vagin wrote:
> +int open_related_ns(struct ns_common *ns,
> + struct ns_common *(*get_ns)(struct ns_common *ns))
> +{
> + struct path path = {};
> + struct file *f;
> + void *err;
> + int fd;
> +
> + fd = get_unused_fd_flags(O_CLOEXEC);
> + if (fd < 0)
> + return fd;
> +
> + while (1) {
> + struct ns_common *parent;
I think you want to rename this variable to ‘relative’ or some other
more-generic term [1] to echo ‘related’ in the function name.
Cheers,
Trevor
[1]: https://github.com/avagin/linux-task-diag/commit/7fad8ff3fc4110bebf0920cec2388390b3bd2238#commitcomment-18223391
--
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[PATCH 3/5] nsfs: add ioctl to get an owning user namespace for ns file descriptor Andrey Vagin <avagin@openvz.org> - 2016-07-14 20:30 +0200 Re: [PATCH 3/5] nsfs: add ioctl to get an owning user namespace for ns file descriptor "W. Trevor King" <wking@tremily.us> - 2016-07-14 20:50 +0200
csiph-web