> On Fri, Aug 4, 2017 at 2:12 AM, Yasunori Goto <y-g...@jp.fujitsu.com> wrote:
> > 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.
> 




_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

Reply via email to