Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-03-23 Thread Dmitry Osipenko
23.03.2021 22:57, Dmitry Osipenko пишет: > 23.03.2021 21:21, Thierry Reding пишет: >> On Sat, Feb 27, 2021 at 02:19:39PM +0300, Dmitry Osipenko wrote: >>> 03.02.2021 14:18, Mikko Perttunen пишет: >>> ... > I'll need more time to think about it. > How about something like this: >>>

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-03-23 Thread Dmitry Osipenko
23.03.2021 21:21, Thierry Reding пишет: > On Sat, Feb 27, 2021 at 02:19:39PM +0300, Dmitry Osipenko wrote: >> 03.02.2021 14:18, Mikko Perttunen пишет: >> ... I'll need more time to think about it. >>> >>> How about something like this: >>> >>> Turn the syncpt_incr field back into an array

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-03-23 Thread Thierry Reding
On Sat, Feb 27, 2021 at 02:19:39PM +0300, Dmitry Osipenko wrote: > 03.02.2021 14:18, Mikko Perttunen пишет: > ... > >> I'll need more time to think about it. > >> > > > > How about something like this: > > > > Turn the syncpt_incr field back into an array of structs like > > > > #define DRM_TEGR

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-03-01 Thread Mikko Perttunen
On 2/27/21 1:19 PM, Dmitry Osipenko wrote: 03.02.2021 14:18, Mikko Perttunen пишет: ... I'll need more time to think about it. How about something like this: Turn the syncpt_incr field back into an array of structs like #define DRM_TEGRA_SUBMIT_SYNCPT_INCR_REPLACE_SYNCOBJ    (1<<0) #def

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-02-27 Thread Dmitry Osipenko
03.02.2021 14:18, Mikko Perttunen пишет: ... >> I'll need more time to think about it. >> > > How about something like this: > > Turn the syncpt_incr field back into an array of structs like > > #define DRM_TEGRA_SUBMIT_SYNCPT_INCR_REPLACE_SYNCOBJ    (1<<0) > #define DRM_TEGRA_SUBMIT_SYNCPT_

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-02-03 Thread Mikko Perttunen
On 1/29/21 7:30 PM, Dmitry Osipenko wrote: 28.01.2021 19:58, Thierry Reding пишет: On Thu, Jan 28, 2021 at 01:08:54PM +0200, Mikko Perttunen wrote: On 1/27/21 11:20 PM, Dmitry Osipenko wrote: 26.01.2021 05:45, Mikko Perttunen пишет: 2. We will probably need a dedicated drm_tegra_submit_cmd fo

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-01-30 Thread Dmitry Osipenko
28.01.2021 19:58, Thierry Reding пишет: > On Thu, Jan 28, 2021 at 01:08:54PM +0200, Mikko Perttunen wrote: >> On 1/27/21 11:20 PM, Dmitry Osipenko wrote: >>> 26.01.2021 05:45, Mikko Perttunen пишет: > 2. We will probably need a dedicated drm_tegra_submit_cmd for sync point > increments.  Th

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-01-28 Thread Thierry Reding
On Thu, Jan 28, 2021 at 01:08:54PM +0200, Mikko Perttunen wrote: > On 1/27/21 11:20 PM, Dmitry Osipenko wrote: > > 26.01.2021 05:45, Mikko Perttunen пишет: > > > > 2. We will probably need a dedicated drm_tegra_submit_cmd for sync point > > > > increments.  The job's sync point will be allocated dy

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-01-28 Thread Mikko Perttunen
On 1/27/21 11:20 PM, Dmitry Osipenko wrote: 26.01.2021 05:45, Mikko Perttunen пишет: 2. We will probably need a dedicated drm_tegra_submit_cmd for sync point increments.  The job's sync point will be allocated dynamically when job is submitted.  We will need a fag for the sync_incr and wait_sync

Re: [PATCH v5 00/21] Host1x sync point UAPI should not be used for tracking DRM jobs

2021-01-28 Thread Dmitry Osipenko
26.01.2021 05:45, Mikko Perttunen пишет: >> 2. We will probably need a dedicated drm_tegra_submit_cmd for sync point >> increments.  The job's sync point will be allocated dynamically when job >> is submitted.  We will need a fag for the sync_incr and wait_syncpt >> commands, saying "it's a job's s