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 B9A4AA0544; Fri, 23 Sep 2022 10:15:46 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A5D35400D7; Fri, 23 Sep 2022 10:15:46 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mails.dpdk.org (Postfix) with ESMTP id EC5244003C for ; Fri, 23 Sep 2022 10:15:44 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1663920945; x=1695456945; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=d9quuBxB4Bg0uunajak+5zvdYSi5EvgZK85y4mcjyyI=; b=jtJD3SB1QhDrd0LFKPu45qYB/qCH73z7nLVR3zj0ettm8DhFIeoskRRp NTrWFJtfMCsZe92val7h9S25YjzPFZ6A3ivLlXZectqB4k5A0OoaGe7o7 BUZhHcjQOA0nswXdmkEEtpdDvTgv3utFKeeXO20taCppxmVWsNYNBDrUA wPi2wlPbhPUYpqMjS5ketSJCNNOmVvjW82Rbq2Nbu9p4iwk5O1TZwnZIS 0CldcQR04fwUKA1saAbjVLPV5tsnHXD0eYpNPykLIzkVDReyy8WQ82SNh 7PBrRGQeFliYNrDlZjRGc8n5LJFMzMz/epYJoBACnBZaIyKH86cYOKGrX g==; X-IronPort-AV: E=McAfee;i="6500,9779,10478"; a="283637920" X-IronPort-AV: E=Sophos;i="5.93,337,1654585200"; d="scan'208";a="283637920" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Sep 2022 01:15:23 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.93,337,1654585200"; d="scan'208";a="709222740" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by FMSMGA003.fm.intel.com with ESMTP; 23 Sep 2022 01:15:22 -0700 Received: from orsmsx609.amr.corp.intel.com (10.22.229.22) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Fri, 23 Sep 2022 01:15:22 -0700 Received: from orsmsx611.amr.corp.intel.com (10.22.229.24) by ORSMSX609.amr.corp.intel.com (10.22.229.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Fri, 23 Sep 2022 01:15:21 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx611.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31 via Frontend Transport; Fri, 23 Sep 2022 01:15:21 -0700 Received: from NAM04-MW2-obe.outbound.protection.outlook.com (104.47.73.170) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2375.31; Fri, 23 Sep 2022 01:15:21 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KHGFGOfmXceUuhIrzr61j0mAp6aF81ENfbqmQwCESlu7y8UkWW6wWrnVIqEZSGiy9G3zVmDD8An3U8MaDalHQG0LplNfy24XNU3A6oeeCd4V4lhzShGgDgLniZwuDeoCQO6u0b1vYf7KR7gyhC24VHZt8Qc1h1P2Ytn8k+mR1HqOFdg6t3Rl5E4FUAfp5ewWkXvI118xfPVrcqZOIEGkJjNHVZ9H3kIGYALia3tehiYk7MUo+oou8uj6Q3MNwyWgt3YQCXG/fTm5XcuYQmB9Dqa4Hcz4nWxlBhwo87kGFtBcuSAUrEn842gPlw0eYRUAdVn5NHLWEGksVEG647xKJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=shcbpUiDSNvzZJIztNw4JvAgj+JfnHolUUw4pfhYRcA=; b=i6Id82oIa9128SbMck2jdO419djD+o3z0ngBzcFxwMPQHQNPY5Otxn8q2RTZxCJxr7AC7Sa9+sjfUsHjnXUun0xuOIHFc85rgcQfqSqbKuyiT8JzHXdiQmiMb2QaiKFfhGlUWo8EaPrt1EV2SnEXwDvUFNx0617JpNVVvJQs+hhb1G0eTbUbpO0Ypp/orAoYQeBpQ7Pb2ONc7DdgXKHCfZynYtW8jympxytEA+SzBtjCgqu4gJc/FB64klJqpXP8UaAOq0el/SLpVi0YpThWIIFtpDUsqTEHK9dub4Uid1oovK8DOK/Fw5azGeDlivMBWJFbA6pNuaVjJwUvkjPlaw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from MWHPR11MB1629.namprd11.prod.outlook.com (2603:10b6:301:d::21) by DM4PR11MB5328.namprd11.prod.outlook.com (2603:10b6:5:393::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.20; Fri, 23 Sep 2022 08:15:14 +0000 Received: from MWHPR11MB1629.namprd11.prod.outlook.com ([fe80::13c:8120:d994:16d2]) by MWHPR11MB1629.namprd11.prod.outlook.com ([fe80::13c:8120:d994:16d2%6]) with mapi id 15.20.5654.020; Fri, 23 Sep 2022 08:15:14 +0000 Date: Fri, 23 Sep 2022 09:15:07 +0100 From: Bruce Richardson To: Juraj =?utf-8?Q?Linke=C5=A1?= CC: Stanislaw Kardach , "thomas@monjalon.net" , "david.marchand@redhat.com" , "ronan.randles@intel.com" , "Honnappa.Nagarahalli@arm.com" , "ohilyard@iol.unh.edu" , "lijuan.tu@intel.com" , "dev@dpdk.org" Subject: Re: [PATCH v4 4/9] dts: add ssh pexpect library Message-ID: References: <20220728100044.1318484-1-juraj.linkes@pantheon.tech> <20220729105550.1382664-1-juraj.linkes@pantheon.tech> <20220729105550.1382664-5-juraj.linkes@pantheon.tech> <20220914094222.vprahs3kfhnsklyh@toster> <7f360965ca2c47b29e2413e3207234db@pantheon.tech> <20220922143226.3d5e552cylcie665@toster> <115446c94a7c4eda877c8a968adce100@pantheon.tech> Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <115446c94a7c4eda877c8a968adce100@pantheon.tech> X-ClientProxiedBy: LO4P123CA0027.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:151::14) To MWHPR11MB1629.namprd11.prod.outlook.com (2603:10b6:301:d::21) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: MWHPR11MB1629:EE_|DM4PR11MB5328:EE_ X-MS-Office365-Filtering-Correlation-Id: 385f0992-fc9b-4491-d50e-08da9d3bbe13 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: yfaSirwpoHBGnvYRineUBMqEMbWWRgpBhEVIemWYEq2iAg3f1MVAA20OCS/aj2NWgKR0J/atKdjS6f3+WxMmnR+sAv90zIQXNXu75s1rFqAGz5Tgoin5UmzznLsN9D+x0yuT3XO0uTcGEDx2FbRXbTBD2Gm+Ze6gxgGZEqzKBI1qvdrhWwAb8k7UBWpu55Ce98aycwK4qyzX1YkxvBrXU21sQvBG4IaxDvr3tx8qm0qHIyVMmE8BEvXpp+Jhb9+xNIw+iGaiIOQLdRM4kled95aV9G+e4jfAY4qKKTQFdnZsR48a/6pBIzsApg+aLxfdjtxRDBQMgYqHfpNODkja2ovPJ/PVPgggJicC6pUrRl9h63o0fXRfVbmamJPVLBQ3ULO7ER/tF9oc4qCHViuHJJ+Riyz7fY5nqZNv8B/32qx9hnhQD3iZuorFZHKva7V7ElH+w5AyaQ7V9frU0AElxst5hInutGVJnYYa8aisxaHrK3FCWTVsg4E99iKqgWDzFSP0pbXJMWSBt5CLrV1TCTBZ7GuEoHZqj+ZlL9oH/3wWwyLL20LFrLKhv8s9y96VfitXMvRy30hiyhS/kRvG55AdrRwxHxaPehQ24CtFQrvLvyoYyItF302+g4o1B/+34c7fEjpaAB4kp8oBxuZrdhzqaHwYPggQcuGpFdnlMoWH8A97dq79xBeWjhL1JQu6yLO1d+eyMIXFzceIsQzwLw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR11MB1629.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(39860400002)(346002)(376002)(136003)(366004)(396003)(451199015)(186003)(6666004)(26005)(41300700001)(6512007)(44832011)(2906002)(53546011)(6506007)(5660300002)(8936002)(82960400001)(38100700002)(86362001)(83380400001)(6486002)(316002)(478600001)(54906003)(4326008)(8676002)(6916009)(66946007)(66556008)(66476007); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Um45bXJRQStkaXJzaDZocjlpMGp2S1FEN0t1SWpKSytUQUhENWppeEtTNjBM?= =?utf-8?B?dkRhQVpaL0RhelI0czluVTFXcVBWMVRwRkVpS04zN0ZnaWpwbU9oZ3N6cGIx?= =?utf-8?B?Wit6T2JVK1A2akpFY3hnMEI3UjgyL2xTNHltSFNEb0h1elpPeHNLZ2RGZGw5?= =?utf-8?B?enBJcjgvS0xjNHB1MXJ0aGEzS3JNRnRCQVR2VC9xNXdKMFFLUW5OTHZtcHpS?= =?utf-8?B?dFk0Z2VQM0oxN0RxTEhpSDczcjNOTEYvY1FXUzlWQzY3VUZrMUlSK05Mckd5?= =?utf-8?B?QWxvdGxYdmtVYTh5TnFzWmdES0xyWlVSOWIxT2dpMldDLy9ONVJ3Q0Q5Vy9F?= =?utf-8?B?VmFJazl5cXc1UDIyQzdaeTJsY3k5d1ZIWS9IQ2ttM1Z6dEVqdm03ZVZjdW5I?= =?utf-8?B?d2tLTEtyRmlSUERkOHpwQ3FoTCtYcllBZzZIYW1mMUZuQW9HelJTb1ZSVVM3?= =?utf-8?B?TXVuN2w5ZmNlbkJRMC9QekVMRVJIS2xaSWVaNGpSQk9KNEwrS1cweVppNllk?= =?utf-8?B?QXBuTjRjdnRuY0lkenN4SUw2YnRCN25uaVRTZ1VETEphb0ZjaFYyRHJxc1ZJ?= =?utf-8?B?RlltQTd1YXRzVG5PVHZMVUR5TVkrekNQZjAwdVcvVURiRjdTOXV3cHI1aXlh?= =?utf-8?B?NHF1b2tZU08wVWQwT3cySGIrL2tCWmNISzJCYS9mUTB2cEY1OUd5MUpnQTUr?= =?utf-8?B?U2h4VFI5RFJvdzFmbnMxM1FpRzAwYWI2WmxoS25kOUtBU0NGR2ZRem9xWDR3?= =?utf-8?B?ZjVPNFl6Rk4zbDlwNnJUaFg4bWVHZnUwZitIeW1MTHB1empzcTlXV2ZYdUs3?= =?utf-8?B?cThUdWhmS010QWMvRTduaWdhT1lFdm5SYkJBZHhQdEFaREVOV3Z6VnBGSDZH?= =?utf-8?B?bmFnTXlvWG80WmZxQ3hucU1UWHo5N2RIMW05ck5ibkFCRnh1NGxUZXVjMDd4?= =?utf-8?B?c3Vwa2lPSmkxOW9aSVNqbWlXdEw4SnF2TjZyV3o2ZVpFendITW05bXE3Uk00?= =?utf-8?B?c29WbldBd3FWRjlBdkRlaWNkbU5NQjJ5SnhxNGtZc2tjY3Y2ZTdUUGtpZkxP?= =?utf-8?B?eWxtRUw3Nm5sTmh3TXF5dG1TM2pWVVBDN1dyalgxellqY05YMUxYV0h0dzJq?= =?utf-8?B?QThpVDRYOGNvZjNlOFVSSHdyNVFLWGQyaStuOEllVHdqRXozWFRkazE3SUV2?= =?utf-8?B?d3VRTkFsM2h6aDYzaXlkYUdDV3F0SjE3YldmUk9Md280bk55KzJwSHZaWVpI?= =?utf-8?B?NzNtZkVzQkNyRVlEQkdNTCtBL3AycHREK25jbk54YWw5eTIwbGpBaXdaK1Fw?= =?utf-8?B?dTI2d2RhK3RIMGpGYlNyWDdEbVlRNGk0dUhjcitxenV3dGY1VzNsVFdVSUFW?= =?utf-8?B?VVBSOVVQREw3M1pUOUtPWmtXOUZmT1U1aVMzQUJXeC9XUXl6OUlnVmV5Z21L?= =?utf-8?B?dStFaVlBS1U4aG1zR0kwRmhxZHVpa1kyNzVsN0pIWWttNm9BcGpjZDFqdUFK?= =?utf-8?B?ZHpuazRseHFlakZiYVg5L1Y1dXZZRXV2VXNqTGFiYUtRZm9VMzBNK2hpdTlS?= =?utf-8?B?V0U2L1Z6R3RvT3U3Skg0NmdVbkdZeXJQQ2VVSE9EeTUySXFBLzVxZEFVM2x5?= =?utf-8?B?aTBxVjA1anorSk9RYy9DRUJDdGYxeW16aHdIbmFXcUlvdmkyYzFPRzlEVUNu?= =?utf-8?B?UUxIV244bHo0S2lvT1VBVU1Sc3JKaXFtbERlUEgzSFpPQUdZb1ZFYUJrekIy?= =?utf-8?B?eG41MzNIMWhIdkRWNnhQK2ZvcVllNGdoQTdocTZpVUlXdm9iaUVVcDAvckxw?= =?utf-8?B?d3FDbjB3Z1hGeXdmYmgvUEx6RktpU0xjc1p4akFDZDI4dUFCZFdrbE9DclRP?= =?utf-8?B?czhzeEVjVUVmbSsyRkpuZDZLSXFCVGp0eXBsRXFBZEcxUUhzVU01WUM0MVky?= =?utf-8?B?S0FnMEE4NnZJTU8vYXlQQVQrU2RlNzNaRVBlL2s3andnZmFFR2Zsc1l2Y2N4?= =?utf-8?B?ODJoOHhob0MxRXVKbzZQb0VXa0srclJpbEVLWS8yd3BCNjJ5M0RHWGJnajVo?= =?utf-8?B?SUd0QjJqaEIwbGFOcU4xbWt1VXBXbU1kQXVZN0xoc256T3M3eTdQRUpXd3NP?= =?utf-8?B?d3RBWW9mN0VoS21ENFNwWmJBRkxvUk4vN0hCM3RBc0FGYUNlSEYreWlkRTFp?= =?utf-8?B?clE9PQ==?= X-MS-Exchange-CrossTenant-Network-Message-Id: 385f0992-fc9b-4491-d50e-08da9d3bbe13 X-MS-Exchange-CrossTenant-AuthSource: MWHPR11MB1629.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Sep 2022 08:15:14.5823 (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: KQvltVdEtZuaXvTa7QuakesCYH67i71xFROeZVDoV24P492WoAI1jUSdvubvsVcEwg9Kdz1afoaFJZraUom9zsIQLqsyutuRDxB3EYQA8Cc= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB5328 X-OriginatorOrg: intel.com 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 On Fri, Sep 23, 2022 at 07:22:26AM +0000, Juraj Linkeš wrote: > > > > -----Original Message----- > > From: Stanislaw Kardach > > Sent: Thursday, September 22, 2022 4:32 PM > > To: Juraj Linkeš > > Cc: thomas@monjalon.net; david.marchand@redhat.com; > > ronan.randles@intel.com; Honnappa.Nagarahalli@arm.com; > > ohilyard@iol.unh.edu; lijuan.tu@intel.com; dev@dpdk.org > > Subject: Re: [PATCH v4 4/9] dts: add ssh pexpect library > > > > On Thu, Sep 22, 2022 at 09:41:40AM +0000, Juraj Linkeš wrote: > > > Hi Stanislaw, > > First a preface. I understand that the DTS rework is sponsored by someone and > > there may be people waiting with their labs for this job to be done. > > Everything that I'll write below is from a point of view of a developer who'd like > > to utilize DTS as a test suite for DPDK when adding support for new PMDs or > > architectures/boards. This might be in conflict with time-to-market metric at this > > point in time but I'm more focused on the state of DPDK+DTS in the long run. > > So feel free to disregard my comments if there are higher priorities. > > > > > > Neither of the current DTS maintainer nor me are the author of the code, so > > we can only speculate as to why certain parts were implemented the way they > > are. > > > > > > We've thought a lot about replacing pexpect with something else, such as > > Fabric. Fabric is supposedly faster, which is the biggest draw and instead of > > fixing/reworking pexpect code, it makes sense to switch our focus on Fabric. For > > this PoC version though, we'd like to stay with this pexpect code and work on > > other appoaches in the next release cycle. The code is well tested so there's not > > much point in poking in it if it's to be replaced. > > I have a nasty experience of code staying without re-factoring for long "because > > it works". When it comes to DTS my experience is that it works only if used > > exactly on the setups it was meant for. Adapting it to a new setup, new PMD or > > "even" running in the cloud shows that parts of it are held together with a string. > > I'm not blaming DTS devs here. Such approach is often needed for various > > reasons (usually time-to-market) and it's hard to be forward-compatible. > > > > That said I would suggest to use this opportunity to refactor DTS while it's still > > not merged. Otherwise we'll be left with code that we're uncertain why it works. > > That's not a quality-first approach and it'll bite us in the backside in the future. > > > > Let's do things right, not fast. > > Absolutely, but effective time use is also something to consider. Our current plan doesn't won't really have to contend with problems in the future, as we want to add the Farbic implementation in the next release cycle. I'm also working on refactoring the code a bit - I'm adding an abstraction that would allow us to easily replace the pexpect implementation with Fabric (with no impact on DTS behavior - the same APIs will need to be implemented). Also, we'll remove the pexpect implementation once Fabric is in place (unless we can think of a reason for pexpect to stay, in which case we'll need to refactor it). I think that instead of focusing on pexpect we could focus on making sure the replacement won't cause any issues. What do you think? > Personally, I would be very keen to get the move of DTS to the main repo underway, and so I wouldn't look to have too many massive changes required before we start seeing patches merged in. Basic code cleanup and refactoring is fine, but I would think that requiring massive changes like replacing expect with fabric may be too big an ask. After all, the rest of DPDK is moving on, meaning more and more DTS content is being added to the separate DTS repo every release, making the job bigger each time. :-( Tl;dr - I'm ok to leave fabric replacement for a release next year. /Bruce