[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 08/10] vpci/rebar: Free Rebar resources when init_rebar() fails


  • To: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • From: "Chen, Jiqian" <Jiqian.Chen@xxxxxxx>
  • Date: Fri, 6 Jun 2025 08:32:35 +0000
  • Accept-language: en-US
  • 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
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=1GpoamSMunsvFeR74MN8OcfzZgcA+45yqhKevz0JsSs=; b=aQfPYdDlW/99RVgeqwzZRBIbeL9eHNjW/D9lzOj5B59sKDuJIMQPSA3AxD2RJ5Sb9KBuGEMlPe7CshcqbdgwtteNV7JapdfZyWPPnHaz7hbmT3wqFppjg96sBRknHq0YUlD1WzxByWL+XWWc3kpTNjITT2ZRw/NbZgOg63tn9Zo1CV03YMCU5qXLbI4KRP5Z1xgX4kaNfNndrmYHScuFS+i2F0Nps1T8dLtAqycsllDoQUTOX5VWlyMuB+3RgXTpCJrjDcSplLaVSTMxuaIa0xByN5ewrZyZztK+WKYTysH/p9X92JT0NB0PP5bQPxc17rTedWc2HE6keHkDNCoyuw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ax9NaYCk2HAXr155pC/gberOicZIj/zHtoXzYHpKobnuldNGkkjynnaFwmLN6Kqi5gVVDNNC+Buj1vNe2gRbrWFWeaVUy2L4dZ++Lg/vNHK09iAoNUxmNzlwKxhcfGnOhF6+4m/cKntamnhfluaW21piwdiOBj2exlFNWpQw0jwcr3TI1ZhojmP7Z5j2OAcn19ebww2ltRxdtwW+fqk7JW9P/VfCU6jUR+WL1BCgSFTvyChlo4eFzTLfbzC6nj1BsCberNVhjBCh9Bl6yHpy0Jmrg9O4S+30DdNcndU+6ZBONVJyGiD4gMI7hhXssp16a+mXuwOhg8O/nRnQ0JjDCQ==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Huang, Ray" <Ray.Huang@xxxxxxx>, "Chen, Jiqian" <Jiqian.Chen@xxxxxxx>
  • Delivery-date: Fri, 06 Jun 2025 08:32:42 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHbziMRPzOlFqwgb0K+e9/00F6hg7P0vO6AgAGnwoA=
  • Thread-topic: [PATCH v5 08/10] vpci/rebar: Free Rebar resources when init_rebar() fails

On 2025/6/5 23:14, Roger Pau Monné wrote:
> On Mon, May 26, 2025 at 05:45:57PM +0800, Jiqian Chen wrote:
>> When init_rebar() fails, current logic return fail and free Rebar-related
>> resources in vpci_deassign_device(). But the previous new changes will
>> hide Rebar capability and return success, it can't reach
>> vpci_deassign_device() to remove resources if hiding success, so those
>> resources must be removed in cleanup function of Rebar.
>>
>> To do that, implement cleanup function for Rebar.
>>
>> Signed-off-by: Jiqian Chen <Jiqian.Chen@xxxxxxx>
> 
> LGTM, just one nit about a bounds check.
> 
>> ---
>> cc: "Roger Pau Monné" <roger.pau@xxxxxxxxxx>
>> ---
>> v4->v5 changes:
>> * Change definition "static void cleanup_rebar" to "static int cf_check 
>> cleanup_rebar" since cleanup hook is changed to be int.
>>
>> v3->v4 changes:
>> * Change function name from fini_rebar() to cleanup_rebar().
>> * Change the error number to be E2BIG and ENXIO in init_rebar().
>>
>> v2->v3 changes:
>> * Use fini_rebar() to remove all register instead of in the failure path of 
>> init_rebar();
>>
>> v1->v2 changes:
>> * Called vpci_remove_registers() to remove all possible registered registers 
>> instead of using a array to record all registered register.
>>
>> Best regards,
>> Jiqian Chen.
>> ---
>>  xen/drivers/vpci/rebar.c | 35 ++++++++++++++++++++++++-----------
>>  1 file changed, 24 insertions(+), 11 deletions(-)
>>
>> diff --git a/xen/drivers/vpci/rebar.c b/xen/drivers/vpci/rebar.c
>> index 9cafd80ca2c9..4b1892fab3d6 100644
>> --- a/xen/drivers/vpci/rebar.c
>> +++ b/xen/drivers/vpci/rebar.c
>> @@ -49,6 +49,26 @@ static void cf_check rebar_ctrl_write(const struct 
>> pci_dev *pdev,
>>      bar->guest_addr = bar->addr;
>>  }
>>  
>> +static int cf_check cleanup_rebar(struct pci_dev *pdev)
>> +{
>> +    uint32_t ctrl;
>> +    unsigned int nbars;
>> +    unsigned int rebar_offset = pci_find_ext_capability(pdev->sbdf,
>> +                                                        
>> PCI_EXT_CAP_ID_REBAR);
>> +
>> +    if ( !rebar_offset || !is_hardware_domain(pdev->domain) )
> 
> I think you could check rebar_offset < PCI_CFG_SPACE_SIZE to be more
> accurate?
OK.
Do I need to change in init_rebar()?

> 
> Thanks, Roger.

-- 
Best regards,
Jiqian Chen.

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.