[+cc Alex, Don]
On Wed, Jul 22, 2020 at 05:44:28PM +0800, Hanjun Guo wrote:
> The acpi_get_table() should be coupled with acpi_put_table() if
> the mapped table is not used at runtime to release the table
> mapping.
>
> In pci_quirk_amd_sb_acs(), IVRS table is just used for checking
> AMD IOMMU is supported, not used at runtime, put the table after
> using it.
>
> Signed-off-by: Hanjun Guo <guohanjun@xxxxxxxxxx>
Applied to pci/virtualization for v5.9, thanks!
I added this:
Fixes: 15b100dfd1c9 ("PCI: Claim ACS support for AMD southbridge devices")
but I didn't add a stable tag. Does this cause any issue that would
warrant a stable tag?
> ---
> drivers/pci/quirks.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 812bfc3..487ed4d 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -4409,6 +4409,8 @@ static int pci_quirk_amd_sb_acs(struct pci_dev *dev, u16 acs_flags)
> if (ACPI_FAILURE(status))
> return -ENODEV;
>
> + acpi_put_table(header);
> +
> /* Filter out flags not applicable to multifunction */
> acs_flags &= (PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_EC | PCI_ACS_DT);
>
> --
> 1.7.12.4
>
