[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPcyv4h-8bm4nxO=5K6vRb3720rtaW8A-VHWRaZAhyw0_tjrvg@mail.gmail.com>
Date: Fri, 11 Dec 2015 10:18:59 -0800
From: Dan Williams <dan.j.williams@...el.com>
To: Jerry Hoemann <Jerry.Hoemann@....com>
Cc: Ross Zwisler <ross.zwisler@...ux.intel.com>,
"Rafael J. Wysocki" <rjw@...ysocki.net>,
Len Brown <lenb@...nel.org>,
"Elliott, Robert (Persistent Memory)" <elliott@....com>,
jmoyer <jmoyer@...hat.com>,
Dmitry Krivenok <krivenok.dmitry@...il.com>,
Linda Knippers <linda.knippers@....com>,
"linux-nvdimm@...ts.01.org" <linux-nvdimm@...ts.01.org>,
Linux ACPI <linux-acpi@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions
On Fri, Dec 11, 2015 at 10:09 AM, Jerry Hoemann <jerry.hoemann@....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@...el.com>
Lv Zheng <lv.zheng@...el.com>
Rafael J. Wysocki <rafael.j.wysocki@...el.com>
<linux-acpi@...r.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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists