linux-kernel - Re: [PATCH v11 2/4] drm: Add API for capturing frame CRCs

Open Source and information security mailing list archives

Date:   Tue, 11 Oct 2016 12:12:24 +0200
From:   Benjamin Gaignard <benjamin.gaignard@...aro.org>
To:     Emil Velikov <emil.l.velikov@...il.com>
Cc:     Tomeu Vizoso <tomeu.vizoso@...labora.com>,
        linux-doc@...r.kernel.org, Jonathan Corbet <corbet@....net>,
        "Linux-Kernel@...r. Kernel. Org" <linux-kernel@...r.kernel.org>,
        ML dri-devel <dri-devel@...ts.freedesktop.org>,
        Daniel Vetter <daniel.vetter@...el.com>
Subject: Re: [PATCH v11 2/4] drm: Add API for capturing frame CRCs

Hello,

I have been able to test it on my setup and, after implementing
set_crc_source function,
I have crc/data and crc/control entries for each of my crtc.

"cat /sys/kernel/debug/dri/0/crtc-0/crc/data" is blocking but I'm
really sure of my driver implementation...

Anyway you could add my ack on this patch

Regards,
Benjamin

2016-10-10 15:12 GMT+02:00 Emil Velikov <emil.l.velikov@...il.com>:
> On 6 October 2016 at 16:21, Tomeu Vizoso <tomeu.vizoso@...labora.com> wrote:
>> Adds files and directories to debugfs for controlling and reading frame
>> CRCs, per CRTC:
>>
>> dri/0/crtc-0/crc
>> dri/0/crtc-0/crc/control
>> dri/0/crtc-0/crc/data
>>
>> Drivers can implement the set_crc_source callback() in drm_crtc_funcs to
>> start and stop generating frame CRCs and can add entries to the output
>> by calling drm_crtc_add_crc_entry.
>>
>> v2:
>>     - Lots of good fixes suggested by Thierry.
>>     - Added documentation.
>>     - Changed the debugfs layout.
>>     - Moved to allocate the entries circular queue once when frame
>>       generation gets enabled for the first time.
>> v3:
>>     - Use the control file just to select the source, and start and stop
>>       capture when the data file is opened and closed, respectively.
>>     - Make variable the number of CRC values per entry, per source.
>>     - Allocate entries queue each time we start capturing as now there
>>       isn't a fixed number of CRC values per entry.
>>     - Store the frame counter in the data file as a 8-digit hex number.
>>     - For sources that cannot provide useful frame numbers, place
>>       XXXXXXXX in the frame field.
>>
>> v4:
>>     - Build only if CONFIG_DEBUG_FS is enabled.
>>     - Use memdup_user_nul.
>>     - Consolidate calculation of the size of an entry in a helper.
>>     - Add 0x prefix to hex numbers in the data file.
>>     - Remove unnecessary snprintf and strlen usage in read callback.
>>
>> v5:
>>     - Made the crcs array in drm_crtc_crc_entry fixed-size
>>     - Lots of other smaller improvements suggested by Emil Velikov
>>
>> v7:
>>     - Move definition of drm_debugfs_crtc_crc_add to drm_internal.h
>>
>> v8:
>>     - Call debugfs_remove_recursive when we fail to create the minor
>>       device
>>
>> v9:
>>     - Register the debugfs directory for a crtc from
>>       drm_crtc_register_all()
>>
>> v10:
>>     - Don't let debugfs failures interrupt CRTC registration (Emil
>>       Velikov)
>>
>> v11:
>>     - Remove extra brace that broke compilation. Sorry!
>>
>> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@...labora.com>
>
> Reviewed-by: Emil Velikov <emil.velikov@...labora.com>
>
> Emil
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Benjamin Gaignard

Graphic Study Group

Linaro.org │ Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog

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