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 8CA6A42FB0; Tue, 1 Aug 2023 11:18:11 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6444940A7D; Tue, 1 Aug 2023 11:18:11 +0200 (CEST) Received: from mgamail.intel.com (unknown [134.134.136.24]) by mails.dpdk.org (Postfix) with ESMTP id 553F3400D5 for ; Tue, 1 Aug 2023 11:18:09 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1690881489; x=1722417489; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=pOaG/CltyB0hSXYcTvYg+w9rVit9hGXTBi/IWOzIIQ4=; b=VnzWqK2eyJlNCUdXCt7aVhLcisGXvN9MFdS9HKzFE8hI0s2vet5Ta/dK Ork1czoSAXzVq8+O8vYn+6K/xp2qxxT4DyUD/2giFoWDlTUpEpAbpV3xN owDX7kYacpoSpBbbNqdoksA+lr0pnqY5K6Xh6Xj++bIkd77w4uVQxdYyQ vXIi8D9EFSxoMZAsPUdoMEPvnXRqZNRLje2DMyb3q729BWqvmCqA5TQrt oNioxnSyvEw+zQcuYRd1Z02V0dVKL2ivlPKmmdav15J09CgaK1f3EeGK3 qnzmFAkZT3oqo0ZvPt8RhqVzEWbKgnC7XgpA4d6Cpd6TzLpds/ofILTkD A==; X-IronPort-AV: E=McAfee;i="6600,9927,10788"; a="371958930" X-IronPort-AV: E=Sophos;i="6.01,246,1684825200"; d="scan'208";a="371958930" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Aug 2023 02:18:08 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10788"; a="798594916" X-IronPort-AV: E=Sophos;i="6.01,246,1684825200"; d="scan'208";a="798594916" Received: from orsmsx602.amr.corp.intel.com ([10.22.229.15]) by fmsmga004.fm.intel.com with ESMTP; 01 Aug 2023 02:18:08 -0700 Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) 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.27; Tue, 1 Aug 2023 02:18:07 -0700 Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) by ORSMSX612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Tue, 1 Aug 2023 02:18:07 -0700 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27 via Frontend Transport; Tue, 1 Aug 2023 02:18:07 -0700 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (104.47.66.47) 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.27; Tue, 1 Aug 2023 02:18:05 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=daY2UtPTufI2YTGZrnZ+pEqcZ6MIdSebSbgtrp25FVFfHFY6nxOnaHxC83y0Um9WCA0euT9ZGpiBRJfXIFJ8aE1LDqDq5TNauFm32Aq6C1nXZGg/htIrTFNtV/ls9ig4FehGyMm1JSnaNWyPoKMwOoYF/ZTo4PE6Lux3uI6FbSvbmXxURv5GUJF+wiiTzYM3H83ebsQzqwbdnGgByVKqFeTntVSDpgXZlyXrgzxjWj/8qF3WwcBe3GMZFxF7cxSs75fCYYJ5/uFh4lF7QqMJpu0SXfuy01V5JXRFbrVoiF3efTQYC9lNm/veGzQO0Mln8V6VwmjdGLtHfnvUUz3zMw== 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=z1Q8rlr8H6JAMo/i2154n+/ykSuJavSBNpkvYUBLPOw=; b=AeALqoNCknRUkJp0cy739IJmKIGiKrTtMgg00phYy3/VgTuOCSBmCjO07ocL7+6jZTC0wWmeANaRQ6G2RwDDm+7aBYKejIvpr32kKwNxlMGkrJ5u32WwUs2Qce5qJjYPqdc9m0dHUm1+42EWI7QDm47zLJE/KhtZWXYy0/bbyP56aeEihuk9vP33yxb5zYcbH/WDU2OIq8CyiqwRIgjp/oVOQlOoyepDe2L0cC23Ybu6viDHywtz+EmaRkbx7oqyB7LrFnCuI960EYmTqkaArISEMrkrxJLS8OzKXxvnhNI/qSV+czu6He6hKT/EveZdssIGRh2O23n8qd88ABBL3w== 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 IA0PR11MB7694.namprd11.prod.outlook.com (2603:10b6:208:409::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6631.43; Tue, 1 Aug 2023 09:18:04 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::9802:65f0:c441:780f]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::9802:65f0:c441:780f%7]) with mapi id 15.20.6631.043; Tue, 1 Aug 2023 09:18:04 +0000 Date: Tue, 1 Aug 2023 10:17:57 +0100 From: Bruce Richardson To: David Marchand CC: , Timothy McDaniel Subject: Re: [PATCH v4 2/9] event/dlb2: skip configuration if no eventdev lib Message-ID: References: <20230622134840.3225975-1-bruce.richardson@intel.com> <20230623150708.2203918-1-bruce.richardson@intel.com> <20230623150708.2203918-3-bruce.richardson@intel.com> Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-ClientProxiedBy: DB8PR06CA0027.eurprd06.prod.outlook.com (2603:10a6:10:100::40) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|IA0PR11MB7694:EE_ X-MS-Office365-Filtering-Correlation-Id: c50cd996-86d5-4358-30d2-08db92703582 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: +IZLCreDDv0zVe+eEB9YbYAUwOjI+KazKQxzS+crtOPwvnK1RiAuvra1OQ9qjFKfhu1jlA2C6A8OZYezyBExn/ZHQ70uMBy6d01IRrRIITtRaDqd9gF3YlZPeo6tU8TKYkzKbZe7PqqLP6daafxfloz2LPe19JVvYQf8kTtItRB8pZ+UUARB/dqpl7Q6qNfX/svHPFYln0VjJro59O38rWLGSFNBFAczf5/PemIQu/ou28n18HBmFLhD4otOPCZc4l0pk+sljVB5KYjOD5ce+gNlOQpxNUU/h/vHWi0qq2A6O5QYvZdM/gBbuJwkdE9S2aZe39rn8mmqUpn5wbIq214udm9UFCuAJsc24GRJLxMevlvsggBd1buRXNICNfo4CSgTJWB1kFoCtdHYGINdXiDe/xxvWxFpeRI+O/xNCR9cp1f6fCNkB7UyHYW9l/IPi1WDjJ9K3z5vuq67kRf0QzUuJRCF62ixpMTqPTF+GPPA5rrKrdxK1IYeyrMHz51lJwOsZgdznVCCn0NJg+EO0y7FqeNe+Nhnlgtm5de1h/M= 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:(13230028)(376002)(366004)(346002)(136003)(39860400002)(396003)(451199021)(38100700002)(82960400001)(86362001)(6512007)(966005)(478600001)(6666004)(6486002)(186003)(107886003)(53546011)(26005)(8676002)(8936002)(6506007)(44832011)(5660300002)(66556008)(66946007)(66476007)(6916009)(4326008)(41300700001)(316002)(2906002)(83380400001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?akZvVmsxNWNZUGNjMFFwYzdveHFuUFNvbVRkKy9QbFk2VDVabWdnM3AycTBS?= =?utf-8?B?L3hSaE1nRndEeDR6VkRBMEpvL055a0I3elQrT3dVL2V1dUtqVFhReHh4cVBC?= =?utf-8?B?dnJnWXRaRTBzdURZanJNY2VqSXh5eUg1K2c3bUZsOUMzL2J6S1BqdzYxRC92?= =?utf-8?B?UnFSRStnbElWS091eTZwRmdTMUx5d2lDQkNDaWFDMnFsbjNLOEFGSHlERUxW?= =?utf-8?B?MWlqMlhmK0FxeFkvYWFqdmpyWjFQZE52TjBPdkVlVFBmeW1lU0ttQVBTeUg2?= =?utf-8?B?cENWcTRYYTluZjlSeGZHZVRNRUVYVVF4V2ZqNnBFSnVMSHMzanJNeUdJbGRu?= =?utf-8?B?Uy9LNEx3cm5KMzRONmV6TDhSWEd5TEdEN0s1NXhQUEo4YlRFTDZTWHBkVkRi?= =?utf-8?B?aTFSQ0xEby9LUkxaRUtyS0Z4d1JEaDQwZFN6TEdidmkrKzg0bm9OdXF4MGs2?= =?utf-8?B?d0R4OTdPR2VBUnJabnJmS0JoU2hwVnJETWdDWUhPdGdRQmFSU2RicmorVkxm?= =?utf-8?B?NW9TM0huUWlYeEJkazZjenNtWENVVEtscFpqdDNnNDFIRFA5d2NiZ2MrbFpM?= =?utf-8?B?OFVZMitmL0pMbW8ydUxxekRLNWhYNE93VXk3ZXZGZXp6eTNLMk5Xc0Fjd2Ux?= =?utf-8?B?MHFCMHN6L0hBN0h2bEFCMmtXZnNyZ0xkb0FTWjNJallMZ3dtUjl5R2ROMStm?= =?utf-8?B?MlNGUERESENseVNEVTVhek82ek1VVzZCb1lYQ0d1cVZiTmVaZWNRKzRLQlBV?= =?utf-8?B?eitQc2lobWh4QjNoMUw5RVR5L0txUmpjS3o4ZkhSUUFEUUNtNzgwRFROTWph?= =?utf-8?B?WGJqR2ZjRXI3VGZBcnJ0QVBKL1dsL1RzaGtLNGlRdWNaYWRRS0RNT1JDZXdm?= =?utf-8?B?MXdQRTdkS1ZPWTdCV01FOWRaQ0JSYzE2OENVQzZFSCtQZHgyOFIrYlN0eVFC?= =?utf-8?B?ME13UWl4dWJ6M0xrY0MvY2loejZHTmpzNzQvVDNuUHdvREtvRWNIQnl4VkN4?= =?utf-8?B?TEhVYndGV0ZjSUFJRHVHRll5NytFTnpiWTRPb3VtYzY2dUg2Z1pDMjRrNlRY?= =?utf-8?B?Njk5T3pPSDVydGwybk9DeWdsQlIzZFJiK2l0Nk5Qd2JqZlhhbHpxREJOU2Vt?= =?utf-8?B?VHMyTTNZTXJEdXAwc1ZkVGJldmRvQkphR3l5VS9FYkNPUFBGY093S0V6Z1Rj?= =?utf-8?B?MVVOcERNUmQ4YnNvUHpQc2E0b2M0d3MyOUJaQ0VHNmlzVTJQWXZPYUlmZnh6?= =?utf-8?B?bThlZDdWWEFMU3NIWTRRdERtZTVjeHo3dUw1eFhOR3hYVmJEU0lIL0V1RTRW?= =?utf-8?B?czh0NzdoY2dFN2NwNEtzb2VXUWJCa2VzR0xkSndpcGh4TjV5TFNKNWRhdTVV?= =?utf-8?B?OGtXSUZFbVdUc2ZVd3M2RGY0NnRJbzE5c00yTXRQMDJLbk9oRFpBSko5UFhF?= =?utf-8?B?bEJMQS9odVBOanVOYS82WnFQcDVza1dRNmJpdFcxQW5WWk5KRjlpVFdXa1g0?= =?utf-8?B?M2JDakp4TnRtQWhBZEd3Y1JNREFCd25BeWJaaVU5TTA2SWpqaDlGR2VEK3Jo?= =?utf-8?B?L2JvdGFlRHM3SFhod3EyVHd1b1BRZTlBNStCMmJqaVZwb2R1WHlYS1pTUENG?= =?utf-8?B?VUFVTmFlTXpmRURuTkdGVHpVVERLaG95aGl4NElic3c5NitwY3VKNEZoL2ZR?= =?utf-8?B?d0V5eW92MEFybDFNMHdLeTdmMlFLLzVwY1BIajFRdk5LQVMwUzFidE11dEw1?= =?utf-8?B?b04wVDE3R2NWSnJNQlNIRGN3NVZGeFVLL0NjUkVBOFFNUmI5eG54ZXBlcHRv?= =?utf-8?B?YnBEbnNMTE1yKzF3TE1mb1BJTGh0V210bXZYc0Q2UjgzN2FvYmE3Q1ZaRXpI?= =?utf-8?B?a3d0ZXVhR2JpMG1HZkhFV0l0NTFXR29sN3duK0pwc2VNb1RpbVllbHQ3TkFz?= =?utf-8?B?YS9ESVRBbG9iZ01malhvb24ycm9FczZ6UjJrbG8zVnlJOGM0WkVGdU1OYmdj?= =?utf-8?B?UElCOGsra3lLSllYbHZnVDh4UjN6aHdmYkVYSlZNNjNLT0JXTHo3UFVySy9Q?= =?utf-8?B?Z0dDVCtVOFErWlJudG43N0cwUENXY1k2NU5xY29pVVRDV2xVS2Y0R05aRTJS?= =?utf-8?B?K1ZUUG9FUU5KVk9mNVdXcGFiWXlrRDZxQ3dNd1lFR0NiU1N6L1A2UUpFckhP?= =?utf-8?B?TUE9PQ==?= X-MS-Exchange-CrossTenant-Network-Message-Id: c50cd996-86d5-4358-30d2-08db92703582 X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Aug 2023 09:18:03.6963 (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: tsvyC5xZgS7gN/cSzRbpDmQe745hz2s/s1fvZQfQp9ISv0y4C/jaIgqZjSb1BYKiI29GckrB2Gx1cheZYNJfDA1JuHEETkFmKoE7lE9ejS4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR11MB7694 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 Tue, Aug 01, 2023 at 10:36:30AM +0200, David Marchand wrote: > Hello Bruce, > > On Tue, Jul 18, 2023 at 11:10 AM Bruce Richardson > wrote: > > > > On Thu, Jun 29, 2023 at 11:39:53AM +0200, David Marchand wrote: > > > On Wed, Jun 28, 2023 at 12:19 PM David Marchand > > > wrote: > > > > > > > > On Fri, Jun 23, 2023 at 5:07 PM Bruce Richardson > > > > wrote: > > > > > > > > > > While the build system will skip building most libs and drivers when a > > > > > dependency is missing for a component, for DLB2 driver, the > > > > > "static_rte_eventdev" object is referenced inside the meson.build file > > > > > itself, which will cause crashes if it doesn't exist i.e. if eventdev is > > > > > disabled. Prevent this issue by skipping processing the file if no > > > > > eventdev. [The build system will still report missing dependency, as the > > > > > dependency is set by default for all eventdev drivers] > > > > > > > > > > Signed-off-by: Bruce Richardson > > > > > > > > Could we evaluate the class "std_deps" before jumping to each driver > > > > meson.build? > > > > > > Hum, with my suggestion, we lose the opportunity for drivers to > > > rewrite completely their "deps". > > > I doubt we have cases where it really matters, but if this revealed to > > > be necessary, such driver may be directly referenced in > > > drivers/meson.build like we do for common/cnxk & friends. > > > > > > To illustrate the idea, I pushed your series along patches of mine > > > (target is v23.11) in my github repo. > > > https://github.com/david-marchand/dpdk/commit/enable_libs~8 > > > > > I don't think we should need to worry too much about the case of a driver > > needing to rewrite deps. The standard deps should really just be the > > minimal deps for a class. > > > > However, for the implementation of that check, I'm not sure I like the > > approach of checking std_deps in a loop for each driver. Firstly, the logic > > inside the per-driver block is already pretty complicated, but secondly, I > > think we can skip that loop entirely if the standard deps for a class are > > not met. I think instead it would be better to process std_deps value once > > immediately after the "subdir(class)" line, and if not all deps are > > present, just loop through all the driver directories for that class > > immediately and mark them as unbuildable, before just moving on to the next > > class. > > Skipping this loop makes the change smaller and the logic in > drivers/meson.build easier to understand. > We lose the exhaustive summary for disabled content (before, every > disabled driver was logged with the associated reason), but we can > still log the reason why a class is disabled. > When we "subdir" into the driver class directory to get the standard dependencies, we also get the list of drivers of that class. We can then loop through those drivers to individually mark them as disabled for the summary. It would be a fairly small loop, and would keep things consistent with what we have now. On the other hand, having a single line for a whole class of drivers may make the summary more useful! /Bruce