From patchwork Mon Oct 16 11:04:11 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Matti Vaittinen X-Patchwork-Id: 15417 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:612c:2908:b0:403:3b70:6f57 with SMTP id ib8csp3377108vqb; Mon, 16 Oct 2023 04:04:40 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHqRPwlAcoAJt66cTfMOanJEieEwZ4D1pKj7mzBkgvloY8jDpe6kqsjnjDdfmpPBRUlkIw3 X-Received: by 2002:a17:90a:17c5:b0:27d:3b8:a08a with SMTP id q63-20020a17090a17c500b0027d03b8a08amr14049285pja.1.1697454280066; Mon, 16 Oct 2023 04:04:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1697454280; cv=none; d=google.com; s=arc-20160816; b=BHv7PysiICMj/k4awEOa4Zp2JeyQHgDJZrskChR3DTjHQ2MmeXqOkdBxvGVAVV94Iu i+O5sGmtPy5cBIHPn+mDXVE84aXv2bxa5GAFNQ2EoJauiOa1XYIQFTP0AgIKNStYqLpX IAYiqWqGPma5dvHFgNQLp54iQMZeBVVV8NjOttHs5fA+eBMLezHWTVIU0HbA0+34m4rC dPlXomsbNiFEIJsxoKdg72saNqo9BHkTKgXpn8IzIf1hrchdaCzCUQCx1wVwYhGIh99e 1lzf5DTfxgFnLukqYhjpGTIYXN1QiCqHF0DLSb7/lzjfUa56aBVGTXxbnwo1ydG2VQ0l m8VA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-disposition:mime-version:message-id :subject:cc:to:from:date:dkim-signature; bh=/nDb8K3NJwk1obZ1SsU2lxQGhz5tCjrcotcdhCQHsm0=; fh=jC3Zr14IfogOQHFLVGA9DtjKypPgomeJbjRJgp8dR94=; b=FCt30hfVc1TFcG4OH5sPqwWYfIhP40EqmUu+bSJol6cepeXXC1+fRLtydQGBkMVZ3H uXddYHyjqlM6IW3TOoSrqXOFq++WAvjOLr20mrDLtqr+E4ulWCvIqVCqnaUTDZukXhxe Dej7tyEWdlQSmMH3NdAS/M/oUanYz3QR4xlJJtXYPlMWQ+Cvmtad4aNUPIQE4AifdATv loBboMipLQN4IkugXGlyidOepA6fO7MdCLNMLrUO7utyUF6jY/UHvzXsTjs3QyHclAg9 p8cNMKca8UypNQbrBA7NwRg1RQ8HLIbZYfuCUM2uY8100ROyaHtRnSB36ONgDsv3D/IQ dzxw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=AUS8P6Th; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from fry.vger.email (fry.vger.email. [23.128.96.38]) by mx.google.com with ESMTPS id lx15-20020a17090b4b0f00b0027cef9ad081si6484840pjb.164.2023.10.16.04.04.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Oct 2023 04:04:40 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) client-ip=23.128.96.38; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=AUS8P6Th; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id 8E75D804B5F9; Mon, 16 Oct 2023 04:04:37 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232488AbjJPLEX (ORCPT + 18 others); Mon, 16 Oct 2023 07:04:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34316 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230116AbjJPLEX (ORCPT ); Mon, 16 Oct 2023 07:04:23 -0400 Received: from mail-lj1-x22e.google.com (mail-lj1-x22e.google.com [IPv6:2a00:1450:4864:20::22e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 11AC583; Mon, 16 Oct 2023 04:04:21 -0700 (PDT) Received: by mail-lj1-x22e.google.com with SMTP id 38308e7fff4ca-2c515527310so21690071fa.2; Mon, 16 Oct 2023 04:04:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1697454259; x=1698059059; darn=vger.kernel.org; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=/nDb8K3NJwk1obZ1SsU2lxQGhz5tCjrcotcdhCQHsm0=; b=AUS8P6ThC7Ni6xfq5vKGDZWmAprcI4E9Z2YY7wprmMw1L155luPP4Aw6Vh4SM+Y7Sb OrEE3hKjWPlFGxVwqUV++fTmqQhLmkbVOmc4h1Nsfl503tkMPWM9DjLxuX8l7fvGAgJT frEynMsxyX7/PTJvO2TWxRBeZDl8+YVL73CpUcHlKCL+K7bSrrpvKFUmqC3rMbEmaJ/M cLgKUMzb2zQT1UfQbGPmC11Imk7BZDxiWKZA4KBraa28N6D1imnTKG/u8YOKqnG9oyLP gv437OgdUr+3N24Nqgi/KrrD3BdCEHtFiL/XiM4m31WLQVXgEP5IjhEsSaRbAd2WqQhm QI+w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697454259; x=1698059059; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=/nDb8K3NJwk1obZ1SsU2lxQGhz5tCjrcotcdhCQHsm0=; b=j8hRtZbVh8izNSvLGXVhqYvks8KmAElBWX841ocN63Y9FQTSi/S0XkZmhpdRN3PcLP OMT5elgENw9PCKgE2ZGnLgobk1Grb+7htmAaNaFCfve5FW8gddsUqAUhcY0jZzLZnMlS wJCKuMAtRfkXCT8xtxcejltd6VBqxjdbTQbzVj9og+c/JG7U3Fb/Q/F4M5g5zza2e+Ml GUNvVkKRjp3aXpu8o42hklzpdOfnOEf9sAc2OXBlqTDl5S7IhJN5QyKT7ZNX2xIjRAO2 RrjoLv7NHa4HnuuxMZnHcIw3PC2A8RMGUZpjH6/3dP+TKioBkxrW7LGa424AjMCiFuxp sALQ== X-Gm-Message-State: AOJu0YyNfII6RC7d9tHkVo5tAGtA9lfNZDo3+LmLV2djRUugJ20qXPC7 aspbZvO0wqeW0N3PoPO9Tg0= X-Received: by 2002:a05:651c:1a24:b0:2c5:1bd9:f95c with SMTP id by36-20020a05651c1a2400b002c51bd9f95cmr3540664ljb.53.1697454258987; Mon, 16 Oct 2023 04:04:18 -0700 (PDT) Received: from dc78bmyyyyyyyyyyyyydt-3.rev.dnainternet.fi (dc78bmyyyyyyyyyyyyydt-3.rev.dnainternet.fi. [2001:14ba:16f8:1500::7]) by smtp.gmail.com with ESMTPSA id e20-20020a2e8ed4000000b002bfff335ca1sm1267503ljl.79.2023.10.16.04.04.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Oct 2023 04:04:17 -0700 (PDT) Date: Mon, 16 Oct 2023 14:04:11 +0300 From: Matti Vaittinen To: Matti Vaittinen , Matti Vaittinen Cc: Jonathan Cameron , Lars-Peter Clausen , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 0/2] Sanity-check available_scan_masks array Message-ID: MIME-Version: 1.0 Content-Disposition: inline X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Mon, 16 Oct 2023 04:04:37 -0700 (PDT) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1779909819028685324 X-GMAIL-MSGID: 1779909819028685324 Sanity-check available_scan_masks array The available_scan_masks is an array of bitmasks representing the channels which can be simultaneously(*) scanned by a driver from the device. Following special characteristics apply: - When IIO is scanning through the array it will use the first mask which can be used to scan all enabled channels. This means drivers should order the array in the order of the preference. This does also mean that a mask which is a subset of a mask located earler in array will never be used because the earlier one will be picked by the core. - Masks wider than size of long are supported only to some extent. The code scanning through the array will interpret the first mask with first long zeroed as end-of-array terminator. Changing this behaviour would make mask-arrays for multi-long masks to be terminated by more than one zero long. Failure to do so would result kernel to read beyond the array generating a potentially hazardous bug. Add a sanity-check to IIO-device registration emitting a warning if available_scan_mask array is misordered or if mask width is larger than a long while available_scan_mask-array is populated. Currently there should be no in-tree drivers with available_scan_mask populated and mask wider than a long. Revision history: v1 => v2: - Add patch 2/2 documenting why iio_scan_mask_match() checks only a long worth of bits while searching for the end of the available_scan_mask-array. - Styling of patch 1/2 as per comments from Jonathan v1 and related discussion here: https://lore.kernel.org/lkml/ZRvjuZaQWdZw1U1I@dc78bmyyyyyyyyyyyyydt-3.rev.dnainternet.fi/ Matti Vaittinen (2): iio: sanity check available_scan_masks array iio: buffer: document known issue drivers/iio/industrialio-buffer.c | 16 ++++++++ drivers/iio/industrialio-core.c | 63 +++++++++++++++++++++++++++++++ 2 files changed, 79 insertions(+)