Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <3ffb6891-aa32-6835-2cbd-e9b2f204ac8f@intel.com>
Date:   Thu, 22 Sep 2022 10:52:15 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.2.2
Subject: Re: [PATCH 5/5] selftests/resctrl: Flush stdout file buffer before
 executing fork()
Content-Language: en-US
To:     Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>,
        Fenghua Yu <fenghua.yu@intel.com>,
        Shuah Khan <shuah@kernel.org>
CC:     <linux-kernel@vger.kernel.org>, <linux-kselftest@vger.kernel.org>
References: <20220914015147.3071025-1-tan.shaopeng@jp.fujitsu.com>
 <20220914015147.3071025-7-tan.shaopeng@jp.fujitsu.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20220914015147.3071025-7-tan.shaopeng@jp.fujitsu.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bWtEdThsRnlyNGk4bVFxOUxGZGEyYmxMd1A5ZUp6ZXE0b0lQRzdOOSs0TEpY?=
 =?utf-8?B?S1BmazFwRFRkUHNqMSt1U0hSVkgwS1d6aWkvSmdiTnV5MmZuS2tlMFF3SnVC?=
 =?utf-8?B?N3dkL1VPNFAwWndyUlN4N0ZHN3BsR3lPa3BCRXUwLzA5SzdiSUF2enhTUEFW?=
 =?utf-8?B?aERxS25NUEpPUHZVb2wyVmFmWlNsckVPWFZXajN3amlzMytYRnMyUFF1Wm5Y?=
 =?utf-8?B?QlFJajFvM29GRnF2LzFhbGxzN3AxOUtRWXN6a0ZKUkNVcUwrRzAzajg2VXFQ?=
 =?utf-8?B?ZC92anRZRjVqU0RNR3Ryekd1WGxPY0E4eHVZU3BNTlNiakRYaWsvVHQxTlJ6?=
 =?utf-8?B?U1d0Z2NVcmhrb0N3aEJSL3VKS1k3NW11YUY4dlNQaE95cExnckpERjZqdk95?=
 =?utf-8?B?aTd6MlBVSysweEZQVTVvTzdJS0U1UGxObUpoNjhlR3hDV2NaaGRrK2lyY1hW?=
 =?utf-8?B?MEUwVEZWbEpzMDdUdVRmQUlOZzl5aGhYS1I2MVdwLy83NG9NWlJabmhKRXV1?=
 =?utf-8?B?NHhYTGhUWDRqK28rVGsySG81TzRoeHUxS1luZ3Q0bHY4bFVzNmdlRmdjZCt0?=
 =?utf-8?B?U0daRXVBZUNpMDl3ZWNMeWpTaitUUzhMYSt2WjNoU2M4d2tBMmlLOTlSeW5h?=
 =?utf-8?B?ZFpRMzd0WGo2aElpT2hWZHp0RlF1UUVwMW9iOVdaVEk1cWtxM3JnbWFQYzNY?=
 =?utf-8?B?cEpDNGpNeXZmUGlvTWNMRjlkTUdRRHhQWXk1dXhJRGF5WEJiNFg2LzViNUNZ?=
 =?utf-8?B?U0FOekNFYjQ3OUQzUGpCREovSmV0a1h0SWUxNUlnWUJESW80SlJGVzJzMy9I?=
 =?utf-8?B?d3VhbnQ3MVFnVGEyU1J1MlpQenpmQ0lTaXhJRW1kS25ZQVY5eVNMKy9YMHpT?=
 =?utf-8?B?MVd4QVJtSjNFNjhmSzNCaFFzcEdwRXlzY0lMZTNCREhDRmhTdGhyQlpBU0g4?=
 =?utf-8?B?SUU5bzVRVFdvT1BKVDJjakZmN250ZklFb043Tzhqc21tSS9uTWlTNmtrM0Zk?=
 =?utf-8?B?eGhpU3ZHcENKSXMvV2ZvTXIxRGRpNXVFZnovaW9zWnJWREZYSXJQM2t0QVZX?=
 =?utf-8?B?WHBqWjlEQ2FOcmtnQ1A5dmRiSHRvdFRoQXRYanB5M0JSTmFhRkphZHNMei9G?=
 =?utf-8?B?MW92b2w4SEZBR1ovaVp6TzY1bHNMUjFMZ09Icld6WVFBTkZYc0tuVWRsQUlH?=
 =?utf-8?B?MEdiNnAzUStIWS9VSGdQZENGc3RyOTZJRG5NVE93eE55NGRsaWRkQmpMZXVz?=
 =?utf-8?B?cGF4b2RoQ0tPZWpmcVpZRnBzbDZnYnhCa2JJQ2NoUE56cmpEVGxhMWxURUlB?=
 =?utf-8?B?dEM0ZU9XRVYxWXpzbm9XRk5vUXltdWNIOXNheUtKdWhiamtWd3lWMjFTaS9l?=
 =?utf-8?B?ZkFaa0w5cm5hc2lVQ0FUNklSZUJ2ZDFSTm9RTVJLT2tLajd5UG51S3ZZcHRT?=
 =?utf-8?B?RXFQVFB0QkhlQk5ISHJpbVpaK2hSYWVVRk9mc2ZSYmhEMXY3RVY0eFYvK1h6?=
 =?utf-8?B?SHJDM3pjbUZtQTFFUFFaaVE0NGo2TGlRa1pnVkQ5VE1OSGgwTmVCSmU3Y0pq?=
 =?utf-8?B?U2Jpc3hUbW5IS05DNlo4TFNQM2ZSK005aTBXbFJBdXRGU3FlWWhVc0NWZWt6?=
 =?utf-8?B?V0w4T3JDd3hBUFA2eWd5d1BoZW1icHh6VUlrT0o1ZlppSHgvN21XV2F3UElX?=
 =?utf-8?B?VVhuYXZQS050TU9RSlNhbUo4L2Fxem1yTithQlpiTXg0dHM3VWxxVzZGNUxC?=
 =?utf-8?B?NFpBckdyRnNmUjlKZGZZWElZRmVnQmVkaXFhQkZ3czZ6L3p4eUg1Qm9iakh6?=
 =?utf-8?B?ejh0ZkM5dklNQlBXYTJVenZMYjRYampRU1drcmRKWlRjM0x3cGZZRkRkTXVu?=
 =?utf-8?B?ci9ESGJreTBmUHVXazZobEtaRkdrOWJrK295ODB6cUxPUVU1ekMxTUpCTjVL?=
 =?utf-8?B?b3l5aDAzU1pSL1JneXVlRmk4enNMKzZ1dGNtdlZMdTR6Ymh3Smp6UkdDUmJU?=
 =?utf-8?B?SCtTaXp0RUFETUg0Ni9wZVYxdnB0MlFMK2R0elJxYk0wclBJaXhkb1drTUxo?=
 =?utf-8?B?eGRSdHlyY29qMXFXSTRwYUZBQ0xzOGJsK1hSbDFaNXJZUndjWmk4NkoxUG1M?=
 =?utf-8?B?Ym5nTXgvQ1QrOGw5MUVHZmVzL0tVUCtmZGZnTzFQbUNMWXFzRDFGVFRoanBZ?=
 =?utf-8?B?S2c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 5ceeff4c-517f-4a71-da8a-08da9cc330e6
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Sep 2022 17:52:18.1056
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: gkjn54RQYLX9S6lmskBBwSYDzLrnzoTQ0AafH60llxwuYY+/ylVq24nrNmRdbtpJAz1Pj9NbQyXuEvqU9LatSMjKzQFqCQe9QOJfS8XTvKQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR11MB7282
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-6.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 9/13/2022 6:51 PM, Shaopeng Tan wrote:
> When a process has buffered output, a child process created by fork()
> will also copy buffered output. When using kselftest framework,
> the output (resctrl test result message) will be printed multiple times.
> 
> Add fflush() to flush out the buffered output before executing fork().
> 
> Signed-off-by: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>

Thank you Shaopeng.

Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette