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 72F9D41CA8; Wed, 15 Feb 2023 19:28:54 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5397B410DD; Wed, 15 Feb 2023 19:28:54 +0100 (CET) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2079.outbound.protection.outlook.com [40.107.223.79]) by mails.dpdk.org (Postfix) with ESMTP id 3861F40A7D; Wed, 15 Feb 2023 19:28:52 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=L6pAFxqzFMnqTNVVnu12v2PWd3jZqWkC2e2WmBTeEVCQnJ3fiEaf5EnZH8opelA8w4atHdOu+Zd3CkJBMTst1MPvsaxhN4Vdv2ZdA6WjH8zOgy4bLUP843zKKWHgWfaVfkzIvBQ8T+9AyE2MKYMIlH1l1uClAQAfuHnJxJtQeeF6mIL2oWiGR/VoPZzyldNuPmsfjhxzlenyJs/5LcZQbHOp/mh/jGXzM8r4Brgxk27hKVcLfX5q8h+AScQSEKgUR+Nep8TaxDYlL6hAlMNghC1jPC6h1T1nSib8OH9ap3ZSP/TyA2ayONKS6ffWKEMymNWWh1GmAgKZvkrZHM3UsQ== 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=5mesGoNmL2sUy4NMEezHEljpplQ+uj4KODgcrz4Pgkk=; b=DC0EGutU2tpF7Z7oAKFOoPQlTsLacB/x+xSxRz7TPJr7vgz2AlbiADEmO2iRexWyj73kMzBJ4bIR3RhwIg/5UZEqY8JyOfR+XFaKwTJgh1T0hGa+/AHU+Wio96A+TJbB8BMC19heZhTs9IQk1EYHas/aJpvH1ylOyzPLNtxgwTDxJB7iAaaSh7jKhB9mJcHKzYzVmAOpm5xX/TA0sOXCW8bs+VsqouBFmLQ2v2jnwTbwTr0udz06sYQv53YP/tQ9h4A/zZZTgz5RRj+S68RsDpAi3TEJ+cEU4qtjxETUfINb/qwAY/VKvsGFiWC6Ojzj9T1XQ7hm+eXn1xxQvRF4NQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5mesGoNmL2sUy4NMEezHEljpplQ+uj4KODgcrz4Pgkk=; b=Yd2PaVKNND5TdU7RcTjlGVuk5aFdtS9dbXGmVBBQ6qPM4RHDjxNd9C6Sig17lFXnZSElYbL1g4Z5MfO51TaxQjXUtf42jY351k/dhXkliWLrs5OZTxVTgpKfwM3XwrnWI3F4HVAHS8UqAi3Ca/BRrShMhmcBDlrtVpEjh5tpZnQ= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) by PH7PR12MB6883.namprd12.prod.outlook.com (2603:10b6:510:1b9::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6086.26; Wed, 15 Feb 2023 18:28:50 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::3614:22ed:ed5:5b48]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::3614:22ed:ed5:5b48%8]) with mapi id 15.20.6086.026; Wed, 15 Feb 2023 18:28:50 +0000 Message-ID: <1856a555-e7d5-6dfa-6cf0-c4e170b81482@amd.com> Date: Wed, 15 Feb 2023 18:28:44 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.7.2 Content-Language: en-US To: =?UTF-8?Q?Niklas_S=c3=b6derlund?= Cc: Chaoyong He , dev@dpdk.org, Luca Boccassi , Kevin Traynor , oss-drivers@corigine.com, Peng Zhang , jin.liu@corigine.com, stable@dpdk.org References: <20230208091544.22122-1-chaoyong.he@corigine.com> From: Ferruh Yigit Subject: Re: [PATCH] net/nfp: support 48-bit DMA address for firmware with NFDk In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: LO4P265CA0140.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:2c4::13) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|PH7PR12MB6883:EE_ X-MS-Office365-Filtering-Correlation-Id: 239f4ffa-579a-421e-aed0-08db0f827b98 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: x1kxoHTjMBLBOwLwASpZdWpurdQe0JSpPi4tuCdFLBR6FwbGO5YraLyLnyMofBOTv1pFE1MesZaM6+Wxbl8dEJhs8Wd/jcja1chXzgSROfSvn8wN2qbMD7067yL1xTZS0qkndofM5YP/9DUmSRYl/QcsysfNXK3G5+ybxdKuFOKdT2RoITXv8nPTihrCjckHkSW5tzXn1ysYsxwOpwqk9nFlwopX1EJBCBrYdyh6trSP76qaP0yBdlQcDoWq6Xr3+ZhiRuALg8Od+F9LBWBleTlOEjvr8Igopyh+MVZmrBuP+Vr4IMFAyMb8m/F7QuDIl3f7ZbSxp48oOdq8B+Xb8ZRJKW0nEeMVGG8AvBBrql+EzR0fhfN6mm0BG7kIer0BhHeJuALMfcJZEKVMrT+Oax91U694WgLtfkTXG8EIzTYtBu892dSdiCmAuNqS3VgvFQzggZOu428t5CTFzKERb4e+Qnnxd209hRjNW+sGKamZuOrcSFN7fzZAeUCDDuyIp/yM0AYKU7dtnHx2DMK/jysPobMAMm5TNUICHsRyq9vZ+9fQh75e0orby2lMTv8PfKDgaY7m9L6RbGJnFOY1VASWu9i7bHd8/JXc4CU32m+c/trGYevOnYGm82dqa0igpXLS+lKyVzBcOghXCIy0plQxxkUZFJocTXO0G8OZIwSNnVDZ1V+yrkDVwCgh3tG+RWRvCYy0JHZSZW532Vqy2M/sW/yxhUN5CIRlcYL8uFY= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR12MB4294.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(4636009)(396003)(366004)(39860400002)(136003)(376002)(346002)(451199018)(66574015)(2616005)(54906003)(316002)(41300700001)(5660300002)(8936002)(31696002)(86362001)(66946007)(8676002)(4326008)(6916009)(66476007)(66556008)(83380400001)(38100700002)(36756003)(44832011)(6486002)(66899018)(478600001)(6666004)(31686004)(186003)(2906002)(6512007)(53546011)(26005)(6506007)(43740500002)(45980500001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NW9yUWZFdlRDTDFicTRvejl3dlkyclNlcW80SWxQVG16eFg0aTM3c2RnWExl?= =?utf-8?B?MzFYMnVuM2hzYWR1VTdIbkY4UzQ4dGZ1ZGpJYUxBNE5uK21vY1RDVXU3WklN?= =?utf-8?B?aG43WitnM0J6T2FKNlRPZ3hOOHV1Szh1bGhSZWRhb3JqS0M1aEVuaUlKK1hM?= =?utf-8?B?dVZwTU11cEYxbUZOZEVieUtkMitzWDhNaGptTzZxcHpxTzNzc3hRVVArSnVk?= =?utf-8?B?ME44NHNlbDBZbnQwc0g2L3B2b2dnbDNSRzFFL2JobG9SbFppVmtRN1ArTVJG?= =?utf-8?B?VW1MdmZIYzFFdnRMNmI4S2huRGVnRllScm82VGJacUF4TGtpU3I1VTQ1eU5m?= =?utf-8?B?dzZyb2NoNjFUVTRlOFVURm9xWTNTbmJhTzcvdjlxaUZYalI2SkNXUldJL3ZG?= =?utf-8?B?Q2c0MjFaZWhsMEhwd2lHanJnclpQSzZWMkd2QmlmaEdoRXA3VlJnT3JIUnAz?= =?utf-8?B?eHM1M01ITnd2RlRmVHl2NW9ScmFHdkplU0M4eW92elI0OWl2L1pxSFYxbXVu?= =?utf-8?B?NmZFdmVSenpOVDZ0N0JMTG1XK0VyK0VYZ0N2amVWcFpqdlozN045MDdwampn?= =?utf-8?B?OXR5Qyt6c1NCRUt0RkVtdUlxeDFnQVFhdUw1cE9GaFY0NTl3SWpzc3g5YlND?= =?utf-8?B?WEtCZ2dHUVN0bnE0VTJQN3F5Wit2amUrWlVCcHkrZU1zZVpKVG1SL1lKZ05B?= =?utf-8?B?YVlGOFRUekhzek1sa2dOUEtrSWR4TkdEVW1iZXYyeVpuN0g4ZDJNN1BCL3lX?= =?utf-8?B?QWg5Q295RjJvaEJWV1ZyUFBBSGFHQWtlUis5Rko0bE9YM1g4Q2tEYWRJeWwz?= =?utf-8?B?dnRwVXV0eXo3TnpqcDdNUmd0T1ZhOWpWdWZyUWpwVkQ0TWM1MGI1ZDhHRDgr?= =?utf-8?B?cnhYNDdSOUhCbTRGcnRzdThuMU5ycmFIRVVUY0xMUjUrcVphcFFNVU9zQ1pH?= =?utf-8?B?aTJsOExtZGlLR2xEamRYWlpIRy9RN2pLMmpva05yOEFMb1RWZXBUT0tjeE5v?= =?utf-8?B?MHNzWEFmTUM3bjFxeVVWc054Mnhxa25OVGJ3Y2Vaak5mc05SSm1zcG4vbGVB?= =?utf-8?B?dk1zU1c5a2VGeHdzZGVtVWlrYU9NYk5uWlM0YkRBd2lOUlRlTmRzcGZSZlYw?= =?utf-8?B?anEydTh5Y21LZlVlQmVvN3A1WUVjeDNpMEJyM1V4c1B4UWt2SkVsV3ZxbGNG?= =?utf-8?B?bVFNeDRZcGxYem80U3dsR3loL0g0MEZSTE92ZGQ3U2ZhamhjL1UraWpnS21o?= =?utf-8?B?UzBGNU1UYjk3MzYvSXhKVC9YY1k4R1BIWEErZ21GYTBWOUJOWVpCQjQ1RlFN?= =?utf-8?B?M0pWZHNDZlJMY3BZaDFRWklhTTREMjBVcVBZRVJqakRibnhrbUY1N0ZtcmJk?= =?utf-8?B?Qm5SSjdGZVRqZUJmQ0ZmU0JNU2VSYUg0NkFvZEJLaTNmcDlGd1BZUTRocEps?= =?utf-8?B?cmMrd3F3M2Z6TXdHZGFrQ3VSNExpWnA1Zmd6c1I1NHlFYm4rTm1ubktSUjQv?= =?utf-8?B?MWVwLzFPTmo4ajZuR0FxdE1aYW5jL21JazZObFZrdUZqQzZ5eXBPQnNEYUV2?= =?utf-8?B?UFgvMHMrL0tWU3NQK3UwbGEyQjBJc3dySzQxMkNUTEJzc2JQdFUzU3h5akJI?= =?utf-8?B?TlZnNGNSNkRyVWpmU25ETURORHBTaExXTEdjSmMrV05GRitUUDNhRXU5bTRC?= =?utf-8?B?ckQySURKRTRpeHJOYjhlaFdLbG5HQk44N2I1SEVJTWJRL3hDRnhkTmprRnI5?= =?utf-8?B?WmluajM3VkpXL0xwWjVXTHhEUkkvb2h5K3ZLWm9HaDU4aGdlUjB3WC9SNllU?= =?utf-8?B?WFdHRkswcDBVaEFhcTdiL1RIWFVyVEpjTWRCemdtQ1lJd2s5Z2NxbVZXa1BP?= =?utf-8?B?TmZrUjZSdmp4cFNXYnMvN1dWVURVQld5YmttRTU3WUMrTmltWloySnZaWlE5?= =?utf-8?B?bU1xRVN2ZUJnUThMLytpcGFOT3lVRzlTdytZdjVVMHFLNEVIanFNeTQvVXJs?= =?utf-8?B?VXM4QU0wUWJNT3B2ZytQQTlZS092R3lnQjlmSFY2Vml3ajNPRWMrUGJLQlg1?= =?utf-8?B?SkhwS3RqUVZrRGNGUXBxVTduQkJtakdoNFlUSUR4MjVXUzVpVlFxVHpnUTd3?= =?utf-8?Q?sAlGdl1OEuwyPGAsGZ4edxuA8?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 239f4ffa-579a-421e-aed0-08db0f827b98 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Feb 2023 18:28:49.9013 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 8j64nYhR3OS+BykIo4sw2YVOBTkiSDjpzkcgq83eFv39b08FY+RYDDMuN4x29hX4 X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6883 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 2/15/2023 5:47 PM, Niklas Söderlund wrote: > Hi Ferruh, > > Thanks for your continues effort in dealing with NFP patches. > > On 2023-02-15 13:42:01 +0000, Ferruh Yigit wrote: >> On 2/8/2023 9:15 AM, Chaoyong He wrote: >>> From: Peng Zhang >>> >>> 48-bit DMA address is supported in the firmware with NFDk, so enable >>> this feature in PMD now. But the firmware with NFD3 still just >>> support 40-bit DMA address. >>> >>> RX free list descriptor, used by both NFD3 and NFDk, is also modified >>> to support 48-bit DMA address. That's OK because the top bits is always >>> set to 0 when assigned with 40-bit DMA address. >>> >>> Fixes: c73dced48c8c ("net/nfp: add NFDk Tx") >>> Cc: jin.liu@corigine.com >>> Cc: stable@dpdk.org >>> >> >> Why a backport is requested? As far as I understand this is not fixing >> anything but extending device capability. Is this a fix? > > I agree this is a bit of a grey zone. We reasoned this was a fix as we > should have done this from the start in the commit that added support > for NFDk. Are you OK moving forward with this as a fix or would you > prefer we resubmit without the request to backport? > I am not sure, is this change have any potential to change behavior for existing users? Like if one of your user is using 22.11.1 release, and if this patch backported to next LTS version, 22.11.2, will user notice any difference? @Luca, @Kevin, what is your comment as LTS maintainers? >> >>> Signed-off-by: Peng Zhang >>> Reviewed-by: Chaoyong He >>> Reviewed-by: Niklas Söderlund >> >