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 5AE214368D; Wed, 6 Dec 2023 16:50:52 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 094DA402A7; Wed, 6 Dec 2023 16:50:52 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.24]) by mails.dpdk.org (Postfix) with ESMTP id C66F04021E for ; Wed, 6 Dec 2023 16:50:49 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1701877850; x=1733413850; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=oqCRmsgS7VJiuQcUVmOBLoiyZrnOQYCqdczlm86dFUY=; b=nEMhHEBnM8jRqfrr8ZCi7OkxAmqO2fVb1INFkUvx5I8V4CAw6Ayueg1r ywEnyzmbpqjSt+NB5wGizXVM1F0LlCATEzLiahf0eOKN9zhFVScdhBGc2 S2N6fZsW3J+AeAVNltnIYKbXYl+pVnNx1bsXUrs1lXTKmLq0qrdAdvAZE zhUkU9UagADRFeI5SpCZAp7o8iAwTNCsZovTFzUWykuO5fPn2NYhql+8p wA6afhmM3pqLzJMTyrWMudBoPvK3Q8Ow2Zpi2rHvwZde2gJrfpUFQnxeE QFreBgP6LTgWNVcTrVygKycS2aRpwnpl7llwWvVLHGXnUtnfjmO7Jna+t A==; X-IronPort-AV: E=McAfee;i="6600,9927,10916"; a="396868645" X-IronPort-AV: E=Sophos;i="6.04,255,1695711600"; d="scan'208";a="396868645" Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Dec 2023 07:50:48 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10916"; a="1102871610" X-IronPort-AV: E=Sophos;i="6.04,255,1695711600"; d="scan'208";a="1102871610" Received: from orsmsx602.amr.corp.intel.com ([10.22.229.15]) by fmsmga005.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 06 Dec 2023 07:50:47 -0800 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX602.amr.corp.intel.com (10.22.229.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Wed, 6 Dec 2023 07:50:43 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Wed, 6 Dec 2023 07:50:43 -0800 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (104.47.58.168) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Wed, 6 Dec 2023 07:50:42 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GNHEgzi9MOPjSL+iXFZl19Dwzxyo+WNV/FOGWsPxB0efj4smn3VWTCsU7aDtoq4W9ojgQ2/LSZxjypLSCoPikwNIrF2r3/krn1oe7H26G/vuAo5laHFiiLKFZRZMOV6TduEpXD5WaDPS/yAhcgGmowQ4qmJAgRVnEUw3sdrrQRUvk5MKckrodGupjBi7XAVJmA5w56MpyHqD0AFn4ywOXBi6Rp9u1/a5TALq0PrBTQqYtmjHWs+Nw5/cvv97ZY/uKb3Q6fTKk8gGLP/+AekqrOiM9FOhnIYvHgmRkXNvehUXVA59B6rVdYpBBaBWxYWftj/sblPRYXVjB7ghG3TLtg== 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=9wAwhHZhnHKC3o4YqwE4BySbd5/FIOsDyxLFV6QIJeU=; b=SnpsH3Ahzzrdd9ZyeKunuCRh1ypgCyN9cr2TscZvd08QwfuhykQKI7weQgSuRaP835iuAh2aZfQW4TvkK//NmR1Psw4ktVR8SsSRuKf81EJNvH5N/W0dVzpufXs1lr8iAE/VTd1jFJ6e9TnoVCtDiOuhnShg9OLQsqY3o07HpY84BOfGczi8PPIkI6vT2l2l8RS7y8k0P6LJvTtAgQZx20jqe8RoJ7GxaBgvye34NLAkYQQWxecALT+Qbpd2WxR9JxKk+fEgknkk/vxuj2kL3okI1sI2xl+iT6krPJJOLPYWIC0dAGN89d4imI1QWLfq03nHkBxhX6muVqL3HtjvmQ== 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 DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) by PH7PR11MB8456.namprd11.prod.outlook.com (2603:10b6:510:2fe::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7046.34; Wed, 6 Dec 2023 15:50:39 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::4782:d54a:209d:cb49]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::4782:d54a:209d:cb49%7]) with mapi id 15.20.7046.034; Wed, 6 Dec 2023 15:50:39 +0000 Date: Wed, 6 Dec 2023 15:50:31 +0000 From: Bruce Richardson To: "Varghese, Vipin" CC: "Dumitrescu, Cristian" , "dev@dpdk.org" , "Yigit, Ferruh" , "Parikh, Neerav" Subject: Re: [PATCH] cfgfile: increase value length Message-ID: References: <20231206112952.1588-1-vipin.varghese@amd.com> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: DUZPR01CA0041.eurprd01.prod.exchangelabs.com (2603:10a6:10:468::19) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|PH7PR11MB8456:EE_ X-MS-Office365-Filtering-Correlation-Id: a0a6e8a1-2b5b-4a86-0a6c-08dbf673184d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 3Zo9uomOG1YtLuh/qOx9ToAsnpHDz0GnrR9u1NtccWphULi1Owv46IFS9L85eFl3wjbCo9aZnO9USwzE9I6cVztzWr17r8vKC2JpJik7VEOie1VxMgYbAcsFp++2aEHQeWoIVn3KrQLvR2yPoLkTs+ozRqZqaBvjX1MstT93Yv44iH6AnfMfRRMExDSeoR/9UF7FDwYbv635gbzKE7/t4QmL5VrH7bbuT/Xz0jGkEtZK/UhO0ywUcLulN7TB+Wibj9GjO51Wx0hnWFz4XS2IZoYe+ZtcEEolQKsqHpz3FwnjQynAi0CusJRZc0rrh/Aj0/AiX93PWvLHZojF4l+Fsmr4CfTLlrPfiUCpM/fYHKTl+LRg/O0yilJBe/6iIV23Flw0CXbTQb5xeAu3yxT+dZsQBnpUmYMW1IOx1TBatFzWD3yYPqCM4M5HsvqYd9T2ZW8hU5xCOVlvbjEABYjcIYIifDlMbGzR2jahJWh7IiGes0EFgyon0wGGtXllDC5gvSn2Bv9yXKMLfFmuzv1FuV2BiBx5QP2PZeizE6dFB4Tv4e48qluKuSXTX+kmG6RjzhtaTCAHWaDkI1AVWkCXlg== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DS0PR11MB7309.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(366004)(396003)(39860400002)(376002)(136003)(346002)(230922051799003)(64100799003)(186009)(451199024)(1800799012)(26005)(6512007)(6506007)(8676002)(8936002)(4326008)(66946007)(316002)(66556008)(66476007)(6916009)(54906003)(82960400001)(478600001)(6486002)(6666004)(83380400001)(38100700002)(2906002)(41300700001)(5660300002)(86362001)(44832011); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?WEJh5WhKv75E2g26iqwxbIRRmfeO5ooS+OfcKP7LHzgXtuhLa/3gvUqiwdgs?= =?us-ascii?Q?FoZodl/5Dh7VZZsJzAawzibMm3if012pezuQ10Fp/vbFT/IimgSpWfZAZAJw?= =?us-ascii?Q?gifIh3R3qvwggrOETxmc7IfQs2rhPDd4K+4onsEWwi5WvcCzw/nrhYnqwn6a?= =?us-ascii?Q?7b45uvQxIyR5aAq38uXDZN0JfkW4cfEDB+t9YgfSEjadtVNCcHFQWbKU2WZx?= =?us-ascii?Q?LPjMk09OxRc6y9ac9rRv3PBLnWn3jGc2csFytEm2QfJhKpBmVSVAZ6KE2uCu?= =?us-ascii?Q?IXjtEMGoT3ejQQFb13nWXC5OQnJdl+GnjZO2rW0r9TPUEPXZEuM3T0gR0nwU?= =?us-ascii?Q?O346pnL8X6uJ3svtMkFXwkuB8m89SZNXj7nFE81YtMON9jE505/RtHqYBDVa?= =?us-ascii?Q?ETZBl9d3GZ/NlXDh9gp1hC4WSKp+093s0f6NO0Y7ZbG9RtooUzEoqSnt4OKP?= =?us-ascii?Q?MHXgDiZv0T77OPX79b4KFFBKAK/Jh2EnWWEEYwv07fT0UBx6yMBuqcOv6gPz?= =?us-ascii?Q?M07Q+9U4nLAKAkbluZWNQopRQATT67Ro0DVCe7neaI5X2v8dFHR8q2aHRKFj?= =?us-ascii?Q?j4r2sVyJD+5ySIn93Mu9+NYhQMB08bRQMGFuwDsPGd9slPP60qBcXwoJD/F3?= =?us-ascii?Q?KNRdRCaQxZ2GxUqbrR9E5dwUMyXTUVekhCN6oYS6yv3UVvvr7qfMPIi36J+5?= =?us-ascii?Q?9q4r24lb1nGyDNvkbEu362KDwu/ZeTuYGynn00FfgJ6DTlcU4xDDLZSv+0Ae?= =?us-ascii?Q?UgMQBHIofzICPwn9QDX6ymvhKJN4jTyLQz2s0bCH4KFJ9ix6to4lBSiQRsxg?= =?us-ascii?Q?lArfy1uXi7WocR97IEUij3MdAYdw5QaQswXBLV4RXQXwlIJqeaEY5gcDGtUR?= =?us-ascii?Q?Mk0fAezIMvGi7JEI36f/F5S2CdDGkZZT7L+Q7W2sbH/fzirVqhHvTCbcnDd0?= =?us-ascii?Q?9jscgOLpc4RHh8BQE6H06cl+SgEFUWvVSUVZU+L+0f1nW0uvdklRfk9/w23S?= =?us-ascii?Q?WT6p/oh0eoCdPEwh1DcCCC9JUqIpt2QwEhf9cadGNmOqaaf+hbpmkoeZtA6h?= =?us-ascii?Q?LSfkiCthlWNVeHCZ6kwGCGwtdh5p6bIatAODNak8fuDwl1rO392uqWvydli6?= =?us-ascii?Q?ln3Iv9BJLJD8v2oQxVWPkmhcNGXXdcslL3+buV2H8kpP0YQVMLU4LKuywfqm?= =?us-ascii?Q?om58l6h2U8ojuuSi3Mcq9nNA6E+JSScKqp6h8ZcPLjP+B2ErQqcoLVi8Oikm?= =?us-ascii?Q?z+91+vDiFl7k1tYYMqSkNSwdKAZ5Q0U7YMfBf/QLbqG5HwdsuDp7/24lqGqe?= =?us-ascii?Q?2Hz9qhRANpqbjk/VP9oYmmGY9SuiAkeCHLSIvdTlzhWbUvaPG9dCNppW68rx?= =?us-ascii?Q?+I7P0PcIlpUnMAR564gro0YXxdSQV8S9DbzebqHVdkSlkUgcOE2frjh9+vOJ?= =?us-ascii?Q?WPVvjWwl2lQqqQk09Jjql9cjSN4mdPVbMHVkSueQVz7kXFo6YxBbz3/wApC/?= =?us-ascii?Q?h5xoUWHHlNfUDjKK3sTeHg9RoJnkdCMZwPCDESTdS4/i2zX3OBpccEfadbMG?= =?us-ascii?Q?E6apgjAacYYZzqMuYDZd/agMhQvrMNuYnaJDnCkTJpyN+OOtx6zxLFzmuBHh?= =?us-ascii?Q?UQ=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: a0a6e8a1-2b5b-4a86-0a6c-08dbf673184d X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Dec 2023 15:50:39.4238 (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: Je69R4h/9gI6EhfnEO4ElgOqMOOf63SnszXhQ28wc3hT4qmGwSTq447QpGweh1JNLGcltRDutmEfHth7sJzbEeDKrgGxWMf2tU16y4oBPos= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB8456 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 Wed, Dec 06, 2023 at 03:22:41PM +0000, Varghese, Vipin wrote: > [AMD Official Use Only - General] > > Thanks Bruce & Cristian for the comments. > An increase seems ok to me, but is an 8x increase really necessary? If > lines in the config files are over 1k in size, then it implies that > some > other mechanism would surely be better for configuration. > Can we make do with an increase to 512 only? > VV> We encountered this issue > [1]https://bugs.dpdk.org/show_bug.cgi?id=1333 trying to use multiple > queue with DSA. But I hear you 256 to 2048 is big jump. > Happy to compromise with 1K. > VV> Sure let me update this is v2. > Since there is a ABI breakage, > [2]https://patchwork.dpdk.org/project/dpdk/patch/20231206112952.1588-1- > vipin.varghese@amd.com/ I will re work and share v2. Looking at the bugzilla, I still think even an increase to 1k is too much, and that we should look to adjust the config file format in some way to accomodate these longer line settings. Taking the specific example in the bug: * each entry in the line is wasting 5 characters with the string "lcore" * each PCI ID is starting with 0000, so we can do like EAL does and assume PCI domain is "0000" if just the BDF is given. This may involve changes to the PCI driver in question, since i assume these queues are names rather than addresses, but it would make them easier to address. If we did these two items, then: lcore_dma=lcore10@0000:00:04.2-q0,lcore12@0000:00:04.2-q1,lcore13@0000:00:04.2-q2,lcore14@0000:00:04.2-q3,lcore15@0000:00:04.2-q4,lcore16@0000:00:04.2-q5,lcore17@0000:00:04.2-q6,lcore18@0000:00:04.2-q7,lcore19@0000:00:04.4-q0,lcore20@0000:00:04.4-q1,lcore21@0000:00:04.4-q2,lcore22@0000:00:04.4-q3,lcore23@0000:00:04.4-q4,lcore24@0000:00:04.4-q5,lcore25@0000:00:04.4-q6,lcore26@0000:00:04.4-q7 becomes: lcore_dma=10@00:04.2-q0,12@00:04.2-q1,13@00:04.2-q2,14@00:04.2-q3,15@00:04.2-q4,16@00:04.2-q5,17@00:04.2-q6,18@00:04.2-q7,19@00:04.4-q0,20@00:04.4-q1,21@00:04.4-q2,22@00:04.4-q3,23@00:04.4-q4,24@00:04.4-q5,25@00:04.4-q6,26@00:04.4-q7 a reduction from 394 chars to 234 (10 chars per device). However, a better alternative may be to split up the lcore_dma item entirely, so that we always have one element per line: lcore_dma0=lcore10@0000:04.2-q0 lcore_dma1=lcore12@0000:04.2-q1 .... lcore_dma15=lcore26@0000:04.4-q7 What is especially good about using this is a solution is that it can be fully backward compatible and has no ABI issues. If an "lcore_dma=" line exists, we can split it, otherwise look for lcore_dma0, lcore_dma1 etc. As I stated above, the need for really long lines in a config file is a symptom of a more serious config problem, rather than an issue with the cfgfile lib itself. /Bruce