Re: [PATCH 3/5] nvme: refactor namespace probing

Sagi Grimberg
2022-07-13T11:30:01Z

>> Maybe nvme_ns_info can have a id reference such that
>> nvme_update_ns_info_block can reuse it?
>
> I thought about it and, and it does seem doable. But it just seems
> like an awful lack of abstraction, all for avoiding a single command
> on some setups. True, those are the absolute majority right now, but
> with more and more information going to go into the comman set
> independent identify page it will become less and less so going
> forward.

I don't think its that bad, and it saves fetching the same data
twice. But I don't really care either way...