| From | Dan Williams <> | Date | Fri, 21 Feb 2020 08:24:18 -0800 | Subject | Re: [PATCH v3 00/27] Add support for OpenCAPI Persistent Memory devices |
| |
On Fri, Feb 21, 2020 at 8:21 AM Dan Williams <dan.j.williams@intel.com> wrote: > > On Thu, Feb 20, 2020 at 7:28 PM Alastair D'Silva <alastair@au1.ibm.com> wrote: > > > > From: Alastair D'Silva <alastair@d-silva.org> > > > > This series adds support for OpenCAPI Persistent Memory devices, exposing > > them as nvdimms so that we can make use of the existing infrastructure. > > A single sentence to introduce: > > 24 files changed, 3029 insertions(+), 97 deletions(-) > > ...is inadequate. What are OpenCAPI Persistent Memory devices? How do > they compare, in terms relevant to libnvdimm, to other persistent > memory devices? What challenges do they pose to the existing enabling? > What is the overall approach taken with this 27 patch break down? What > are the changes since v2, v1? If you incorporated someone's review > feedback note it in the cover letter changelog, if you didn't
Assumptions and tradeoffs the implementation considered are also critical for reviewing the approach.
|