From mboxrd@z Thu Jan  1 00:00:00 1970
From: Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [PATCH v2 5/6] virtio-gpu: add basic prime support
Date: Tue, 22 Sep 2015 17:24:29 +0200
Message-ID: <1442935469.13084.36.camel@redhat.com>
References: <1442828417-6165-1-git-send-email-kraxel@redhat.com>
 <1442828417-6165-6-git-send-email-kraxel@redhat.com>
 <20150922094027.GF3383@phenom.ffwll.local>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Return-path: <dri-devel-bounces@lists.freedesktop.org>
Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28])
 by gabe.freedesktop.org (Postfix) with ESMTPS id 61B7E6EAAB
 for <dri-devel@lists.freedesktop.org>; Tue, 22 Sep 2015 08:24:33 -0700 (PDT)
In-Reply-To: <20150922094027.GF3383@phenom.ffwll.local>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Errors-To: dri-devel-bounces@lists.freedesktop.org
Sender: "dri-devel" <dri-devel-bounces@lists.freedesktop.org>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Dave Airlie <airlied@redhat.com>, open list <linux-kernel@vger.kernel.org>, "open list:VIRTIO GPU DRIVER" <dri-devel@lists.freedesktop.org>, "open list:VIRTIO GPU DRIVER" <virtualization@lists.linux-foundation.org>
List-Id: dri-devel@lists.freedesktop.org

PiA+ICtpbnQgdmlydGdwdV9nZW1fcHJpbWVfbW1hcChzdHJ1Y3QgZHJtX2dlbV9vYmplY3QgKm9i
aiwKPiA+ICsJCSAgICAgICBzdHJ1Y3Qgdm1fYXJlYV9zdHJ1Y3QgKmFyZWEpCj4gPiArewo+ID4g
KwlXQVJOX09OQ0UoMSwgIm5vdCBpbXBsZW1lbnRlZCIpOwo+ID4gKwlyZXR1cm4gRU5PU1lTOwo+
IAo+IFRoaXMgY2FuIGdldCBjYWxsZWQgYnkgdXNlcnNwYWNlLCBzbyBwbGVhc2UgZG9uJ3QgV0FS
TiBoZXJlLiBBbHNvIG1pc3NpbmcKPiBuZWdhdGUgc2lnbjoKPiAKPiAJcmV0dXJuIC1FTk9TWVM7
CgpIbW0gbm93IGNoZWNrcGF0Y2ggdGhyb3dzIGEgd2FybmluZyBhdCBtZToKCjxxdW90ZT4KICAg
V0FSTklORzogRU5PU1lTIG1lYW5zICdpbnZhbGlkIHN5c2NhbGwgbnInIGFuZCBub3RoaW5nIGVs
c2UKICAgIzEyOiBGSUxFOiBkcml2ZXJzL2dwdS9kcm0vdmlydGlvL3ZpcnRncHVfcHJpbWUuYzo3
MDoKICAgKyAgICAgICByZXR1cm4gLUVOT1NZUzsKPC9xdW90ZT4KCkkgZ3Vlc3MgSSBzaG91bGQg
dXNlIHNvbWV0aGluZyBlbHNlIHRoZW4gKGhlcmUgYW5kIGVsc2V3aGVyZSBpbiB0aGUKZmlsZSk/
ICBNYXliZSAtRUlOVkFMPyAgT3RoZXIgc3VnZ2VzdGlvbnM/Cgp0aGFua3MsCiAgR2VyZAoKCl9f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBt
YWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMu
ZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751839AbbIVPYe (ORCPT <rfc822;w@1wt.eu>);
	Tue, 22 Sep 2015 11:24:34 -0400
Received: from mx1.redhat.com ([209.132.183.28]:47538 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1751262AbbIVPYc (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Tue, 22 Sep 2015 11:24:32 -0400
Message-ID: <1442935469.13084.36.camel@redhat.com>
Subject: Re: [PATCH v2 5/6] virtio-gpu: add basic prime support
From: Gerd Hoffmann <kraxel@redhat.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: dri-devel <dri-devel@lists.freedesktop.org>,
        open list <linux-kernel@vger.kernel.org>,
        "open list:VIRTIO GPU DRIVER" <dri-devel@lists.freedesktop.org>,
        "open list:VIRTIO GPU DRIVER" 
	<virtualization@lists.linux-foundation.org>,
        Dave Airlie <airlied@redhat.com>
Date: Tue, 22 Sep 2015 17:24:29 +0200
In-Reply-To: <20150922094027.GF3383@phenom.ffwll.local>
References: <1442828417-6165-1-git-send-email-kraxel@redhat.com>
	 <1442828417-6165-6-git-send-email-kraxel@redhat.com>
	 <20150922094027.GF3383@phenom.ffwll.local>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> > +int virtgpu_gem_prime_mmap(struct drm_gem_object *obj,
> > +		       struct vm_area_struct *area)
> > +{
> > +	WARN_ONCE(1, "not implemented");
> > +	return ENOSYS;
> 
> This can get called by userspace, so please don't WARN here. Also missing
> negate sign:
> 
> 	return -ENOSYS;

Hmm now checkpatch throws a warning at me:

<quote>
   WARNING: ENOSYS means 'invalid syscall nr' and nothing else
   #12: FILE: drivers/gpu/drm/virtio/virtgpu_prime.c:70:
   +       return -ENOSYS;
</quote>

I guess I should use something else then (here and elsewhere in the
file)?  Maybe -EINVAL?  Other suggestions?

thanks,
  Gerd