Re: [PATCH 2/3] drm/amdgpu: Remove two ! operations in an if condition
Am 30.05.2017 um 23:47 schrieb Alex Xie: Make the code easier to understand. Signed-off-by: Alex Xie--- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c index 7a323f9..9fdeb82 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c @@ -680,9 +680,11 @@ bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring, if (amdgpu_vm_had_gpu_reset(adev, id)) return true; - if (!vm_flush_needed && !gds_switch_needed) + + if (vm_flush_needed || gds_switch_needed) + return true; + else return false; - return true; Actually that should even use the canonical form, e.g. something like "return vm_flush_needed || gds_switch_needed;". Apart from that looks good to me, Christian. } /** ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 2/3] drm/amdgpu: Remove two ! operations in an if condition
On 31/05/17 06:47 AM, Alex Xie wrote: > Make the code easier to understand. > > Signed-off-by: Alex Xie> --- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > index 7a323f9..9fdeb82 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -680,9 +680,11 @@ bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring > *ring, > > if (amdgpu_vm_had_gpu_reset(adev, id)) > return true; > - if (!vm_flush_needed && !gds_switch_needed) > + > + if (vm_flush_needed || gds_switch_needed) > + return true; > + else > return false; This can be further simplified to return vm_flush_needed || gds_switch_needed; With that, Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 2/3] drm/amdgpu: Remove two ! operations in an if condition
On 2017年05月31日 05:47, Alex Xie wrote: Make the code easier to understand. Signed-off-by: Alex Xieok to me, Reviewed-by: Chunming Zhou --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c index 7a323f9..9fdeb82 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c @@ -680,9 +680,11 @@ bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring, if (amdgpu_vm_had_gpu_reset(adev, id)) return true; - if (!vm_flush_needed && !gds_switch_needed) + + if (vm_flush_needed || gds_switch_needed) + return true; + else return false; - return true; } /** ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx