From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-vc0-f174.google.com ([209.85.220.174]:49253 "EHLO mail-vc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755436AbbCRKSQ (ORCPT ); Wed, 18 Mar 2015 06:18:16 -0400 Received: by mail-vc0-f174.google.com with SMTP id la4so16660786vcb.5 for ; Wed, 18 Mar 2015 03:18:15 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20150318025238.GC10105@dastard> References: <1425909612-28034-1-git-send-email-drysdale@google.com> <1425909612-28034-3-git-send-email-drysdale@google.com> <20150316232434.GJ28557@dastard> <20150318025238.GC10105@dastard> From: David Drysdale Date: Wed, 18 Mar 2015 10:17:55 +0000 Message-ID: Subject: Re: [PATCHv3 xfstests 2/3] generic: test openat and new O_BENEATH flag Content-Type: text/plain; charset=UTF-8 Sender: fstests-owner@vger.kernel.org To: Dave Chinner Cc: Kees Cook , LKML , Alexander Viro , "Eric W. Biederman" , Greg Kroah-Hartman , Meredydd Luff , Will Drewry , Jorge Lucangeli Obes , Ricky Zhou , Lee Campbell , Julien Tinnes , Mike Depinet , James Morris , Andy Lutomirski , Paolo Bonzini , Paul Moore , Christoph Hellwig , Michael Kerrisk , Linux API , linux-security-module , fstests@vger.kernel.org List-ID: [resend, remembering to tick the Plaintext button this time -- sorry] On Wed, Mar 18, 2015 at 2:52 AM, Dave Chinner wrote: > On Tue, Mar 17, 2015 at 08:33:27AM -0700, Kees Cook wrote: >> On Mon, Mar 16, 2015 at 4:24 PM, Dave Chinner >> wrote: >> > On Mon, Mar 09, 2015 at 02:00:11PM +0000, David Drysdale wrote: >> >> Test basic openat(2) behaviour. >> >> >> >> Test that if O_BENEATH flag is set, openat() will only open >> >> paths that have no .. component and do not start with /. >> >> Symlinks are also checked for the same restrictions. >> >> >> >> Signed-off-by: David Drysdale --- >> >> .gitignore | 1 + common/openat | 61 >> >> ++++++++++++++++++++++++++++++ src/Makefile | 3 +- >> >> src/openat.c | 100 >> >> +++++++++++++++++++++++++++++++++++++++++++++++++ >> > >> > This strikes me as something that shoul dbe added to xfs_io for >> > testing, as it already supports a heap of other open flags and >> > xfstests is already dependent on it. >> >> While I don't see a problem adding this to xfs_io, I'd still like >> to see this test live in the kernel tree too. > > You can do whatever you want with the kernel tree - it doesn't > concern me at all. > >> Having it in the >> same source means more testing, IMO. > > That's complete bunk, though. This helper binary is a limited use, > one shot test that can't be combined with anything else. "sharing" > the source code across multiple different test suites doesn't change > that. However, putting support into xfs_io means it will get more > testing because we can *easily* use the functionality in many, many > different ways. It's useful to have a simple self-test in the kernel tree, so that kernel developers (and their buildbots) can catch any obvious regressions that might arise when working in related areas, without requiring any other dependencies. (Personally, I also think it's good that new kernel APIs come with demonstrations of their use in userspace in a nearby commit.) OTOH, testing openat() behaviour should clearly be a part of the xfstests suite (as a general/comprehensive filesystem test repository), and Dave makes a good point that putting the openat() functionality into xfs_io allows for more flexible test combinations. So I guess ideally we should end up with: a) Reinstate the kernel-tree selftests from the earlier iteration. b) Add openat test functionality into xfs_io c) Migrate the new xfstests cases to use xfs_io rather than a local openat utility program However, given that I've already moved the test code once, and that a)-c) involve three different source trees, I'd like to de-couple things: - Concentrate on kernel changes and a) on the LKML. - If & when the kernel changes make progress, return to b) and c). Sound sensible? Thanks, David. > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com