From nobody Wed Apr 1 09:46:39 2026 Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) (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 CC7CE396B73 for ; Mon, 30 Mar 2026 20:44:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.171 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774903459; cv=none; b=twrerxR0z+ts3Qfh/5veXbk/JMIAN/d56j9sXiRcRgBgLQ86ddtGBMjj4IFWIxPAYr53jAT76t3O/q3EHDHoWT3q+N5bnEJ+T42+yHKvhW1dDX4O/rR9APZMmL2R/kG9dQLLc1O7iR+dPfwMhr/8D6KRuls2Hv7gEICYPECSY2k= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774903459; c=relaxed/simple; bh=V/J/NqbhMxWKwCmcNFvflHQXMGBiNkLsB+ZbVAlGsAY=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=S3biWXSDI1m3NblLf0x30flyx/Tvgzp3dUrK+e+BkDhGIqDvOL6+IkagTZ0FljIotFJ80sDoRf3YEoyETcNZ/fuTHLodvJiDwF4WFN+0LBAM8FNq2JlGCZkZin5rbh44kYtXfLCSyzWhEYNvpWEyln9bg0LOyZ963rQ5IXdd08w= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=pMPMLe5i; arc=none smtp.client-ip=209.85.214.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="pMPMLe5i" Received: by mail-pl1-f171.google.com with SMTP id d9443c01a7336-2aae4816912so31520025ad.2 for ; Mon, 30 Mar 2026 13:44:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1774903457; x=1775508257; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=cE1sZcA1XZypV//xVG+QNtow8cQCW7Hc34/dIAf9N1g=; b=pMPMLe5ij5lAX8tC9T5lVyKmgSzQ3qxVbX9+n78FpE+EMotP88VfWxFmxzSTMzoKiM vKRwgDgDD5VcppmwlYcjKx1NSGj2IQX/BeqFIh7+NqjVunS4T4OTb307lMHZ+FBtPYVQ ZwNAoBfIQtTvM1ptJC9aKRHLUT8RlUH0ETHIqI65CzbgfXF9X0vwEtropwldd/N9BqBs r+EijLK/xHHbO3rtDj9zxuGmZgPcl9BhQ7bQ+JAczhOor1m5yJN5fkgaSi3yBjgwt4ok 3eowV6jcyEUdH0Cqem7+lOTUWetkYsUUtG9ogQ8vXr9PE4HICWtmetNKUVDTm9+AGYXU ZbJA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774903457; x=1775508257; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=cE1sZcA1XZypV//xVG+QNtow8cQCW7Hc34/dIAf9N1g=; b=NpdLDovibmu1WrYSJ0BxNpSPBBXLt1RMmiP6hgvrrAvAbfQk3dBRSECdAmh8TYa/ri QzBv9100AssLuD5KWhf2CBxv4PnITCdsf/H5yjPt4DUrOQ8/fXXyOk1lpNKm2Ec8++F9 jGWTSA9Xiar4kkT7LQmiq2GV6eadLinT93m76XEqGzSf7VuYj3xgY8kxQ2TUkuCiJyDx BOPRIPTqBmKGKjSvW2N6groS8GdYZoPuYtg8GrQm6X2FMtws7qYKpjXjy4Uqsd+gC5Dk xcvze5YGLH3KdPUWyg7gZ4ke91RCAFgqG/zjatUQOT+887OQ3zWC62i332k6BUMuz+1L 3ZqQ== X-Forwarded-Encrypted: i=1; AJvYcCXHUJkPJEBTXli5SnCnDAHlTETPbRMKSDi63EXgxtRaM9dtYnZ+Eell3Wr57ywtdEqwgPhzx6b8tiD5rgQ=@vger.kernel.org X-Gm-Message-State: AOJu0YwbNZM705TJJU9S4ZIrz75cGaCCbzzNiusg8d9pWH3JjdxNktqx fY4siEEqqJaq+VbyxZXP4Xkw0ewka7/qxdpBeZddt50TG6zVpkzd1dfy X-Gm-Gg: ATEYQzxCCWyuU2otg8wwp5chsGTUkInEZcH9MhFtO/Iprs5abOyQgFiyuP9QorAoPHY eyOU48zrsN4fVUZeuioAARHFaoOgkpu0Zk7xRS1EwgazZLUtGAQNDB4FxWGm3MdFCg3GAocuVt4 +qT0N/D8I/R+40sOIpOoXYTJWul1diCj0cevjBlQP38dfdihV6j2QOq+7auRz2UlFoaW4ykIwRT TH/Sv/2IBHeJ/WMC8Uf5H2NiRaU6ctJhi9ozmk7K2RBE6msvPQdggjSbizJPw3ZzBjrY0ZuTfAh 0YrumEVV5677ygFhFeR6Opz3iykVe+1a2s02lzt6cVVSQo6f6zvYzsPHz3uX8eRg/2E5JD9IWgN dR4j8ODk7ylVzAV/ivEYFCrQCK6IOeFPM/n/mkMgOsxDTtA/bXB4gSewFDszQi3p+Imx2hSxTo3 /NAOoYDlezL8H1fgRAfmjl8acGzo7yfopcYgRg8veYw0F2ju3ZrMaDvnE= X-Received: by 2002:a17:903:19ee:b0:2b0:6e60:9586 with SMTP id d9443c01a7336-2b0cdc2badbmr142210825ad.17.1774903457148; Mon, 30 Mar 2026 13:44:17 -0700 (PDT) Received: from ryzen ([2601:644:8000:5b5d::8bd]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b2427c5f5bsm92656805ad.82.2026.03.30.13.44.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Mar 2026 13:44:16 -0700 (PDT) From: Rosen Penev To: dmaengine@vger.kernel.org Cc: Andy Shevchenko , Vinod Koul , Frank Li , Kees Cook , "Gustavo A. R. Silva" , linux-kernel@vger.kernel.org (open list:INTEL MID (Mobile Internet Device) PLATFORM), linux-hardening@vger.kernel.org (open list:KERNEL HARDENING (not covered by other areas):Keyword:\b__counted_by(_le|_be)?\b) Subject: [PATCHv3] dmaengine: hsu: use kzalloc_flex() Date: Mon, 30 Mar 2026 13:43:57 -0700 Message-ID: <20260330204357.4476-1-rosenp@gmail.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Simplifies allocations by using a flexible array member in this struct. Remove hsu_dma_alloc_desc(). It now offers no readability advantages in this single usage. Add __counted_by to get extra runtime analysis. Assign counting variable after allocation as required by __counted_by. Apply the exact same treatment to struct hsu_dma and devm_kzalloc(). Signed-off-by: Rosen Penev --- v3: update description. v2: address review comments. drivers/dma/hsu/hsu.c | 45 ++++++++++++------------------------------- drivers/dma/hsu/hsu.h | 4 ++-- 2 files changed, 14 insertions(+), 35 deletions(-) diff --git a/drivers/dma/hsu/hsu.c b/drivers/dma/hsu/hsu.c index f62d60d7bc6b..78a2352ada8c 100644 --- a/drivers/dma/hsu/hsu.c +++ b/drivers/dma/hsu/hsu.c @@ -241,28 +241,10 @@ int hsu_dma_do_irq(struct hsu_dma_chip *chip, unsigne= d short nr, u32 status) } EXPORT_SYMBOL_GPL(hsu_dma_do_irq); -static struct hsu_dma_desc *hsu_dma_alloc_desc(unsigned int nents) -{ - struct hsu_dma_desc *desc; - - desc =3D kzalloc_obj(*desc, GFP_NOWAIT); - if (!desc) - return NULL; - - desc->sg =3D kzalloc_objs(*desc->sg, nents, GFP_NOWAIT); - if (!desc->sg) { - kfree(desc); - return NULL; - } - - return desc; -} - static void hsu_dma_desc_free(struct virt_dma_desc *vdesc) { struct hsu_dma_desc *desc =3D to_hsu_dma_desc(vdesc); - kfree(desc->sg); kfree(desc); } @@ -276,10 +258,15 @@ static struct dma_async_tx_descriptor *hsu_dma_prep_s= lave_sg( struct scatterlist *sg; unsigned int i; - desc =3D hsu_dma_alloc_desc(sg_len); + desc =3D kzalloc_flex(*desc, sg, sg_len, GFP_NOWAIT); if (!desc) return NULL; + desc->nents =3D sg_len; + desc->direction =3D direction; + /* desc->active =3D 0 by kzalloc */ + desc->status =3D DMA_IN_PROGRESS; + for_each_sg(sgl, sg, sg_len, i) { desc->sg[i].addr =3D sg_dma_address(sg); desc->sg[i].len =3D sg_dma_len(sg); @@ -287,11 +274,6 @@ static struct dma_async_tx_descriptor *hsu_dma_prep_sl= ave_sg( desc->length +=3D sg_dma_len(sg); } - desc->nents =3D sg_len; - desc->direction =3D direction; - /* desc->active =3D 0 by kzalloc */ - desc->status =3D DMA_IN_PROGRESS; - return vchan_tx_prep(&hsuc->vchan, &desc->vdesc, flags); } @@ -428,22 +410,19 @@ int hsu_dma_probe(struct hsu_dma_chip *chip) { struct hsu_dma *hsu; void __iomem *addr =3D chip->regs + chip->offset; + unsigned short nr_channels; unsigned short i; int ret; - hsu =3D devm_kzalloc(chip->dev, sizeof(*hsu), GFP_KERNEL); + /* Calculate nr_channels from the IO space length */ + nr_channels =3D (chip->length - chip->offset) / HSU_DMA_CHAN_LENGTH; + hsu =3D devm_kzalloc(chip->dev, struct_size(hsu, chan, nr_channels), GFP_= KERNEL); if (!hsu) return -ENOMEM; - chip->hsu =3D hsu; - - /* Calculate nr_channels from the IO space length */ - hsu->nr_channels =3D (chip->length - chip->offset) / HSU_DMA_CHAN_LENGTH; + hsu->nr_channels =3D nr_channels; - hsu->chan =3D devm_kcalloc(chip->dev, hsu->nr_channels, - sizeof(*hsu->chan), GFP_KERNEL); - if (!hsu->chan) - return -ENOMEM; + chip->hsu =3D hsu; INIT_LIST_HEAD(&hsu->dma.channels); for (i =3D 0; i < hsu->nr_channels; i++) { diff --git a/drivers/dma/hsu/hsu.h b/drivers/dma/hsu/hsu.h index 3bca577b98a1..f6ca1014bccf 100644 --- a/drivers/dma/hsu/hsu.h +++ b/drivers/dma/hsu/hsu.h @@ -71,11 +71,11 @@ struct hsu_dma_sg { struct hsu_dma_desc { struct virt_dma_desc vdesc; enum dma_transfer_direction direction; - struct hsu_dma_sg *sg; unsigned int nents; size_t length; unsigned int active; enum dma_status status; + struct hsu_dma_sg sg[] __counted_by(nents); }; static inline struct hsu_dma_desc *to_hsu_dma_desc(struct virt_dma_desc *v= desc) @@ -115,8 +115,8 @@ struct hsu_dma { struct dma_device dma; /* channels */ - struct hsu_dma_chan *chan; unsigned short nr_channels; + struct hsu_dma_chan chan[] __counted_by(nr_channels); }; static inline struct hsu_dma *to_hsu_dma(struct dma_device *ddev) -- 2.53.0