Message-ID: <HE1PR0602MB335553397CB175052E00EF70DAA40@HE1PR0602MB3355.eurprd06.prod.outlook.com>
Date:   Wed, 6 May 2020 15:16:40 +0000
From:   Fabio Comolli <fabio.comolli@...l.polimi.it>
To:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:     "luzmaximilian@...il.com" <luzmaximilian@...il.com>,
        "robert.moore@...el.com" <robert.moore@...el.com>,
        "rik.kaneda@...el.com" <rik.kaneda@...el.com>,
        "rafael.j.wysocki@...el.com" <rafael.j.wysocki@...el.com>
Subject: [Regression] [Bisected] Commit
 6d232b29cfce65961db4a668c2c6c6987cd24d45 breaks some of the Fn-keys on my old
 Sony Vaio VPCM13M1E.

Hi,

I updated my old Sony Vaio a few days ago and discovered that a few Fn-keys on my old Sony Vaio had stopped working. 
The bisection points at the commit in the subject and reverting it makes everything work again even with current
Linus' git.

Full bug report below. Please CC: me as I'm not subscribed to lkml.

Regards,
Fabio


-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-

BIOS table is:

[    0.000000] DMI: Sony Corporation VPCM13M1E/VAIO, BIOS R0120Z4 05/12/2010
[    0.014701] ACPI: RSDP 0x00000000000F0410 000024 (v03 Sony  )
[    0.014709] ACPI: XSDT 0x000000007F4DB078 00004C (v01 Sony   VAIO     20100512 AMI  00010013)
[    0.014724] ACPI: FACP 0x000000007F4E1388 0000F4 (v04 Sony   VAIO     20100512 AMI  00010013)
[    0.014740] ACPI: DSDT 0x000000007F4DB150 006231 (v02 Sony   VAIO     20100512 INTL 20051117)
[    0.014751] ACPI: FACS 0x000000007F509F40 000040
[    0.014758] ACPI: FACS 0x000000007F509F80 000040
[    0.014766] ACPI: APIC 0x000000007F4E1480 000072 (v01 Sony   VAIO     20100512 AMI  00010013)
[    0.014777] ACPI: MCFG 0x000000007F4E14F8 00003C (v01 Sony   VAIO     20100512 MSFT 00000097)
[    0.014787] ACPI: SLIC 0x000000007F4E1538 000176 (v01 Sony   VAIO     20100512 Sony 01000000)
[    0.014797] ACPI: HPET 0x000000007F4E16B0 000038 (v01 Sony   VAIO     20100512 AMI. 00000003)

There are 7 Fn-keys available on this model:

Mute
Volume Down
Volume Up
Brightness Down
Brightness Up
External Video Output Switch
Hibernate

-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-

Working versions write this in the messages log at boot time:

[   20.175244] input: Sony Vaio Keys as /devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:01/SNY5001:00/input/input5
[   20.186497] input: Sony Vaio Jogdial as /devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:01/SNY5001:00/input/input6
[   20.217724] sony_laptop: SNC setup done.

The acpi_listen output is:

button/mute MUTE 00000080 00000000 K
button/volumedown VOLDN 00000080 00000000 K
button/volumeup VOLUP 00000080 00000000 K
video/brightnessdown BRTDN 00000087 00000000 K
sony/hotkey SNY5001:00 00000001 00000010
sony/hotkey SNY5001:00 00000001 0000003b
video/brightnessup BRTUP 00000086 00000000 K
sony/hotkey SNY5001:00 00000001 00000011
sony/hotkey SNY5001:00 00000001 0000003b
video/switchmode VMOD 00000080 00000000 K
sony/hotkey SNY5001:00 00000001 00000012
sony/hotkey SNY5001:00 00000001 0000003b

-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-

Broken versions write this in the messages log at boot time:

[   19.143613] input: Sony Vaio Keys as /devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:01/SNY5001:00/input/input6
[   19.158941] input: Sony Vaio Jogdial as /devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:01/SNY5001:00/input/input7
[   19.180118] sony_laptop: Invalid acpi_object: expected 0x1 got 0x3
[   19.184342] sony_laptop: Invalid acpi_object: expected 0x1 got 0x3
[   19.186183] sony_laptop: Invalid acpi_object: expected 0x1 got 0x3
[   19.188688] sony_laptop: SNC setup done.

The acpi_listen output is:

