From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f174.google.com (mail-qk1-f174.google.com [209.85.222.174]) (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 54F61313522 for ; Wed, 15 Apr 2026 03:28:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776223695; cv=none; b=XaISEhEc5C3ZPPDefKtzUvGBabFBa2XjRS4K6kS4oXrzRhDL3mAbl588I1bPXnwJp+WUu0la0njFrVzVij3OuNF+5QwXa98xmhsRDpTviM1o+alqbG+Ey8QeeXzQwG79qnnyXx78vumH112/fnohGad7T/lvXCJyBzl9USVQmgo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776223695; c=relaxed/simple; bh=tCSW40mdJktOK1h1VfrP3QQE4ZWe+0q0/kLHOByHApk=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=GTAPXA1qu2nMCeSh6A0tWoTcXprFVdQy/+NgEBHOgajkaZgOYS1VI2lkPNF3/QuYcBSIQcyTqW3UzpkGUH8aOjNwoDmMyRBv7sgEdWR8Yd0KDJ3Adx+U/BZJEA9CbXMejy/CkyEIi3mJkAlvR3k+Vof4QApSOJ5DvqktlaboR8g= 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=scHr2tsG; arc=none smtp.client-ip=209.85.222.174 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="scHr2tsG" Received: by mail-qk1-f174.google.com with SMTP id af79cd13be357-8cb40149037so590665685a.2 for ; Tue, 14 Apr 2026 20:28:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1776223693; x=1776828493; 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=rLKjOgt0prGno2etyoUy7FVuq/6O60livyoOWpMDLDg=; b=scHr2tsGPNkK0eklazc83dke/VS8irTYyGPxqKBfopPNepWMbvV1fpaf3p580DjFEm 5rMXdhaysAAsPgts9ydlq4uyDJUJ6RNIvTR1Yv+3qwYvjjWljVoDv8TdO2OiMYJYRAnU +Xldqv0bawAhi07m21K5vtdwMWJDgc3SeVvEJE9xGRDVogJWypYSRx5oUPMQJYH1wo8H y207mW+yyiBzEuDmVUfCvK9QBokXDZfelieqPWMISNAKCZSvkbMWcSn7A/+htwsvLLvB yCJFq7eaSjAMqANQnS6VLVCydPbPPDvkwx7aXPT/riVTe/CHd2XUrp7o99SKX1+KT+r9 DVZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776223693; x=1776828493; 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=rLKjOgt0prGno2etyoUy7FVuq/6O60livyoOWpMDLDg=; b=WLMT4lmgU5VkUefB8h7nImjSFUc7oDN7c0wX1rtz72t4JAF75cM+Qqg5X3U1pncGqn UL/e4bq8znIHS0FKgvY97+Vn8chneNJOjVJ6ZE3Mm6wm12CBs3/AwDnfOLSW+F7lK5ov L6EyUY1V8hojgtnNWAXOsinXEP+xUUe4mpnrPHjqGCU+5hcRdSrKsTSgkX2Y/NO/ja1q 6hdxbSLhS+dbXdbeMdjdS6CilslhiPd5bV7z1z1R2Q75/LRW9IuCgRI2dvXyDJiEQt2M oMg5EdbHfBeyUcbhDZjJ2mjyeSY80tqDBqwkNoyVHtU8hF+2dWcTVOl6NJaiUS8gkuu1 O76A== X-Forwarded-Encrypted: i=1; AFNElJ+EogeN25R2upEo03GlendwbWlvAjHN5X8wBAo+Cb3JICMu4tPXEZqNjHXUgmx68J/xmb+Ga575vabgs1c=@vger.kernel.org X-Gm-Message-State: AOJu0YxkVLXF91BroZdtA7gGu17IXDz6XcB7W/znZMYud98GW9Rpdfjm Vch8PWwOvdId8BxiKuHOXeszL9+9PSCtC+5mallkY7j3+Kl+WX7uGjIa X-Gm-Gg: AeBDietXRi9K2MexKgKaZULahsJPwNEeXj1U4We4K0gSs5lr6ETxA2IFZmvwUCUaZBt DqjZ78pKExxg6e0L+9iM5eh2prYF/m9PLJFZhfvX/VBj8lW9NUhqCqrDJ7oPCqTGvLvRg+Leraw hMoiP1WgSiW34kntqUlpWjoyutig2d0Ww6HROogF3vZGrxB2DG5Ty6N/YxIvsq/6nnXJUHY671n YD5x3KZGrSAid4ey3QJSANNk2WGn2bFLjrN06WjLkeqlkF15cxrZ7uAH0ACDjiseSFoeJXI+HoA Y/SKTTAIiSLm6MyUhm4K0Brdrk2/lO4Kb+wdugkXzvyJ8qH2l2ZuKnHwpCN+LRskJGFDxtTk7gV 1SMBzwmPQDfRkEZMZdEgwbI4o+HMneMk1YCfZXb13qQbCRDm0tDJT2H8NhfAWDvj0eKCHKkpm59 JSJ60/45dOI1NKw8g1xloMHEx/ltCE5vTtX0NybBA+niAhi/bYaBY97u8= X-Received: by 2002:ac8:5f88:0:b0:50d:6838:964c with SMTP id d75a77b69052e-50dd5af0776mr330945371cf.18.1776223693139; Tue, 14 Apr 2026 20:28:13 -0700 (PDT) Received: from ryzen ([2601:644:8000:5b5d::8bf]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-8ae6c974c2esm2578556d6.17.2026.04.14.20.28.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Apr 2026 20:28:12 -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: [PATCHv4] dmaengine: hsu: use kzalloc_flex() Date: Tue, 14 Apr 2026 20:27:53 -0700 Message-ID: <20260415032753.6006-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: 8bit 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. Apply the exact same treatment to struct hsu_dma and devm_kzalloc(). Signed-off-by: Rosen Penev --- v4: move back counting variable assignment, as requested. v3: update description. v2: address review comments. drivers/dma/hsu/hsu.c | 43 +++++++++++-------------------------------- drivers/dma/hsu/hsu.h | 4 ++-- 2 files changed, 13 insertions(+), 34 deletions(-) diff --git a/drivers/dma/hsu/hsu.c b/drivers/dma/hsu/hsu.c index f62d60d7bc6b..b1dd4fd1109b 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, unsigned 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 = kzalloc_obj(*desc, GFP_NOWAIT); - if (!desc) - return NULL; - - desc->sg = 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 = to_hsu_dma_desc(vdesc); - kfree(desc->sg); kfree(desc); } @@ -276,10 +258,15 @@ static struct dma_async_tx_descriptor *hsu_dma_prep_slave_sg( struct scatterlist *sg; unsigned int i; - desc = hsu_dma_alloc_desc(sg_len); + desc = kzalloc_flex(*desc, sg, sg_len, GFP_NOWAIT); if (!desc) return NULL; + desc->nents = sg_len; + desc->direction = direction; + /* desc->active = 0 by kzalloc */ + desc->status = DMA_IN_PROGRESS; + for_each_sg(sgl, sg, sg_len, i) { desc->sg[i].addr = sg_dma_address(sg); desc->sg[i].len = sg_dma_len(sg); @@ -287,11 +274,6 @@ static struct dma_async_tx_descriptor *hsu_dma_prep_slave_sg( desc->length += sg_dma_len(sg); } - desc->nents = sg_len; - desc->direction = direction; - /* desc->active = 0 by kzalloc */ - desc->status = 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 = chip->regs + chip->offset; + unsigned short nr_channels; unsigned short i; int ret; - hsu = devm_kzalloc(chip->dev, sizeof(*hsu), GFP_KERNEL); + /* Calculate nr_channels from the IO space length */ + nr_channels = (chip->length - chip->offset) / HSU_DMA_CHAN_LENGTH; + hsu = devm_kzalloc(chip->dev, struct_size(hsu, chan, nr_channels), GFP_KERNEL); if (!hsu) return -ENOMEM; chip->hsu = hsu; - /* Calculate nr_channels from the IO space length */ - hsu->nr_channels = (chip->length - chip->offset) / HSU_DMA_CHAN_LENGTH; - - hsu->chan = devm_kcalloc(chip->dev, hsu->nr_channels, - sizeof(*hsu->chan), GFP_KERNEL); - if (!hsu->chan) - return -ENOMEM; + hsu->nr_channels = nr_channels; INIT_LIST_HEAD(&hsu->dma.channels); for (i = 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 *vdesc) @@ -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