Re: [PATCH 8/8] nvme-pci: rework the build time assert for NVME_MAX_NR_DESCRIPTORS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 8/8] nvme-pci: rework the build time assert for NVME_MAX_NR_DESCRIPTORS
From: Keith Busch <kbusch@xxxxxxxxxx>
Date: Mon, 23 Jun 2025 09:44:29 -0600
Cc: Jens Axboe <axboe@xxxxxxxxx>, Sagi Grimberg <sagi@xxxxxxxxxxx>, Chaitanya Kulkarni <kch@xxxxxxxxxx>, Kanchan Joshi <joshi.k@xxxxxxxxxxx>, Leon Romanovsky <leon@xxxxxxxxxx>, Nitesh Shetty <nj.shetty@xxxxxxxxxxx>, Logan Gunthorpe <logang@xxxxxxxxxxxx>, linux-block@xxxxxxxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx, Daniel Gomez <da.gomez@xxxxxxxxxxx>, Leon Romanovsky <leonro@xxxxxxxxxx>
In-reply-to: <20250623141259.76767-9-hch@lst.de>
References: <20250623141259.76767-1-hch@lst.de> <20250623141259.76767-9-hch@lst.de>

On Mon, Jun 23, 2025 at 04:12:30PM +0200, Christoph Hellwig wrote:
> The current use of an always_inline helper is a bit convoluted.
> Instead use macros that represent the arithmetics used for building
> up the PRP chain.

Looks good.

Reviewed-by: Keith Busch <kbusch@xxxxxxxxxx>

References:
- new DMA API conversion for nvme-pci v2
  - From: Christoph Hellwig
- [PATCH 8/8] nvme-pci: rework the build time assert for NVME_MAX_NR_DESCRIPTORS
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 7/8] nvme-pci: replace NVME_MAX_KB_SZ with NVME_MAX_BYTE
Next by Date: Re: [PATCH 4/8] nvme-pci: merge the simple PRP and SGL setup into a common helper
Previous by thread: [PATCH 8/8] nvme-pci: rework the build time assert for NVME_MAX_NR_DESCRIPTORS
Next by thread: [PATCH 1/8] block: don't merge different kinds of P2P transfers in a single bio
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID] [Linux SCSI] [Linux ATA RAID] [IDE] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Device Mapper]