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.