From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) (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 1600830B501 for ; Wed, 19 Nov 2025 17:13:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763572417; cv=none; b=FU0VK8WJqRz/YslSKzexyUcl2YelUeRBYrxonlhMkCmf5zBdJgVMdYnuN+Lxonko5+h4O3KX70NocxWry0EYVoJ/RWYuHNWOSQr4vOGmCoBKt/zBI1U6wFVwfaobfnAnSNaw5kpJg1et2wVeaP6vfTQx/gZBTkISRQO80/J1yO0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763572417; c=relaxed/simple; bh=xsooH6y8NKa6xW7ZhKkSuEnIqZND+wVpEwXj3cRihtI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=VTsdS8c8OcHvSIG/MpzAeB2a7wTCostV30u797pXrhAjsLUzgzf3lcZEls8RmYxM8Ib9obRnVp1NUYtjCI4j3aB+dVRtoad4wsmVDcP6lZQlnItG4US3zQFUg06eMVvI+TllwkDNwsQaRTyzE0AB3oURPhW8fnPvBK8MpHAcFBc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=sZbxol95; arc=none smtp.client-ip=209.85.128.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="sZbxol95" Received: by mail-wm1-f50.google.com with SMTP id 5b1f17b1804b1-4779d47be12so34759295e9.2 for ; Wed, 19 Nov 2025 09:13:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763572412; x=1764177212; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=2vhmhQ6Om+Pbwbn3RAZfO5ZRyCOOKsXFMN4D6avcC+M=; b=sZbxol95izRnIdqG6q0bFri8Y8QurHvdXELw58VABkUfu4q+W0r5HCZRBvtdRhTbkx GKs5Ax1ifBmOfuHacJCQssoon21dD7m7CBSnGhxhlZAzNdoxC7RH5fA+P7fHOYFb7Ohd XS3sBoOZ0tj+lE/lF1E9O4gOa0rU+mAA0FHGedMCSVx50WrBfykTOp52RzqxwUnrp0+h dIBdviXChYrYNUg9woA24I4SbgQKhnP2szb1pbVDaRjR07BSzqIAP/k5UxH+LWcHiZZe 3XWbG8fBtoQMi91+7HCem6hNsrwSjtAmnpK/asyBrDGJ5t0hGa1VGSMNNpBQhc6nMdgr WJKw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763572412; x=1764177212; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=2vhmhQ6Om+Pbwbn3RAZfO5ZRyCOOKsXFMN4D6avcC+M=; b=p3n30s2lfocRwybZFdS8bG6PWYusUpwLlegydiHMzOoMQfDWBzkxZoYph5UssWPMvZ JcV/dlo89/tB30RrmmHLMKDWFKR6G0IeJnONVtm87ReduZMC54u3gxe+GuRUcOG4/xbf I7sUE4EhE4Ss//VtNKm1DtdD6vGv0P/gCaeZ3a7cVlzUNK5+7WPbRg1VMGgus7OCxLMH fHk3YAgseRywFTTj3kTrgpS3F7j3XZOjhP+/IprXcXr9pVE1I9NGjMrl95P3reE3mRhF E6xOsWZMbGEvpbB7Fezp8a6mTWKpZSp1hrDIQs66q5zHoQ0hH1SoHW1WJHsjdN08lHua 2Fyg== X-Forwarded-Encrypted: i=1; AJvYcCWN4YNYCrbV3ejLeIVDTlrW4T9GkeQE9n10k7t0eaHmHSBRXkgMFqm4KKz7zFf/lGRl8J1GDvu7moyYmMY=@vger.kernel.org X-Gm-Message-State: AOJu0YxVHchEI9A5vul6+9Xu7NF5J47sS1e0NNDFUBl0J6EjE2rTQKKn hOJJAU53D3EvTT+zDRlv0BGbjbkxRFMZWc96tbA49EK9cv0FQxV1tn0QLOdywCiEvUA= X-Gm-Gg: ASbGncupiqcCI0+jT1iwJbW+HADa0ZH9T6/kPpL0IxL4wEemD+f1y8WrbM8sM70UZuT z8eky6hAQtqgVmZEmBZs79IGLTlpdbqvBUETD7Lt2r8bAjcFLeYD1qrXiD5ZcF4kgfvQ4UouPDF sjLDqiHBfZ7aDfsLTuDi7XYSmgPAakOjy0rzuYDZRjAjCMT1xOilVS+EC2+/nt2B01F0DFKcdp1 ylDEgTkT9clV+NAiyLtvvaSyN+/KkEFocqYtt3kOqyBbSwCCNcP3t5nxVWAo5mimqF3TsGP2+Xn OWONd+RoK5id76qF6d9IhPwQ/9UikmPJs1tIJDPdHTeceKvEwUFKGe25X82yPDSFMDixPhimWK3 Bx40OHDxMvzIpzHUM1IkroeT0Hq8FDrSIuQfxqgdD9cypohzDTzmyiWkI0r0R8Su+lCcWaBPwkw s69JKKXApMqVTf1uYhtI072xAhcDw= X-Google-Smtp-Source: AGHT+IH4OATo6lVvjNQJLgOC+refWkXUUq/dSK8LEPdFqw3Sf5cp0g+sJKtr1ngfEOnJg54Edk+hnA== X-Received: by 2002:a05:600c:1986:b0:46e:35a0:3587 with SMTP id 5b1f17b1804b1-4778fea7037mr190606645e9.27.1763572412224; Wed, 19 Nov 2025 09:13:32 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with UTF8SMTPSA id 5b1f17b1804b1-477b82d8251sm1082725e9.6.2025.11.19.09.13.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Nov 2025 09:13:31 -0800 (PST) Date: Wed, 19 Nov 2025 20:13:28 +0300 From: Dan Carpenter To: Steven Rostedt Cc: Josh Poimboeuf , linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH] unwind: Fix signedness bug in unwind_deferred_request() Message-ID: References: <20251119092418.7daaec1d@batman.local.home> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251119092418.7daaec1d@batman.local.home> On Wed, Nov 19, 2025 at 09:24:58AM -0500, Steven Rostedt wrote: > On Wed, 19 Nov 2025 11:10:43 +0300 > Dan Carpenter wrote: > > > The "bit" value comes from "work->bit". It is set to -1 if we cancel > > the work in unwind_deferred_cancel(). It needs to be signed because > > we check for negative values. Change the type from unsigned long to int. > > > > Fixes: 357eda2d7450 ("unwind deferred: Use SRCU unwind_deferred_task_work()") > > Signed-off-by: Dan Carpenter > > --- > > kernel/unwind/deferred.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/kernel/unwind/deferred.c b/kernel/unwind/deferred.c > > index a88fb481c4a3..8dc11e0d9374 100644 > > --- a/kernel/unwind/deferred.c > > +++ b/kernel/unwind/deferred.c > > @@ -232,8 +232,7 @@ int unwind_deferred_request(struct unwind_work *work, u64 *cookie) > > struct unwind_task_info *info = ¤t->unwind_info; > > int twa_mode = TWA_RESUME; > > unsigned long old, bits; > > - unsigned long bit; > > - int ret; > > + int bit, ret; > > I'm fine with the change to bit, but please do not put them on the same > row. I only do that when the values are related. bit and ret are *not* > related. Sure, I've sent a v2. regards, dan carpenter