Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Christian König
Am 13.05.22 um 12:38 schrieb Charan Teja Kalla: On 5/13/2022 3:59 PM, Christian König wrote: Am 13.05.22 um 12:18 schrieb Charan Teja Kalla: On 5/13/2022 3:41 PM, Greg KH wrote: Reported-by: kernel test robot The trest robot did not say that the dmabuf stat name was being duplicated, did

Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Charan Teja Kalla
On 5/13/2022 3:59 PM, Christian König wrote: > Am 13.05.22 um 12:18 schrieb Charan Teja Kalla: >> On 5/13/2022 3:41 PM, Greg KH wrote: Reported-by: kernel test robot >>> The trest robot did not say that the dmabuf stat name was being >>> duplicated, did it? >>> >> It reported a printk warni

Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Christian König
Am 13.05.22 um 12:18 schrieb Charan Teja Kalla: On 5/13/2022 3:41 PM, Greg KH wrote: Reported-by: kernel test robot The trest robot did not say that the dmabuf stat name was being duplicated, did it? It reported a printk warning on V2[1]. Should we remove this on V3? We only add the kernel

Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Greg KH
On Fri, May 13, 2022 at 03:48:23PM +0530, Charan Teja Kalla wrote: > > On 5/13/2022 3:41 PM, Greg KH wrote: > >> Reported-by: kernel test robot > > The trest robot did not say that the dmabuf stat name was being > > duplicated, did it? > > > > It reported a printk warning on V2[1]. Should we rem

Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Charan Teja Kalla
On 5/13/2022 3:41 PM, Greg KH wrote: >> Reported-by: kernel test robot > The trest robot did not say that the dmabuf stat name was being > duplicated, did it? > It reported a printk warning on V2[1]. Should we remove this on V3? @Christian: Could you please drop this tag while merging? [1] ht

Re: [PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Greg KH
On Fri, May 13, 2022 at 03:08:09PM +0530, Charan Teja Kalla wrote: > The dmabuf file uses get_next_ino()(through dma_buf_getfile() -> > alloc_anon_inode()) to get an inode number and uses the same as a > directory name under /sys/kernel/dmabuf/buffers/. This directory is > used to collect the dmabu

[PATCH V3] dma-buf: ensure unique directory name for dmabuf stats

2022-05-13 Thread Charan Teja Kalla
The dmabuf file uses get_next_ino()(through dma_buf_getfile() -> alloc_anon_inode()) to get an inode number and uses the same as a directory name under /sys/kernel/dmabuf/buffers/. This directory is used to collect the dmabuf stats and it is created through dma_buf_stats_setup(). At current, failur