From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from m16.mail.126.com (m16.mail.126.com [220.197.31.8]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D0AE81FB1 for ; Mon, 22 Jun 2026 01:31:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=220.197.31.8 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782091920; cv=none; b=cm7BvHm/wIKTa5ZnG++Fl9KQm3uCmr5fpZSEbZcQLNANXoL5vhGF7oZOpwJqLzCA19YAIAnDrLh2ShSyeLQuNT3Jzb2DhLy7ZWQHljYNIgitBfiHxpYH7lk5EdvKCl6ITk4ZEBvq7Xc82QN+F/5Vwyt8Acahvol5iG5SKeJfARM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782091920; c=relaxed/simple; bh=00RONsDT5X1+/qHIajMnApH07BVKPv/E83MC/naREIQ=; h=Message-ID:Date:From:MIME-Version:To:CC:Subject:References: In-Reply-To:Content-Type; b=GbLmEwlrZ6ccCDnjmLgeJhbdUHoou7bhf8lTBnifVL8Vw2ZWheMfct4N49P+87bWZGh+au1LsiSLgmZi+A3A4vk/SdRTBwm97R3UVOtB/yAJfaWrQ48W1M+JbSSN+JHnE8EW5e1TRz60AE2gEr9QPwvq4XfB7RaHUTwws0Kn0uU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=126.com; spf=pass smtp.mailfrom=126.com; dkim=pass (1024-bit key) header.d=126.com header.i=@126.com header.b=Tdn9R6c3; arc=none smtp.client-ip=220.197.31.8 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=126.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=126.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=126.com header.i=@126.com header.b="Tdn9R6c3" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=126.com; s=s110527; h=Message-ID:Date:From:MIME-Version:To:Subject: Content-Type; bh=u3ZmX/mjzL9Zl05VnH4RZ3rLeDtK6oQHdc6LujnLA90=; b=Tdn9R6c3y1nVFFiNKPjTCpU6hgPK9uN7+uduy6sAIRPMd3Ot+OzE9X8iBLcMBQ 6lhSLBdpA560O6d7XihN2t05aQUS+xiy8qBs3kLNVTpzeprQY/+1NetBYuEO8uay Zp77BpSvVxvdbalgivW88TDj7vPAAQqKTEc/skE+/HduY= Received: from localhost.localdomain (unknown []) by gzga-smtp-mtada-g0-3 (Coremail) with SMTP id _____wDnryQvkDhq+HghCA--.54830S2; Mon, 22 Jun 2026 09:30:23 +0800 (CST) Message-ID: <6A38902B.2080808@126.com> Date: Mon, 22 Jun 2026 09:30:19 +0800 From: Hongling Zeng User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 To: Danilo Krummrich , Hongling Zeng CC: lyude@redhat.com, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, airlied@gmail.com, simona@ffwll.ch, airlied@redhat.com, ttabi@nvidia.com, bskeggs@nvidia.com, dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 0/5] nouveau/gsp: Clean up IS_ERR vs IS_ERR_OR_NULL usage References: <20260601095403.228220-1-zenghongling@kylinos.cn> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-CM-TRANSID:_____wDnryQvkDhq+HghCA--.54830S2 X-Coremail-Antispam: 1Uf129KBjvdXoWrtF48Kr15AFWDur4DJw1fZwb_yoWfXFg_Wr W7AayfJ3yIkFWDAr4qkF4aqFW2ga4kXr40v39a9wnI9FW7Zry3WrsIkry3Xrn7trZ29rna gFn8J39YgF17KjkaLaAFLSUrUUUUjb8apTn2vfkv8UJUUUU8Yxn0WfASr-VFAUDa7-sFnT 9fnUUvcSsGvfC2KfnxnUUI43ZEXa7IU8n2-5UUUUU== X-CM-SenderInfo: x2kr0wpolqwiqxrzqiyswou0bp/xtbBrhDYqWo4kDAXrgAA3E Hi Danilo, I've reviewed Sashiko's feedback and she's absolutely right about the documentation issues. I will send the v3 patch, However, I want to clarify that these are pre-existing issues in the code, not problems introduced by my patch series， Unlike the previous kernel panic issue that we identified and fixed. Thanks, Hongling 在 2026年06月21日 21:00, Danilo Krummrich 写道: > On Mon Jun 1, 2026 at 11:53 AM CEST, Hongling Zeng wrote: >> >Hongling Zeng (5): >> > nouveau/gsp/rpc: Document RPC function return value contracts >> > nouveau/gsp/rpc: Cleanup incorrect IS_ERR_OR_NULL in rpc.c >> > nouveau/gsp/rm/alloc: Cleanup IS_ERR_OR_NULL usage >> > nouveau/gsp/rm/bar: Cleanup IS_ERR_OR_NULL usage >> > nouveau/gsp: Cleanup IS_ERR_OR_NULL in nvkm_gsp_rpc_rd() > There are a few more comments from Sashiko, did you check them? > > Thanks, > Danilo