* [PATCH] drm/amdkfd: Remove unused entries in table
@ 2021-11-18 20:18 Amber Lin
2021-11-18 20:36 ` Sider, Graham
0 siblings, 1 reply; 2+ messages in thread
From: Amber Lin @ 2021-11-18 20:18 UTC (permalink / raw)
To: amd-gfx; +Cc: Amber Lin
Remove unused entries in kfd_device_info table: num_xgmi_sdma_engines
and num_sdma_queues_per_engine. They are calculated in
kfd_get_num_sdma_engines and kfd_get_num_xgmi_sdma_engines instead.
Signed-off-by: Amber Lin <Amber.Lin@amd.com>
---
drivers/gpu/drm/amd/amdkfd/kfd_device.c | 58 -------------------------
drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 2 -
2 files changed, 60 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
index 3fea47e37c17..e1294fba0c26 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
@@ -68,8 +68,6 @@ static const struct kfd_device_info kaveri_device_info = {
.supports_cwsr = false,
.needs_iommu_device = true,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -87,8 +85,6 @@ static const struct kfd_device_info carrizo_device_info = {
.supports_cwsr = true,
.needs_iommu_device = true,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -105,8 +101,6 @@ static const struct kfd_device_info raven_device_info = {
.supports_cwsr = true,
.needs_iommu_device = true,
.needs_pci_atomics = true,
- .num_sdma_engines = 1,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
#endif
@@ -126,8 +120,6 @@ static const struct kfd_device_info hawaii_device_info = {
.supports_cwsr = false,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
#endif
@@ -145,8 +137,6 @@ static const struct kfd_device_info tonga_device_info = {
.supports_cwsr = false,
.needs_iommu_device = false,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -163,8 +153,6 @@ static const struct kfd_device_info fiji_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -181,8 +169,6 @@ static const struct kfd_device_info fiji_vf_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -200,8 +186,6 @@ static const struct kfd_device_info polaris10_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -218,8 +202,6 @@ static const struct kfd_device_info polaris10_vf_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -236,8 +218,6 @@ static const struct kfd_device_info polaris11_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -254,8 +234,6 @@ static const struct kfd_device_info polaris12_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -272,8 +250,6 @@ static const struct kfd_device_info vegam_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -290,8 +266,6 @@ static const struct kfd_device_info vega10_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -308,8 +282,6 @@ static const struct kfd_device_info vega10_vf_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -326,8 +298,6 @@ static const struct kfd_device_info vega12_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -344,8 +314,6 @@ static const struct kfd_device_info vega20_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -362,8 +330,6 @@ static const struct kfd_device_info arcturus_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 6,
.num_sdma_queues_per_engine = 8,
};
@@ -380,8 +346,6 @@ static const struct kfd_device_info aldebaran_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 3,
.num_sdma_queues_per_engine = 8,
};
@@ -398,8 +362,6 @@ static const struct kfd_device_info renoir_device_info = {
.supports_cwsr = true,
.needs_iommu_device = false,
.needs_pci_atomics = false,
- .num_sdma_engines = 1,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -417,8 +379,6 @@ static const struct kfd_device_info navi10_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 145,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -436,8 +396,6 @@ static const struct kfd_device_info navi12_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 145,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -455,8 +413,6 @@ static const struct kfd_device_info navi14_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 145,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -474,8 +430,6 @@ static const struct kfd_device_info sienna_cichlid_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 92,
- .num_sdma_engines = 4,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -493,8 +447,6 @@ static const struct kfd_device_info navy_flounder_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 92,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -512,8 +464,6 @@ static const struct kfd_device_info vangogh_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 92,
- .num_sdma_engines = 1,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -531,8 +481,6 @@ static const struct kfd_device_info dimgrey_cavefish_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 92,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -550,8 +498,6 @@ static const struct kfd_device_info beige_goby_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 92,
- .num_sdma_engines = 1,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
@@ -569,8 +515,6 @@ static const struct kfd_device_info yellow_carp_device_info = {
.supports_cwsr = true,
.needs_pci_atomics = true,
.no_atomic_fw_version = 92,
- .num_sdma_engines = 1,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 2,
};
@@ -587,8 +531,6 @@ static const struct kfd_device_info cyan_skillfish_device_info = {
.needs_iommu_device = false,
.supports_cwsr = true,
.needs_pci_atomics = true,
- .num_sdma_engines = 2,
- .num_xgmi_sdma_engines = 0,
.num_sdma_queues_per_engine = 8,
};
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
index 1054fedd7b3c..836ec8860c1b 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
@@ -208,8 +208,6 @@ struct kfd_device_info {
bool needs_iommu_device;
bool needs_pci_atomics;
uint32_t no_atomic_fw_version;
- unsigned int num_sdma_engines;
- unsigned int num_xgmi_sdma_engines;
unsigned int num_sdma_queues_per_engine;
};
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* RE: [PATCH] drm/amdkfd: Remove unused entries in table
2021-11-18 20:18 [PATCH] drm/amdkfd: Remove unused entries in table Amber Lin
@ 2021-11-18 20:36 ` Sider, Graham
0 siblings, 0 replies; 2+ messages in thread
From: Sider, Graham @ 2021-11-18 20:36 UTC (permalink / raw)
To: Lin, Amber, amd-gfx@lists.freedesktop.org; +Cc: Lin, Amber
[AMD Official Use Only]
Reviewed-by: Graham Sider <Graham.Sider@amd.com>
> Remove unused entries in kfd_device_info table: num_xgmi_sdma_engines
> and num_sdma_queues_per_engine. They are calculated in
> kfd_get_num_sdma_engines and kfd_get_num_xgmi_sdma_engines
> instead.
>
> Signed-off-by: Amber Lin <Amber.Lin@amd.com>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_device.c | 58 -------------------------
> drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 2 -
> 2 files changed, 60 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> index 3fea47e37c17..e1294fba0c26 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> @@ -68,8 +68,6 @@ static const struct kfd_device_info kaveri_device_info =
> {
> .supports_cwsr = false,
> .needs_iommu_device = true,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -87,8 +85,6 @@ static const struct kfd_device_info carrizo_device_info =
> {
> .supports_cwsr = true,
> .needs_iommu_device = true,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -105,8 +101,6 @@ static const struct kfd_device_info raven_device_info
> = {
> .supports_cwsr = true,
> .needs_iommu_device = true,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 1,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, }; #endif @@ -126,8 +120,6 @@
> static const struct kfd_device_info hawaii_device_info = {
> .supports_cwsr = false,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, }; #endif @@ -145,8 +137,6 @@
> static const struct kfd_device_info tonga_device_info = {
> .supports_cwsr = false,
> .needs_iommu_device = false,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -163,8 +153,6 @@ static const struct kfd_device_info fiji_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -181,8 +169,6 @@ static const struct kfd_device_info fiji_vf_device_info
> = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -200,8 +186,6 @@ static const struct kfd_device_info
> polaris10_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -218,8 +202,6 @@ static const struct kfd_device_info
> polaris10_vf_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -236,8 +218,6 @@ static const struct kfd_device_info
> polaris11_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -254,8 +234,6 @@ static const struct kfd_device_info
> polaris12_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -272,8 +250,6 @@ static const struct kfd_device_info
> vegam_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -290,8 +266,6 @@ static const struct kfd_device_info
> vega10_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -308,8 +282,6 @@ static const struct kfd_device_info
> vega10_vf_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -326,8 +298,6 @@ static const struct kfd_device_info
> vega12_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -344,8 +314,6 @@ static const struct kfd_device_info
> vega20_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -362,8 +330,6 @@ static const struct kfd_device_info
> arcturus_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 6,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -380,8 +346,6 @@ static const struct kfd_device_info
> aldebaran_device_info = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 3,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -398,8 +362,6 @@ static const struct kfd_device_info renoir_device_info
> = {
> .supports_cwsr = true,
> .needs_iommu_device = false,
> .needs_pci_atomics = false,
> - .num_sdma_engines = 1,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -417,8 +379,6 @@ static const struct kfd_device_info
> navi10_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 145,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -436,8 +396,6 @@ static const struct kfd_device_info
> navi12_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 145,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -455,8 +413,6 @@ static const struct kfd_device_info
> navi14_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 145,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -474,8 +430,6 @@ static const struct kfd_device_info
> sienna_cichlid_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 92,
> - .num_sdma_engines = 4,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -493,8 +447,6 @@ static const struct kfd_device_info
> navy_flounder_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 92,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -512,8 +464,6 @@ static const struct kfd_device_info
> vangogh_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 92,
> - .num_sdma_engines = 1,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -531,8 +481,6 @@ static const struct kfd_device_info
> dimgrey_cavefish_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 92,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -550,8 +498,6 @@ static const struct kfd_device_info
> beige_goby_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 92,
> - .num_sdma_engines = 1,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> @@ -569,8 +515,6 @@ static const struct kfd_device_info
> yellow_carp_device_info = {
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> .no_atomic_fw_version = 92,
> - .num_sdma_engines = 1,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 2, };
>
> @@ -587,8 +531,6 @@ static const struct kfd_device_info
> cyan_skillfish_device_info = {
> .needs_iommu_device = false,
> .supports_cwsr = true,
> .needs_pci_atomics = true,
> - .num_sdma_engines = 2,
> - .num_xgmi_sdma_engines = 0,
> .num_sdma_queues_per_engine = 8, };
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> index 1054fedd7b3c..836ec8860c1b 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> @@ -208,8 +208,6 @@ struct kfd_device_info {
> bool needs_iommu_device;
> bool needs_pci_atomics;
> uint32_t no_atomic_fw_version;
> - unsigned int num_sdma_engines;
> - unsigned int num_xgmi_sdma_engines;
> unsigned int num_sdma_queues_per_engine; };
>
> --
> 2.17.1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-11-18 20:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-18 20:18 [PATCH] drm/amdkfd: Remove unused entries in table Amber Lin
2021-11-18 20:36 ` Sider, Graham
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.