From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 70523C52D7C for ; Wed, 21 Aug 2024 05:09:29 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id 1380760812; Wed, 21 Aug 2024 05:09:29 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id qTGg-u19TPI5; Wed, 21 Aug 2024 05:09:28 +0000 (UTC) X-Comment: SPF check N/A for local connections - client-ip=140.211.166.34; helo=ash.osuosl.org; envelope-from=intel-wired-lan-bounces@osuosl.org; receiver= DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org 1765060ACE DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=osuosl.org; s=default; t=1724216968; bh=Kdvr3vsjNyF52JtCUkc2L1/v/zvdzK9Ic64vH1PZOq8=; h=Date:To:References:From:In-Reply-To:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Cc:From; b=bEiOTwN5SmKUuCaKGAe5niD9vpOIsg6oV7dHAb4XKyH0wTuleWe+E0scWQRCJVdhj FHYpbZ1Peca5szLjNUf9T4H0/IcpOHkIphDuWF8RHaRS1B0P4dQTvD9/S8VB1jVnv1 CYXbXiQcnWjYNLqYx8qKCKZt1WhS/Y4RAYKaBs1S5X+Fd6HjW4wxwddKvCN+iyO5fd oZPwmIROLJ/XobA/YsSsrf8g5+BYRVCDzPLhyV8J2EMYAiD43sqIVDTvGYpRVG7bhl VCL5JHK2AbvJHUKBZjn908aY2Y5zck5+knB0032ckLzyJTB7mYBu9yx4ybc/KwvO1D u0gmDBDEeiEAg== Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp3.osuosl.org (Postfix) with ESMTP id 1765060ACE; Wed, 21 Aug 2024 05:09:28 +0000 (UTC) Received: from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138]) by ash.osuosl.org (Postfix) with ESMTP id 9D66B1BF389; Wed, 21 Aug 2024 05:09:26 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 89531812EC; Wed, 21 Aug 2024 05:09:26 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id kXv25WumMbXZ; Wed, 21 Aug 2024 05:09:25 +0000 (UTC) Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=209.85.221.42; helo=mail-wr1-f42.google.com; envelope-from=jirislaby@gmail.com; receiver= DMARC-Filter: OpenDMARC Filter v1.4.2 smtp1.osuosl.org 7EFF8812ED DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org 7EFF8812ED Received: from mail-wr1-f42.google.com (mail-wr1-f42.google.com [209.85.221.42]) by smtp1.osuosl.org (Postfix) with ESMTPS id 7EFF8812ED for ; Wed, 21 Aug 2024 05:09:24 +0000 (UTC) Received: by mail-wr1-f42.google.com with SMTP id ffacd0b85a97d-371b97cfd6fso2127582f8f.2 for ; Tue, 20 Aug 2024 22:09:24 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1724216962; x=1724821762; h=content-transfer-encoding:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Kdvr3vsjNyF52JtCUkc2L1/v/zvdzK9Ic64vH1PZOq8=; b=oOp1PAznWqjWoEhIbwIgiW/n/z7otMhmAgbTP2MoNcexfcMD5bHp2K8Llzn/TWVcqK SRTLB4nFgqd6vtp/vRoftaCM4BKSj5mX/UyteYgj1B7jEiT2exZYXVYQ2vn/UB7MjS7/ HMHkfZhKPzZmDflOB97c3G0/099bjRMFCAuZyVtpPJ647WCMuXxik5O0snLJ2lOQvaZs C4eFKECfKyYNMd8BigZjucdtoLZiGgIv0GuplfecYZtPvDS1O0lfOcqdtzrEHus4oMWv 5FS8iDAemmbvnxePKxpFAiYwDqUC7qi0ch4cVZDrstW6bElK6uLB/669jSkhGd+1h6Q8 7hgQ== X-Forwarded-Encrypted: i=1; AJvYcCUpmCgHPMV4noVCgvaRjD3KEW9gVRtRFk86HHQ7DlvaWFAAo/v+/wFjrWCW2kYMmbzdXCMY4c/r9ja/lwasKGo=@lists.osuosl.org X-Gm-Message-State: AOJu0Ywoo1aQCDDfx2QgaRfs/j9LpzF9WVE6Tdz534/etFXeAEwYkk34 7PHBsCIShMpcMwNBdOYpyFhNj82ekNHNa2CMsmNslobtzUJaLDsO X-Google-Smtp-Source: AGHT+IFC+H4F04UVve3eU5bbnNUszYbRb6wUtv+vOGJqJygUV5UOMblQx6KJUIV8GGsZjVSSaKInLQ== X-Received: by 2002:a5d:6483:0:b0:371:8eaf:3c49 with SMTP id ffacd0b85a97d-372fd82bf2bmr698882f8f.40.1724216962083; Tue, 20 Aug 2024 22:09:22 -0700 (PDT) Received: from ?IPV6:2a0b:e7c0:0:107::aaaa:69? ([2a0b:e7c0:0:107::aaaa:69]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-37195c57d48sm12533357f8f.53.2024.08.20.22.09.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 20 Aug 2024 22:09:21 -0700 (PDT) Message-ID: Date: Wed, 21 Aug 2024 07:09:20 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: Bjorn Helgaas , Petr Valenta References: <20240820213035.GA226181@bhelgaas> Content-Language: en-US From: Jiri Slaby Autocrypt: addr=jirislaby@kernel.org; keydata= xsFNBE6S54YBEACzzjLwDUbU5elY4GTg/NdotjA0jyyJtYI86wdKraekbNE0bC4zV+ryvH4j rrcDwGs6tFVrAHvdHeIdI07s1iIx5R/ndcHwt4fvI8CL5PzPmn5J+h0WERR5rFprRh6axhOk rSD5CwQl19fm4AJCS6A9GJtOoiLpWn2/IbogPc71jQVrupZYYx51rAaHZ0D2KYK/uhfc6neJ i0WqPlbtIlIrpvWxckucNu6ZwXjFY0f3qIRg3Vqh5QxPkojGsq9tXVFVLEkSVz6FoqCHrUTx wr+aw6qqQVgvT/McQtsI0S66uIkQjzPUrgAEtWUv76rM4ekqL9stHyvTGw0Fjsualwb0Gwdx ReTZzMgheAyoy/umIOKrSEpWouVoBt5FFSZUyjuDdlPPYyPav+hpI6ggmCTld3u2hyiHji2H cDpcLM2LMhlHBipu80s9anNeZhCANDhbC5E+NZmuwgzHBcan8WC7xsPXPaiZSIm7TKaVoOcL 9tE5aN3jQmIlrT7ZUX52Ff/hSdx/JKDP3YMNtt4B0cH6ejIjtqTd+Ge8sSttsnNM0CQUkXps w98jwz+Lxw/bKMr3NSnnFpUZaxwji3BC9vYyxKMAwNelBCHEgS/OAa3EJoTfuYOK6wT6nadm YqYjwYbZE5V/SwzMbpWu7Jwlvuwyfo5mh7w5iMfnZE+vHFwp/wARAQABzSFKaXJpIFNsYWJ5 IDxqaXJpc2xhYnlAa2VybmVsLm9yZz7CwXcEEwEIACEFAlW3RUwCGwMFCwkIBwIGFQgJCgsC BBYCAwECHgECF4AACgkQvSWxBAa0cEnVTg//TQpdIAr8Tn0VAeUjdVIH9XCFw+cPSU+zMSCH eCZoA/N6gitEcnvHoFVVM7b3hK2HgoFUNbmYC0RdcSc80pOF5gCnACSP9XWHGWzeKCARRcQR 4s5YD8I4VV5hqXcKo2DFAtIOVbHDW+0okOzcecdasCakUTr7s2fXz97uuoc2gIBB7bmHUGAH XQXHvdnCLjDjR+eJN+zrtbqZKYSfj89s/ZHn5Slug6w8qOPT1sVNGG+eWPlc5s7XYhT9z66E l5C0rG35JE4PhC+tl7BaE5IwjJlBMHf/cMJxNHAYoQ1hWQCKOfMDQ6bsEr++kGUCbHkrEFwD UVA72iLnnnlZCMevwE4hc0zVhseWhPc/KMYObU1sDGqaCesRLkE3tiE7X2cikmj/qH0CoMWe gjnwnQ2qVJcaPSzJ4QITvchEQ+tbuVAyvn9H+9MkdT7b7b2OaqYsUP8rn/2k1Td5zknUz7iF oJ0Z9wPTl6tDfF8phaMIPISYrhceVOIoL+rWfaikhBulZTIT5ihieY9nQOw6vhOfWkYvv0Dl o4GRnb2ybPQpfEs7WtetOsUgiUbfljTgILFw3CsPW8JESOGQc0Pv8ieznIighqPPFz9g+zSu Ss/rpcsqag5n9rQp/H3WW5zKUpeYcKGaPDp/vSUovMcjp8USIhzBBrmI7UWAtuedG9prjqfO wU0ETpLnhgEQAM+cDWLL+Wvc9cLhA2OXZ/gMmu7NbYKjfth1UyOuBd5emIO+d4RfFM02XFTI t4MxwhAryhsKQQcA4iQNldkbyeviYrPKWjLTjRXT5cD2lpWzr+Jx7mX7InV5JOz1Qq+P+nJW YIBjUKhI03ux89p58CYil24Zpyn2F5cX7U+inY8lJIBwLPBnc9Z0An/DVnUOD+0wIcYVnZAK DiIXODkGqTg3fhZwbbi+KAhtHPFM2fGw2VTUf62IHzV+eBSnamzPOBc1XsJYKRo3FHNeLuS8 f4wUe7bWb9O66PPFK/RkeqNX6akkFBf9VfrZ1rTEKAyJ2uqf1EI1olYnENk4+00IBa+BavGQ 8UW9dGW3nbPrfuOV5UUvbnsSQwj67pSdrBQqilr5N/5H9z7VCDQ0dhuJNtvDSlTf2iUFBqgk 3smln31PUYiVPrMP0V4ja0i9qtO/TB01rTfTyXTRtqz53qO5dGsYiliJO5aUmh8swVpotgK4 /57h3zGsaXO9PGgnnAdqeKVITaFTLY1ISg+Ptb4KoliiOjrBMmQUSJVtkUXMrCMCeuPDGHo7 39Xc75lcHlGuM3yEB//htKjyprbLeLf1y4xPyTeeF5zg/0ztRZNKZicgEmxyUNBHHnBKHQxz 1j+mzH0HjZZtXjGu2KLJ18G07q0fpz2ZPk2D53Ww39VNI/J9ABEBAAHCwV8EGAECAAkFAk6S 54YCGwwACgkQvSWxBAa0cEk3tRAAgO+DFpbyIa4RlnfpcW17AfnpZi9VR5+zr496n2jH/1ld wRO/S+QNSA8qdABqMb9WI4BNaoANgcg0AS429Mq0taaWKkAjkkGAT7mD1Q5PiLr06Y/+Kzdr 90eUVneqM2TUQQbK+Kh7JwmGVrRGNqQrDk+gRNvKnGwFNeTkTKtJ0P8jYd7P1gZb9Fwj9YLx jhn/sVIhNmEBLBoI7PL+9fbILqJPHgAwW35rpnq4f/EYTykbk1sa13Tav6btJ+4QOgbcezWI wZ5w/JVfEJW9JXp3BFAVzRQ5nVrrLDAJZ8Y5ioWcm99JtSIIxXxt9FJaGc1Bgsi5K/+dyTKL wLMJgiBzbVx8G+fCJJ9YtlNOPWhbKPlrQ8+AY52Aagi9WNhe6XfJdh5g6ptiOILm330mkR4g W6nEgZVyIyTq3ekOuruftWL99qpP5zi+eNrMmLRQx9iecDNgFr342R9bTDlb1TLuRb+/tJ98 f/bIWIr0cqQmqQ33FgRhrG1+Xml6UXyJ2jExmlO8JljuOGeXYh6ZkIEyzqzffzBLXZCujlYQ DFXpyMNVJ2ZwPmX2mWEoYuaBU0JN7wM+/zWgOf2zRwhEuD3A2cO2PxoiIfyUEfB9SSmffaK/ S4xXoB6wvGENZ85Hg37C7WDNdaAt6Xh2uQIly5grkgvWppkNy4ZHxE+jeNsU7tg= In-Reply-To: <20240820213035.GA226181@bhelgaas> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Mailman-Original-Authentication-Results: smtp1.osuosl.org; dmarc=fail (p=quarantine dis=none) header.from=kernel.org Subject: Re: [Intel-wired-lan] ACPI IRQ storm with 6.10 X-BeenThere: intel-wired-lan@osuosl.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Wired Ethernet Linux Kernel Driver Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Linux regressions mailing list , "Rafael J. Wysocki" , przemyslaw.kitszel@intel.com, Linux kernel mailing list , "linux-acpi@vger.kernel.org" , Tony Nguyen , Bjorn Helgaas , intel-wired-lan@lists.osuosl.org, Len Brown Errors-To: intel-wired-lan-bounces@osuosl.org Sender: "Intel-wired-lan" On 20. 08. 24, 23:30, Bjorn Helgaas wrote: > On Tue, Aug 20, 2024 at 11:13:54PM +0200, Petr Valenta wrote: >> Dne 20. 08. 24 v 20:09 Bjorn Helgaas napsal(a): >>> On Mon, Aug 19, 2024 at 07:23:42AM +0200, Jiri Slaby wrote: >>>> On 19. 08. 24, 6:50, Jiri Slaby wrote: >>>>> CC e1000e guys + Jesse (due to 75a3f93b5383) + Bjorn (due to b2c289415b2b) >>>> >>>> Bjorn, >>>> >>>> I am confused by these changes: >>>> ========================================== >>>> @@ -291,16 +288,13 @@ static int e1000_set_link_ksettings(struct net_device >>>> *net >>>> dev, >>>> * duplex is forced. >>>> */ >>>> if (cmd->base.eth_tp_mdix_ctrl) { >>>> - if (hw->phy.media_type != e1000_media_type_copper) { >>>> - ret_val = -EOPNOTSUPP; >>>> - goto out; >>>> - } >>>> + if (hw->phy.media_type != e1000_media_type_copper) >>>> + return -EOPNOTSUPP; >>>> >>>> if ((cmd->base.eth_tp_mdix_ctrl != ETH_TP_MDI_AUTO) && >>>> (cmd->base.autoneg != AUTONEG_ENABLE)) { >>>> e_err("forcing MDI/MDI-X state is not supported when >>>> lin >>>> k speed and/or duplex are forced\n"); >>>> - ret_val = -EINVAL; >>>> - goto out; >>>> + return -EINVAL; >>>> } >>>> } >>>> >>>> @@ -347,7 +341,6 @@ static int e1000_set_link_ksettings(struct net_device >>>> *netde >>>> v, >>>> } >>>> >>>> out: >>>> - pm_runtime_put_sync(netdev->dev.parent); >>>> clear_bit(__E1000_RESETTING, &adapter->state); >>>> return ret_val; >>>> } >>>> ========================================== >>>> >>>> So no more clear_bit(__E1000_RESETTING in the above fail paths. Is that >>>> intentional? >>> >>> Not intentional. Petr, do you have the ability to test the patch >>> below? I'm not sure it's the correct fix, but it reverts the pieces >>> of b2c289415b2b that Jiri pointed out. >> >> I tested the patch below but it didn't help. After the first boot with new >> kernel it looked promising as the irq storm only appeared for a few seconds, >> but with subsequent reboots it was the same as without the patch. > > Thank you very much for testing that! >> To be sure, I also send the md5sum of ethtool.c after applying the patch: >> >> a25c003257538f16994b4d7c4260e894 ethtool.c > > Thanks, that matches what I get when applying the patch on v6.10. > > I'm at a loss. You could try reverting the entire b2c289415b2b commit > (patch for that is below). FWIW he already tested with b2c289415b2b reverted (I provided him with a built kernel). It behaves the same. So you are not the breaker. > If that doesn't help, I guess you could try reverting the other > commits Jiri mentioned: > > 76a0a3f9cc2f e1000e: fix force smbus during suspend flow > c93a6f62cb1b e1000e: Fix S0ix residency on corporate systems > bfd546a552e1 e1000e: move force SMBUS near the end of enable_ulp function > 6918107e2540 net: e1000e & ixgbe: Remove PCI_HEADER_TYPE_MFD duplicates > 1eb2cded45b3 net: annotate writes on dev->mtu from ndo_change_mtu() > b2c289415b2b e1000e: Remove redundant runtime resume for ethtool_ops > 75a3f93b5383 net: intel: implement modern PM ops declarations > > If you do this, I would revert 76a0a3f9cc2f, test, then revert > c93a6f62cb1b in addition, test, then revert bfd546a552e1 in addition, > etc. Or perhaps easier to do: git bisect v6.10 v6.9 -- drivers/net/ethernet/intel/e1000e/ directly. But that assumes one of the above commits broke it. If they did not, as a last resort, you can still do full bisect (without the "-- drivers" part). I would take v6.10 suses config. Would boot 6.10. do lsmod > /tmp/lsmod make LSMOD=/tmp/lsmod localyesconfig make bzImage and use that bzImage. Note that graphics, wireless and other stuff will be defunct unless you build in firmwares for them (EXTRA_FIRMWARE config). Alternatively use localmodconfig and build and install also modules (now limited to your machine). thanks, -- js suse labs