On Fri, Dec 11, 2015 at 10:18:59AM -0800, Dan Williams wrote: > On Fri, Dec 11, 2015 at 10:09 AM, Jerry Hoemann <jerry.hoemann@xxxxxxx> 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@xxxxxxxxx> > Lv Zheng <lv.zheng@xxxxxxxxx> > Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > <linux-acpi@xxxxxxxxxxxxxxx> > > 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. Wasn't sure the level of lint checking being done on the linux kernel. The examples were all of the variety of a caller using and int that would be passed to a function with a parameter that would now be an u64. I think Lint can detect such issues. In addition to acpi_evaluate_dsm, there are also: acpi_check_dsm, acpi_evaluate_dsm_typed using int for these. Then there were other functions outside of acpica that passed value in ints that went to one of the three functions above. But, I can certainly limit the change to the acpica function declaration/definition. From what I could see the other code would still work, just have potential lint issues. > > > 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. -- ----------------------------------------------------------------------------- Jerry Hoemann Software Engineer Hewlett Packard Enterprise ----------------------------------------------------------------------------- -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
