Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S968189AbdD0Hlv (ORCPT ); Thu, 27 Apr 2017 03:41:51 -0400 Received: from mail.fireflyinternet.com ([109.228.58.192]:58656 "EHLO fireflyinternet.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S968111AbdD0Hlr (ORCPT ); Thu, 27 Apr 2017 03:41:47 -0400 X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=78.156.65.138; Date: Thu, 27 Apr 2017 08:41:37 +0100 From: Chris Wilson To: Laura Abbott Cc: Daniel Vetter , Sean Paul , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Sumit Semwal Subject: Re: [RFC PATCHv2 0/3] dma_buf import support for vgem Message-ID: <20170427074137.GX11432@nuc-i3427.alporthouse.com> Mail-Followup-To: Chris Wilson , Laura Abbott , Daniel Vetter , Sean Paul , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Sumit Semwal References: <1493241150-21742-1-git-send-email-labbott@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1493241150-21742-1-git-send-email-labbott@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 874 Lines: 22 On Wed, Apr 26, 2017 at 02:12:27PM -0700, Laura Abbott wrote: > Hi, > > This is v2 of my proposal to add dma_buf import functions for vgem. > Big changes from v1: > > - A device is required for dma_buf attach to work. The existing vgem driver > intentionally does not use one as it provides a good way to test the DRM > framework. This approach instead puts a dummy platform device in the existing > drm_device->platformdev field and uses that for attaching. > - Native vgem buffers can still be faulted in a page at a time without > requiring the entire buffer be resident in memory. > > I'm still marking this as RFC as I haven't had a chance to finish > a userspace test that can be integrated into igt. Note, that it will be good to cc:intel-gfx@ so that our CI does run it over the existing vgem tests. -Chris -- Chris Wilson, Intel Open Source Technology Centre