linux-kernel - Re: [PATCH V1 5/5] accel/amd_vpci: Add communication channel service

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <05f478f4-b492-4f4a-8b87-a3224e604702@amd.com>
Date: Wed, 12 Nov 2025 20:53:18 -0800
From: Yidong Zhang <yidong.zhang@....com>
To: Christophe JAILLET <christophe.jaillet@...adoo.fr>, <ogabbay@...nel.org>,
	<quic_jhugo@...cinc.com>, <maciej.falkowski@...ux.intel.com>,
	<dri-devel@...ts.freedesktop.org>
CC: <linux-kernel@...r.kernel.org>, <sonal.santan@....com>,
	<mario.limonciello@....com>, <lizhi.hou@....com>, Nishad Saraf
	<nishads@....com>
Subject: Re: [PATCH V1 5/5] accel/amd_vpci: Add communication channel service



On 11/11/25 01:59, Christophe JAILLET wrote:
> Caution: This message originated from an External Source. Use proper 
> caution when opening attachments, clicking links, or responding.
> 
> 
> Le 11/11/2025 à 02:15, David Zhang a écrit :
>> This patch introduces the communication channel (comm_chan) service, 
>> which
>> enables communication between the management PF driver and the user PF
>> driver.
>>
>> The comm_chan service provides a shared-memory-based command channel
>> located in a BAR-mapped region. The user PF driver issues requests by
>> writing command messages into this region. The management PF driver
>> periodically polls the region and process any pending requests.
>>
>> Supported operations include firmware (xclbin) reloads, where the
>> management PF invokes 'versal_pci_load_xclbin' to reprogram the embedded
>> firmware through the remote management queue service.
>>
>> This service provides the foundation for dynamic firmware updates and 
>> other
>> management requests from the user PFs.
>>
>> Co-developed-by: Nishad Saraf <nishads@....com>
>> Signed-off-by: Nishad Saraf <nishads@....com>
>> Signed-off-by: David Zhang <yidong.zhang@....com>
> 
> ...
> 
>> @@ -160,10 +222,13 @@ static const struct config_item_type 
>> versal_pci_cfs_table = {
>>   static int versal_pci_cfs_init(struct versal_pci_device *vdev)
>>   {
>>       struct configfs_subsystem *subsys = &vdev->cfs_subsys;
>> +     char dev_name[64] = "";
> 
> No need to init I think.
> 

Ok, I will fix this.

>> +
>> +     snprintf(dev_name, sizeof(dev_name), "%s%x", DRV_NAME, 
>> versal_pci_devid(vdev));
> 
> This hunk could already be in patch 1. This would make this patch smaller.
> 
>>
>>       snprintf(subsys->su_group.cg_item.ci_namebuf,
>>                sizeof(subsys->su_group.cg_item.ci_namebuf),
>> -              "%s%x", DRV_NAME, versal_pci_devid(vdev));
>> +              "%s", dev_name);
>>
>>       subsys->su_group.cg_item.ci_type = &versal_pci_cfs_table;
>>
> 
> ...