* [PATCH v2 7/8] dma-buf: remove useless FMODE_LSEEK flag [not found] <20220625110115.39956-1-Jason@zx2c4.com> @ 2022-06-25 11:01 ` Jason A. Donenfeld 2022-06-27 9:08 ` Daniel Vetter 0 siblings, 1 reply; 4+ messages in thread From: Jason A. Donenfeld @ 2022-06-25 11:01 UTC (permalink / raw) To: linux-kernel, viro, Jens Axboe, linux-fsdevel Cc: Jason A. Donenfeld, Sumit Semwal, dri-devel, Christian König This is already set by anon_inode_getfile(), since dma_buf_fops has non-NULL ->llseek, so we don't need to set it here too. Suggested-by: Al Viro <viro@zeniv.linux.org.uk> Cc: Sumit Semwal <sumit.semwal@linaro.org> Cc: Christian König <christian.koenig@amd.com> Cc: dri-devel@lists.freedesktop.org Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> --- drivers/dma-buf/dma-buf.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c index 32f55640890c..3f08e0b960ec 100644 --- a/drivers/dma-buf/dma-buf.c +++ b/drivers/dma-buf/dma-buf.c @@ -549,7 +549,6 @@ struct dma_buf *dma_buf_export(const struct dma_buf_export_info *exp_info) goto err_dmabuf; } - file->f_mode |= FMODE_LSEEK; dmabuf->file = file; mutex_init(&dmabuf->lock); -- 2.35.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 7/8] dma-buf: remove useless FMODE_LSEEK flag 2022-06-25 11:01 ` [PATCH v2 7/8] dma-buf: remove useless FMODE_LSEEK flag Jason A. Donenfeld @ 2022-06-27 9:08 ` Daniel Vetter 2022-06-27 9:38 ` Jason A. Donenfeld 2022-06-29 10:30 ` Sumit Semwal 0 siblings, 2 replies; 4+ messages in thread From: Daniel Vetter @ 2022-06-27 9:08 UTC (permalink / raw) To: Jason A. Donenfeld Cc: Jens Axboe, linux-kernel, dri-devel, Christian König, viro, linux-fsdevel, Sumit Semwal On Sat, Jun 25, 2022 at 01:01:14PM +0200, Jason A. Donenfeld wrote: > This is already set by anon_inode_getfile(), since dma_buf_fops has > non-NULL ->llseek, so we don't need to set it here too. > > Suggested-by: Al Viro <viro@zeniv.linux.org.uk> > Cc: Sumit Semwal <sumit.semwal@linaro.org> > Cc: Christian König <christian.koenig@amd.com> > Cc: dri-devel@lists.freedesktop.org > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> I'm assuming this is part of a vfs cleanup and lands through that tree? For that: Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/dma-buf/dma-buf.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c > index 32f55640890c..3f08e0b960ec 100644 > --- a/drivers/dma-buf/dma-buf.c > +++ b/drivers/dma-buf/dma-buf.c > @@ -549,7 +549,6 @@ struct dma_buf *dma_buf_export(const struct dma_buf_export_info *exp_info) > goto err_dmabuf; > } > > - file->f_mode |= FMODE_LSEEK; > dmabuf->file = file; > > mutex_init(&dmabuf->lock); > -- > 2.35.1 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 7/8] dma-buf: remove useless FMODE_LSEEK flag 2022-06-27 9:08 ` Daniel Vetter @ 2022-06-27 9:38 ` Jason A. Donenfeld 2022-06-29 10:30 ` Sumit Semwal 1 sibling, 0 replies; 4+ messages in thread From: Jason A. Donenfeld @ 2022-06-27 9:38 UTC (permalink / raw) To: linux-kernel, viro, Jens Axboe, linux-fsdevel, Sumit Semwal, dri-devel, Christian König Hi Daniel, On Mon, Jun 27, 2022 at 11:08:32AM +0200, Daniel Vetter wrote: > On Sat, Jun 25, 2022 at 01:01:14PM +0200, Jason A. Donenfeld wrote: > > This is already set by anon_inode_getfile(), since dma_buf_fops has > > non-NULL ->llseek, so we don't need to set it here too. > > > > Suggested-by: Al Viro <viro@zeniv.linux.org.uk> > > Cc: Sumit Semwal <sumit.semwal@linaro.org> > > Cc: Christian König <christian.koenig@amd.com> > > Cc: dri-devel@lists.freedesktop.org > > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> > > I'm assuming this is part of a vfs cleanup and lands through that tree? > For that: > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> With the exception of the first patch (which is more urgent), yes, that is my assumption too. Jason ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 7/8] dma-buf: remove useless FMODE_LSEEK flag 2022-06-27 9:08 ` Daniel Vetter 2022-06-27 9:38 ` Jason A. Donenfeld @ 2022-06-29 10:30 ` Sumit Semwal 1 sibling, 0 replies; 4+ messages in thread From: Sumit Semwal @ 2022-06-29 10:30 UTC (permalink / raw) To: Jason A. Donenfeld, linux-kernel, viro, Jens Axboe, linux-fsdevel, Sumit Semwal, dri-devel, Christian König On Mon, 27 Jun 2022 at 14:38, Daniel Vetter <daniel@ffwll.ch> wrote: > > On Sat, Jun 25, 2022 at 01:01:14PM +0200, Jason A. Donenfeld wrote: > > This is already set by anon_inode_getfile(), since dma_buf_fops has > > non-NULL ->llseek, so we don't need to set it here too. > > > > Suggested-by: Al Viro <viro@zeniv.linux.org.uk> > > Cc: Sumit Semwal <sumit.semwal@linaro.org> > > Cc: Christian König <christian.koenig@amd.com> > > Cc: dri-devel@lists.freedesktop.org > > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> > > I'm assuming this is part of a vfs cleanup and lands through that tree? > For that: > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> FWIW, please feel free to add Acked-by: Sumit Semwal <sumit.semwal@linaro.org> > > > --- > > drivers/dma-buf/dma-buf.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c > > index 32f55640890c..3f08e0b960ec 100644 > > --- a/drivers/dma-buf/dma-buf.c > > +++ b/drivers/dma-buf/dma-buf.c > > @@ -549,7 +549,6 @@ struct dma_buf *dma_buf_export(const struct dma_buf_export_info *exp_info) > > goto err_dmabuf; > > } > > > > - file->f_mode |= FMODE_LSEEK; > > dmabuf->file = file; > > > > mutex_init(&dmabuf->lock); > > -- > > 2.35.1 > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch Best, Sumit. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-06-29 20:39 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20220625110115.39956-1-Jason@zx2c4.com>
2022-06-25 11:01 ` [PATCH v2 7/8] dma-buf: remove useless FMODE_LSEEK flag Jason A. Donenfeld
2022-06-27 9:08 ` Daniel Vetter
2022-06-27 9:38 ` Jason A. Donenfeld
2022-06-29 10:30 ` Sumit Semwal
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox