On Fri, Aug 4, 2017 at 2:12 AM, Yasunori Goto
> ndctl:libndctl Make interfaces to use Translate SPA.
A process note, please separate kernel patches and ndctl patches into
a different set. The expectation is that all the [PATCH 1/n] to [PATCH
n/n] should apply to the same project.
I have typically been marking my ndctl patches with a prefix like
[ndctl PATCH 1/n], where the kernel patches just say [PATCH].
> This patch makes 3 new interfaces :
> - to ask bus has translate SPA feature.
> - to call translate SPA.
> - to find DIMM by SPA address.
> Note) I'm not sure how many buffer should be prepared, because
> it depends on max # of mirroring way.
> This patch assume maxmum # is 4 way.
I think it is ok for now to just reserve space for 1-DIMM. When / if
mirroring support materializes in the kernel you should then be able
to detect how many DIMMs are in the mirror set. Since mirror support
would require changes to the Namespace Label specification I think we
have time to circle back and fix up ndctl for the multiple DIMM per
Translate SPA request case.
I see. I'll reserve space for 1-DIMM.
Otherwise, the kernel will handle the case where the provided buffer
is too small and return an error.