From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [PATCH v2 00/16] omapdrm: Implement dma_buf import Date: Wed, 16 Dec 2015 18:18:18 +0200 Message-ID: <56718ECA.9070806@ti.com> References: <1450125584-31701-1-git-send-email-laurent.pinchart@ideasonboard.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1731528146==" Return-path: Received: from arroyo.ext.ti.com (arroyo.ext.ti.com [192.94.94.40]) by gabe.freedesktop.org (Postfix) with ESMTPS id B40946E034 for ; Wed, 16 Dec 2015 08:18:24 -0800 (PST) In-Reply-To: <1450125584-31701-1-git-send-email-laurent.pinchart@ideasonboard.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Laurent Pinchart , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1731528146== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="69WrJms4OEtlAeB8rCC41okQR24Vi5kML" --69WrJms4OEtlAeB8rCC41okQR24Vi5kML Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi Laurent, On 14/12/15 22:39, Laurent Pinchart wrote: > Hello, >=20 > This patch series implements support for dma_buf import in the omapdrm = driver. > The patches are based on top of the latest drm-next branch and can be f= ound in > my git tree at >=20 > git://linuxtv.org/pinchartl/fbdev.git omapdrm/dmabuf-import >=20 > The first two patches are unrelated fixes and enhancements, but I've in= cluded > them in the series to avoid merge conflicts. >=20 > The next 13 patches are miscellaneous fixes, cleanups and refactoring t= o > prepare for patch 16/16 that implements dma_buf import. >=20 > The code has been successfully tested with the vivid driver as an expor= ter, > using a hacked version that uses uncached CPU mappings in vivid when fi= lling > the buffers. vivid is a test driver that generates a test pattern using= the > CPU with cached mappings by default, resulting in corruption on the scr= een due > to missing cache handling. As the problem doesn't occur when sharing bu= ffers > not touched by the CPU or touched through uncached mappings only, it wi= ll be > addressed separately. I've picked up these patches to my omapdrm-4.5 branch: drm: omapdrm: move omap_plane_reset() drm: omapdrm: Fix plane state free in plane reset handler drm: omapdrm: Make fbdev emulation optional drm: omapdrm: gem: Remove unused function prototypes drm: omapdrm: gem: Remove forward declarations drm: omapdrm: gem: Group functions by purpose drm: omapdrm: gem: Move global usergart variable to omap_drm_private drm: omapdrm: gem: Mask out private flags passed from userspace drm: omapdrm: gem: Free the correct memory object drm: omapdrm: gem: Fix GEM object destroy in error path drm: omapdrm: gem: Don't free mmap offset twice drm: omapdrm: gem: Simplify error handling when creating GEM object drm: omapdrm: gem: Remove check for impossible condition I did split the "Fix plane state free in plane reset handler" into two, moving the function first. And as I dropped few patches from between the others, they're not exactly as they were in your series. The remaining patches are: drm: omapdrm: gem: Remove omap_drm_private has_dmm field drm: omapdrm: gem: Clean up GEM objects memory flags drm: omapdrm: gem: Refactor GEM object allocation drm: omapdrm: gem: Implement dma_buf import Those are more complex, and I haven't fully reviewed them yet. I wanted to get the bulk of the patches already, so that I can ensure we get at least those to 4.5. Tomi --69WrJms4OEtlAeB8rCC41okQR24Vi5kML Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJWcY7KAAoJEPo9qoy8lh71BUkP/3JdQFajvUikWVdfLb7XfyZT Jta8TzLU1NhwRurFv8/58zh6hp5QP6FfmZkOyJG7mGFEGdkoi5BEXXinAdq8aKHO 387glHh8KtAdNYPrcIE5I5pJ1nb6RGDmIiFv9spNEVEqHbuzvrFtYZiDFwnuTdxY DdST8mkwXwQojFs2d5UWV5eDgPevMFMKKyyUmaQdCk35LpUOu6edr3nUFij3FG7o 6ihYUboTLx+RseGsSkQnz+oM2VlVM9NFodmCtJlmMoIEacH/bWBr92L0xAAlOx+r 8RBqEHlUqcjSP7Bvdllq1XzvxwDqsfdj5K5LV+sp2c+T5lgulfUMw+4jmy7ysFUz 1lL6eEdniEMbrzdmg8RzZ1j37ZZpriyGJOJ2vME5dw+zXZO6eQEyfr4+404hszDa dn+zD4DKhA8Cz8/jFeMTJVcLR4D738Kpequ1iwncE0aq1gaimMbEi7bG3jpiUovf E4TiGeOXRn+jaQvGd5c9NQ/Gk3HDWMoOyrmBkPnDdemFO6ciSuwI9/GoBq0a7/na a6ParXL4nu3jEUJQrMa/XyDur5uE+7vn/bXEqQICdS7JIWyWzcc44vlj/+rSsL29 GtkNrNtrzYF6ctjCaYb8fCAKAICUm+URTUIq9ZNjyWj/5iaoAgMb8blgX3bkUjDm 73yVvEQ7GjQdUElbKLUk =V3em -----END PGP SIGNATURE----- --69WrJms4OEtlAeB8rCC41okQR24Vi5kML-- --===============1731528146== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK --===============1731528146==--