lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 12 Sep 2022 08:44:27 +0200 From: Christian König <christian.koenig@....com> To: Arvind Yadav <Arvind.Yadav@....com>, andrey.grodzovsky@....com, shashank.sharma@....com, amaranath.somalapuram@....com, Arunpravin.PaneerSelvam@....com, sumit.semwal@...aro.org, gustavo@...ovan.org, airlied@...ux.ie, daniel@...ll.ch, linux-media@...r.kernel.org, dri-devel@...ts.freedesktop.org, linaro-mm-sig@...ts.linaro.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v3 1/6] dma-buf: Remove the signaled bit status check Am 09.09.22 um 19:08 schrieb Arvind Yadav: > Remove the signaled bit status check because it is returning > early when the fence is already signaled and > __dma_fence_enable_signaling is checking the status of signaled > bit again. > > Signed-off-by: Arvind Yadav <Arvind.Yadav@....com> Reviewed-by: Christian König <christian.koenig@....com> > --- > > Changes in v1, v2: This new patch was not part of previous series. > > --- > drivers/dma-buf/dma-fence.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c > index 066400ed8841..64c99739ad23 100644 > --- a/drivers/dma-buf/dma-fence.c > +++ b/drivers/dma-buf/dma-fence.c > @@ -601,9 +601,6 @@ void dma_fence_enable_sw_signaling(struct dma_fence *fence) > { > unsigned long flags; > > - if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags)) > - return; > - > spin_lock_irqsave(fence->lock, flags); > __dma_fence_enable_signaling(fence); > spin_unlock_irqrestore(fence->lock, flags);
Powered by blists - more mailing lists