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 3D76F46091; Wed, 15 Jan 2025 13:53:38 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F2EB04025F; Wed, 15 Jan 2025 13:53:37 +0100 (CET) Received: from EUR02-DB5-obe.outbound.protection.outlook.com (mail-db5eur02on2068.outbound.protection.outlook.com [40.107.249.68]) by mails.dpdk.org (Postfix) with ESMTP id 5D172400D7 for ; Wed, 15 Jan 2025 13:53:36 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=zIXqMvCO44b8JvwaEeuTHfVEwzfLmmmvyaD91RzkTPV3B7F3BIrKlXfl3nnaa22uSG0TZwzpfZFhUH8r5afJy7qOsXh+WnzwgalZGFGZyXjHXb4mMCQYe8lug3YvPr5b3l8/ZyB8etu0ogVB4nYhuIfEysikvEXZYIIXbOzbRrh9MPWbbFPZGoI92p0aNJ0njkvVRoPivV1LGOrI+eMlbeMJNKO1EfqvYp0vz+AuWNahgB/EDhZ+i9kYvLo82GJR7Cedqeq3TxnY8fefhhtwYpxE8Q7y+NwVlwRa5c11t9y/LBWKp/dIOnzBnojnqHZWTRtmyBW9j3vH2G/LI9XH6Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=iNrPyOFQacitonZbe1Xe66bp5mzYnSHYf32ejtzkLtc=; b=ou1ebalsBCvjRtP6kgUzHlN3jtSc3F12xubjjM2/vOndiHE5XSJis3JKyHhCZlMFHRikC8El4Ya7FU2kHgEKMWD3CwYahe3wM76G2OQciMrHzElSz/m14WTDDo9WQaAssp5zBrn2J00gS5SGk5Rir9QtgRVX46Hs97g/h79zDBYnlkVp8WtPppZGEonLz2B6udarxDspUavI0FYQt8mOXrgyUMqDGcZhkaAac/+pI7UCzXCj9oEYzXocT7rPtmE/n6LJL1WGwOERtHULyAozK6NTynEC3lE5F3fslSiKNGlD88SSpEStvj8kreW+frkrXdgogRUaOHpUH+Z0t30Y0A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arm.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=iNrPyOFQacitonZbe1Xe66bp5mzYnSHYf32ejtzkLtc=; b=bcV0P/Rnl7KPg1nAp8+VLF1acvvHQ2ifZK2QuawPvlAqY/lYJVAV6YEIuOQaaNauf+r6Ge9iTTwFayPy6G73HgSIYKwszKidMnUZt024Up8p2m8xbe4gbSlGT/htseFhOg9zI9aO5JCLydQeXBesvMV+Ol2Qq6RD/2fR5wv3Zdc= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; Received: from DB4PR08MB8151.eurprd08.prod.outlook.com (2603:10a6:10:381::16) by DU0PR08MB8018.eurprd08.prod.outlook.com (2603:10a6:10:3e5::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8335.18; Wed, 15 Jan 2025 12:53:34 +0000 Received: from DB4PR08MB8151.eurprd08.prod.outlook.com ([fe80::2dd2:fd4d:8ff5:3733]) by DB4PR08MB8151.eurprd08.prod.outlook.com ([fe80::2dd2:fd4d:8ff5:3733%4]) with mapi id 15.20.8335.017; Wed, 15 Jan 2025 12:53:34 +0000 Message-ID: <2dd4cb4a-f26c-49e9-a6f3-62736c56c9a5@arm.com> Date: Wed, 15 Jan 2025 12:53:32 +0000 User-Agent: Mozilla Thunderbird Cc: nd@arm.com, dev@dpdk.org Subject: Re: [PATCH v5] dts: reform hello world test suite Content-Language: en-US To: Luca Vizzarro , Dean Marx , probb@iol.unh.edu, npratte@iol.unh.edu, yoan.picchi@foss.arm.com, Honnappa.Nagarahalli@arm.com References: <20250107213434.31595-1-dmarx@iol.unh.edu> <20250110192855.21941-1-dmarx@iol.unh.edu> <977b0f3a-daf8-490b-a731-824cd9ef03e1@arm.com> From: Paul Szczepanek In-Reply-To: <977b0f3a-daf8-490b-a731-824cd9ef03e1@arm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: ZR2P278CA0036.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:47::14) To DB4PR08MB8151.eurprd08.prod.outlook.com (2603:10a6:10:381::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB4PR08MB8151:EE_|DU0PR08MB8018:EE_ X-MS-Office365-Filtering-Correlation-Id: 7db43d16-e58e-42dd-e85c-08dd35639f2a x-checkrecipientrouted: true NoDisclaimer: true X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|376014; X-Microsoft-Antispam-Message-Info: =?utf-8?B?cDhDQ2UvK09MaUlVZHRrblA5Rk5OTVkydTROenlCRHgxY2VhVUpEMTlSM1lU?= =?utf-8?B?UVI4K21Zd2sybXlXY2xWOWlsY1RTZHFjYW5hNHQyU3d6YW9GZzZlVU5reUVH?= =?utf-8?B?WTVOMEhiZDBJVnVMQmc4K2wzNjJZVmR1Zy81cytWZmpUNDVxbXovSGE4dnV6?= =?utf-8?B?ZHZqT25EMzhiWkZVOHd3N0IzZllTMHM0d2EyalEyT2tjdnFpMzErcXQ3SGcv?= =?utf-8?B?bkE4UlZoM3pIdzBjaks1dVUzeHlZRnZsOTBRWXpIcGxKblc2ZklMMFBFSnI5?= =?utf-8?B?ZUxkZHNFK1N4a0NhS1dhTTdLMDVLNXNmRHU1UUhDd3g4clM1anZHdGpsVmd0?= =?utf-8?B?R25iK01GbjBPTTVPWVNNK2FSNVBCcy9sWXJMVGw0VDZvN0szNGhMWmFSSEdR?= =?utf-8?B?aksvRzUzbEJWK3M4eHEra0crcTNoSUFucWRiallOMEFBZmpCdC8vbFJtczY2?= =?utf-8?B?QWpBOGNYRTN4bnk3VEJHRjN5aGMrb3VwNmo4TkRUTit2UGh0Y3NJcnllS2M1?= =?utf-8?B?MWw4am4wdHV4a2cwN0RVSmk3UjBmcGpmQTR6Z0VNeDhLc1Iya3B2K3lTbjV3?= =?utf-8?B?YW1LeE1DODNOSDFSQU5zYUxOQUFtd0lZcDlhZ0RQVVd5eHJXRnJFS2gwQlJS?= =?utf-8?B?UFJsTjBrci9yMi9UZ1RTalZLRUtOSDZDZnpydVRyeUcrOEpsMHorQkM0Tzlz?= =?utf-8?B?MC9LSVM1S3NSLzZ1Qk9CbXFBaGlycmtQT3lkUmEyVDNtKzJUbDh2RVJ0VUwz?= =?utf-8?B?aWNxNVgyZFdEYjZ3WEF2WU0veTJSTXNVeWtzcDFOSXlhL0FVVmtSWkdXWmxI?= =?utf-8?B?SzkzeWlqdkM5bDhWKzcxY1c2Sm14REdjTmV4dyt1L3YwL0ZOVUN6QVNwcWRs?= =?utf-8?B?RHJoRm1IbTVmZ3E3UGlCRUhicndMbVZPanpvdW1JcTR1ZW5ZbXBDNmVXWUta?= =?utf-8?B?Q3dhckdKQitPVytOemlMd2VnYTFGTjk3cDJyTHl6Q1MrWnlIY2J0T01KWk5P?= =?utf-8?B?MmJINU80amRZOFZqWndrTThuMDNMbWhLcnlPcHlyWm1sdFlDWTNtejkrYkZy?= =?utf-8?B?U2VPMVkxR2VPbzUrZk1Eei9EbjZlM1duOFRINXFCOUEyR0toR0UrZFlnWTlY?= =?utf-8?B?cHFMRTJMUWpHNlJyWHZad3hkMUtFb1BzWEkva21ObnJlTmJGSWJ4S3dsa2lU?= =?utf-8?B?TWhFckxpNkNaMlNqUklOUGZEaHM3TnJXZksrMXFSNS93MU4zdnFOY3BXQ0o4?= =?utf-8?B?NHZMNCtYemNabG1xVTc3WUZ4RXd2S3VMeUFBT3ZNR1JZbEZiUFlyRWJ2eTQz?= =?utf-8?B?QytsTmVpQTBsM3B5UzJwaWNwTE9sNmgrZHk5TUl4L0N3YXhYa09FMGxzRHZu?= =?utf-8?B?RVZZa3FObktwWlYrQ2dVM0U0ZHliS0hUTVBTSVVWdnJzNTlSaUxXY3ZuUUVa?= =?utf-8?B?cldYZHlkR0NrWVA1dWQwWGlxWFBUWEk0VHRTWEl4bmVTUHM2Y1B6eXl6dFRj?= =?utf-8?B?SUZGRW1sU0ZxRkt6cHhCUnM3QjA4Y2Y0NG4vSG1GbGV6NWErdTdWTXJaclJG?= =?utf-8?B?VHRwd3RRN25vRWtubGZHbGprNklZeVNsc2lyRTk4M1lmNzV4c0MwTVJRYVNE?= =?utf-8?B?Y08wUkRGVkhpRDNuNzFqK2V2ZVlrWUpKMUVsN2NHaWRxQlRUeHVRaVRvQWtO?= =?utf-8?B?Qk9SRWUvdmo4YW1pajU4cHBKMm9UdHZUaTlYTTN2MHBIaC9kZHMyQWZTYm9C?= =?utf-8?B?MmxwTlZKSDRBZ1daa1lIY0c2VTY2SjlyKzhGS1dDVUcrd2RTbGVWYVZXU0FF?= =?utf-8?B?RVhDNSt1OG1XaEZQZWJBdHR0R0t0MHRrUnFIS1JNQU4zcWd2U2g2cXRxbEtx?= =?utf-8?Q?7WDSISVG+/IfG?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB4PR08MB8151.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(366016)(1800799024)(376014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ck1PMkI4OTN5TDdhWXloTGVMbFE4d2tNN21RalRYSnBBMjhnMTlJblh5VlRp?= =?utf-8?B?ZlhpSjJPbXJkZ29DRW1LRmVobWV5SUpXSTJYdEwwYXVGcFk0enh5MElkUzNP?= =?utf-8?B?S1QzUVpaMDF5Ri81Mm4xWVpmQVpITUROWS9Dd1lVU29lK2tyRUdXWWlQMnVO?= =?utf-8?B?UEc4d0hFQVo3U0cxVlV1dTFZTmFRb1FJYk4vRkNGSEdPRjJzVzB3WVQyZ2lq?= =?utf-8?B?aGFXUzVsSXJMUUx3Z0thNW1wZURiQkZUUS9WR0loTE91ZE0xb0xrQTErL3Zx?= =?utf-8?B?bFh4SkpmWFE0ektVSlp5ejhQbVkyWjNWR0dsWXgwaGlJUU11L1VqWS9PUDZQ?= =?utf-8?B?U00vMnAyTVU1ME90TDVSZmZWbFgxdGdXa2pIUDZ5djBMcHQwcUNYSzVzZWQ5?= =?utf-8?B?SHFLSHJqa0tVOEpEUlNKTG4zT24xUHVqT0ZQQVVLQk9xQi8wV3VCZk5Qay9R?= =?utf-8?B?NG41dXdyVVNNcG5XVmhFNHI1bzFjeHJTaFNaeER2aXhUZUZ5L0dlNHNxV0U5?= =?utf-8?B?a0NOQlpTb3pEeXUwanBwY2RndC9CRi9EbnMwajdFSEZBRzdjTjV4WlR3bVQz?= =?utf-8?B?alBtd29SeUEvR3YvYXR1cWhIblhqeEh2cExvdEc3Z3dSWTlsUzNaOGFtSkNC?= =?utf-8?B?NVpwMzZPR3JJTHJ2SmtjNSt5bFRYQ3I0TlhQMHo4d3hRZHBRaEJCNTlCbE1v?= =?utf-8?B?cmIrZXhGQW1sNTlIQXE0YUdVT25aeG9FNmVXOWdDVWRvNHlOS3dtVFN3dWs2?= =?utf-8?B?RlZSMi9FcnpHZ1k2R2hMUmsyWjlPSkE5RTBSVSs0L245ZHQvNEpkUHUxMXJZ?= =?utf-8?B?S3l2bXlUb2l6SFlEQlMvVjNZOHE5UnhhUG1XQUtzTnpyMzFrOTY4bFNXcTND?= =?utf-8?B?d3kwWjI2WllUcklYd25POFUyWFdpdFAxczJPYU5jZUI0Qk9QNTl3Y1VaUnpY?= =?utf-8?B?TGJkdzBxMG55UmtFY3VWbCtEeDkrSVFnY2RxUnJuVEU0UVRYSGpnNFg3anFo?= =?utf-8?B?NHJXNFhzMFVVaWlVYzZBaUQ2ZDdGalpyZzVETEtFK3g0Q0N2OGVNd0RXUkZ6?= =?utf-8?B?Y3lxK3l2OC80QlN4blhYWjNVT09Uc1VFU2IwanErUjh2OGtmS1RoSXQ5VkxX?= =?utf-8?B?WXVCcW5CS2lFVU9za2VnL3BkTm5aczM5Tlc2RElkRGZQdHNPN0Y0aGFTNEVx?= =?utf-8?B?aE9JYlVLc3Q1SlVTZ09EMURTMTM2YjFueGpWL25nZ21pVTFJV1NxeXV2OHlt?= =?utf-8?B?Z1grVDVCZkF1UElwNkFYN3Uydk5sQW5ndlZrM2M3VVlJK0ZCVmQ5RFR6ZHFN?= =?utf-8?B?UXRrVjA0dWxJUzl2QU5nRlRvNUpzakVSVGZxUEVpamxGSWx1cUcyeUFYYm1i?= =?utf-8?B?WVR4MVZlYXNrckFuSml0cHM5RmYybmFSZTZMKzFxS3JYOTRrbUZvbzIrWHFs?= =?utf-8?B?dUpacldRWW5MU1F2UXRkMUVlVlVSZVhJNkpwN1hnTno5RGpOZk84T1pCYWxV?= =?utf-8?B?RjVrRjdleE5EVUNlTVZaYVhiQXJYck9HN1U5ZzI2M2J1YVdQVzR6a3hSUzYw?= =?utf-8?B?R1cyQTNybTZrb01oRVhEdkYxWUxuR3pvVXlvbUR4RnREdWpXMDNmMVdudUZ3?= =?utf-8?B?YzF1YXBLVWVHdXVYUStZSWhwV0pBRG4rMnJ6TUY0RmRPVnk3WXZlS3RlUmhh?= =?utf-8?B?QzdkV1ZOd2NiYWx5dll2MFBlTmdDakRmcXVqc1BrL0E4WHhOUlZpVVVlcC93?= =?utf-8?B?SXAzdFJYb0xtUXZFTFVYTHdBQ2tZT0Vwc3JsZ2g4UGtvV3BONWlkVmN4MVVV?= =?utf-8?B?eXl4b3p3Q0NaUk4ySnVPTkFXd24rbjR4YWFSenp6UDZuaVhrN3dBZUlZZWpK?= =?utf-8?B?cXhzVFBmM3FGTTFVSDU3MWxSWEwxbkFkRklzejhleVJFTmhFUGhXbGFrSlhu?= =?utf-8?B?V0VkZnVsbkpWbnNxaEFTVGwzVVZ6dFdjaWk4dmVCdml3c3pnVkFuK0dOOGZH?= =?utf-8?B?R3RWQURPTmNXRkZWOGhRTWt2dFNqbHVwdXBGazZDYVU3WFAyMStUY3dMUGdy?= =?utf-8?B?MWRsYWpNMklUa3VIZHVIZ3IyKzQzS2UzZEt5RmFwbVlFMm1HdkhoUnltMmZ6?= =?utf-8?B?SXorczNnSWk2K05XQzJHcHh3UGJ2YjQzaVVIQ1gxMXpzYUxLLzFXY05GdVZY?= =?utf-8?B?RWc9PQ==?= X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7db43d16-e58e-42dd-e85c-08dd35639f2a X-MS-Exchange-CrossTenant-AuthSource: DB4PR08MB8151.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jan 2025 12:53:34.7229 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: /sw0NNnUl8Pdc5ln0Fj1ST1h6o3R/v5uY9k5P8fBo0YlJrTYEF1uFdAWyLSQ3fyQeVX1qCKKTGP791I9+J49SQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR08MB8018 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 I think it makes sense to keep logger private and separate. I can see one reason - we can change the logger in the framework without affecting the tests. We can also decide to not use the logger to print test suites messages - for example save per test artifacts. On 15/01/2025 10:17, Luca Vizzarro wrote: > Sorry for the late reply Dean. This is a bit redundant in my opinion, > there is no reason not to give full access to the test suite's own > logger. I'd just change TestSuite._logger into TestSuite.logger to > indicate that it's a public member, and do what you initially did.