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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 C2C65C4345F for ; Tue, 23 Apr 2024 13:45:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=w/xE5984PxCBpcV+VsBMNJp9FdPJnd5LagmQ1jyVCTA=; b=QxNLVn47Xi3qN0 jzpKOeLRYg1psr+0b+of5YnTwCWmf9n3I0CHISFcRDdygJ8os2kbYyF7m/c2A2vUyiDz4iInObC8e xOGoF+X4erRDmDPbg9ONh4KJaf3d0YyFDcPMGkNecs5r9u3mMjIStMIDJKowkmDG3BHWrvN3IBuCs IDUvvSNfU8PDsNoZK1NQo71yD3EtLOwwA16K9iyKTqiS+1FxfZ2LIpLcOJPv5T4ouDyGLmilVX6D+ hb9NoP+Dg5CeMZ73Q/nFn0st+nOc9zgi1PNoSK26TaF3DYwCsCvLOupuHDDBZswqc6aIsYRsQIbwI rlLh/gcrehZRMnDTeiNg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rzGSs-000000008I5-0Ivu; Tue, 23 Apr 2024 13:45:42 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rzFVE-0000000HScj-1Dnt for linux-arm-kernel@bombadil.infradead.org; Tue, 23 Apr 2024 12:44:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=Content-Transfer-Encoding:Content-Type :In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date:Message-ID: Sender:Reply-To:Content-ID:Content-Description; bh=+qnKAwgvBHM35XCDpuBit4GUKItb+LTK0jOfybUvb7k=; b=His4cW3FcNP82ArD7RU+Qj1Fi8 Atx2dXhhmeFQ5v5GbZkvPIL8ssGbZ2yO8Qly9vxChaInjFN7v6zzw+bbfiEcjqzccOz9nOKe0DVLe hkuq3/8sOEp4dFehwmbs679n1sjwCFB5YTWHVb2JKU54hn5jcSG67JoIpgBDOWT793oZYmm67sB44 jBriHsPddaoXDuQD7feUbXI0iAyA3oln7l9htlcMaCd3h1fNBJEEnkNFvqWsS/7sLyHjeipkUg8Gm MHlgDomWHxGGG11To8+ZFdLy/5OFJVe5C10G/3MU+1lQRSFmLrVcvIvuVuTn89wRU6oUVVy51IIxP J4Q2/7mw==; Received: from mail-wr1-x42d.google.com ([2a00:1450:4864:20::42d]) by desiato.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rzFV9-0000000E1zx-3kZg for linux-arm-kernel@lists.infradead.org; Tue, 23 Apr 2024 12:44:03 +0000 Received: by mail-wr1-x42d.google.com with SMTP id ffacd0b85a97d-3476dcd9c46so4226806f8f.0 for ; Tue, 23 Apr 2024 05:43:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1713876232; x=1714481032; darn=lists.infradead.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=+qnKAwgvBHM35XCDpuBit4GUKItb+LTK0jOfybUvb7k=; b=f+RJEq9sMNPjy8lw04VZ4rSrk41opytRN+5MrQJ/Z0hCtym5PoH/YEzwoMRYKX2/Nk PqQnVt4ezYcLa4FYfQTRfifBlCHqMa4B0dh63JTfqCw9KmbLlqBnjO0wVzJKAix0hhj0 aVz2EEvIJzF2ASn81+PIFgbd2NIlFo8NapSJLOtpNc/rG3wWX5xzyjflns3rzBwDKs5O 8dLrSRXi0MeyRGUJccG0AkiO5V8O6YJAKCT+8EH2g8iseEx3V30/Azpr3DyEu2P3d7V4 OYe1m4wYNiU1IMr9RmVW1y+z6IpPrwmjReZ0cBzXdgGTomB49yVHvP7CKnWHsJWqyn8U EW1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713876232; x=1714481032; h=content-transfer-encoding:in-reply-to: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=+qnKAwgvBHM35XCDpuBit4GUKItb+LTK0jOfybUvb7k=; b=DvYUKaD63ZsY5ljKU8E4EBQGV3KcDkHgN6zlfa97aDqDSYNEeWi6y5xuul7uQxMI5w t4ft9GLC5B6iCGvBQRE/bKPjloPF12ViIvqrUuUcKnotc8WZybFgYpeRAIzYd/xggucy Rttp0x34/FbaZL7rGU3BHlpxZG24xX5+e5Z2HUHCOoDg1nI93iZBBc6G2uhQpLrxlcc/ k5lZPsk/9RHmIS4OnExVfjP++e4+6ffnrYtHCBA2gpP1BFZWuumbjr5m0u7raS33p/mk jNgabvNFY/TydB+B528N5j7kqG//WBvHjQgu6kxZWPSENKCR0FjzTiDneaMktOvfgxZ7 +RCA== X-Forwarded-Encrypted: i=1; AJvYcCVQhd6O9G+zfihBrSQxY7ms+5MwzD9345RvhvKTWXKY5JU86A4uGdQ4tMjhQ9cLRTPd19aqr/f1nNXmKOYcnCGE7a2AJatpYHK9TWBU1d8RGUlw0xI= X-Gm-Message-State: AOJu0Yz01SWjQ0m3N5Un5UAwy1uOcDQTkVsw4y/gltzKS4rQ6M7gUXFA 4iZL72kmoZa0UR23pibOiZZxmTW4SSgD7VtcAeNW9y4ZFOMM3ELDL8C9cSfRVGY= X-Google-Smtp-Source: AGHT+IHx5NN+z23VludiFeHTrKmrJCe4gLG+r1lTx6C4j7QEMysWQLMdtEpvLnQqAlOL1NQMbBj3Zg== X-Received: by 2002:a5d:6a8d:0:b0:34b:5caf:6342 with SMTP id s13-20020a5d6a8d000000b0034b5caf6342mr2073298wru.67.1713876232556; Tue, 23 Apr 2024 05:43:52 -0700 (PDT) Received: from [192.168.0.102] ([176.61.106.227]) by smtp.gmail.com with ESMTPSA id r16-20020a5d6950000000b003477d26736dsm14506412wrw.94.2024.04.23.05.43.50 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 23 Apr 2024 05:43:52 -0700 (PDT) Message-ID: Date: Tue, 23 Apr 2024 13:43:49 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 11/35] media: s2255: Use refcount_t instead of atomic_t for num_channels To: Ricardo Ribalda , Martin Tuma , Mauro Carvalho Chehab , Laurent Pinchart , Hans Verkuil , Hugues Fruchet , Alain Volmat , Maxime Coquelin , Alexandre Torgue , Paul Kocialkowski , Greg Kroah-Hartman , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Sakari Ailus , Thierry Reding , Jonathan Hunter , Sowjanya Komatineni , Luca Ceresoli , Matthias Brugger , AngeloGioacchino Del Regno , Hans Verkuil , Sergey Kozlov , Abylay Ospan , Ezequiel Garcia , Dmitry Osipenko , Stanimir Varbanov , Vikash Garodia , Bryan O'Donoghue , Bjorn Andersson , Konrad Dybcio , Benjamin Mugnier , Sylvain Petinot , Jacopo Mondi , Kieran Bingham , Laurent Pinchart , =?UTF-8?Q?Niklas_S=C3=B6derlund?= , Pavel Machek Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev, linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-msm@vger.kernel.org References: <20240415-fix-cocci-v1-0-477afb23728b@chromium.org> <20240415-fix-cocci-v1-11-477afb23728b@chromium.org> Content-Language: en-US From: Bryan O'Donoghue In-Reply-To: <20240415-fix-cocci-v1-11-477afb23728b@chromium.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240423_134400_710511_71010BBB X-CRM114-Status: GOOD ( 11.09 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 15/04/2024 20:34, Ricardo Ribalda wrote: > Use an API that resembles more the actual use of num_channels. > > Found by cocci: > drivers/media/usb/s2255/s2255drv.c:2362:5-24: WARNING: atomic_dec_and_test variation before object free at line 2363. > drivers/media/usb/s2255/s2255drv.c:1557:5-24: WARNING: atomic_dec_and_test variation before object free at line 1558. Hmm, that commit log needs more detail. "Convert from atomic_t to refcount_t because refcount_t has memory ordering guarantees which atomic does not, hence the WARNING for the free after the atomic dec." Something like that. I'll leave it up to yourself to decide if this warrants a Fixes: I don't think so myself because the previous code doesn't seem to matter to the decrement and free. --- bod _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel