Re: tid_blockno() and tid_offset() accessor functions

[Greg Sabino Mullane]

On Sun, Mar 8, 2026 at 3:31 PM Tomas Vondra <to...@vondra.me> wrote:

> No opinion. For displaying the bogus TID value (like "(-1,0)") it's
> probably OK to show values that are a bit weird. If anything, we should
> be more careful on input, it's too late for tid_block() to decide what to
> do with an "impossible" TID value.
>

This one doesn't sit right with me. I think it's not too late. No reason
why tid_block cannot be stricter here than tid itself and complain. Other
than that, the patch looks good to me.


Cheers,
Greg