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 3AB60A0C47; Mon, 26 Jul 2021 14:27:58 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ABE7940F35; Mon, 26 Jul 2021 14:27:57 +0200 (CEST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mails.dpdk.org (Postfix) with ESMTP id 3629D40DDA for ; Mon, 26 Jul 2021 14:27:56 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10056"; a="273308978" X-IronPort-AV: E=Sophos;i="5.84,270,1620716400"; d="scan'208";a="273308978" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Jul 2021 05:27:53 -0700 X-IronPort-AV: E=Sophos;i="5.84,270,1620716400"; d="scan'208";a="516079364" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.8.178]) by fmsmga002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 26 Jul 2021 05:27:52 -0700 Date: Mon, 26 Jul 2021 13:27:48 +0100 From: Bruce Richardson To: John Levon Cc: dev@dpdk.org Message-ID: References: <20210726121627.2324730-1-john.levon@nutanix.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210726121627.2324730-1-john.levon@nutanix.com> Subject: Re: [dpdk-dev] [PATCH] app/test: flush stdout after forking 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 Mon, Jul 26, 2021 at 01:16:27PM +0100, John Levon wrote: > meson test was not capturing the intended output from the child > process; force a flush to ensure it reaches the test log. > > Signed-off-by: John Levon > --- > app/test/process.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/app/test/process.h b/app/test/process.h > index a09a088477..0ed91a939e 100644 > --- a/app/test/process.h > +++ b/app/test/process.h > @@ -110,6 +110,7 @@ process_dup(const char *const argv[], int numargs, const char *env_value) > for (i = 0; i < num; i++) > printf("'%s' ", argv_cpy[i]); > printf("\n"); > + fflush(stdout); > I thought that "\n" normally flushes in most cases, but if explicit flush is necessary for some cases, let's add it. Acked-by: Bruce Richardson