button/mute MUTE 00000080 00000000 K
button/volumedown VOLDN 00000080 00000000 K
button/volumeup VOLUP 00000080 00000000 K
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092
sony/hotkey SNY5001:00 00000001 00000092

i.e. the brightness keys, the external output toggle and the hibernate button are not recognised anymore. Every keypress generates two identical "sony/hotkey SNY5001:00 00000001 00000092" in the acpi_listen output and the following lines in the messages log:

May  4 20:09:34 vaio kernel: sony_laptop: Invalid acpi_object: expected 0x1 got 0x3
May  4 20:09:34 vaio fcomolli[17675]: ACPI group/action undefined: sony/hotkey / SNY5001:00.

The volume and mute button always work.

-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-

Bisection output:

[fcomolli@...o linux]$ git bisect good
6d232b29cfce65961db4a668c2c6c6987cd24d45 is the first bad commit
commit 6d232b29cfce65961db4a668c2c6c6987cd24d45
Author: Maximilian Luz <luzmaximilian@...il.com>
Date:   Tue Dec 17 11:35:22 2019 -0800

    ACPICA: Dispatcher: always generate buffer objects for ASL create_field() operator
    
    ACPICA commit 79a466b64e6af36cc83102f05915e56cb7dd89ab
    
    According to table 19-419 of the ACPI 6.3 specification, buffer_fields
    created using the ASL create_field() Operator have been treated as
    integers if the buffer_field is small enough to fit inside of an ASL
    integer (32-bits or 64-bits depending on the definition block
    revision). If they are larger, buffer fields are treated as ASL
    Buffer objects. However, this is not true for other AML interpreter
    implementations.
    
    It has been discovered that other AML interpreters always treat
    buffer fields created by create_field() as a buffer regardless of the
    length of the buffer field.
    
    More specifically, the Microsoft AML interpreter always treats buffer
    fields created by the create_field() operator as buffer. ACPICA
    currently does this only when the field size is larger than the
    maximum integer width. This causes problems with AML code shipped in
    Microsoft Surface devices.
    
    More details:
    
    The control methods in these devices determine the success of an ASL
    control method execution by examining the type resulting from storing
    a buffer field created by a create_field() operator. On success, a
    Buffer object is expected, on failure an Integer containing an error
    code. This buffer object is created with a dynamic size via the
    create_field() operator. Due to the difference in behavior, Buffer
    values of small size are however converted to Integers and thus
    interpreted by the control method as having failed, whereas in
    reality it succeeded. Below is an example of a control method called
    TEST that illustrates this behavior.
    
    Method (CBUF) // Create a Buffer field
    {
        /*
         * Depending on the value of RAND, ACPICA interpreter will treat
         * BF00 as an integer or buffer.
         */
        create_field (BUFF, 0, RAND, BF00)
        return (BF00)
    }
    
    Method (TEST)
    {
        /*
         * Storing the value returned by CBUF to local0 will result in
         * implicit type conversion outlined in the ACPI specification.
         *
         * ACPICA will treat local0 like an ASL integer if RAND is less
         * than or equal to 64 or 32 (depending on the definition_block
         * revision). If RAND is greater, it will be treated like an ASL
         * buffer. Other implementations treat local0 like an ASL buffer
         * regardless of the value of RAND.
         */
        local0 = CBUF()
    
        /*
         * object_type of 0x03 represents an ASL Buffer
         */
        if (object_type (Local0) != 0x03)
        {
            // Error on ACPICA if RAND is small enough
        }
        else
        {
            /*
             * Success on APICA if RAND is large enough
             * Other implementations always take this path because local0
             * is always treated as a buffer.
             */
        }
    }
    
    This change prohibits the previously mentioned integer conversion to
    match other AML interpreter implementations (Microsoft) that do not
    conform to the ACPI specification.
    
    Link: https://github.com/acpica/acpica/commit/79a466b6
    Signed-off-by: Maximilian Luz <luzmaximilian@...il.com>
    Signed-off-by: Bob Moore <robert.moore@...el.com>
    Signed-off-by: Erik Kaneda <erik.kaneda@...el.com>
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>

 drivers/acpi/acpica/acobject.h |  3 ++-
 drivers/acpi/acpica/dsopcode.c |  2 ++
 drivers/acpi/acpica/exfield.c  | 10 ++++++++--
 3 files changed, 12 insertions(+), 3 deletions(-)

-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-

