Re: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV
On 12/9/2021 2:46 PM, Chen, Guchun wrote: [Public] Hi Lijo, The check is not necessary. It has a guard by for loop in the caller. for (i = 0; i < adev->vcn.num_vcn_inst; ++i) { ... if (amdgpu_vcn_is_disabled_vcn(adev, VCN_ENCODE_RING, i)) { .. } Thanks for the clarification Guchun. Reviewed-by: Lijo Lazar Regards, Guchun -Original Message- From: Lazar, Lijo Sent: Thursday, December 9, 2021 4:53 PM To: Shi, Leslie ; amd-gfx@lists.freedesktop.org Cc: Chen, Guchun Subject: Re: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV On 12/9/2021 1:56 PM, Leslie Shi wrote: Guest OS will setup VCN instance 1 which is disabled as an enabled instance and execute initialization work on it, but this causes VCN ib ring test failure on the disabled VCN instance during modprobe: amdgpu :00:08.0: amdgpu: ring vcn_enc_1.0 uses VM inv eng 5 on hub 1 amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test failed on vcn_dec_0 (-110). amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test failed on vcn_enc_0.0 (-110). [drm:amdgpu_device_delayed_init_work_handler [amdgpu]] *ERROR* ib ring test failed (-110). v2: drop amdgpu_discovery_get_vcn_version and rename sriov_config to vcn_config v3: modify VCN's revision in SR-IOV and bare-metal Fixes: 36b7d5646476 ("drm/amdgpu: handle SRIOV VCN revision parsing") Signed-off-by: Leslie Shi --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 29 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h | 2 -- drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 15 +++--- drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.h | 2 +- 4 files changed, 14 insertions(+), 34 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c index 552031950518..f31bc0187394 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c @@ -380,18 +380,15 @@ int amdgpu_discovery_reg_base_init(struct amdgpu_device *adev) ip->revision); if (le16_to_cpu(ip->hw_id) == VCN_HWID) { - if (amdgpu_sriov_vf(adev)) { - /* SR-IOV modifies each VCN’s revision (uint8) -* Bit [5:0]: original revision value -* Bit [7:6]: en/decode capability: -* 0b00 : VCN function normally -* 0b10 : encode is disabled -* 0b01 : decode is disabled -*/ - adev->vcn.sriov_config[adev->vcn.num_vcn_inst] = - (ip->revision & 0xc0) >> 6; - ip->revision &= ~0xc0; - } + /* Bit [5:0]: original revision value +* Bit [7:6]: en/decode capability: +* 0b00 : VCN function normally +* 0b10 : encode is disabled +* 0b01 : decode is disabled +*/ + adev->vcn.vcn_config[adev->vcn.num_vcn_inst] = + ip->revision & 0xc0; + ip->revision &= ~0xc0; adev->vcn.num_vcn_inst++; } if (le16_to_cpu(ip->hw_id) == SDMA0_HWID || @@ -485,14 +482,6 @@ int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, int n return -EINVAL; } - -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int vcn_instance, -int *major, int *minor, int *revision) -{ - return amdgpu_discovery_get_ip_version(adev, VCN_HWID, - vcn_instance, major, minor, revision); -} - void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev) { struct binary_header *bhdr; diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h index 0ea029e3b850..14537cec19db 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h @@ -33,8 +33,6 @@ void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev); int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, int number_instance, int *major, int *minor, int *revision); -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int vcn_instance, -
Re: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV
On 12/9/2021 1:56 PM, Leslie Shi wrote: Guest OS will setup VCN instance 1 which is disabled as an enabled instance and execute initialization work on it, but this causes VCN ib ring test failure on the disabled VCN instance during modprobe: amdgpu :00:08.0: amdgpu: ring vcn_enc_1.0 uses VM inv eng 5 on hub 1 amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test failed on vcn_dec_0 (-110). amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test failed on vcn_enc_0.0 (-110). [drm:amdgpu_device_delayed_init_work_handler [amdgpu]] *ERROR* ib ring test failed (-110). v2: drop amdgpu_discovery_get_vcn_version and rename sriov_config to vcn_config v3: modify VCN's revision in SR-IOV and bare-metal Fixes: 36b7d5646476 ("drm/amdgpu: handle SRIOV VCN revision parsing") Signed-off-by: Leslie Shi --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 29 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h | 2 -- drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 15 +++--- drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.h | 2 +- 4 files changed, 14 insertions(+), 34 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c index 552031950518..f31bc0187394 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c @@ -380,18 +380,15 @@ int amdgpu_discovery_reg_base_init(struct amdgpu_device *adev) ip->revision); if (le16_to_cpu(ip->hw_id) == VCN_HWID) { - if (amdgpu_sriov_vf(adev)) { - /* SR-IOV modifies each VCN’s revision (uint8) -* Bit [5:0]: original revision value -* Bit [7:6]: en/decode capability: -* 0b00 : VCN function normally -* 0b10 : encode is disabled -* 0b01 : decode is disabled -*/ - adev->vcn.sriov_config[adev->vcn.num_vcn_inst] = - (ip->revision & 0xc0) >> 6; - ip->revision &= ~0xc0; - } + /* Bit [5:0]: original revision value +* Bit [7:6]: en/decode capability: +* 0b00 : VCN function normally +* 0b10 : encode is disabled +* 0b01 : decode is disabled +*/ + adev->vcn.vcn_config[adev->vcn.num_vcn_inst] = + ip->revision & 0xc0; + ip->revision &= ~0xc0; adev->vcn.num_vcn_inst++; } if (le16_to_cpu(ip->hw_id) == SDMA0_HWID || @@ -485,14 +482,6 @@ int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, int n return -EINVAL; } - -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int vcn_instance, -int *major, int *minor, int *revision) -{ - return amdgpu_discovery_get_ip_version(adev, VCN_HWID, - vcn_instance, major, minor, revision); -} - void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev) { struct binary_header *bhdr; diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h index 0ea029e3b850..14537cec19db 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h @@ -33,8 +33,6 @@ void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev); int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, int number_instance, int *major, int *minor, int *revision); -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int vcn_instance, -int *major, int *minor, int *revision); int amdgpu_discovery_get_gfx_info(struct amdgpu_device *adev); int amdgpu_discovery_set_ip_blocks(struct amdgpu_device *adev); diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c index 2658414c503d..38036cbf6203 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c @@ -284,20 +284,13 @@ int amdgpu_vcn_sw_fini(struct amdgpu_device *adev) bool amdgpu_vcn_is_disabled_vcn(struct amdgpu_device *adev, enum vcn_ring_type type, uint32_t vcn_instance) { bool ret = false; + int vcn_config =
RE: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV
[Public] Hi Lijo, The check is not necessary. It has a guard by for loop in the caller. for (i = 0; i < adev->vcn.num_vcn_inst; ++i) { ... if (amdgpu_vcn_is_disabled_vcn(adev, VCN_ENCODE_RING, i)) { .. } Regards, Guchun -Original Message- From: Lazar, Lijo Sent: Thursday, December 9, 2021 4:53 PM To: Shi, Leslie ; amd-gfx@lists.freedesktop.org Cc: Chen, Guchun Subject: Re: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV On 12/9/2021 1:56 PM, Leslie Shi wrote: > Guest OS will setup VCN instance 1 which is disabled as an enabled > instance and execute initialization work on it, but this causes VCN ib > ring test failure on the disabled VCN instance during modprobe: > > amdgpu :00:08.0: amdgpu: ring vcn_enc_1.0 uses VM inv eng 5 on hub > 1 amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test > failed on vcn_dec_0 (-110). > amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test > failed on vcn_enc_0.0 (-110). > [drm:amdgpu_device_delayed_init_work_handler [amdgpu]] *ERROR* ib ring test > failed (-110). > > v2: drop amdgpu_discovery_get_vcn_version and rename sriov_config to > vcn_config > v3: modify VCN's revision in SR-IOV and bare-metal > > Fixes: 36b7d5646476 ("drm/amdgpu: handle SRIOV VCN revision parsing") > Signed-off-by: Leslie Shi > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 29 ++- > drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h | 2 -- > drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 15 +++--- > drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.h | 2 +- > 4 files changed, 14 insertions(+), 34 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c > index 552031950518..f31bc0187394 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c > @@ -380,18 +380,15 @@ int amdgpu_discovery_reg_base_init(struct amdgpu_device > *adev) > ip->revision); > > if (le16_to_cpu(ip->hw_id) == VCN_HWID) { > - if (amdgpu_sriov_vf(adev)) { > - /* SR-IOV modifies each VCN’s revision > (uint8) > - * Bit [5:0]: original revision value > - * Bit [7:6]: en/decode capability: > - * 0b00 : VCN function normally > - * 0b10 : encode is disabled > - * 0b01 : decode is disabled > - */ > - > adev->vcn.sriov_config[adev->vcn.num_vcn_inst] = > - (ip->revision & 0xc0) >> 6; > - ip->revision &= ~0xc0; > - } > + /* Bit [5:0]: original revision value > + * Bit [7:6]: en/decode capability: > + * 0b00 : VCN function normally > + * 0b10 : encode is disabled > + * 0b01 : decode is disabled > + */ > + adev->vcn.vcn_config[adev->vcn.num_vcn_inst] = > + ip->revision & 0xc0; > + ip->revision &= ~0xc0; > adev->vcn.num_vcn_inst++; > } > if (le16_to_cpu(ip->hw_id) == SDMA0_HWID || @@ -485,14 > +482,6 @@ > int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, > int n > return -EINVAL; > } > > - > -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int > vcn_instance, > - int *major, int *minor, int *revision) > -{ > - return amdgpu_discovery_get_ip_version(adev, VCN_HWID, > -vcn_instance, major, minor, > revision); > -} > - > void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev) > { > struct binary_header *bhdr; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h > b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h > index 0ea029e3b850..14537cec19db 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h > @@ -33,8 +33,6 @@ void amdgpu_discovery_harvest_ip(struct amdgpu_device > *adev); > in
RE: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV
[Public] Reviewed-by: Guchun Chen Regards, Guchun -Original Message- From: Shi, Leslie Sent: Thursday, December 9, 2021 4:27 PM To: Lazar, Lijo ; amd-gfx@lists.freedesktop.org Cc: Chen, Guchun ; Shi, Leslie Subject: [PATCH v3] drm/amdgpu: fix incorrect VCN revision in SRIOV Guest OS will setup VCN instance 1 which is disabled as an enabled instance and execute initialization work on it, but this causes VCN ib ring test failure on the disabled VCN instance during modprobe: amdgpu :00:08.0: amdgpu: ring vcn_enc_1.0 uses VM inv eng 5 on hub 1 amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test failed on vcn_dec_0 (-110). amdgpu :00:08.0: [drm:amdgpu_ib_ring_tests [amdgpu]] *ERROR* IB test failed on vcn_enc_0.0 (-110). [drm:amdgpu_device_delayed_init_work_handler [amdgpu]] *ERROR* ib ring test failed (-110). v2: drop amdgpu_discovery_get_vcn_version and rename sriov_config to vcn_config v3: modify VCN's revision in SR-IOV and bare-metal Fixes: 36b7d5646476 ("drm/amdgpu: handle SRIOV VCN revision parsing") Signed-off-by: Leslie Shi --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 29 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h | 2 -- drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 15 +++--- drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.h | 2 +- 4 files changed, 14 insertions(+), 34 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c index 552031950518..f31bc0187394 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c @@ -380,18 +380,15 @@ int amdgpu_discovery_reg_base_init(struct amdgpu_device *adev) ip->revision); if (le16_to_cpu(ip->hw_id) == VCN_HWID) { - if (amdgpu_sriov_vf(adev)) { - /* SR-IOV modifies each VCN’s revision (uint8) -* Bit [5:0]: original revision value -* Bit [7:6]: en/decode capability: -* 0b00 : VCN function normally -* 0b10 : encode is disabled -* 0b01 : decode is disabled -*/ - adev->vcn.sriov_config[adev->vcn.num_vcn_inst] = - (ip->revision & 0xc0) >> 6; - ip->revision &= ~0xc0; - } + /* Bit [5:0]: original revision value +* Bit [7:6]: en/decode capability: +* 0b00 : VCN function normally +* 0b10 : encode is disabled +* 0b01 : decode is disabled +*/ + adev->vcn.vcn_config[adev->vcn.num_vcn_inst] = + ip->revision & 0xc0; + ip->revision &= ~0xc0; adev->vcn.num_vcn_inst++; } if (le16_to_cpu(ip->hw_id) == SDMA0_HWID || @@ -485,14 +482,6 @@ int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, int n return -EINVAL; } - -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int vcn_instance, -int *major, int *minor, int *revision) -{ - return amdgpu_discovery_get_ip_version(adev, VCN_HWID, - vcn_instance, major, minor, revision); -} - void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev) { struct binary_header *bhdr; diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h index 0ea029e3b850..14537cec19db 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.h @@ -33,8 +33,6 @@ void amdgpu_discovery_harvest_ip(struct amdgpu_device *adev); int amdgpu_discovery_get_ip_version(struct amdgpu_device *adev, int hw_id, int number_instance, int *major, int *minor, int *revision); -int amdgpu_discovery_get_vcn_version(struct amdgpu_device *adev, int vcn_instance, -int *major, int *minor, int *revision); int amdgpu_discovery_get_gfx_info(struct amdgpu_device *adev); int amdgpu_discovery_set_ip_blocks(struct amdgpu_device *adev); diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c index 2658414c503d..38036cbf6203 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c @@