Received: by 2002:ab2:3141:0:b0:1ed:23cc:44d1 with SMTP id i1csp307992lqg; Fri, 1 Mar 2024 06:03:12 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCVFCLLimMGEsctu1Spz5uThvo8MkZ7hFZWgX5wKJ908DeWKJafqRDgiKaplY0KBrZx0S8ufIxW3sUiEQiGU1WLs0Zezj7uePvhkeVG+0Q== X-Google-Smtp-Source: AGHT+IEMyveIxOhRWM/2vGgesB19jJpntpI3CXb5skGsFCS10b7lxkntddHdTBcX0H+FT3+ZNYAD X-Received: by 2002:a05:6602:1228:b0:7c7:a21d:ffbf with SMTP id z8-20020a056602122800b007c7a21dffbfmr1844243iot.10.1709301791929; Fri, 01 Mar 2024 06:03:11 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709301791; cv=pass; d=google.com; s=arc-20160816; b=lMdFNSFojcnznoVoHfYQdDEN4BALKba0zz04AaCL/+W/Okd65bFFiXrofSZcM8B4FM dsu6Gw9B3BGWV/ogrEeQuj498ZpVxW5DTr5t/9ORqUlyjx04aWSvLZ7ptPxsYbSTeQb8 Lpvqs/Pl9g+hffTYRBf7mrflXLDa52D43HhBan3mXblMv1cHmmMWFmLIMNPLXxFe7N8T olCafA8upIYKwSrxOXZhxBU0fqE06EROdcPsi5A7DuzFkTJ4AQ9Sln48dqh1mAnu/F3e nyE+UU2dmQ/E2/K0QSgSuL743abrJ4EMpkXDTOrOlNAm2bF+nWWiJVmEJ7Rqs9M09JTi iIMQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:list-unsubscribe:list-subscribe:list-id:precedence :references:message-id:subject:cc:to:from:date:dkim-signature; bh=N0GBKYx6YHUDYdDi4omP9/JhVsdS8Z2tMZ/w6xbci6U=; fh=V+TJET6Ly37MLkFSCRCfyJq0bPyk3RtsggRmyyPzu/w=; b=mcvWISyQF9Qz3KxEshdiy+Rh+O+f4uFfeZjNlF/1HECfaldXxwMsAlfndoP7TokdUQ Y8PRMlm+QM5ppADiupxTEqISGN+YLBGceZrxxNwacBBTy22kSVV2UxENDsxV0NKpABaR cM4zAfZrBo8QEkkN6S+9uLWK58YJ/a2G8LP9imYolewy87CpAfGhDmgaVlT/lz65yI/8 DDVwoytN1BETiUQydrroMZLBtgTyGTRXP5jI0B8+lqSSuxxlamTtzrREEiPk31gZG3S2 h7QEaex4suXbh3KQ4hOQtfhF0F2ZJVkhzbIl6fbKuJbaz7hsqyCT0vkjo+/hczybqkD6 xG1A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="XQHJigX/"; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-88531-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-88531-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id q13-20020a02a30d000000b004748afe854esi1177587jai.15.2024.03.01.06.03.11 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 06:03:11 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-88531-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="XQHJigX/"; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-88531-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-88531-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org 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 sv.mirrors.kernel.org (Postfix) with ESMTPS id 884D0281EAD for ; Fri, 1 Mar 2024 14:03:11 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id B47BC6EEF7; Fri, 1 Mar 2024 14:03:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="XQHJigX/" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 D73016E60C; Fri, 1 Mar 2024 14:03:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709301784; cv=none; b=FkY8/jxLDYDS60L3/+mF7opqyTC8euBMNhaxVXPB1q0t2ZNdgiaB2Q9E4LbyvvHhLuZeXxYavk+sOHRvuRlFY6TiB6XzHZrqS6Zv/p0tEThpkK8OMt887qUKJd34uddLh8HLJk4oh+6eUp97dD/rAoSrahrZ3rJndYcmf0wyMcM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709301784; c=relaxed/simple; bh=0zgjWQSZJu5FIVJHU/jQgShZVUpbLN0aAKYRK+7+DyI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=pNAH9iDhl8Bx/uZnPiGPn8ztqDLX43PSlLBIJQwz0kqxN1W43jNmK1qXs5jYajk98wr/6rNx7F+HtIbyOzOxJEtk4OVaZlT2A5vPAyDvipHCE6OCIinIwZMe4QOB6c3yfbeytQA1dsUeMYK1NerdrzMrdb14nrJ4nwnhonrgqVI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=XQHJigX/; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5D61AC433F1; Fri, 1 Mar 2024 14:03:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1709301783; bh=0zgjWQSZJu5FIVJHU/jQgShZVUpbLN0aAKYRK+7+DyI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=XQHJigX/2HJz/xIXEi3vh311K81ea1fNXH82TV47vNBN5vfvLalJkPJwkJ3Oe7doH xnGMuoNjs98iO3V2VBfhe10MpB6eraCKKII1WeeNoiiJvjHdmgU3HjU3dCwZqJcnAf Hnbsjh64tErTWssu9pgT3ACEsQc2pQ71rCHWFhqPCAaE+S/mN9sioiAirOPR2PxKqj PjDP+X2HGeR+5+Rn2dr9oux5eRYvubutmAX0Q3ZbIAPORacEvZuI3j/9OSk05G+EL9 hwlhNLfIKrr2TA0D7q+GomqVNSk+W095TcB2fOqhj4ZqhGJLwjf1zLS7O8ZgI/64yR 3Cuy4au7n2qgQ== Received: from johan by xi.lan with local (Exim 4.97.1) (envelope-from ) id 1rg3Tn-000000001Qd-0bmX; Fri, 01 Mar 2024 15:03:15 +0100 Date: Fri, 1 Mar 2024 15:03:15 +0100 From: Johan Hovold To: Gabor Juhos Cc: Bjorn Andersson , Konrad Dybcio , Michael Turquette , Stephen Boyd , Sricharan Ramabadhran , Varadarajan Narayanan , Gokul Sriram Palanisamy , Devi Priya , Anusha Rao , Bryan O'Donoghue , Georgi Djakov , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/7] clk: qcom: gcc-ipq5018: fix terminating of frequency table arrays Message-ID: References: <20240229-freq-table-terminator-v1-0-074334f0905c@gmail.com> <20240229-freq-table-terminator-v1-1-074334f0905c@gmail.com> <91b36da5-637d-4156-8be4-5aed55fc3c5d@gmail.com> 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=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <91b36da5-637d-4156-8be4-5aed55fc3c5d@gmail.com> On Fri, Mar 01, 2024 at 02:37:01PM +0100, Gabor Juhos wrote: > Hi Johan, > > 2024. 03. 01. 10:40 keltezéssel, Johan Hovold írta: > > On Thu, Feb 29, 2024 at 07:07:46PM +0100, Gabor Juhos wrote: > >> The frequency table arrays are supposed to be terminated with an > >> empty element. Add such entry to the end of the arrays where it > >> is missing in order to avoid possible out-of-bound access when > >> the table is traversed by functions like qcom_find_freq() or > >> qcom_find_freq_floor(). > >> > >> Fixes: e3fdbef1bab8 ("clk: qcom: Add Global Clock controller (GCC) driver for IPQ5018") > > > > Good find! > > > > Looks like these should be backported to the stable kernels as well so > > someone should add: > > > > Cc: stable@vger.kernel.org > > > > to all patches except possibly the sc8280xp one (that camera clock > > controller was added in 6.8-rc1 so that patch does not need it in case > > you can these fixes in before 6.8 is released). > > You are right maybe, although I did not find strong enough reasons for adding > the stable tags. > > Only the changes of the gcc-ipq5018 driver has been tested on real hardware the > others are not. So those does not fit into the "It must be obviously correct and > tested." rule. Since this looks like a straight-forward and obviously correct fix for a bug which could have bad consequences, not being able to test each patch on actual hardware is not a problem. Johan