[fcomolli@...o linux]$ git bisect log
git bisect start
# bad: [7111951b8d4973bda27ff663f2cf18b663d15b48] Linux 5.6
git bisect bad 7111951b8d4973bda27ff663f2cf18b663d15b48
# good: [84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d] Linux 4.19
git bisect good 84df9525b0c27f3ebc2ebb1864fa62a97fdedb7d
# good: [ee7dd7733b201c6201174f384a92dbadc9f3129c] sis900: remove TxIDLE
git bisect good ee7dd7733b201c6201174f384a92dbadc9f3129c
# good: [44bf67f32a6803339ac1ba721b158c3e2272cabe] Merge drm/drm-next into drm-misc-next
git bisect good 44bf67f32a6803339ac1ba721b158c3e2272cabe
# good: [7214618c60e947b8cea12b47d8279bd4220f21bc] Merge tag 'riscv/for-v5.5-rc3' of git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux
git bisect good 7214618c60e947b8cea12b47d8279bd4220f21bc
# bad: [4cadc60d6bcfee9c626d4b55e9dc1475d21ad3bb] Merge tag 'for-v5.6' of git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply
git bisect bad 4cadc60d6bcfee9c626d4b55e9dc1475d21ad3bb
# bad: [a78208e2436963d0b2c7d186277d6e1a9755029a] Merge branch 'linus' of git://git.kernel.org/pub/scm/linux/kernel/git/herbert/crypto-2.6
git bisect bad a78208e2436963d0b2c7d186277d6e1a9755029a
# good: [9e1af7567b266dc6c3c8fd434ea807b3206bfdc1] Merge tag 'mmc-v5.6' of git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc
git bisect good 9e1af7567b266dc6c3c8fd434ea807b3206bfdc1
# bad: [d99391ec2b42d827d92003dcdcb96fadac9d862b] Merge branch 'core-rcu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
git bisect bad d99391ec2b42d827d92003dcdcb96fadac9d862b
# good: [6d277aca488fdf0a1e67cd14b5a58869f66197c9] Merge tag 'pm-5.6-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
git bisect good 6d277aca488fdf0a1e67cd14b5a58869f66197c9
# bad: [534b0a8b677443c0aa8c4c71ff7887f08a2b9b41] Merge tag 'core-debugobjects-2020-01-28' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
git bisect bad 534b0a8b677443c0aa8c4c71ff7887f08a2b9b41
# bad: [22a8f39c520fc577c02b4e5c99f8bb3b6017680b] Merge tag 'for-5.6/drivers-2020-01-27' of git://git.kernel.dk/linux-block
git bisect bad 22a8f39c520fc577c02b4e5c99f8bb3b6017680b
# bad: [34dabd81160f7bfb18b67c1161b3c4d7ca6cab83] Merge tag 'pnp-5.6-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
git bisect bad 34dabd81160f7bfb18b67c1161b3c4d7ca6cab83
# bad: [3dd855147feff375dfa6737331628ea919e9da59] Merge branches 'acpi-battery', 'acpi-video', 'acpi-fan' and 'acpi-drivers'
git bisect bad 3dd855147feff375dfa6737331628ea919e9da59
# bad: [ff7a672f83b355365478a1fdfb60933ef34d8d02] Merge branch 'acpica'
git bisect bad ff7a672f83b355365478a1fdfb60933ef34d8d02
# bad: [ae6252d8dfeb21f5067a09a8f4a6dd30851d70c1] ACPICA: Update version to 20191213
git bisect bad ae6252d8dfeb21f5067a09a8f4a6dd30851d70c1
# good: [5ddbd77181dfca61b16d2e2222382ea65637f1b9] ACPICA: Disassembler: create buffer fields in ACPI_PARSE_LOAD_PASS1
git bisect good 5ddbd77181dfca61b16d2e2222382ea65637f1b9
# bad: [6d232b29cfce65961db4a668c2c6c6987cd24d45] ACPICA: Dispatcher: always generate buffer objects for ASL create_field() operator
git bisect bad 6d232b29cfce65961db4a668c2c6c6987cd24d45
# good: [69e86e59ad2a2518704a31c35530e6e99963c358] ACPICA: acpisrc: add unix line ending support for non-windows build
git bisect good 69e86e59ad2a2518704a31c35530e6e99963c358
# first bad commit: [6d232b29cfce65961db4a668c2c6c6987cd24d45] ACPICA: Dispatcher: always generate buffer objects for ASL create_field() operator

Powered by blists - more mailing lists