Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1678242

Re: [PATCH v3 7/7] acpi, nfit: override mask

Show all headers | View raw

On Thu, Jun 29, 2017 at 02:16:17PM -0700, Dan Williams wrote:
> On Thu, Jun 29, 2017 at 9:56 AM, Jerry Hoemann <jerry.hoemann@hpe.com> wrote:
> > Have module parameter override_dsm_mask override the dsm_mask for
> > root calls like it does for non-root dsm calls.
> >
> > Signed-off-by: Jerry Hoemann <jerry.hoemann@hpe.com>
> > ---
> >  drivers/acpi/nfit/core.c | 7 ++++++-
> >  1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/acpi/nfit/core.c b/drivers/acpi/nfit/core.c
> > index 7d2f1a0..87acaf2 100644
> > --- a/drivers/acpi/nfit/core.c
> > +++ b/drivers/acpi/nfit/core.c
> > @@ -1627,6 +1627,7 @@ static void acpi_nfit_init_dsms(struct acpi_nfit_desc *acpi_desc)
> >         struct nvdimm_bus_descriptor *nd_desc = &acpi_desc->nd_desc;
> >         const u8 *uuid = to_nfit_uuid(NFIT_DEV_BUS);
> >         struct acpi_device *adev;
> > +       unsigned long dsm_mask;
> >         int i;
> >
> >         nd_desc->cmd_mask = acpi_desc->bus_cmd_force_en;
> > @@ -1638,7 +1639,11 @@ static void acpi_nfit_init_dsms(struct acpi_nfit_desc *acpi_desc)
> >                 if (acpi_check_dsm(adev->handle, uuid, 1, 1ULL << i))
> >                         set_bit(i, &nd_desc->cmd_mask);
> >         set_bit(ND_CMD_CALL, &nd_desc->cmd_mask);
> > -       for (i = 0; i < ND_CMD_CALL; i++)
> > +
> > +       dsm_mask = 0x3bf;
> > +       if (override_dsm_mask)
> > +               dsm_mask = override_dsm_mask;
> > +       for_each_set_bit(i, &dsm_mask, BITS_PER_LONG)
> >                 if (acpi_check_dsm(adev->handle, uuid, 1, 1ULL << i))
> >                         set_bit(i, &nd_desc->bus_dsm_mask);
> >  }
> 
> I don't think we need this patch. 'override_dsm_mask' is there to make
> it easier for vendor-specific DSM testing and debug for DIMM-level
> DSMs. The root bus is not vendor specific and the command set is not
> evolving at the same rate we are seeing change at DIMM-level DSMs.

Override_dsm_mask is there to allow using old kernels with new
firmware/hardware.

It takes a long time to get even simple changes upstreamed, backported
to distros, released, distributed to customers, installed.

In testing, for months we have had to work around the inability to call
these functions from linux.  A waste of effort.

-- 

-----------------------------------------------------------------------------
Jerry Hoemann                  Software Engineer   Hewlett Packard Enterprise
-----------------------------------------------------------------------------

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 0/7] Enable DSM pass thru for root functions Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 19:00 +0200
  [PATCH v3 7/7] acpi, nfit: override mask Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 19:00 +0200
    Re: [PATCH v3 7/7] acpi, nfit: override mask Dan Williams <dan.j.williams@intel.com> - 2017-06-29 23:20 +0200
      Re: [PATCH v3 7/7] acpi, nfit: override mask Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-30 00:20 +0200
        Re: [PATCH v3 7/7] acpi, nfit: override mask Dan Williams <dan.j.williams@intel.com> - 2017-06-30 01:00 +0200
  [PATCH v3 4/7] acpi, nfit: Use bus_dsm_mask for passthru Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 19:00 +0200
    Re: [PATCH v3 4/7] acpi, nfit: Use bus_dsm_mask for passthru Dan Williams <dan.j.williams@intel.com> - 2017-06-29 23:40 +0200
      Re: [PATCH v3 4/7] acpi, nfit: Use bus_dsm_mask for passthru Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 23:50 +0200
        Re: [PATCH v3 4/7] acpi, nfit: Use bus_dsm_mask for passthru Dan Williams <dan.j.williams@intel.com> - 2017-06-30 00:00 +0200
          Re: [PATCH v3 4/7] acpi, nfit: Use bus_dsm_mask for passthru Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-30 01:30 +0200
            Re: [PATCH v3 4/7] acpi, nfit: Use bus_dsm_mask for passthru Dan Williams <dan.j.williams@intel.com> - 2017-06-30 03:30 +0200
  [PATCH v3 6/7] libnvdimm: New ACPI 6.2 DSM functions Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 19:00 +0200
  [PATCH v3 3/7] libnvdimm: Add bus level dsm mask. Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 19:00 +0200
    Re: [PATCH v3 3/7] libnvdimm: Add bus level dsm mask. Dan Williams <dan.j.williams@intel.com> - 2017-06-29 23:30 +0200
  [PATCH v3 1/7] libnvdimm: passthru functions clear to send Jerry Hoemann <jerry.hoemann@hpe.com> - 2017-06-29 19:00 +0200

csiph-web