Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752746AbbLKTAN (ORCPT ); Fri, 11 Dec 2015 14:00:13 -0500 Received: from g4t3426.houston.hp.com ([15.201.208.54]:56631 "EHLO g4t3426.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751449AbbLKTAJ (ORCPT ); Fri, 11 Dec 2015 14:00:09 -0500 Date: Fri, 11 Dec 2015 12:00:05 -0700 From: Jerry Hoemann To: Dan Williams Cc: Ross Zwisler , "Rafael J. Wysocki" , Len Brown , "Elliott, Robert (Persistent Memory)" , jmoyer , Dmitry Krivenok , Linda Knippers , "linux-nvdimm@lists.01.org" , Linux ACPI , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v3 3/3] nvdimm: Add IOCTL pass thru functions Message-ID: <20151211190005.GA58528@tevye.fc.hp.com> Reply-To: Jerry.Hoemann@hpe.com References: <20151210002442.GA104756@tevye.fc.hp.com> <20151211180950.GA57513@tevye.fc.hp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2316 Lines: 55 On Fri, Dec 11, 2015 at 10:18:59AM -0800, Dan Williams wrote: > On Fri, Dec 11, 2015 at 10:09 AM, Jerry Hoemann 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 > Lv Zheng > Rafael J. Wysocki > > > 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-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/