From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 78F0AC43331 for ; Wed, 13 Nov 2019 12:20:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5C90E22466 for ; Wed, 13 Nov 2019 12:20:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727148AbfKMMUJ (ORCPT ); Wed, 13 Nov 2019 07:20:09 -0500 Received: from youngberry.canonical.com ([91.189.89.112]:59812 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727122AbfKMMUI (ORCPT ); Wed, 13 Nov 2019 07:20:08 -0500 Received: from [79.140.120.64] (helo=wittgenstein) by youngberry.canonical.com with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.86_2) (envelope-from ) id 1iUrcy-0005VB-82; Wed, 13 Nov 2019 12:20:04 +0000 Date: Wed, 13 Nov 2019 13:20:03 +0100 From: Christian Brauner To: Naresh Kamboju Cc: Christian Kellner , open list , linux-api@vger.kernel.org, Jann Horn , Christian Kellner , Christian Brauner , Shuah Khan , Andrew Morton , "Peter Zijlstra (Intel)" , Ingo Molnar , Michal Hocko , Thomas Gleixner , Elena Reshetova , Roman Gushchin , Andrea Arcangeli , Al Viro , Aleksa Sarai , "Dmitry V. Levin" , "open list:KERNEL SELFTEST FRAMEWORK" , joel@joelfernandes.org Subject: Re: [PATCH v4 2/2] pidfd: add tests for NSpid info in fdinfo Message-ID: <20191113122002.5xrac2dwmeavdwnw@wittgenstein> References: <20191011122323.7770-1-ckellner@redhat.com> <20191014162034.2185-1-ckellner@redhat.com> <20191014162034.2185-2-ckellner@redhat.com> <20191015100743.t6gowsic7c347ldv@wittgenstein> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 13, 2019 at 05:22:33PM +0530, Naresh Kamboju wrote: > Hi Christian, Hi Naresh, [+Cc Joel since this is _not related_ to the fdinfo patches but rather the polling tests] Thanks for following up here. See for more comments below. > > On Tue, 15 Oct 2019 at 15:37, Christian Brauner > wrote: > > > > On Mon, Oct 14, 2019 at 06:20:33PM +0200, Christian Kellner wrote: > > > From: Christian Kellner > > > > > > Add a test that checks that if pid namespaces are configured the fdinfo > > > file of a pidfd contains an NSpid: entry containing the process id in > > > the current and additionally all nested namespaces. In the case that > > > a pidfd is from a pid namespace not in the same namespace hierarchy as > > > the process accessing the fdinfo file, ensure the 'NSpid' shows 0 for > > > that pidfd, analogous to the 'Pid' entry. > > > > > > Signed-off-by: Christian Kellner > > > > That looks reasonable to me. > > on arm64 Juno-r2, Hikey (hi6220) and dragonboard 410c and arm32 > Beagleboard x15 test pidfd_test failed. > and on x86_64 and i386 test fails intermittently with TIMEOUT error. > Test code is being used from linux next tree. > > Juno-r2 test output: > -------------------------- > # selftests pidfd pidfd_test > pidfd: pidfd_test_ # > # TAP version 13 > version: 13_ # > # 1..4 > : _ # > # # Parent pid 10586 > Parent: pid_10586 # > # # Parent Waiting for Child (10587) to complete. > Parent: Waiting_for # > # # Time waited for child 0 > Time: waited_for # > # Bail out! pidfd_poll check for premature notification on child > thread exec test Failed > out!: pidfd_poll_check # > # # Planned tests != run tests (4 != 0) > Planned: tests_!= # > # # Pass 0 Fail 0 Xfail 0 Xpass 0 Skip 0 Error 0 > Pass: 0_Fail # > [FAIL] 1 selftests pidfd pidfd_test # exit=1 > selftests: pidfd_pidfd_test [FAIL] > > arm32 x15 output log, > ----------------------------- > # selftests pidfd pidfd_test > pidfd: pidfd_test_ # > [FAIL] 1 selftests pidfd pidfd_test # TIMEOUT > selftests: pidfd_pidfd_test [FAIL] > > x86_64 output log, > ------------------------- > # selftests pidfd pidfd_test > pidfd: pidfd_test_ # > [FAIL] 1 selftests pidfd pidfd_test # TIMEOUT > selftests: pidfd_pidfd_test [FAIL] > > Test results comparison, > https://qa-reports.linaro.org/lkft/linux-mainline-oe/tests/kselftest/pidfd_pidfd_test > https://qa-reports.linaro.org/lkft/linux-next-oe/tests/kselftest/pidfd_pidfd_test > > link, > https://lkft.validation.linaro.org/scheduler/job/993549#L17835 Note, that this failure is _not_ related to the fdinfo and NSpid patches here. It's rather related to the polling testing code that Joel added. Iirc, then it is timing sensitive. I'll try to make some room this week to look into this. Christian