Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions

From	Dan Williams <dan.j.williams@intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions
Date	2015-12-11 19:20 +0100
Message-ID	<qEAsp-81R-7@gated-at.bofh.it> (permalink)
References	(1 earlier) <qBmP2-47N-59@gated-at.bofh.it> <qDCwi-25G-21@gated-at.bofh.it> <qDXhn-76y-1@gated-at.bofh.it> <qDXAJ-7eK-3@gated-at.bofh.it> <qEAsp-81R-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH v3 0/3] nvdimm: Add an IOCTL pass thru for DSM calls Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-02 22:10 +0100
  [PATCH v3 2/3] nvdimm: Add wrapper for IOCTL pass thru Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-02 22:10 +0100
  [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-02 22:10 +0100
    Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Dan Williams <dan.j.williams@intel.com> - 2015-12-09 03:20 +0100
      Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-10 01:30 +0100
        Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Dan Williams <dan.j.williams@intel.com> - 2015-12-10 01:50 +0100
          Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-11 19:20 +0100
            Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Dan Williams <dan.j.williams@intel.com> - 2015-12-11 19:20 +0100
              Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-11 20:10 +0100
  [PATCH v3 1/3] nvdimm: Clean-up access mode check. Jerry Hoemann <jerry.hoemann@hpe.com> - 2015-12-02 22:10 +0100

csiph-web