Received-SPF: pass (google.com: domain of linux-kernel+bounces-136450-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <56a5734a-076c-4274-9d54-7da8a11fce89@amd.com>
Date: Tue, 9 Apr 2024 18:25:48 +1000
User-Agent: Mozilla Thunderbird Beta
Subject: Re: [PATCH kernel v3] PCI/DOE: Support discovery version 2
Content-Language: en-US
To: Bjorn Helgaas <bhelgaas@google.com>
Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
 Ira Weiny <ira.weiny@intel.com>, Dan Williams <dan.j.williams@intel.com>,
 Jonathan Cameron <Jonathan.Cameron@huawei.com>,
 Lukas Wunner <lukas@wunner.de>,
 Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>,
 Sathyanarayanan Kuppuswamy Natarajan <sathyanarayanan.nkuppuswamy@gmail.com>
References: <20240307022006.3657433-1-aik@amd.com>
 <32ac2ee6-26b1-4f37-bd88-db1ef2a31267@linux.intel.com>
 <16120a31-c3d6-48a8-9355-7b063e28d242@amd.com>
 <CAC41dw9x8H2m=YB66+PA-GwiVkpCyT0PobdWvFXLHpAsQOE0ew@mail.gmail.com>
 <8d542410-d8e1-4243-939c-6ce7743a66f1@amd.com>
 <8d076ed4-5c3c-499a-bca0-9bd51d5454cc@linux.intel.com>
From: Alexey Kardashevskiy <aik@amd.com>
In-Reply-To: <8d076ed4-5c3c-499a-bca0-9bd51d5454cc@linux.intel.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?VGdacHpGRU9YNVJ0OXg1OUc4cTlld093Z2o4bnRWem1GYjNzUS9odkpYcGtp?=
 =?utf-8?B?WUw0MnNIWVZBd2Z4ckZJU2xrNjhJUkRMNXpNbVpXUGRwaFF4NUl1cHNWV1NB?=
 =?utf-8?B?QTZCd3BIZGs2cktQK3dJWE5PeFNNYmxuSEhoeWd4UnNOM2lMOTZWZ1F1VkZT?=
 =?utf-8?B?SnBkQ0w5UExuZjFXZzR6WW54bWZuL1QzaFFuK010THZxNmZTV0swRVY5MXow?=
 =?utf-8?B?bnBaMmV2RTBEeFJ2eks3STZNa3BjcSsvK25pMEpqVkxDc2hNRHNJaXVmdisz?=
 =?utf-8?B?a0JtTGd5dW9JMHYyS0VSNUlzL2VQNGF6Q0s2N2N1dTVCTmxxZTNwRnh6SlVV?=
 =?utf-8?B?b04wK0pSNnBndDBoTVU4dTlPQitTbXZJdFQxYm1oamJsK3NWRVJjS0I2NzZM?=
 =?utf-8?B?MzlHWnhiMVphSkV6VmJveUN1QUUzSzJNcktHZkk3RklMZkovbzdSbkZPUGtr?=
 =?utf-8?B?SDEzaUN1UlZld0JGcWhqUnVuTlZVSXhIbWw5bVRMUktCOTcwcjNSazlwbXFF?=
 =?utf-8?B?d1FqREEzSWw2NVRzWTVOdXR6Wng0bGNUeFFyaGNjNjFQbkd2RkhPNjNQcGNV?=
 =?utf-8?B?b1hxcS9TdHpGblA2aUVGR1lvb1dtM0tpRUxsTC9ROFQzKzhFa3J5R2RMZWVm?=
 =?utf-8?B?R0VuVklTNGpxOC9WWnFwVFhzVlo5MU1nbm9GRk54MndCdmROcnpFZ0o4NlV6?=
 =?utf-8?B?KzhhRGVVM0NBRkJJbi9ETHFnUWNCZ0J0bWQ0cGttV1BsZ2RkOFU4R1ltMWQ4?=
 =?utf-8?B?b2czTTZOOXpkUlZlV0U0V3YzZDU2TWs3eTRUTHBwNThCdGdVSkVKVEl6QTBZ?=
 =?utf-8?B?QVhFUDhNNURGZkZmSHNrU2pTZlYyQ0V5aWU2UkhHanRVNHA4K1F3cTFiSmxY?=
 =?utf-8?B?RHVTenl5Z2FnTWVSZ1F2Q1ZNNzUvQk0vRE85UzJxaDRRUVBySERnU2JXY0Uv?=
 =?utf-8?B?UTIzdmRHbFNIVVZPRWpDU2oyb2Y5TE10Y2poaEM1YmNnNHNQNGVHbnFmUWN0?=
 =?utf-8?B?dkdwSkErMFBxdWVEc2lhY08xR2FHTUVMZDR6VmE4YVlla0V6bFhtQ0ticW1i?=
 =?utf-8?B?SGxHZ3lwWGRSL0NRSzJmekU3Smh5ekE4VnBIZERzcGYxcUV1eHl2aVpsQ0lI?=
 =?utf-8?B?NFN6TlB2NXZFRytlcHBxQWZQcnZaQzVQRDlzekZMVXZuWkx6SnVwUnp0dnpp?=
 =?utf-8?B?NjR4Mjl5V1ZrNkVpOUNRWXBMS1dhOHJuTS91V2Q0clBUZms3YUxaY2txMFVs?=
 =?utf-8?B?S2RYQjZ5Nk5ZeDBmZElhaXZmZTZ1NjRXb1U2SFV0ZCs2SXFjUDNPc2lYQTBz?=
 =?utf-8?B?QU05UUJoTmlpeFVoSHBmUDhibTZhSkROUXg4cVpQZXRuUnMveUtPUnZFeHRW?=
 =?utf-8?B?M2FDTXJMK0REU3pJRXZkS0xHTmJkb1ZVUmRaVDcwRXdTQmdVTUt3VTNPY3M2?=
 =?utf-8?B?NklWeU40ekZ6RkNUcldKZmNGUGFSOFVTUXFqS0VkMWxUUm9PT3Z5QzkwUG1z?=
 =?utf-8?B?cmVZb2ZvazZmNG5UZ3BPTDNCVGFrUG5kaXEwd1JmeEFjL2k1UHVndXBROUpz?=
 =?utf-8?B?ZjNRN0E1eWZZVThXR1lWeDNVY2k1bktDRnFFMjhpQndDZHpZS2ZCa2xia0Jo?=
 =?utf-8?B?L0ZwMXY5aWprblJtRW54aDFqUnVyZlNab3NBYTRPTDh3dW1DRlFPd0RWTHp0?=
 =?utf-8?B?YitkTVJ5elF5d012RzdiYUswVFdsZG5Mc2tsejJ6ZG4rbVhVc1JwTU52dlNy?=
 =?utf-8?B?L2p1WEtZcGMwSVAvaVVTS1hXdDBzSlBVREVkVld1YXFxYWpkSmxxNXcxWUdX?=
 =?utf-8?B?ZFN0RC9CVjVUQWh6REgyTm1EcGo4cmtQWjFQMUlESU5vN2xhVGpwYUFJS2hY?=
 =?utf-8?B?NU5DK0RrazErcTROQ2dEMi8xMTBwdkVPQnNqcGRTejV0cEI5MUE2Q21XYTIz?=
 =?utf-8?B?dThsWE1yczVpOWZ4U2o4bEtsY0phYldzSXJlb08yWi9BdE5WN0wxTTBKNVBD?=
 =?utf-8?B?QktJZWhFZ1dlYXBvaXcrSk9DMzRRQm83M290cDhiUGIzdlZ5V3FIOUhGQk0r?=
 =?utf-8?B?K2xTWHJBbVlxWVdleVIwWmtxb1Mxam9jWmFvM0RHaTA0VU5rcUNyL2kxdWUr?=
 =?utf-8?Q?/2PpTtQn69MeJT4WTuNK9jneB?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 54565b2f-eb1a-4cc0-b253-08dc586ead71
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB9194.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Apr 2024 08:25:56.2813
 (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: wTMx2EHW6IRUGlvLohkBh1A2k+1fDhxbo5NCnV/HcKAB/wn1hbuuCS/BRfeHfsGT82qdydBAhpRALFRMwZoCyw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR12MB8904

Ping? Is this going anywhere? Thanks,


On 14/3/24 17:49, Kuppuswamy Sathyanarayanan wrote:
> 
> On 3/13/24 5:59 PM, Alexey Kardashevskiy wrote:
>>
>>
>> On 7/3/24 17:22, Sathyanarayanan Kuppuswamy Natarajan wrote:
>>> On Wed, Mar 6, 2024 at 10:09 PM Alexey Kardashevskiy <aik@amd.com> wrote:
>>>>
>>>>
>>>>
>>>> On 7/3/24 16:06, Kuppuswamy Sathyanarayanan wrote:
>>>>>
>>>>> On 3/6/24 6:20 PM, Alexey Kardashevskiy wrote:
>>>>>> PCIe r6.1, sec 6.30.1.1 defines a "DOE Discovery Version" field in
>>>>>> the DOE Discovery Request Data Object Contents (3rd DW) as:
>>>>>>
>>>>>> 15:8 DOE Discovery Version – must be 02h if the Capability Version in
>>>>>> the Data Object Exchange Extended Capability is 02h or greater.
>>>>>
>>>>> Does this change fix any functional issue? If yes, please add Fixes:
>>>>> tag and probably CC stable?
>>>>
>>>> The issue is that before this patch the DOE driver was following PCIe
>>>> r6.0 and it was not working on a device with DOE cap v2 added in r6.1.
>>>> Is it "fixes"?
>>>>
>>>
>>> Since you are enabling support for the newer spec version, I think it will not
>>> fall under bug fix. But it might be qualified for the stable branch.
>>> It depends on
>>> whether you want to support this device in older(stable) kernels.
>>
>>
>> I am going to need lot more stuff (TSM, IDE, TDISP) in addition to this but it is coming much later so there is no point in pushing this alone to older kernels. Thanks,
>>
>>
> Got it.
> 
> Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
> 
> 

-- 
Alexey