From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1256FC9EC79 for ; Mon, 12 Jan 2026 11:33:47 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 81B0010E3B7; Mon, 12 Jan 2026 11:33:46 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="c0IXWhxe"; dkim-atps=neutral Received: from sea.source.kernel.org (sea.source.kernel.org [172.234.252.31]) by gabe.freedesktop.org (Postfix) with ESMTPS id A632910E3B4; Mon, 12 Jan 2026 11:33:45 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 45C9E43B8B; Mon, 12 Jan 2026 11:33:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B51EBC16AAE; Mon, 12 Jan 2026 11:33:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1768217625; bh=gewNW9JfOu+wmTeMqWf/D1iX1aeq3pa4b5o6CPOafNg=; h=Date:From:Subject:Cc:To:References:In-Reply-To:From; b=c0IXWhxeuCF+OyQaC2AIspZ3WuoilRsLIYZSy/hMcduANQB5xgWOLKcQUZAQ7yVFv FsYA7BMGWx/PVQa5vvKWGjX/GXOLi5u3AhYKH+3BV6qH/ZzJMF3LRqzQ50wrt+WhBi dtpWFgzBw8X7l07UYzb6YTH1fYNv7sp3ADNSG0lKHP2jVnItR/q3UwWYNGz1mmjXAh 6O58N4kwElLaemzKoyg61kVmuMfXFeGzidbWiWKZTkS+r6AttKeojhsqdH/GBJdjaX s+14oFIiTInrV5OzEKYS/vNRyFZuEaJrPgA8+Nb82TgpKvF5f2cUGAsGs07AZOX9G9 oWXerIZh9E1Bg== Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Mon, 12 Jan 2026 12:33:40 +0100 Message-Id: From: "Danilo Krummrich" Subject: Re: [RFC 3/3] drm/sched: Disallow initializing entities with no schedulers Cc: , , , , =?utf-8?q?Christian_K=C3=B6nig?= , "Matthew Brost" To: "Tvrtko Ursulin" References: <20260107124351.94738-1-tvrtko.ursulin@igalia.com> <20260107124351.94738-4-tvrtko.ursulin@igalia.com> <340d0ce2-85e6-4fd8-992c-c35dda9b0cbb@igalia.com> In-Reply-To: <340d0ce2-85e6-4fd8-992c-c35dda9b0cbb@igalia.com> X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" On Mon Jan 12, 2026 at 11:29 AM CET, Tvrtko Ursulin wrote: > It looks completely logical to me to have both lines dealing with the=20 > same scheduler list, accessing the same input parameter even, next to=20 > each other: > > entity->num_sched_list =3D num_sched_list; > entity->sched_list =3D num_sched_list > 1 ? sched_list : NULL; > > No? Agreed! Yet, please don't add unrelated changes to patches.