From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C770FA0546; Tue, 6 Apr 2021 14:58:42 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8E177406A2; Tue, 6 Apr 2021 14:58:42 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mails.dpdk.org (Postfix) with ESMTP id AF72F4067C for ; Tue, 6 Apr 2021 14:58:41 +0200 (CEST) IronPort-SDR: loyIetOr8DDlTZ2S1A2sA/gunup9r4jkjTaokTVSYPy7E6pk3coXujYTh6u8iHpTA14oVEFgp1 S1T6oOqmFhTA== X-IronPort-AV: E=McAfee;i="6000,8403,9946"; a="180599498" X-IronPort-AV: E=Sophos;i="5.81,309,1610438400"; d="scan'208";a="180599498" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Apr 2021 05:58:40 -0700 IronPort-SDR: HknFIVl8PvlgnroCx6QF6QGvIMpRllh2w5fqJBMBRsfolVWls0r6gDdKFWrd9s9BRJ97yIHhMI 1VnNftC/cA/A== X-IronPort-AV: E=Sophos;i="5.81,309,1610438400"; d="scan'208";a="457878388" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.31.235]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 06 Apr 2021 05:58:39 -0700 Date: Tue, 6 Apr 2021 13:58:35 +0100 From: Bruce Richardson To: Aaron Conole Cc: Thomas Monjalon , dev@dpdk.org, David Marchand , anatoly.burakov@intel.com Message-ID: <20210406125835.GB551@bricha3-MOBL.ger.corp.intel.com> References: <20210317144409.288346-1-aconole@redhat.com> <2080877.JXBMkDg2Jp@thomas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-dev] [PATCH] test: make hugepage check more robust under Linux X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Apr 06, 2021 at 08:33:07AM -0400, Aaron Conole wrote: > Thomas Monjalon writes: > > > 17/03/2021 15:44, Aaron Conole: > >> The hugepage test really needs to check multiple things on Linux: > >> > >> 1. Are hugepages reserved in the system? > >> > >> 2. Is the hugepage mountpoint available so that we can allocate them? > >> > >> 3. Do we have permissions to write into the hugepage mountpoint? > >> > >> The existing hugepage check only verifies the first. On some setups, > >> a non-root user won't have access to the mountpoint for hugepages to > >> be allocated and that needs to be reflected in the test as well. Add > >> such checks for Linux OS to give a more check when running test suites. > > > > Requirements 2 & 3 are optional. > > You don't need a mount point if using the option --in-memory. > > That's true, but it seems to break a few of the unit tests without. > I'll clarify the commit message. > > Additionally, I thought it would be simple to just incorporate your > suggestions - but it seems that meson / ninja doesn't have cascading > dependencies the way 'make' does (or, I haven't figured out from the > syntax how to do that) - a 'run_command' gets resolved at configure > time and it doesn't seem that we can make a run_target depend on another > run_target since dependencies are on file outputs. Maybe we do some > kind of trickery here where we write a file that the build script reads? > > I am trying to figure out how best to accomplish this - suggestions > welcome. > Sorry that I'm late to this thread. Can you perhaps explain what you mean by cascading dependencies in this instance, or what you are trying to do exactly that is not supported? In terms of output files that build script reads, yes that is possible - the file just needs to be in the standard .d format that gcc produces and then ninja can use that to track file dependencies. The script "call-sphinx-build.py" does this, for example, and doc/guides/meson.build tells meson/ninja to look for dependencies in file ".html.d" generated by that script. /Bruce