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 BDA4FA0C4D; Mon, 4 Oct 2021 13:50:47 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 616A2410DB; Mon, 4 Oct 2021 13:50:47 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id D4280410D7 for ; Mon, 4 Oct 2021 13:50:45 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10126"; a="225713103" X-IronPort-AV: E=Sophos;i="5.85,345,1624345200"; d="scan'208";a="225713103" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Oct 2021 04:50:44 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,345,1624345200"; d="scan'208";a="567015018" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by fmsmga002.fm.intel.com with ESMTP; 04 Oct 2021 04:50:43 -0700 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Mon, 4 Oct 2021 04:50:43 -0700 Received: from fmsmsx602.amr.corp.intel.com (10.18.126.82) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Mon, 4 Oct 2021 04:50:43 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Mon, 4 Oct 2021 04:50:43 -0700 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (104.47.56.171) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Mon, 4 Oct 2021 04:50:43 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kIAoGOXbHuddHerxlacR2506xd/F6M65mLPTrZx4g4E0veeD3rTMR05agj0jJi5pfGcxbWUU50fH6MwuY8FgWe+BN2FxPLhd1fJV28iHXkDdjfRN1+fA37ItGiD0WfKRrMckIAuTtXFnGxJxD5NAiwt7ikbi2TNyQ1mc69uxl8OnmAPayrIBK/Lbju2rVR6I8CwGMZgS9P+u1t0DD/EK5VJBaV1ELbZIajkmQD+FNK1BnlSO8xeInB0iy+zcTywmjunJ67QCiNPoG834KmQOe2QY+16wTQuktELWhMR12rXIzPUIUvo5bldhNAFRR9SwF40gQqecmGqVIJWPZDqARA== 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=PJPavYOhufNYQDyx5gZ2kGO5MRLOwAKMi7pmWuumpsM=; b=eYNLTESa2NpzfyXHjmwYgPAva18UGxBN9nSwyVF/LppQHaiaQWj2QcgOv7ocMvPTRLFJNWrGntme3z+t7H/E5puWlKVRxMkDEJK88yED49NML5ow1Fb0oZyuzIuICDtc3au/ncFaMn4umA89Dxvs/Su5RlRP8lel3nklg295LCgc/WaWsC8O5RdrNKTUMLVFqOrkiHpZS/J5a1HE2LUuWE/4wPA0rfLf2xMCfeLDjDG2hXW6bCcU4HuTFx21h4MjFCWvEwD49gHrXPghj8dEzQZUKDRziMIKHt3pGNCHUcuSlWXvHSPPpa7A34KvOsVVKObXSTkAKYHVZF15evq5AA== 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 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=PJPavYOhufNYQDyx5gZ2kGO5MRLOwAKMi7pmWuumpsM=; b=K0lKwHkiCpNGgwT1qiMMTvDf7y6YZ2Nt40tDSW4FpX+NjcXPy9EHGBAF2Jm0sPxH8Axs+A3j6oiBQhAIolm3Pw2bRPVZs/kS4FiQOyWy4KGwtIv9jgnup8gAbr8EQdQwbaeBHpaDWVgldasY4umV03H6JZs3F0gO3bsUVZvustA= Authentication-Results: dpdk.org; dkim=none (message not signed) header.d=none;dpdk.org; dmarc=none action=none header.from=intel.com; Received: from PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) by PH0PR11MB5014.namprd11.prod.outlook.com (2603:10b6:510:31::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4566.14; Mon, 4 Oct 2021 11:50:42 +0000 Received: from PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::747b:3a08:d1ec:31fc]) by PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::747b:3a08:d1ec:31fc%4]) with mapi id 15.20.4566.022; Mon, 4 Oct 2021 11:50:42 +0000 To: Pavan Nikhilesh Bhagavatula , "Jerin Jacob Kollanukkaran" , "Maciej Czekaj [C]" CC: "dev@dpdk.org" References: From: Ferruh Yigit X-User: ferruhy Message-ID: <625c0729-7716-471d-e563-faa42fef08be@intel.com> Date: Mon, 4 Oct 2021 12:50:36 +0100 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit X-ClientProxiedBy: DB6PR07CA0065.eurprd07.prod.outlook.com (2603:10a6:6:2a::27) To PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) MIME-Version: 1.0 Received: from [192.168.0.206] (37.228.236.146) by DB6PR07CA0065.eurprd07.prod.outlook.com (2603:10a6:6:2a::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4587.9 via Frontend Transport; Mon, 4 Oct 2021 11:50:41 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f87cd0f2-92be-4f79-afc0-08d9872d3190 X-MS-TrafficTypeDiagnostic: PH0PR11MB5014: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7691; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: SyD9efdX/5w4YlUzW9rwcL/twPsruqHnf5WbHbcvN/po1lWHlOqbpyyXb0R3NajJ8iREUPcGRTBuVpeTaNRQqGWE8FHtr6t4yhmwi/ezfpfyKYgGArSBJMbA2bQhMlHHibPUgrNd9hYHh09wvDunRwIsJ1AwK3ktG0iRQh1lp672f94kM0sXs9aNgLdImXu8OpzomZCbSFh1qGa84JNAaRbqcQmKkjWKo6eOlWxqQ3LqSU313axD7UkzDQYn9wf+zlBUDTifjn+FWr/nhBnDvZmHStGhy2NqpUyQsj5znhh4GG57cgMwmWVvrggemYvlWJ2j2cXrZPZRtufyxbtr2lPYw2An4gzX36KQe4IIsyhw+GBCIw1ejSd+U3fqR4F17cU4qdKl0fo5l0yNjGIFoRH0D13rvensUkexNQPnZNHQuAYmegZxd3CtMQHEDgGJIkiHcL7bBaenR9w/T9QnUgGhRL8qkEsjoVZpDiPl0wB9trTkKOf/47E3j5r5x8oYKHKw5FnS7oPN1gZiD0BewC++OR6NMGxZ64xxANXUcog6uN3KjaWPuyeeIibJEd3uyqThNe4qsukunKReQ4MwOVokHauha8KZ3n77BiqgHsi6GqvmxefQLHnuGX0k6X5hFdPvhgvw3mBmT70RD/jQP1G1uK5OtPgplB6OJtBqM6p3Mp4fRVTJntjWGoHJsuFXWV+0zmXKqh5AbVYYf4Dnkw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5000.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(2616005)(53546011)(5660300002)(186003)(110136005)(26005)(6486002)(8936002)(83380400001)(44832011)(956004)(4326008)(16576012)(38100700002)(31686004)(316002)(36756003)(31696002)(508600001)(86362001)(8676002)(6666004)(66556008)(2906002)(66476007)(66946007)(45980500001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QjBZU3ZWYkJMT3huMWpuZno4SUZJMER0OXhNT0c5N0N1R0g3ZzlOL3BZSzRv?= =?utf-8?B?V2hQZ0hNQXFUQ2U0cktnL29LbHNyMjR6eUUxR3BwbE9sUFVMYXExWGhHTklK?= =?utf-8?B?NmZDNmdLdEhIYXB5Vlh5aU1wbE5KbzdLT05tdWo3TnB3dFo1RVZ1ZE5kcVE0?= =?utf-8?B?WVdXVHFQeWdjZVFBdmRYYUJXMzh3ZjVzeGJ1STJ4VlZUVHlsclZUbHM1SGlI?= =?utf-8?B?b3dTc043OWlxSnNHS3huZXRUek5oTHlrT0l1UHZ4aUc0Nk53NXBhVzNueGdF?= =?utf-8?B?aTFWWGhlYm9vQnZwV0wvNi8rWG9iSk5xNS9kbm5ZZ1lQamN3RGRvc2RTdENJ?= =?utf-8?B?c052MjlDZHgvQ0xyUDdLNlVyUVR1WVB6VTI2dmZqM2RqVTUzWEZ0czJFZUQ5?= =?utf-8?B?OFZKR3dBMHdiVDgwa0EwVnJLaGsxYzM1amVCcndFNEhmdUVmeTNwdi9TR29J?= =?utf-8?B?LzBsM3ZFY0lJMURUZGJLRzhUK1AyUVgwM2FkR0QyU0VEc0xFSlp1WjhUL0xP?= =?utf-8?B?aGJBMXA2RkdUazNzNDNRWGVsemYzcjVMZWEwS05yRDZYUGU2YjBTbzJCV2l6?= =?utf-8?B?OWZrbExHUW5hQVVTVHpvSWZzQnpnN0x6YXJueldGRnA2RU5yNk9pa1lHRkpr?= =?utf-8?B?YlJKd3VBWndNeDdZQk5sZ2pEamtUL3dLZW1aMEtJR0RoalhFVlQxTnhCT3dr?= =?utf-8?B?V0YxeUw1RWV2UmE2M2owYUQ0TWxvZmJ2akdtd2V2Y3AvaTcwdVVlMjRvTWdn?= =?utf-8?B?YlE2blhDaHdCWXY1T0ZRNlhhb0NqaE8zbmp6T1FUVHBuODYxR0gwbUlVNTd3?= =?utf-8?B?Q2JpTTN1d284VkNyb2ZYeVh5cDRwZ2hVNDdCRHh3T1BVNkVmTXhuRGtvYUpt?= =?utf-8?B?S3FKQXl3WTZwN0pEK0kwM1VncE5mQTFaWHNuZWtsWUhkUTVHb0h1bmhmZ1d2?= =?utf-8?B?ZGduUi95eXVjUy83MG8veVUrUTdPL3BxRGJVdDRKemtxOTgzalhReVQ4elBN?= =?utf-8?B?WTIzSVRDSENabUJPZWZ1a2VoZzZSdFVRTHpid0RrUmFCNkZhS1FyOWpEUldw?= =?utf-8?B?dVVxUU4yVnkxUFdzR1pzdmRrb3BQNDFQUmltNHBLc1NFN3IwTVFEVUhxbWtz?= =?utf-8?B?S0NvUStwcmFtVjZvZjNHODVpbzVnYy9UeVV6UmRxSWtqREpLazIyNGk4ZVhB?= =?utf-8?B?cHhSbUJUclVJZWlpNVdYRUMrMDJRTndTZ2ZQUWNEWjJLK2ptR2hXZVZPeDdq?= =?utf-8?B?UUQweGtkN3FXWW9XdUFrUnVEUDFjcnFFa0U4eVQ5aU1ack50SmVOb210UVFS?= =?utf-8?B?RFZtTjkyVnFHWldEc0hXNmZycW5MT0JFZFBSUy9TcXd5b0NlWW1wRXRudzk0?= =?utf-8?B?NEx6OXREYUMrOXJEUXNBMkhIRFE0R3N4RlQzc0NlQmVYUjFIdUY4NXlkVm5J?= =?utf-8?B?MVBPVU4zb3NBUFV4NUVpL202dzUzb0p3QXJyVDBzdERrNEVPRms0WmRtMHBS?= =?utf-8?B?bDgvU3hKdVdaeFpPeG9GTExhSndHaUs3aTdZVU5Lc0RURFJzeXZjeXo4Nmh6?= =?utf-8?B?aXBSQUh3SkYxZU9vM3d5Nnp3Tk8zYXc0WUs4WFEvdG1sNmtSM3hmUVR2RjVx?= =?utf-8?B?SnVldm82REpaZFpkc1pYNVhMam16WGlmbG9IVDMzTFBqZFMyMmdjb1BrdHBy?= =?utf-8?B?YUlhSVdJT3lCRkdhemNKU3pvaEhqT1psM0luR21QQmErU2tWazFQejRuaW5l?= =?utf-8?Q?6+oLPP7vZQt6GJIOEvrQQilhV4koFFBoJAT9IYE?= X-MS-Exchange-CrossTenant-Network-Message-Id: f87cd0f2-92be-4f79-afc0-08d9872d3190 X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5000.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Oct 2021 11:50:42.5714 (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: PTBItW876lyD+3+a2bbNzZyFalBA9f/HMvEjzwMD2bGBE2VMjcXOBoxPf4Ire3LPVdUyuZeIq0z7L8XAJNAMdQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5014 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [PATCH v5 1/3] net/thunderx: enable build only on 64-bit Linux 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 Sender: "dev" On 10/4/2021 12:34 PM, Pavan Nikhilesh Bhagavatula wrote: >> On 10/4/2021 11:02 AM, Pavan Nikhilesh Bhagavatula wrote: >>>> On 10/4/2021 6:56 AM, pbhagavatula@marvell.com wrote: >>>>> From: Pavan Nikhilesh >>>>> >>>>> Due to Linux kernel AF(Admin function) driver dependency, only >>>> enable >>>>> build for 64-bit Linux. >>>>> >>>> >>>> Hi Pavan, >>>> >>>> Isn't it possible to provide a commit log in the kernel side etc, that let >>>> others to verify why only 64 bit is required, or if someone want to >>>> support >>>> 32bit that may help them to investigate the source of the restriction. >>> >>> Arch 32 support is not implemented on ThunderX, so 32bit will not >> run. >>> >> >> I see, is following correct: >> All thunderx, octeonx & octeontx2 only supports VF in the DPDK, and PF >> is >> supported by Linux kernel driver. And Linux kernel driver doesn't >> support arch32. > > AF != PF, AF is something that manages all the shared resources between PF/VF. > I see, I though AF is part of PF functionality. Are there two different kernel modules for PF and AF? So can DPDK driver drive PF? In a way, PF by DPDK, VF by DPDK, AF by Linux kernel driver. >> >> Is something changed in kernel driver side to drop the 32bit support? >> If it was not supported at all, what is the motivation to disable the DPDK >> drivers now? >> > > It was never supported to begin with, motivation is that build will fail if we try to > compile with 32b. > If there is no plan to support 32bit in the kernel side, that is reasonable to disable 32bit build, please provide above details in the commit log. And after above said, how much maintenance cost to support 32bit, if the build error is on the logging format "%lx" etc .. (as we mostly have 32bit build errors), it is better to fix them using 'PRIx64' which is more proper way anyway. If there is more logical issue with 32bit pointers, I agree with you to disable it. Can you please provided the build error in the commit log as record? >>>> >>>>> Signed-off-by: Pavan Nikhilesh >>>>> Acked-by: Jerin Jacob >>>>> --- >>>>> v5 Changes >>>>> - s/fuction/function. >>>>> >>>>> v4 Changes: >>>>> - Update commit message regarding dependency on AF driver. >>>>> >>>>> drivers/net/thunderx/meson.build | 4 ++-- >>>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>>>> >>>>> diff --git a/drivers/net/thunderx/meson.build >>>> b/drivers/net/thunderx/meson.build >>>>> index 4bbcea7f93..da665bd76f 100644 >>>>> --- a/drivers/net/thunderx/meson.build >>>>> +++ b/drivers/net/thunderx/meson.build >>>>> @@ -1,9 +1,9 @@ >>>>> # SPDX-License-Identifier: BSD-3-Clause >>>>> # Copyright(c) 2017 Cavium, Inc >>>>> >>>>> -if is_windows >>>>> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64') >>>>> build = false >>>>> - reason = 'not supported on Windows' >>>>> + reason = 'only supported on 64-bit Linux' >>>>> subdir_done() >>>>> endif >>>>> >>>>> -- >>>>> 2.17.1 >>>>> >>> >