From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 A226D43CE73 for ; Mon, 12 Oct 2026 07:00:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791788445; cv=none; b=npa3/1A9JyVVAybZ+mLqOujLTj1CljiKE5/WBJXpMvdLGhYvIqHsvQgTR0zOUqjeJt+LJfNcDXz6rmz2mPjMIiiHE2cSN9moS/WjxNcaX6QDq0F1VGUuthvrR6z40RzpeoYDExFa9RomvkWmM1njy5nhZ9X7GjfvcGN40iPMEqU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791788445; c=relaxed/simple; bh=+esHrw8A827bwmP7zAwxMk0pslZ0NIIMcR6VJkJHJvU=; h=Message-ID:From:Subject:To:Cc:In-Reply-To:References:Content-Type: Date; b=Yol9n5FqPyEAVoIsVU1yIiis1BE+6ZWOgYOIRgaukGYW1wNeDYCpDdGZ4nBZ/d1yeBkR2bZynRB/0EIH0q55tt5uRwxMd802kOM+4JjYV56PQaTk4/Bnc1Cpn4/ZLwq/TmA8brG87ntrLm/Tg/bBzJuHKlFWoKIlc0EMQ6u3gT0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Lwb9+lqI; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Lwb9+lqI" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 257E41F0089B; Mon, 12 Oct 2026 07:00:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791788444; bh=r19Im8HO0VMd/LtZn9fpu+F06N9cUU5a2NFzYpKq+GA=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=Lwb9+lqIMoBPxopNffzr76/GdktcJ6JLy55t1ss9ESuXbW4EGzexC04uYmdeaBeN/ nxB/bkx9TLySzo++456GGkHNi0bzv9bJsJq5OJ/oP7qWZNp/fihuMv6UWExJ4WLq6J ikEBz0KqrltWJv8fLiCyKVFgn1mcMBknlotQWFsf2LZ0EygvR8o0mQIhj6cLz/TlGm RDobx+UcFxwFX4kErg1H2ffoQVFvN8V4RnKBBHYx/PvypvH2Ka7x+2DYmme0t9S8bs 7oTKnuDJYa56eIdAiMaMbcjxzKbKpePn8UQyfvAf9/MIbyd0GxXs0VNoz8mfdcFNhZ t3owfYy6Uhf6Q== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH v21 09/14] media: qcom: camss: vfe: Add support for VFE 1080 Reply-To: sashiko-reviews@lists.linux.dev To: "Hangxiang Ma" Cc: vkoul@kernel.org, olteanv@gmail.com, media-ci@linuxtv.org, linux-phy@lists.infradead.org, conor+dt@kernel.org, robh@kernel.org, devicetree@vger.kernel.org, neil.armstrong@linaro.org In-Reply-To: <20261011-kaanapali-camss-v21-9-04fd28d815c6@oss.qualcomm.com> References: <20261011-kaanapali-camss-v21-0-04fd28d815c6@oss.qualcomm.com> <20261011-kaanapali-camss-v21-9-04fd28d815c6@oss.qualcomm.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Mon, 12 Oct 2026 07:00:43 +0000 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 2 potential issue(= s) to consider: - [High] media: camss: hardcoded AUP_UPDATE count violates hardware constra= int - [High] media: camss: data race on csid->aup_update and rup_update -- commit bab5581422e8c33728a37abe32648420503fc60c Author: Hangxiang Ma media: qcom: camss: vfe: Add support for VFE 1080 Add support for the VFE (Video Front End) 1080 found on Kaanapali SoC. > Kaanapali requires REG_UPDATE and AUP_UPDATE to be issued only after all > CSID configuration has completed. In addition, the number of AUP_UPDATE > requests must match the number of buffers queued to the write master > while it is being enabled. [Severity: High] Does the current implementation actually enforce this requirement? If userspace queues 1 buffer via V4L2 and calls STREAMON, the vb2_queue lacks a min_queued_buffers limit in msm_video_register(). Consequently, vfe_enable_output_v2() dynamically queues only 1 buffer to the hardware, but csid_configure_stream() in camss-csid-1080.c unconditionally issues CAMSS_INIT_BUF_COUNT (2) AUP_UPDATE commands. Can this mismatch lead to write master state corruption or IOMMU faults due to hardware desynchronization? > diff --git a/drivers/media/platform/qcom/camss/camss-vfe-1080.c b/drivers= /media/platform/qcom/camss/camss-vfe-1080.c > new file mode 100644 > index 0000000000000..016857cd10251 > --- /dev/null > +++ b/drivers/media/platform/qcom/camss/camss-vfe-1080.c [ ... ] > +static void vfe_reg_update(struct vfe_device *vfe, enum vfe_line_id line= _id) > +{ > + int port_id =3D line_id; > + > + camss_reg_update(vfe->camss, vfe->id, port_id, false); > +} [Severity: High] Does this introduce a data race on csid->aup_update and rup_update? This call reaches __csid_aup_update() in process context. While local interrupts are disabled via vfe->output_lock, there is no cross-CPU synchronization. Concurrently, csid_isr() fires in hardirq context (potentially on another C= PU) and calls __csid_aup_rup_clear(), which performs a bitwise clear on the same state variables without holding a shared lock: csid->aup_update &=3D ~CSID_RUP_AUP_RDI(port_id); csid->rup_update &=3D ~CSID_RUP_AUP_RDI(port_id); Can these concurrent non-atomic bitwise operations cause lost updates and corrupt the CSID_AUP_CMD and CSID_RUP_CMD register states, leading to stream stalls or hardware misconfiguration? --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261011-kaanapali-= camss-v21-0-04fd28d815c6@oss.qualcomm.com?part=3D9 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 672DCCA9EC5 for ; Mon, 12 Oct 2026 07:00:46 +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-Transfer-Encoding:Content-Type:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Date:References: In-Reply-To:Cc:To:Subject:From:Message-ID:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=TGRJ4HdUhIYv/Y6w/DRvYlk2MqQmBe27eoQaXMeq1k4=; b=rIPkjMv2moPC2o BlM5ZHvSfG5rSnqRPQG7i+Y6n/EZV8dMeCv7LrlDfRi7dSZruAGZgPMlAzOoCEeeANEzVc/AlmHQv 10AALH5vArgPHu6S23c4poOW6MyqVzrM8kFb/6HYJn2TeCCpnuQ2X3+A+axArxuT2M7CyqDpj8R7f 2Lxq7d9qJ8KBZkWpeCmMJq97ge9/beVGeZR71iH50nTT4WDooSpDxm8bOrRFcknLjM7Ys/T3m4Y3N +HDM6BiZAA5f6iQ+4lAjGwYOL7xXn11/mhAHT2Hg4TH0vwqXh2aU1Icmcjyl4xPIPZ0VfxJ0pVvv6 77DeeAyiYTI78NDzRARA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xGA1e-00000009nTp-0SqV; Mon, 12 Oct 2026 07:00:46 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xGA1c-00000009nTg-3cOK for linux-phy@lists.infradead.org; Mon, 12 Oct 2026 07:00:44 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 9C55740AD1; Mon, 12 Oct 2026 07:00:44 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 257E41F0089B; Mon, 12 Oct 2026 07:00:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791788444; bh=r19Im8HO0VMd/LtZn9fpu+F06N9cUU5a2NFzYpKq+GA=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=Lwb9+lqIMoBPxopNffzr76/GdktcJ6JLy55t1ss9ESuXbW4EGzexC04uYmdeaBeN/ nxB/bkx9TLySzo++456GGkHNi0bzv9bJsJq5OJ/oP7qWZNp/fihuMv6UWExJ4WLq6J ikEBz0KqrltWJv8fLiCyKVFgn1mcMBknlotQWFsf2LZ0EygvR8o0mQIhj6cLz/TlGm RDobx+UcFxwFX4kErg1H2ffoQVFvN8V4RnKBBHYx/PvypvH2Ka7x+2DYmme0t9S8bs 7oTKnuDJYa56eIdAiMaMbcjxzKbKpePn8UQyfvAf9/MIbyd0GxXs0VNoz8mfdcFNhZ t3owfYy6Uhf6Q== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH v21 09/14] media: qcom: camss: vfe: Add support for VFE 1080 To: "Hangxiang Ma" Cc: vkoul@kernel.org, olteanv@gmail.com, media-ci@linuxtv.org, linux-phy@lists.infradead.org, conor+dt@kernel.org, robh@kernel.org, devicetree@vger.kernel.org, neil.armstrong@linaro.org In-Reply-To: <20261011-kaanapali-camss-v21-9-04fd28d815c6@oss.qualcomm.com> References: <20261011-kaanapali-camss-v21-0-04fd28d815c6@oss.qualcomm.com> <20261011-kaanapali-camss-v21-9-04fd28d815c6@oss.qualcomm.com> Date: Mon, 12 Oct 2026 07:00:43 +0000 X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg MiBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6Ci0gW0hpZ2hdIG1lZGlhOiBjYW1zczog aGFyZGNvZGVkIEFVUF9VUERBVEUgY291bnQgdmlvbGF0ZXMgaGFyZHdhcmUgY29uc3RyYWludAot IFtIaWdoXSBtZWRpYTogY2Ftc3M6IGRhdGEgcmFjZSBvbiBjc2lkLT5hdXBfdXBkYXRlIGFuZCBy dXBfdXBkYXRlCi0tCgpjb21taXQgYmFiNTU4MTQyMmU4YzMzNzI4YTM3YWJlMzI2NDg0MjA1MDNm YzYwYwpBdXRob3I6IEhhbmd4aWFuZyBNYSA8aGFuZ3hpYW5nLm1hQG9zcy5xdWFsY29tbS5jb20+ CgptZWRpYTogcWNvbTogY2Ftc3M6IHZmZTogQWRkIHN1cHBvcnQgZm9yIFZGRSAxMDgwCgpBZGQg c3VwcG9ydCBmb3IgdGhlIFZGRSAoVmlkZW8gRnJvbnQgRW5kKSAxMDgwIGZvdW5kIG9uIEthYW5h cGFsaSBTb0MuCgo+IEthYW5hcGFsaSByZXF1aXJlcyBSRUdfVVBEQVRFIGFuZCBBVVBfVVBEQVRF IHRvIGJlIGlzc3VlZCBvbmx5IGFmdGVyIGFsbAo+IENTSUQgY29uZmlndXJhdGlvbiBoYXMgY29t cGxldGVkLiBJbiBhZGRpdGlvbiwgdGhlIG51bWJlciBvZiBBVVBfVVBEQVRFCj4gcmVxdWVzdHMg bXVzdCBtYXRjaCB0aGUgbnVtYmVyIG9mIGJ1ZmZlcnMgcXVldWVkIHRvIHRoZSB3cml0ZSBtYXN0 ZXIKPiB3aGlsZSBpdCBpcyBiZWluZyBlbmFibGVkLgoKW1NldmVyaXR5OiBIaWdoXQpEb2VzIHRo ZSBjdXJyZW50IGltcGxlbWVudGF0aW9uIGFjdHVhbGx5IGVuZm9yY2UgdGhpcyByZXF1aXJlbWVu dD8KCklmIHVzZXJzcGFjZSBxdWV1ZXMgMSBidWZmZXIgdmlhIFY0TDIgYW5kIGNhbGxzIFNUUkVB TU9OLCB0aGUgdmIyX3F1ZXVlCmxhY2tzIGEgbWluX3F1ZXVlZF9idWZmZXJzIGxpbWl0IGluIG1z bV92aWRlb19yZWdpc3RlcigpLiBDb25zZXF1ZW50bHksCnZmZV9lbmFibGVfb3V0cHV0X3YyKCkg ZHluYW1pY2FsbHkgcXVldWVzIG9ubHkgMSBidWZmZXIgdG8gdGhlIGhhcmR3YXJlLApidXQgY3Np ZF9jb25maWd1cmVfc3RyZWFtKCkgaW4gY2Ftc3MtY3NpZC0xMDgwLmMgdW5jb25kaXRpb25hbGx5 IGlzc3VlcwpDQU1TU19JTklUX0JVRl9DT1VOVCAoMikgQVVQX1VQREFURSBjb21tYW5kcy4KCkNh biB0aGlzIG1pc21hdGNoIGxlYWQgdG8gd3JpdGUgbWFzdGVyIHN0YXRlIGNvcnJ1cHRpb24gb3Ig SU9NTVUgZmF1bHRzCmR1ZSB0byBoYXJkd2FyZSBkZXN5bmNocm9uaXphdGlvbj8KCj4gZGlmZiAt LWdpdCBhL2RyaXZlcnMvbWVkaWEvcGxhdGZvcm0vcWNvbS9jYW1zcy9jYW1zcy12ZmUtMTA4MC5j IGIvZHJpdmVycy9tZWRpYS9wbGF0Zm9ybS9xY29tL2NhbXNzL2NhbXNzLXZmZS0xMDgwLmMKPiBu ZXcgZmlsZSBtb2RlIDEwMDY0NAo+IGluZGV4IDAwMDAwMDAwMDAwMDAuLjAxNjg1N2NkMTAyNTEK PiAtLS0gL2Rldi9udWxsCj4gKysrIGIvZHJpdmVycy9tZWRpYS9wbGF0Zm9ybS9xY29tL2NhbXNz L2NhbXNzLXZmZS0xMDgwLmMKWyAuLi4gXQo+ICtzdGF0aWMgdm9pZCB2ZmVfcmVnX3VwZGF0ZShz dHJ1Y3QgdmZlX2RldmljZSAqdmZlLCBlbnVtIHZmZV9saW5lX2lkIGxpbmVfaWQpCj4gK3sKPiAr CWludCBwb3J0X2lkID0gbGluZV9pZDsKPiArCj4gKwljYW1zc19yZWdfdXBkYXRlKHZmZS0+Y2Ft c3MsIHZmZS0+aWQsIHBvcnRfaWQsIGZhbHNlKTsKPiArfQoKW1NldmVyaXR5OiBIaWdoXQpEb2Vz IHRoaXMgaW50cm9kdWNlIGEgZGF0YSByYWNlIG9uIGNzaWQtPmF1cF91cGRhdGUgYW5kIHJ1cF91 cGRhdGU/CgpUaGlzIGNhbGwgcmVhY2hlcyBfX2NzaWRfYXVwX3VwZGF0ZSgpIGluIHByb2Nlc3Mg Y29udGV4dC4gV2hpbGUgbG9jYWwKaW50ZXJydXB0cyBhcmUgZGlzYWJsZWQgdmlhIHZmZS0+b3V0 cHV0X2xvY2ssIHRoZXJlIGlzIG5vIGNyb3NzLUNQVQpzeW5jaHJvbml6YXRpb24uCgpDb25jdXJy ZW50bHksIGNzaWRfaXNyKCkgZmlyZXMgaW4gaGFyZGlycSBjb250ZXh0IChwb3RlbnRpYWxseSBv biBhbm90aGVyIENQVSkKYW5kIGNhbGxzIF9fY3NpZF9hdXBfcnVwX2NsZWFyKCksIHdoaWNoIHBl cmZvcm1zIGEgYml0d2lzZSBjbGVhciBvbiB0aGUgc2FtZQpzdGF0ZSB2YXJpYWJsZXMgd2l0aG91 dCBob2xkaW5nIGEgc2hhcmVkIGxvY2s6CgogICAgY3NpZC0+YXVwX3VwZGF0ZSAmPSB+Q1NJRF9S VVBfQVVQX1JESShwb3J0X2lkKTsKICAgIGNzaWQtPnJ1cF91cGRhdGUgJj0gfkNTSURfUlVQX0FV UF9SREkocG9ydF9pZCk7CgpDYW4gdGhlc2UgY29uY3VycmVudCBub24tYXRvbWljIGJpdHdpc2Ug b3BlcmF0aW9ucyBjYXVzZSBsb3N0IHVwZGF0ZXMKYW5kIGNvcnJ1cHQgdGhlIENTSURfQVVQX0NN RCBhbmQgQ1NJRF9SVVBfQ01EIHJlZ2lzdGVyIHN0YXRlcywgbGVhZGluZwp0byBzdHJlYW0gc3Rh bGxzIG9yIGhhcmR3YXJlIG1pc2NvbmZpZ3VyYXRpb24/CgotLSAKU2FzaGlrbyBBSSByZXZpZXcg wrcgaHR0cHM6Ly9zYXNoaWtvLmRldi8jL3BhdGNoc2V0LzIwMjYxMDExLWthYW5hcGFsaS1jYW1z cy12MjEtMC0wNGZkMjhkODE1YzZAb3NzLnF1YWxjb21tLmNvbT9wYXJ0PTkKCi0tIApsaW51eC1w aHkgbWFpbGluZyBsaXN0CmxpbnV4LXBoeUBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHBzOi8vbGlz dHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LXBoeQo=