Messages in this thread | | | Date | Fri, 11 Dec 2015 10:18:59 -0800 | Subject | Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions | From | Dan Williams <> |
| |
On Fri, Dec 11, 2015 at 10:09 AM, Jerry Hoemann <jerry.hoemann@hpe.com> wrote: > On Wed, Dec 09, 2015 at 04:48:55PM -0800, Dan Williams wrote: >> Yes, updating the acpi_evaluate_dsm() definition seems the best choice. >> > > I have a patch for this. While not big (6 files), these files are > outside of nvdimm and will have a two line of over a dozen reviewers/lists > not previously reviewing this series.
For an acpi change no need to cc all those folks and lists. Just cc the following for that change:
Bob Moore <robert.moore@intel.com> Lv Zheng <lv.zheng@intel.com> Rafael J. Wysocki <rafael.j.wysocki@intel.com> <linux-acpi@vger.kernel.org>
Why 6 files and not 2 for a prototype update? I wouldn't go touch existing callers of acpi_evaluate_dsm() if they have been living with the potential truncation all this time there's no need to change.
> Do you want me to send this patch as one of this series (w/ the > extra reviewers?)
Yes, send that patch with the series so the acpi developers have the context for what motivated the change.
| |