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

Groups > linux.kernel > #1299041

Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is valid

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Tuesday 29 December 2015 13:44:13 Michał Kępień wrote:
> > > > According to Dell WMI document mentioned in ML dicussion
> > > > archived at
> > > > http://www.spinics.net/lists/platform-driver-x86/msg07220.html
> > > > OS should check Dell WMI descriptor structure.
> > > 
> > > "Should" or "can"?  I skimmed through the ACPI-WMI PDF and
> > > Mario's message again and I couldn't find any explicit statement
> > > urging the reader to check the structure in question before
> > > doing anything else.
> > 
> > That's questionable... In "Design flow" is first point that WMI
> > descriptor check.
> 
> Which "Design flow" are you referring to?  Because I found at least
> two: chapter 2.3 and a subsection of chapter 2.3.3.  Funnily enough,
> in both of these locations the WMI Descriptor Method is discussed
> first.
> 
> Personally, I wouldn't use the structure of that document to draw
> cause-effect conclusions.  Just look at the last chapter (2.3.4),
> which shows how to tell whether the BIOS supports the ACPI-WMI
> interface. Shouldn't that be the first thing to check, before doing
> anything else mentioned in that document?  Yet, it's the last thing
> discussed.
> 
> Anyway, while the document mentions in several places that the BIOS
> WMI Descriptor object can be queried, it fails to convince me as to
> why this is necessary at all as all values in the returned buffer
> are constant. Perhaps parsing the buffer is useful as a sanity check
> of some kind, but it certainly isn't a prerequisite for performing
> further actions.
> 
> Given the nature of your patchset, I'd personally rephrase the commit
> message(s) to state that according to your observations, there are
> behavioral differences between models with different versions of the
> WMI Interface, so we parse the WMI Descriptor object to determine
> which WMI Interface version is used on the machine we're running on.
>  Perhaps with an additional word or two that it won't hurt to also
> check the WMI Descriptor object's correctness while we're at it.
> 
> If you feel like I'm nit-picking and none of the above matters,
> please feel free to disregard my input and just follow your gut.

It's ok. We just understand it quite differently. And in this case what 
about changing commit message to something like this?

===
dell-wmi: Check if Dell WMI descriptor structure is valid

After examining existing DSDT ACPI tables of more laptops and looking 
into Dell WMI document mentioned in ML dicussion archived at 
http://www.spinics.net/lists/platform-driver-x86/msg07220.html we will 
parse and check WMI descriptor if contains expected data. It is because 
WMI descriptor contains interface version number and it is needed to 
know in next commit.
===

-- 
Pali Rohár
pali.rohar@gmail.com

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

Thread

[PATCH 0/2] Fixes for dell-wmi Pali Rohár <pali.rohar@gmail.com> - 2015-12-24 22:20 +0100
  [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is valid Pali Rohár <pali.rohar@gmail.com> - 2015-12-24 22:30 +0100
    Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is valid Andy Lutomirski <luto@amacapital.net> - 2015-12-25 02:30 +0100
      Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is valid Pali Rohár <pali.rohar@gmail.com> - 2015-12-25 14:10 +0100
    Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is  valid Michał Kępień <kernel@kempniu.pl> - 2015-12-28 14:40 +0100
      Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is valid Pali Rohár <pali.rohar@gmail.com> - 2015-12-28 15:10 +0100
        Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is  valid Michał Kępień <kernel@kempniu.pl> - 2015-12-29 13:50 +0100
          Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is valid Pali Rohár <pali.rohar@gmail.com> - 2015-12-29 17:10 +0100
            Re: [PATCH 1/2] dell-wmi: Check if Dell WMI descriptor structure is  valid Michał Kępień <kernel@kempniu.pl> - 2015-12-30 12:30 +0100
  [PATCH 2/2] dell-wmi: Process only one event on devices with interface version 0 Pali Rohár <pali.rohar@gmail.com> - 2015-12-24 22:30 +0100
    Re: [PATCH 2/2] dell-wmi: Process only one event on devices with  interface version 0 Michał Kępień <kernel@kempniu.pl> - 2015-12-28 14:50 +0100
      Re: [PATCH 2/2] dell-wmi: Process only one event on devices with interface version 0 Pali Rohár <pali.rohar@gmail.com> - 2015-12-28 14:50 +0100
  Re: [PATCH 0/2] Fixes for dell-wmi Pali Rohár <pali.rohar@gmail.com> - 2015-12-27 14:10 +0100
    Re: [PATCH 0/2] Fixes for dell-wmi Pali Rohár <pali.rohar@gmail.com> - 2015-12-27 14:20 +0100
    Re: [PATCH 0/2] Fixes for dell-wmi Gabriele Mazzotta <gabriele.mzt@gmail.com> - 2015-12-27 14:20 +0100
  Re: [PATCH 0/2] Fixes for dell-wmi Gabriele Mazzotta <gabriele.mzt@gmail.com> - 2015-12-27 14:10 +0100
  Re: [PATCH 0/2] Fixes for dell-wmi Michał Kępień <kernel@kempniu.pl> - 2015-12-28 14:40 +0100
    Re: [PATCH 0/2] Fixes for dell-wmi Pali Rohár <pali.rohar@gmail.com> - 2015-12-28 14:50 +0100
      Re: [PATCH 0/2] Fixes for dell-wmi Michał Kępień <kernel@kempniu.pl> - 2015-12-29 13:20 +0100

csiph-web