Received-SPF: pass (google.com: domain of linux-kernel+bounces-47946-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <895b31de-f7f8-425c-870b-1524be21c688@amd.com>
Date: Thu, 1 Feb 2024 21:16:37 +1100
User-Agent: Mozilla Thunderbird Beta
Subject: Re: [PATCH kernel 1/4] pci/doe: Define protocol types and make those
 public
Content-Language: en-US
To: Lukas Wunner <lukas@wunner.de>
Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
 Bjorn Helgaas <bhelgaas@google.com>,
 Jonathan Cameron <jonathan.cameron@huawei.com>
References: <20240201060228.3070928-1-aik@amd.com>
 <20240201060228.3070928-2-aik@amd.com> <20240201065040.GA31925@wunner.de>
From: Alexey Kardashevskiy <aik@amd.com>
In-Reply-To: <20240201065040.GA31925@wunner.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?d1Bjc0NOWjRBQnhNOEwvWG5qYWRqdGliWUZhcWRrZG1Cd0tsSW50OUY4aHl6?=
 =?utf-8?B?OUlmY1ZrdEhleTRuYVhWa2F4S1pCSll0MjZQUkgwSnJLOHd4KytCWjJBVXFn?=
 =?utf-8?B?emh3SStORHAvSlhxZS9BUjR6Nll6dDZxUlNlblpvWXB0bVNZOWtQZmgydnhl?=
 =?utf-8?B?QytCVytwR0pYWG9KMUg3Y2tYU09KT3QzTmZZbWZaNzNLWk9JZ2dXSURsbG1D?=
 =?utf-8?B?c2tmQ1ltSnB4L3NTbnAzVHhnSUFNZ1cyRzM5d1VaSFdybW85UEtLNldTUzdx?=
 =?utf-8?B?WkUrK1AxVFlRUGJyOG41OXRpWUJVSGVnOENhOTgxVVJnSUhBcXF3T1dkOTZ0?=
 =?utf-8?B?Q3JXOFgxakJxZU9MN3hpampWUUl6eFRsMk9zVnQ4bkhxUnRTVlUvTlE3c0o3?=
 =?utf-8?B?TjMrUFphdWtpTFVMV01vVlpmVHV0ZS9tYVNkeWJ2QW1NMmdmcHBtNmg4Ylp3?=
 =?utf-8?B?NGQ1aC9Yb200YjNFNkVvNkZ5NDVjTkxDUnRoMjF1ZWZTakdXOEw3V1pWcjZX?=
 =?utf-8?B?L2xuYS9SV255Q1lTMjJ2bW5WQkl5aXV4dTNQNFVEaGVTUm9vT1dScTllTVhw?=
 =?utf-8?B?aXQ0RnNUSGwzc0orRnlxMksvd0l0WjltNHdKQ25KWDMzeUN3eWlLMDlqTDZm?=
 =?utf-8?B?UTJLWGpXODRtVExiMFVsYkJqa0RKTlhyY0dab0hiVTJSNEl5ZWR6SE1heWZp?=
 =?utf-8?B?YWZHdVBMR29WWXpLUlE1cEhpTkl6ZUxCaXduQzBRQkJ4cUwySXVtdVpuZ01F?=
 =?utf-8?B?ekZvbjg2V3RzUUwzdTNSdE1yVVVEdTc2d0ova3RjWkp2eEJOYnNWdGpxVlg5?=
 =?utf-8?B?Sjc3SWoraTVocmpibTNqUnZIaW1kSUQyMnArd2g5MjBhODhQKzIzNjhHYUky?=
 =?utf-8?B?dSsyY1hoQlNzb2xORGZzYkZBcjY3eGJlMnJoZUl4SGhjc2tueENUdk92NUlZ?=
 =?utf-8?B?YnprUXdnZ053KzRWUzlKSTNxYjc1cTBoUEg1OEIwZ1FpSkRqWmJzQnJVcUtP?=
 =?utf-8?B?alNCVkhtUDlxVUVPU2Zob2o5SVJGclBJQ3JSSlF0bEpZN3JwVFcrVWw1Qm9B?=
 =?utf-8?B?L1pXTVVGOTZkWTE2S3JxbW1tRmVKUlpaZE8xUDZMd0Y2ajUzSWR6WGpIOWEv?=
 =?utf-8?B?VkwvU0NVVUJpbHVrUVVzTkhKVjYwbWYwNEVEVWdTU0FLUHNnaXlacVdNYmFP?=
 =?utf-8?B?MFl5ZlozSGxyYThETXlnZG1IL2pEaGZ2aFY2S3U5eFUvNFZFY1FCUHB2c1M4?=
 =?utf-8?B?eUJISng3bjZFRU9LbEgvS014MTVQT3V0eVNNYzJKSkloVWFWbjl4cEVhSy9p?=
 =?utf-8?B?WFpWc2ZmUG0rMW1MNVpkbmEyODdZV203WUtIdEhiVGt4bVFYQmpTS2E3dCtW?=
 =?utf-8?B?NE1mWjkzQmh4QWVKUEZTYkNlTXpsRDlTR3dnbXRTRFZ5L0RWUEhmQ05hY3FH?=
 =?utf-8?B?NCtJSWQ5WTQ0N0dpVVlxMS9JUGNhd3l6Ymg5QnNhVGZNOXVRbDJia2w3M3A3?=
 =?utf-8?B?ZmIwS3BwekMrcGtzeVgvazNGRFhBYUVxWkxhS1E0bGNWOE03aEJiVkE2dklQ?=
 =?utf-8?B?M3pkK3FFVjFXdkZaOXdMbWNtU0VMaHB5dGwzZEtvWWR5am43VnhzNksxYWsr?=
 =?utf-8?B?Ty9YZmpXM2c1Y2NvTVdmUHU5YVRBbTlYWk9GZlpKbEJEd2tQb1ZEM0J3c3Ix?=
 =?utf-8?B?QnhuOFZRK2RLRHpkcHhwS2NKVEYwclFPZXJiTXpvaHl5VytjVlIwYjlvL2tS?=
 =?utf-8?B?amFYVmU0OUJmampHaU5CelpUOXpDRzRFbDF1cEJFMW16YlFaYWdRRGdwa3Nz?=
 =?utf-8?B?dTBRRjUvemd1RTBqR3NUazdhRHpqQitzc0tqRnB3VC9zdHN2UnZGcFFnWG11?=
 =?utf-8?B?azlmYXVYbXluRVpuc1RGcmVnSG5NMyt2VTB2anpIQjJyOXNWellUdXVlQ3A1?=
 =?utf-8?B?OERsZDlzR25GYkZuNnQ5b0ZNcHNTSzBScjZrYzRlaFU3SDcyeHd2YWhWV1M3?=
 =?utf-8?B?d0QveUxqa1p2M2RpWHdUYlZ3ZWppSlI1L2tzNjRJNWNGZ2szSnduenNNK1Ft?=
 =?utf-8?B?SGMxTkhRWEcrUDdNZW5BK01sTjdNbHVvUU1YN25KVDFicUZLZlBkbUpOd01X?=
 =?utf-8?Q?X6t2a0tLjLr9lr6gaeA39//hv?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 00430813-4906-4648-897a-08dc230ee3ea
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB9194.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Feb 2024 10:16:44.3091
 (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: 7HNryE/chGGDDnYWdRvPxaVIyg1B9Id0PgyeqQpNl/4XaDkXX0mtIQ7o+3QrZR+sJtJkxiszh6eSjjGMdV0TRA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB7198

On 1/2/24 17:50, Lukas Wunner wrote:
> On Thu, Feb 01, 2024 at 05:02:25PM +1100, Alexey Kardashevskiy wrote:
>> Already public pci_doe() takes a protocol type argument.
>> PCIe 6.0 defines three, define them in a header for use with pci_doe().
> [...]
>> --- a/include/linux/pci-doe.h
>> +++ b/include/linux/pci-doe.h
>> @@ -13,6 +13,10 @@
>>   #ifndef LINUX_PCI_DOE_H
>>   #define LINUX_PCI_DOE_H
>>   
>> +#define PCI_DOE_PROTOCOL_DISCOVERY		0
>> +#define PCI_DOE_PROTOCOL_CMA_SPDM		1
>> +#define PCI_DOE_PROTOCOL_SECURED_CMA_SPDM	2
> 
> These are deliberately defined in the .c files which actually need them,
> i.e. DISCOVERY is defined in drivers/pci/doe.c and CMA_SPDM is defined in
> drivers/pci/cma.c:
> 
> https://lore.kernel.org/all/7721bfa3b4f8a99a111f7808ad8890c3c13df56d.1695921657.git.lukas@wunner.de/
> 
> I don't see why they would have to be public if they're each only needed
> in a single .c file.

SEV TIO needs both CMA and SECURED_CMA for DOE device<->firmware 
bouncing and it is going to use pci_doe() for this. I should have put 
this into the commit log, sorry about that.

Or the plan is to add pci_doe_secure_transport() to cma.c and force 
everyone use that?

The PCI SIG DOE protocol numbers (discovery, CMA, secure CMA) are all 
defined in one place in the PCIe spec and defining them in different 
places (doe.c, cma.c) is weird imho.


> 
> Thanks,
> 
> Lukas

-- 
Alexey