Re: [PATCH] dma-buf: insert memory barrier before updating num_fences

2025-05-14 Thread Christian König
On 5/13/25 04:06, Hyejeong Choi wrote: > smp_store_mb() inserts memory barrier after storing operation. > It is different with what the comment is originally aiming so Null > pointer dereference can be happened if memory update is reordered. > > Signed-off-by: Hyejeong Choi I've reviewed, add CC

Re: [PATCH] dma-buf: insert memory barrier before updating num_fences

2025-05-12 Thread Hyejeong Choi
Thank you for the review. I re-sent updated patch, please take a look. Best regards, Hyejeong On Mon, May 12, 2025 at 03:16:33PM +0200, Christian König wrote: > On 5/12/25 13:12, Hyejeong Choi wrote: > > smp_store_mb() inserts memory barrier after storing operation. > > It is different with what

[PATCH] dma-buf: insert memory barrier before updating num_fences

2025-05-12 Thread Hyejeong Choi
smp_store_mb() inserts memory barrier after storing operation. It is different with what the comment is originally aiming so Null pointer dereference can be happened if memory update is reordered. Signed-off-by: Hyejeong Choi --- drivers/dma-buf/dma-resv.c | 5 +++-- 1 file changed, 3 insertions

[PATCH] dma-buf: insert memory barrier before updating num_fences

2025-05-12 Thread Hyejeong Choi
smp_store_mb() inserts memory barrier after storing operation. It is different with what the comment is originally aiming so Null pointer dereference can be happened if memory update is reordered. Signed-off-by: Hyejeong Choi --- drivers/dma-buf/dma-resv.c | 5 +++-- 1 file changed, 3 insertions

Re: [PATCH] dma-buf: insert memory barrier before updating num_fences

2025-05-12 Thread Christian König
On 5/12/25 13:12, Hyejeong Choi wrote: > smp_store_mb() inserts memory barrier after storing operation. > It is different with what the comment is originally aiming so Null > pointer dereference can be happened if memory update is reordered. > > Signed-off-by: Hyejeong Choi > --- > drivers/dma-b