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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DF99FC433EF for ; Wed, 10 Nov 2021 16:42:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C06A861205 for ; Wed, 10 Nov 2021 16:42:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229531AbhKJQpF (ORCPT ); Wed, 10 Nov 2021 11:45:05 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44344 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231526AbhKJQpE (ORCPT ); Wed, 10 Nov 2021 11:45:04 -0500 Received: from mail-oi1-x232.google.com (mail-oi1-x232.google.com [IPv6:2607:f8b0:4864:20::232]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E8BB8C061764 for ; Wed, 10 Nov 2021 08:42:16 -0800 (PST) Received: by mail-oi1-x232.google.com with SMTP id be32so6249383oib.11 for ; Wed, 10 Nov 2021 08:42:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=swiecki.net; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=H7hoGwWJnykyOW04HONpzk1mM5tsbdn8LVftPEXQKdQ=; b=BT8mtsiiqINuoDDAmo6vdWY1e8h1ko5qEt1PbcTIBNaZFn1DpT6ijehrEQkOX3MjDj qLkNL4BMHorAyNxhg9Fe/6g6Eia6nzuZmLzARCGsCRT7b6BDOstUOHROfey5a5ZpW0YE hra/OLlN7fxeD7ynN++DFNF4486LEcv3hVu70= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=H7hoGwWJnykyOW04HONpzk1mM5tsbdn8LVftPEXQKdQ=; b=rrKFdtwQ2K8eLjd2RgiNSnOTxzOBjZ2y1m3DFgEzmJahRqLZiq/wXhMz8zyYi60nlT j2cmU++gBBCIZ3h1ACtAy2sNDkY2Aea94nfPZp7NVfdd4u4/x9P21Uh3BRsUgL6EolbG osK3ij1n+etmbzHY9LSkFGlPcVarNmeAhqy6i40EgzW16r1QFrpyjlblQ6r9RQ5DxMxO N+pXSaJvtW2+J/i8H2auO35nddRT83oMQShUqcgmDg9VxzGVoZ0C8nEuEoznbqMbv5uZ USbsLc4gy8J1DdiYSVYzqL0xyfdZKAJ3vVXK4BWr+XNvPEh3cTL+VnRPw8D8WTaElMMH 51mw== X-Gm-Message-State: AOAM532CtUD4fWXjjPYbZ/qYdcAxeJHtd3ZJGZpUVL44GWrxRbJpbMEL DVz4MKNfUrKsUPKwZl1dqP5lJ17abgs5QXF8u6rsGQ== X-Google-Smtp-Source: ABdhPJxHHhtgU+Bhr9WOpU8WHUSBmqUhvLThAvCpG3Ve7RGaZhDbrfL8cmNUq4AZv6A5628GWedL6+IBaMnKIKm5fwA= X-Received: by 2002:aca:d989:: with SMTP id q131mr382945oig.167.1636562536292; Wed, 10 Nov 2021 08:42:16 -0800 (PST) MIME-Version: 1.0 References: <20211109165848.GA1155989@bhelgaas> In-Reply-To: From: =?UTF-8?B?Um9iZXJ0IMWad2nEmWNraQ==?= Date: Wed, 10 Nov 2021 17:42:05 +0100 Message-ID: Subject: Re: [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates To: =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= Cc: Arnd Bergmann , Bjorn Helgaas , Jens Axboe , Darren Stevens , mad skateman , "linux-pci@vger.kernel.org" , Damien Le Moal , Olof Johansson , "R.T.Dickinson" , Christian Zigotzky , "bhelgaas@google.com >> Bjorn Helgaas" , Matthew Leaman , linuxppc-dev , Christian Zigotzky Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org > [+CC Adding Robert for visibility] > > Hi Arnd, > > Thank you looking at this! Much appreciated. > > > > > You could attach the kernel config there, too, since it didn't make it > > > > to the mailing list (vger may discard them -- see > > > > http://vger.kernel.org/majordomo-info.html). > > > > > > Bjorn and I looked at which commits that went with a recent Pull Request > > > from us might be causing this, but we are a little bit at loss, and were > > > hoping that you could give us a hand in troubleshooting this. > > > > For reference, these are the patches in that branch that touch any > > interesting files, > > as most of the contents are for pci-controller drivers that are not used on > > powerpc at all: > > > > $ git log --no-merges --oneline 512b7931ad05..dda4b381f05d > > arch/powerpc/ drivers/of/ drivers/pci/*.[ch] include/linux/ > > acd61ffb2f16 PCI: Add ACS quirk for Pericom PI7C9X2G switches > > 978fd0056e19 PCI: of: Allow matching of an interrupt-map local to a PCI device > > 041284181226 of/irq: Allow matching of an interrupt-map local to an > > interrupt controller > > 0ab8d0f6ae3f irqdomain: Make of_phandle_args_to_fwspec() generally available > > 5ec0a6fcb60e PCI: Do not enable AtomicOps on VFs > > 7a41ae80bdcb PCI: pci-bridge-emul: Fix emulation of W1C bits > > fd1ae23b495b PCI: Prefer 'unsigned int' over bare 'unsigned' > > ff5d3bb6e16d PCI: Remove redundant 'rc' initialization > > 3331325c6347 PCI/VPD: Use pci_read_vpd_any() in pci_vpd_size() > > e1b0d0bb2032 PCI: Re-enable Downstream Port LTR after reset or hotplug > > ac8e3cef588c PCI/sysfs: Explicitly show first MSI IRQ for 'irq' > > 88dee3b0efe4 PCI: Remove unused pci_pool wrappers > > b5f9c644eb1b PCI: Remove struct pci_dev->driver > > 2a4d9408c9e8 PCI: Use to_pci_driver() instead of pci_dev->driver > > 4141127c44a9 powerpc/eeh: Use to_pci_driver() instead of pci_dev->driver > > f9a6c8ad4922 PCI/ERR: Reduce compile time for CONFIG_PCIEAER=n > > 43e85554d4ed xen/pcifront: Use to_pci_driver() instead of pci_dev->driver > > 34ab316d7287 xen/pcifront: Drop pcifront_common_process() tests of pcidev, pdrv > > 9f37ab0412eb PCI/switchtec: Add check of event support > > 5a72431ec318 powerpc/eeh: Use dev_driver_string() instead of struct > > pci_dev->driver->name > > ae232f0970ea PCI: Drop pci_device_probe() test of !pci_dev->driver > > 097d9d414433 PCI: Drop pci_device_remove() test of pci_dev->driver > > 8e9028b3790d PCI: Return NULL for to_pci_driver(NULL) > > 357df2fc0066 PCI: Use unsigned to match sscanf("%x") in pci_dev_str_match_path() > > bf2928c7a284 PCI/VPD: Add pci_read/write_vpd_any() > > b2105b9f39b5 PCI: Correct misspelled and remove duplicated words > > 7c3855c423b1 PCI: Coalesce host bridge contiguous apertures > > e0f7b1922358 PCI: Use kstrtobool() directly, sans strtobool() wrapper > > 36f354ec7bf9 PCI/sysfs: Return -EINVAL consistently from "store" functions > > 95e83e219d68 PCI/sysfs: Check CAP_SYS_ADMIN before parsing user input > > af9d82626c8f PCI/ACPI: Remove OSC_PCI_SUPPORT_MASKS and OSC_PCI_CONTROL_MASKS > > 9a0a1417d3bb PCI: Tidy comments > > 06dc660e6eb8 PCI: Rename pcibios_add_device() to pcibios_device_add() > > e3f4bd3462f6 PCI: Mark Atheros QCA6174 to avoid bus reset > > 3a19407913e8 PCI/P2PDMA: Apply bus offset correctly in DMA address calculation > > > > Out of these, I agree that most of them seem harmless, these would > > be the ones I'd try to look at more closely, or maybe revert for testing: > > > > 978fd0056e19 PCI: of: Allow matching of an interrupt-map local to a PCI device > > 041284181226 of/irq: Allow matching of an interrupt-map local to an > > interrupt controller > > e1b0d0bb2032 PCI: Re-enable Downstream Port LTR after reset or hotplug > > 7c3855c423b1 PCI: Coalesce host bridge contiguous apertures > > 3a19407913e8 PCI/P2PDMA: Apply bus offset correctly in DMA address calculation > > Robert would you be able build a kernel without the patches Arnd singled > out as potential curlprits? Might be expidite some troubleshooting saving > a lot of time doing bisect. > > I wonder if this will help you with the following problem: > https://lore.kernel.org/linux-pci/CAP145pjO9zdGgutHP=of0H+L1=nSz097zf73i7ZYm2-NWuwHhQ@mail.gmail.com/ Thanks. I tried removing all of those (latest 5), and I had Windows/qemu boot hangs with all of them removed. But I cannot say for sure, because I did quite a mess with my kernel/qemu setup, but with this code https://github.com/torvalds/linux/commit/cb690f5238d71f543f4ce874aa59237cf53a877c and with patch https://lkml.org/lkml/2021/11/9/836 my qemu/Win11 seems to be booting again (even if dmesg is filled with pci-related errors around initialization timeouts) I think I'll try to concentrate on helping with the pic code in this thread: https://lkml.org/lkml/2021/11/10/684 - and once it works well with the host kernel, I'll try to figure out whether anything still troubles the vfio/kvm code. PS: I assumed you asked me to test it wrt my troubles with qemu/vfio/kvm/Win11.