public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mengchi Cheng <mengcc@amazon.com>
To: <sumit.semwal@linaro.org>, <christian.koenig@amd.com>
Cc: <linux-media@vger.kernel.org>, <kamatam@amazon.com>,
	Mengchi Cheng <mengcc@amazon.com>
Subject: Inconsistent fence info between 6.1 and 5.15 kernel
Date: Mon, 20 Nov 2023 16:50:14 -0800	[thread overview]
Message-ID: <20231121005014.4074003-1-mengcc@amazon.com> (raw)

Hello,                                                                  
                                                                        
We are upleveling kernel from 5.15.32 to 6.1.62 for our system. The     
kernels we use are all vanila kernel from upstream without any extra    
patches. But we see our graphics application crashed in the 6.1.        
                                                                        
And later we found that 6.1 has DMA_BUF_IOCTL_EXPORT_SYNC_FILE option   
added into dma_buf_ioctl. It initialized dma_fence_stub and returned    
with a sync_file. So then our app called sync_file_ioctl_fence_info to  
inquire fence info, it returns info of dma_fence_stub, which was not    
expected. As the result, the app aborted because of it.                 
                                                                        
Meanwhile, with the 5.15 kernel, our app somehow also called            
DMA_BUF_IOCTL_EXPORT_SYNC_FILE. Since it returned -ENOTTY, the app did  
not further ask for fence info.                                         
                                                                        
I am not a graphics engineer. I do not know if userspace is handling    
dma_fence_stub correctly. But since the addition of                     
DMA_BUF_IOCTL_EXPORT_SYNC_FILE is causing differnet behavior, I am      
wondering if there is a way to keep things consistent in the kernel.    
For example, add a macro to provide a choice disabling it? There is
probably a better solution to it. Look forward to your reply.                                                          
                                                                        
                                                                        
Best,                                                                   
Mengchi Cheng

-- 
2.40.1


             reply	other threads:[~2023-11-21  0:50 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-21  0:50 Mengchi Cheng [this message]
2023-11-21  8:17 ` Inconsistent fence info between 6.1 and 5.15 kernel Christian König

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231121005014.4074003-1-mengcc@amazon.com \
    --to=mengcc@amazon.com \
    --cc=christian.koenig@amd.com \
    --cc=kamatam@amazon.com \
    --cc=linux-media@vger.kernel.org \
    --cc=sumit.semwal@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox