linux-kernel - Re: [PATCH v4] drm/panthor: Add DEV_QUERY_TIMESTAMP

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240905093923.64ca39e1@collabora.com>
Date: Thu, 5 Sep 2024 09:39:23 +0200
From: Boris Brezillon <boris.brezillon@...labora.com>
To: Mary Guillemard <mary.guillemard@...labora.com>
Cc: linux-kernel@...r.kernel.org, kernel@...labora.com, Mihail Atanassov
 <mihail.atanassov@....com>, Steven Price <steven.price@....com>, Liviu
 Dudau <liviu.dudau@....com>, Maarten Lankhorst
 <maarten.lankhorst@...ux.intel.com>, Maxime Ripard <mripard@...nel.org>,
 Thomas Zimmermann <tzimmermann@...e.de>, David Airlie <airlied@...il.com>,
 Daniel Vetter <daniel@...ll.ch>, dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v4] drm/panthor: Add DEV_QUERY_TIMESTAMP_INFO dev query

On Wed, 4 Sep 2024 12:19:11 +0200
Boris Brezillon <boris.brezillon@...labora.com> wrote:

> On Fri, 30 Aug 2024 10:03:50 +0200
> Mary Guillemard <mary.guillemard@...labora.com> wrote:
> 
> > Expose timestamp information supported by the GPU with a new device
> > query.
> > 
> > Mali uses an external timer as GPU system time. On ARM, this is wired to
> > the generic arch timer so we wire cntfrq_el0 as device frequency.
> > 
> > This new uAPI will be used in Mesa to implement timestamp queries and
> > VK_KHR_calibrated_timestamps.
> > 
> > Since this extends the uAPI and because userland needs a way to advertise
> > those features conditionally, this also bumps the driver minor version.
> > 
> > v2:
> > - Rewrote to use GPU timestamp register
> > - Added timestamp_offset to drm_panthor_timestamp_info
> > - Add missing include for arch_timer_get_cntfrq
> > - Rework commit message
> > 
> > v3:
> > - Add panthor_gpu_read_64bit_counter
> > - Change panthor_gpu_read_timestamp to use
> >   panthor_gpu_read_64bit_counter
> > 
> > v4:
> > - Fix multiple typos in uAPI documentation
> > - Mention behavior when the timestamp frequency is unknown
> > - Use u64 instead of unsigned long long
> >   for panthor_gpu_read_timestamp
> > - Apply r-b from Mihail
> > 
> > Signed-off-by: Mary Guillemard <mary.guillemard@...labora.com>
> > Reviewed-by: Mihail Atanassov <mihail.atanassov@....com>  
> 
> Reviewed-by: Boris Brezillon <boris.brezillon@...labora.com>

Queued to drm-misc-next.