[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD


  • To: Sébastien Chaumat <euidzero@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • From: Mario Limonciello <mario.limonciello@xxxxxxx>
  • Date: Wed, 6 Mar 2024 11:33:18 -0600
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=6SCUk1blSDm4CBP4y0a2QP95WF0lNlN6ALMTMkwD85c=; b=EwkaDBXQq2bdOW4+MfidivGB0xnOgEtlvfcuPKaJpZOGZuCJU6BPwKjGzgTNW00rh09ezxPyQ8gf+VWFb2lOe/nRZH7wI4YSLQvaZzOf/2EoSDszaGHe9GOdjwn1mYF0tlWpwRR78r5p0nKhcxkAnVk+95nwMfxNFJVmwnRQYo5Zvj/fzfAo3pvD9UwPFOR8UmJcQaZPNafJaeyBghWEUUTXx+7Hlvyt6wzpS1K6J0Ah7V1UHljJ5vWCWHgbUhhSKETKjsMArU0bblSrYtCKuyshrfkmvLrrDPXDe33nmb5tgAaXoycutBVygfoA/3HXIay13xjvki9GjaQTMXpiIA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Dw9V9IpULwYHSjD7nn46QrROZ2tl9Mm6dz2/PJKPrg2Jg04WcJdFOyv/sp9t9icrlltdYdlO3n2mWc2pP/di3xm6oocSJsMhGOmmVbpNBCg5tra2qgJIOHFg4t3Zr3Y6WiN/cqvFnUUlk0rcjN5u5HAs7cpwL+OY/p4NdTb70MijZX0lICZ4QDLS3FvQxpWOI0PQn1usrMckqfoMd1gxqN+lOmGRZhV145DU2ykMWJVu9v9Ajo3Yd8m6UawGoSeKeGFBm35jvqGDutfqpoDU2kno9ARG80uvD2Ycm95DQJ9e1sfSDWkJfSq5lEmlJyGEaDntUflx3es+LKfErCEouQ==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com;
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Wed, 06 Mar 2024 17:33:33 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 3/6/2024 11:28, Sébastien Chaumat wrote:




    Reasoning backward  (using a  kernel without the pinctrl_amd driver
    to ensure xen only is at stake) :
      checking the diff in IOAPIC  between bare metal and xen  (IRQ7 is
    on pin07 on APIC )

    using kernel argument : apic=debug

    bare metal :
    [    0.715330] fedora kernel: ... APIC VERSION: 81050010
    ...
    [    0.715433] fedora kernel:  pin07, disabled, edge , high, V(00),
    IRR(0), S(0), physical, D(0000), M(0)

    xen :
    [    2.249582] fedora kernel: ... APIC VERSION: 00000014
    ...
    [    2.249730] fedora kernel:  pin07, disabled, level, low , V(60),
    IRR(0), S(0), physical, D(0000), M(0)

    So the APIC table is not the same.

    As strange as it looks the  (IOAPIC 0) pin07 is correctly described
    by the APIC in xen but yet differently than in baremetal.
    But the APIC message comes long after the
    [    1.833145] fedora kernel: xen: registering gsi 7 triggering 0
    polarity 1

    so I wonder if the APIC pin07 info had any influence.

Finally found the fix : adding ioapic_ack=new to xen boot parameters.
Not only the trackpad is now working but also the ACPI Embedded Controller which is completely disabled.

Sébastien

That's great news! I'm personally totally unfamiliar with ioapic_ack=new, so I did a quick search and found out it's a Xen parameter (I came across
https://xenbits.xen.org/docs/4.5-testing/misc/xen-command-line.html).

This mentions that "new" should be the default, so why isn't it the case?

Also; I'd be really interested to hear what happens with s2idle with Xen now (if it works).



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.