Message ID | 20231111224236.890431-1-dmitry.osipenko@collabora.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b909:0:b0:403:3b70:6f57 with SMTP id t9csp447118vqg; Sat, 11 Nov 2023 14:44:42 -0800 (PST) X-Google-Smtp-Source: AGHT+IGyZLb/qEALVebGUSU+BZ8DGeMKy4M5289pGu6aRSY6CHTCdeAnUVRN7tMaO7KmK845+0NN X-Received: by 2002:a05:6808:1a08:b0:3af:9851:4d32 with SMTP id bk8-20020a0568081a0800b003af98514d32mr4730062oib.7.1699742682608; Sat, 11 Nov 2023 14:44:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1699742682; cv=none; d=google.com; s=arc-20160816; b=Cn/PYKaD8pLDphRXpuEtvVvzOmiqwsxfWD71H1nlPXHhQyQJJT3fm/5950QsR77r37 nikDWxpmTGy0PGFtMjpZyJftk2yYEsDdoPAwAFdXiqURCv5lTFUqvI2AmiMEq5vk4N6w LJXkBZc/CGruYa7q76i0f7Lxo6UUbCeatWQqhufrM/diblr6Ean+qL8tjnY73i176rzi l5rqhJc3uAVgQ8U/L38aYrjaH2GsvBjl0Plldv0Pb12MrUwSSSXdZG7TjsNteqGAFqEa ZfwavXYRmVGZd6z8ALdeOBzMg7//cFFwjtzqJjp3bLEhQeAnTlQVeu6GZeF89UdAIwr/ l/fg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=S0ax+noH9KufT4TAahbenmyzhuzdHmT2hTPOD+H2/tQ=; fh=U68+6Qc+dbFaYckALFHn/HzxQbVAce5jM1o1vTNcyWU=; b=RZUtSiI6ByTT6To2HVK0rzKUUjvhbtQXenkaZ7w9FJwiiTEi8DW2epCg9uWbFl7Fuy h576cSjHGa9k9onH+/BV0ivnZwOyONUz75IY9IyLPWAIBvWFhXTd/oP98H3fwVV3m5ph vQWlUXKE1Kk2oPMEtJhCmDTngUxXVJGYGgQ5aiNOAHkp24M6GtZn/em88tt9n3STz/RD 540KFW+xyPh2Tk3tx6iO7HX7Ox2HqZzDr6N72aR6ImK0pxtCciIscc6bhcx4uROgaADJ lqfx9kMvgcO8ktY3eo85T3xvOL5LVlFlyPqEH6pp/6yZJON5TQgKx83mxfRODjsvHT7a T4Dw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=kC3L8gWi; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: from agentk.vger.email (agentk.vger.email. [2620:137:e000::3:2]) by mx.google.com with ESMTPS id d7-20020a056a00244700b006c4d3cd9132si2676020pfj.0.2023.11.11.14.44.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 11 Nov 2023 14:44:42 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=kC3L8gWi; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id 9CA678041EAB; Sat, 11 Nov 2023 14:44:40 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229810AbjKKWno (ORCPT <rfc822;heyuhang3455@gmail.com> + 29 others); Sat, 11 Nov 2023 17:43:44 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54978 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229436AbjKKWnm (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Sat, 11 Nov 2023 17:43:42 -0500 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A5169D8 for <linux-kernel@vger.kernel.org>; Sat, 11 Nov 2023 14:43:39 -0800 (PST) Received: from workpc.. (zone.collabora.co.uk [167.235.23.81]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: dmitry.osipenko) by madras.collabora.co.uk (Postfix) with ESMTPSA id 62B1966073B7; Sat, 11 Nov 2023 22:43:37 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1699742618; bh=U10+EkHqkOD4kaEoYu2gK2478uYYFAVFdFMCVmakhG8=; h=From:To:Cc:Subject:Date:From; b=kC3L8gWiZ0wdnhznrSREusgQLxJc+IkSf97mNgdSNHO+XbRywCHmeXoAh4KxLVcwJ hYuALejDVbvWoSmsNsdrt/jpDv+/yse1MSvAQ/NnlUu1nbzHeuBCjUdcKsSHzGFlJ8 4nK6XeRj8OzJ7u8oFrJAVQLGMrUIchXWXjPHDbI8oj7yMmVGi5yee/73HUQAN3X42T 7bPdGGUlStlcReb/KljFTjrYVDLjn5ysuhloyoVpGOnNRKGEXrbHXcAWZtG91i9FDb fNnPoQRD06ZrOAy14Sk5RIRfhK0SPG8stElNplvjx3yh2u24IEzSV/cdgxf4u0IUPf WOj7vKN+ERmUw== From: Dmitry Osipenko <dmitry.osipenko@collabora.com> To: David Airlie <airlied@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>, Gurchetan Singh <gurchetansingh@chromium.org>, Chia-I Wu <olvaffe@gmail.com> Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, kernel@collabora.com Subject: [PATCH v1] drm/virtio: Fix return value for VIRTGPU_CONTEXT_PARAM_DEBUG_NAME Date: Sun, 12 Nov 2023 01:42:36 +0300 Message-ID: <20231111224236.890431-1-dmitry.osipenko@collabora.com> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Sat, 11 Nov 2023 14:44:40 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1782309382833668969 X-GMAIL-MSGID: 1782309382833668969 |
Series |
[v1] drm/virtio: Fix return value for VIRTGPU_CONTEXT_PARAM_DEBUG_NAME
|
|
Commit Message
Dmitry Osipenko
Nov. 11, 2023, 10:42 p.m. UTC
The strncpy_from_user() returns number of copied bytes and not zero on
success. The non-zero return value of ioctl is treated as error. Return
zero on success instead of the number of copied bytes.
Fixes: 7add80126bce ("drm/uapi: add explicit virtgpu context debug name")
Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
---
drivers/gpu/drm/virtio/virtgpu_ioctl.c | 1 +
1 file changed, 1 insertion(+)
Comments
On 11/12/23 01:42, Dmitry Osipenko wrote: > The strncpy_from_user() returns number of copied bytes and not zero on > success. The non-zero return value of ioctl is treated as error. Return > zero on success instead of the number of copied bytes. > > Fixes: 7add80126bce ("drm/uapi: add explicit virtgpu context debug name") > Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com> > --- > drivers/gpu/drm/virtio/virtgpu_ioctl.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c > index 1e2042419f95..e4f76f315550 100644 > --- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c > +++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c > @@ -665,6 +665,7 @@ static int virtio_gpu_context_init_ioctl(struct drm_device *dev, > goto out_unlock; > > vfpriv->explicit_debug_name = true; > + ret = 0; > break; > default: > ret = -EINVAL; Applied to misc-next
diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c b/drivers/gpu/drm/virtio/virtgpu_ioctl.c index 1e2042419f95..e4f76f315550 100644 --- a/drivers/gpu/drm/virtio/virtgpu_ioctl.c +++ b/drivers/gpu/drm/virtio/virtgpu_ioctl.c @@ -665,6 +665,7 @@ static int virtio_gpu_context_init_ioctl(struct drm_device *dev, goto out_unlock; vfpriv->explicit_debug_name = true; + ret = 0; break; default: ret = -EINVAL;