From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tobias Jakobi Subject: Re: [PATCH 00/13] drm/exynos: async G2D and g2d_move() Date: Wed, 7 Oct 2015 20:32:22 +0200 Message-ID: <56156536.6070906@math.uni-bielefeld.de> References: <1442937302-8211-1-git-send-email-tjakobi@math.uni-bielefeld.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.math.uni-bielefeld.de ([129.70.45.10]:41362 "EHLO smtp.math.uni-bielefeld.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751575AbbJGSc1 (ORCPT ); Wed, 7 Oct 2015 14:32:27 -0400 In-Reply-To: <1442937302-8211-1-git-send-email-tjakobi@math.uni-bielefeld.de> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: linux-samsung-soc@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, emil.l.velikov@gmail.com, jy0922.shim@samsung.com, gustavo.padovan@collabora.co.uk, inki.dae@samsung.com Gentle ping! :-) - Tobias Tobias Jakobi wrote: > Hello, > > this series mostly touches G2D code. It introduces the following: > > (1) drmHandleEvent2() is added to enable processing of vendor-specific > events. This will be used to expose asynchronous operation of the > G2D. The necessary kernel infrastructure is already there since > a lot of kernel versions. [This touches libdrm core code!] > > (2) The necessary infrastructure to handle G2D events. This includes > adding g2d_config_event() and g2d_exec2() to the public API. > A test application is provided to ensure that everything works > as expected. > > (3) A small performance test application which can be used to measure > the speed of solid color clear operations. Interesting for > benchmarking and plotting colorful graphs (e.g. through > Mathematica). > > (4) g2d_move() which works similar to g2d_copy() but like the C > memmove() properly handles overlapping buffer copies. > Again a test application is present to check that this > indeed does what it should. > > (5) Various small changes. A framebuffer colorformat fix for the > general G2D test application. Moving the currently unused > g2d_reset() to the public API. Adding a counterpart to > exynos_bo_map() to unmap buffers again. > > (6) Last but not least a small bump of the Exynos version number. > > Please review and let me know what I should change/improve. > > > With best wishes, > Tobias > > P.S.: Most patches were submitted already some time ago but never > made it upstream. So if something looks familiar, don't worry! ;) > > Tobias Jakobi (13): > drm: Implement drmHandleEvent2() > exynos: Introduce exynos_handle_event() > tests/exynos: add fimg2d performance analysis > exynos/fimg2d: add g2d_config_event > exynos: fimg2d: add g2d_exec2 > tests/exynos: add fimg2d event test > tests/exynos: use XRGB8888 for framebuffer > exynos: fimg2d: add g2d_set_direction > exynos/fimg2d: add g2d_move > tests/exynos: add test for g2d_move > exynos/fimg2d: add exynos_bo_unmap() > exynos/fimg2d: add g2d_reset() to public API > exynos: bump version number > > exynos/exynos-symbol-check | 5 + > exynos/exynos_drm.c | 48 ++++++ > exynos/exynos_drm.h | 12 ++ > exynos/exynos_drmif.h | 27 +++ > exynos/exynos_fimg2d.c | 164 +++++++++++++++++-- > exynos/exynos_fimg2d.h | 49 ++++++ > exynos/libdrm_exynos.pc.in | 2 +- > tests/exynos/Makefile.am | 26 ++- > tests/exynos/exynos_fimg2d_event.c | 326 +++++++++++++++++++++++++++++++++++++ > tests/exynos/exynos_fimg2d_perf.c | 320 ++++++++++++++++++++++++++++++++++++ > tests/exynos/exynos_fimg2d_test.c | 134 ++++++++++++++- > xf86drm.h | 21 +++ > xf86drmMode.c | 10 +- > 13 files changed, 1128 insertions(+), 16 deletions(-) > create mode 100644 tests/exynos/exynos_fimg2d_event.c > create mode 100644 tests/exynos/exynos_fimg2d_perf.c >