From: Dan Williams <dan.j.williams@intel.com> To: Jerry Hoemann <Jerry.Hoemann@hpe.com> Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-nvdimm@lists.01.org" <linux-nvdimm@lists.01.org> Subject: Re: [PATCH v4 3/6] libnvdimm, acpi, nfit: Add bus level dsm mask for pass thru. Date: Wed, 5 Jul 2017 22:25:39 -0700 [thread overview] Message-ID: <CAPcyv4jmq9O0ZxbVk_=c-DAoJ5wGwBUgJqA70T-Ke-8ABU3FDQ@mail.gmail.com> (raw) In-Reply-To: <20170705231417.GA27667@anatevka.americas.hpqcorp.net> On Wed, Jul 5, 2017 at 4:14 PM, Jerry Hoemann <jerry.hoemann@hpe.com> wrote: > On Wed, Jul 05, 2017 at 09:35:48AM -0700, Dan Williams wrote: [..] >> It was a mistake to use _DSM for common root-level functionality, and >> we shouldn't double down on that mistake by allowing unfettered > > As to the moral aspects of ACPI's decision to standardiz the DSM for NVDIMM, > I take no position on whether it was a good thing or a bad thing; but it > is a thing. We need to handle it. I see no particular benefit to > making our own lives more difficult. We do handle everything we need to. Making future updates move at the same pace as standard ACPI enabing is the goal as well as not adding any momentum to continue abusing _DSM when we should be creating named methods for bus-level generic functionality. As a maintainer of this subsystem I'm fine with the burden of continuing to touch the code as the specification evolves and that stance matches standard Linux practice. _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm
WARNING: multiple messages have this Message-ID (diff)
From: Dan Williams <dan.j.williams@intel.com> To: Jerry Hoemann <Jerry.Hoemann@hpe.com> Cc: "linux-nvdimm@lists.01.org" <linux-nvdimm@lists.01.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v4 3/6] libnvdimm, acpi, nfit: Add bus level dsm mask for pass thru. Date: Wed, 5 Jul 2017 22:25:39 -0700 [thread overview] Message-ID: <CAPcyv4jmq9O0ZxbVk_=c-DAoJ5wGwBUgJqA70T-Ke-8ABU3FDQ@mail.gmail.com> (raw) In-Reply-To: <20170705231417.GA27667@anatevka.americas.hpqcorp.net> On Wed, Jul 5, 2017 at 4:14 PM, Jerry Hoemann <jerry.hoemann@hpe.com> wrote: > On Wed, Jul 05, 2017 at 09:35:48AM -0700, Dan Williams wrote: [..] >> It was a mistake to use _DSM for common root-level functionality, and >> we shouldn't double down on that mistake by allowing unfettered > > As to the moral aspects of ACPI's decision to standardiz the DSM for NVDIMM, > I take no position on whether it was a good thing or a bad thing; but it > is a thing. We need to handle it. I see no particular benefit to > making our own lives more difficult. We do handle everything we need to. Making future updates move at the same pace as standard ACPI enabing is the goal as well as not adding any momentum to continue abusing _DSM when we should be creating named methods for bus-level generic functionality. As a maintainer of this subsystem I'm fine with the burden of continuing to touch the code as the specification evolves and that stance matches standard Linux practice.
next prev parent reply other threads:[~2017-07-06 5:24 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-06-30 16:09 [PATCH v3 0/7] Enable DSM pass thru for root functions Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-06-30 16:09 ` [PATCH v4 1/6] libnvdimm: passthru functions clear to send Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-06-30 16:09 ` [PATCH v4 2/6] acpi, nfit: Enable DSM pass thru for root functions Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-06-30 16:09 ` [PATCH v4 3/6] libnvdimm, acpi, nfit: Add bus level dsm mask for pass thru Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-07-01 3:55 ` Dan Williams 2017-07-01 3:55 ` Dan Williams 2017-07-01 19:58 ` Jerry Hoemann 2017-07-01 19:58 ` Jerry Hoemann 2017-07-01 20:08 ` Dan Williams 2017-07-01 20:08 ` Dan Williams 2017-07-01 20:10 ` Dan Williams 2017-07-01 20:10 ` Dan Williams 2017-07-01 20:38 ` Jerry Hoemann 2017-07-01 20:38 ` Jerry Hoemann 2017-07-01 20:46 ` Dan Williams 2017-07-01 20:46 ` Dan Williams 2017-07-04 20:08 ` Jerry Hoemann 2017-07-04 20:08 ` Jerry Hoemann 2017-07-04 20:37 ` Dan Williams 2017-07-04 20:37 ` Dan Williams 2017-07-05 15:26 ` Linda Knippers 2017-07-05 15:26 ` Linda Knippers 2017-07-05 16:24 ` Jerry Hoemann 2017-07-05 16:24 ` Jerry Hoemann 2017-07-05 16:35 ` Dan Williams 2017-07-05 16:35 ` Dan Williams 2017-07-05 23:14 ` Jerry Hoemann 2017-07-05 23:14 ` Jerry Hoemann 2017-07-06 5:25 ` Dan Williams [this message] 2017-07-06 5:25 ` Dan Williams 2017-06-30 16:09 ` [PATCH v4 4/6] acpi, nfit: Show bus_dsm_mask in sysfs Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-06-30 16:09 ` [PATCH v4 5/6] libnvdimm: New ACPI 6.2 DSM functions Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-06-30 16:09 ` [PATCH v4 6/6] nfit allow override of root dsm mask Jerry Hoemann 2017-06-30 16:09 ` Jerry Hoemann 2017-07-01 3:49 ` Dan Williams 2017-07-01 3:49 ` Dan Williams
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAPcyv4jmq9O0ZxbVk_=c-DAoJ5wGwBUgJqA70T-Ke-8ABU3FDQ@mail.gmail.com' \ --to=dan.j.williams@intel.com \ --cc=Jerry.Hoemann@hpe.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-nvdimm@lists.01.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.