Received: by 2002:a05:7412:98c1:b0:fa:551:50a7 with SMTP id kc1csp1421472rdb; Sun, 7 Jan 2024 20:16:40 -0800 (PST) X-Google-Smtp-Source: AGHT+IF7T3STUHEDykHNysYTjLD9MiSyKipSK8JGync5V2XzJ3udxQBtkTIpCOqSq8u/z3Cxbjd2 X-Received: by 2002:ad4:4eed:0:b0:680:c771:cab7 with SMTP id dv13-20020ad44eed000000b00680c771cab7mr4975969qvb.106.1704687400181; Sun, 07 Jan 2024 20:16:40 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704687400; cv=none; d=google.com; s=arc-20160816; b=gw4flaMiee94GPoUTuYKef78oufgFCgaiK6Gsxub3lF1E1woHh7K8olzCs1VxNs+bM FNunpOIdNg6VThHjarLQgLCRrlHd0dczYGWniAylZnB1gDkQ6QOrE9oZuXUeh4zd2OIP 3lLaEbL3UPht/oDUaqn/LCsB3lReMiSAy+rl2SLQuQc8uXHlhyJbJF6xrgHaHnSkIFpW OBCGt3ATNQIqeANkmqLttWK2nmIBFzYKPGP9NhIPpjJL1hwtXRIBsqKW+crKWhGWGIfp 94DCBlZBnQdvl1Hj7CP2w3Bt5l0KYqNxQHeOheAvoyB3JMLe42k0UemvvEP5zAR1fA95 nTow== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:autocrypt:from:content-language:references:cc:to :subject:user-agent:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:date:message-id:dkim-signature; bh=G39BbRFm9oOt8R6geou4KSFAkCT2cUdc4etIgTklN44=; fh=35OjPwAlzRnDkxUOSbbeIr382tXPnwD04KE7jOBdE5M=; b=YYsisQbscDu2p9N9s/O8oFYMpZSC3HwxCgqW2LvzJRDZFAXMVjd/Q1q5NExnAwPaTu RCjXLkwGCNWsCvmgClEpZl9qHJWse7Vd/tBucJF8WBKAaw/b1ciqjihaugWIqrUgMyP+ +3RuWXqSZniDh/z6XQWLs3+DZnc4mIK93QbEcn/JHEg8BLshkRFz1npzKWeXysiDi8kg ox55oXsFXFtNN3bwh3OWJoKLTi3xORzZgjqCgwciNRIL/vz3TeEmnR0xI1jd4E99ngzX heqKmqfTgOKFhHi4mVAoylAHgmUb2lSm9/6TauLzAEU+npZAtK1iIeppNF7WHtNgJRJu oRFw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=H96c5Lae; spf=pass (google.com: domain of linux-kernel+bounces-19077-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-19077-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id f30-20020a0caa9e000000b0067f4272cb34si7373132qvb.48.2024.01.07.20.16.40 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 07 Jan 2024 20:16:40 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-19077-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=H96c5Lae; spf=pass (google.com: domain of linux-kernel+bounces-19077-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-19077-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id BA03C1C2154C for ; Mon, 8 Jan 2024 04:16:39 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A15468813; Mon, 8 Jan 2024 04:16:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="H96c5Lae" X-Original-To: linux-kernel@vger.kernel.org Received: from mail-qv1-f47.google.com (mail-qv1-f47.google.com [209.85.219.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 797E27F; Mon, 8 Jan 2024 04:16:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-qv1-f47.google.com with SMTP id 6a1803df08f44-67f5c0be04cso15450016d6.0; Sun, 07 Jan 2024 20:16:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1704687388; x=1705292188; darn=vger.kernel.org; h=in-reply-to:autocrypt:from:content-language:references:cc:to :subject:user-agent:mime-version:date:message-id:from:to:cc:subject :date:message-id:reply-to; bh=G39BbRFm9oOt8R6geou4KSFAkCT2cUdc4etIgTklN44=; b=H96c5LaejS7QBID25VnfOHq617crTlRo92sxNKkZqB0m2hDnFYPGrOqtzaYAPSxn8m bCZqVkAdbk34+dJK0TO7JuO0Z6je/RaKfK1sFxxPkHgr4DRZNpytdtMm99nmTbT6whFl C2T16FHZyUXzjnNLamnCtgoKqYlPanXMp4oDQ9nlrQvfVa8LFfunsLshowKfnO5kQNYH GO21McEaqqIjxy8482h/FAowo4wtZkQhR0r7jqzru+QustEuYrISzdP3wYlQ3k/S4X3/ q1gVusPZyD7rbeOknxQWMPJDB6+l0MS0sOSGdGf/lzLph4+dc1rdR2syTsPLXGLgJ3pj G9cw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1704687388; x=1705292188; h=in-reply-to:autocrypt:from:content-language:references:cc:to :subject:user-agent:mime-version:date:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=G39BbRFm9oOt8R6geou4KSFAkCT2cUdc4etIgTklN44=; b=ZGvXXwnIA2X/YDZxzzuE3+ESwhvTtnupITo1cazci+OVHdMNGgoD1mQJ7GEzYlqAo7 I/i0gCWeJ1cos2ipF8QcPI3MQ4B5L4wD/UbsehimBMhR4L4+bnOfwsAxVo/2urHeHmsw Z65htmkXq4TvTc9+NW3HV0urSyuSSn6Zi5kXWLS82l73ouwbn6rgINl6+1Xo0McUvll/ UbVJc7tHzl5dSHXS/StNinyb9SyoisJNV2sbowDrOgW4+g9b3wmAS7utojLjxnw/yA5f L1OAJWE61+VcS0DoZFZ0DF5fCwcAUXpU3cmd0bMT422YgPIGUpP7EAPuaZ3F5y7kxHaG yV8w== X-Gm-Message-State: AOJu0YyOl0/WWAmOC0J0pQScWnMIP5wlxwmkOioEbf8H92r822mdD2wK VQhejvAAFpAGbjbjQSj91qQ= X-Received: by 2002:ad4:5aab:0:b0:67f:641e:bfc5 with SMTP id u11-20020ad45aab000000b0067f641ebfc5mr4757391qvg.26.1704687388243; Sun, 07 Jan 2024 20:16:28 -0800 (PST) Received: from [192.168.2.14] ([174.88.31.222]) by smtp.gmail.com with ESMTPSA id mx20-20020a0562142e1400b0067f802d373bsm2557852qvb.89.2024.01.07.20.16.27 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 07 Jan 2024 20:16:27 -0800 (PST) Message-ID: <02eea90d-5a3a-415f-9123-36e81ff9511a@gmail.com> Date: Sun, 7 Jan 2024 23:16:18 -0500 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:115.0) Gecko/20100101 Thunderbird/115.6.0 Subject: Re: [PATCH 1/2] drm/sched: One function call less in drm_sched_init() after error detection To: Markus Elfring , dri-devel@lists.freedesktop.org, kernel-janitors@vger.kernel.org, Daniel Vetter , David Airlie , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann Cc: LKML , cocci@inria.fr References: <12b3e9cb-3241-40cc-b7a4-43c45b9eedc9@web.de> <85066512-983d-480c-a44d-32405ab1b80e@web.de> Content-Language: en-CA, en-US From: Luben Tuikov Autocrypt: addr=ltuikov89@gmail.com; keydata= xjMEZTohOhYJKwYBBAHaRw8BAQdAWSq76k+GsENjDTMVCy9Vr4fAO9Rb57/bPT1APnbnnRHN Ikx1YmVuIFR1aWtvdiA8bHR1aWtvdjg5QGdtYWlsLmNvbT7CmQQTFgoAQRYhBJkj7+VmFO9b eaAl10wVR5QxozSvBQJlOiE6AhsDBQkJZgGABQsJCAcCAiICBhUKCQgLAgQWAgMBAh4HAheA AAoJEEwVR5QxozSvSm4BAOwCpX53DTQhE20FBGlTMqKCOQyJqlMcIQ9SO1qPWX1iAQCv3vfy JwktF7REl1yt7IU2Sye1qmQMfJxdt9JMbMNNBs44BGU6IToSCisGAQQBl1UBBQEBB0BT9wSP cCE8uGe7FWo8C+nTSyWPXKTx9F0gpEnlqReRBwMBCAfCfgQYFgoAJhYhBJkj7+VmFO9beaAl 10wVR5QxozSvBQJlOiE6AhsMBQkJZgGAAAoJEEwVR5QxozSvSsYA/2LIFjbxQ2ikbU5S0pKo aMDzO9eGz69uNhNWJcvIKJK6AQC9228Mqc1JeZMIyjYWr2HKYHi8S2q2/zHrSZwAWYYwDA== In-Reply-To: <85066512-983d-480c-a44d-32405ab1b80e@web.de> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="------------akJcGWK3GfgbDEfTJY9Ui4oZ" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --------------akJcGWK3GfgbDEfTJY9Ui4oZ Content-Type: multipart/mixed; boundary="------------j0wM5t9os3GxcVz0WOntpr3w"; protected-headers="v1" From: Luben Tuikov To: Markus Elfring , dri-devel@lists.freedesktop.org, kernel-janitors@vger.kernel.org, Daniel Vetter , David Airlie , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann Cc: LKML , cocci@inria.fr Message-ID: <02eea90d-5a3a-415f-9123-36e81ff9511a@gmail.com> Subject: Re: [PATCH 1/2] drm/sched: One function call less in drm_sched_init() after error detection References: <12b3e9cb-3241-40cc-b7a4-43c45b9eedc9@web.de> <85066512-983d-480c-a44d-32405ab1b80e@web.de> In-Reply-To: <85066512-983d-480c-a44d-32405ab1b80e@web.de> --------------j0wM5t9os3GxcVz0WOntpr3w Content-Type: multipart/mixed; boundary="------------NXadqVeg3RFjjwx8Fg5PFeCc" --------------NXadqVeg3RFjjwx8Fg5PFeCc Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 2023-12-26 10:56, Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 26 Dec 2023 16:30:25 +0100 >=20 > The kfree() function was called in one case by the > drm_sched_init() function during error handling > even if the passed data structure member contained a null pointer. > This issue was detected by using the Coccinelle software. >=20 > Thus adjust a jump target. >=20 > Signed-off-by: Markus Elfring Thank you Markus for this patch. Reviewed-by: Luben Tuikov Pushed to drm-misc-next. --=20 Regards, Luben > --- > drivers/gpu/drm/scheduler/sched_main.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) >=20 > diff --git a/drivers/gpu/drm/scheduler/sched_main.c b/drivers/gpu/drm/s= cheduler/sched_main.c > index 550492a7a031..b99d4e9ff109 100644 > --- a/drivers/gpu/drm/scheduler/sched_main.c > +++ b/drivers/gpu/drm/scheduler/sched_main.c > @@ -1289,7 +1289,7 @@ int drm_sched_init(struct drm_gpu_scheduler *sche= d, > sched->sched_rq =3D kmalloc_array(num_rqs, sizeof(*sched->sched_rq), > GFP_KERNEL | __GFP_ZERO); > if (!sched->sched_rq) > - goto Out_free; > + goto Out_check_own; > sched->num_rqs =3D num_rqs; > for (i =3D DRM_SCHED_PRIORITY_KERNEL; i < sched->num_rqs; i++) { > sched->sched_rq[i] =3D kzalloc(sizeof(*sched->sched_rq[i]), GFP_KERN= EL); > @@ -1314,9 +1314,10 @@ int drm_sched_init(struct drm_gpu_scheduler *sch= ed, > Out_unroll: > for (--i ; i >=3D DRM_SCHED_PRIORITY_KERNEL; i--) > kfree(sched->sched_rq[i]); > -Out_free: > + > kfree(sched->sched_rq); > sched->sched_rq =3D NULL; > +Out_check_own: > if (sched->own_submit_wq) > destroy_workqueue(sched->submit_wq); > drm_err(sched, "%s: Failed to setup GPU scheduler--out of memory\n", = __func__); > -- > 2.43.0 >=20 --------------NXadqVeg3RFjjwx8Fg5PFeCc Content-Type: application/pgp-keys; name="OpenPGP_0x4C15479431A334AF.asc" Content-Disposition: attachment; filename="OpenPGP_0x4C15479431A334AF.asc" Content-Description: OpenPGP public key Content-Transfer-Encoding: quoted-printable -----BEGIN PGP PUBLIC KEY BLOCK----- xjMEZTohOhYJKwYBBAHaRw8BAQdAWSq76k+GsENjDTMVCy9Vr4fAO9Rb57/bPT1A PnbnnRHNIkx1YmVuIFR1aWtvdiA8bHR1aWtvdjg5QGdtYWlsLmNvbT7CmQQTFgoA QRYhBJkj7+VmFO9beaAl10wVR5QxozSvBQJlOiE6AhsDBQkJZgGABQsJCAcCAiIC BhUKCQgLAgQWAgMBAh4HAheAAAoJEEwVR5QxozSvSm4BAOwCpX53DTQhE20FBGlT MqKCOQyJqlMcIQ9SO1qPWX1iAQCv3vfyJwktF7REl1yt7IU2Sye1qmQMfJxdt9JM bMNNBs44BGU6IToSCisGAQQBl1UBBQEBB0BT9wSPcCE8uGe7FWo8C+nTSyWPXKTx 9F0gpEnlqReRBwMBCAfCfgQYFgoAJhYhBJkj7+VmFO9beaAl10wVR5QxozSvBQJl OiE6AhsMBQkJZgGAAAoJEEwVR5QxozSvSsYA/2LIFjbxQ2ikbU5S0pKoaMDzO9eG z69uNhNWJcvIKJK6AQC9228Mqc1JeZMIyjYWr2HKYHi8S2q2/zHrSZwAWYYwDA=3D=3D =3DqCaZ -----END PGP PUBLIC KEY BLOCK----- --------------NXadqVeg3RFjjwx8Fg5PFeCc-- --------------j0wM5t9os3GxcVz0WOntpr3w-- --------------akJcGWK3GfgbDEfTJY9Ui4oZ Content-Type: application/pgp-signature; name="OpenPGP_signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="OpenPGP_signature.asc" -----BEGIN PGP SIGNATURE----- wnsEABYIACMWIQSZI+/lZhTvW3mgJddMFUeUMaM0rwUCZZt3EgUDAAAAAAAKCRBMFUeUMaM0r2Z2 AP9SVEcM87yBgCEEumb7RrTcowF28fJe4at5i5gRwqqH9AD/fIedeYxvDzo+bgS6qnGXXomI7lqZ tF/sjFL/xiKcBAI= =xKWp -----END PGP SIGNATURE----- --------------akJcGWK3GfgbDEfTJY9Ui4oZ--