Re: [PATCH] block: always allocate integrity buffer when required

[Christoph Hellwig <hch@lst.de>]

On Fri, May 09, 2025 at 08:33:00AM -0600, Keith Busch wrote:
> It looks like tag_size just refers to the space for the "application
> tag", which can vary depending on if ref_tag is used or not. But it's
> always going to be smaller than the tuple size.
> 
> I think you mean 'if tuple_size == sizeof(struct {t10|crc64}_pi_tuple)',
> depending on which csum type is used.

Ah yes, of course.

> I introduced a new flag because I
> thought that gen/strip property was just an arbitrary decision that NVMe
> made for PRACT, but if it's a universal thing, then we can totally use
> that. 

There is no other protocol supporting mixed PI / totally user defined
metadata.  But strip/insert can't be supported if there is more metadata
than the PI tuple, so I suspect anyone else picking it up would have
to treat it that way.  If I'm wrong and someone will eventually do
something else we can still go down the flag route when needed.