| To: | Dwight Engen <dwight.engen@xxxxxxxxxx> |
|---|---|
| Subject: | Re: linux-next: build failure after merge of the final tree |
| From: | Jeremy Kerr <jk@xxxxxxxxxx> |
| Date: | Wed, 21 Aug 2013 14:30:04 +0800 |
| Cc: | Arnd Bergmann <arnd@xxxxxxxx>, cbe-oss-dev@xxxxxxxxxxxxxxxx, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, Ben Myers <bpm@xxxxxxx>, linux-next@xxxxxxxxxxxxxxx, Gao feng <gaofeng@xxxxxxxxxxxxxx>, linuxppc-dev@xxxxxxxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20130821010822.220f592a@xxxxxxxxxx> |
| References: | <20130820172052.1f0d89ddf6a1a40ef70333fd@xxxxxxxxxxxxxxxx> <20130820120702.000b044e@xxxxxxxxxx> <201308202246.30869.arnd@xxxxxxxx> <20130821010822.220f592a@xxxxxxxxxx> |
| User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 |
Hi all, > Yes, I agree. The other filesystems that take an Opt_uid as well do use > current_user_ns() and not init_user_ns. They also do a uid_valid() > check and fail the mount (or fallback to GLOBAL_ROOT_UID). So I think > that would look like this: Looks good to me. Builds and mounts as expected. Acked-by: Jeremy Kerr <jk@xxxxxxxxxx> Cheers, Jeremy |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: linux-next: build failure after merge of the final tree, Dwight Engen |
|---|---|
| Next by Date: | Re: [PATCH 1/3] quota: Add a new quotactl command Q_XGETQSTATV, Christoph Hellwig |
| Previous by Thread: | Re: linux-next: build failure after merge of the final tree, Dwight Engen |
| Next by Thread: | Re: linux-next: build failure after merge of the final tree, Ben Myers |
| Indexes: | [Date] [Thread] [Top] [All Lists] |