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 X-Spam-Level: X-Spam-Status: No, score=-10.1 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 23D91C433B4 for ; Mon, 26 Apr 2021 22:40:20 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 395FC600D1 for ; Mon, 26 Apr 2021 22:40:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 395FC600D1 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=roeck-us.net Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4FTfwx4gB9z30F9 for ; Tue, 27 Apr 2021 08:40:17 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20161025 header.b=kZLCwHWT; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=gmail.com (client-ip=2607:f8b0:4864:20::32d; helo=mail-ot1-x32d.google.com; envelope-from=groeck7@gmail.com; receiver=) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20161025 header.b=kZLCwHWT; dkim-atps=neutral Received: from mail-ot1-x32d.google.com (mail-ot1-x32d.google.com [IPv6:2607:f8b0:4864:20::32d]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4FTfwN0FSyz2xMd for ; Tue, 27 Apr 2021 08:39:47 +1000 (AEST) Received: by mail-ot1-x32d.google.com with SMTP id d3-20020a9d29030000b029027e8019067fso51756282otb.13 for ; Mon, 26 Apr 2021 15:39:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=e2srYZCezWerrvydsSZOThr9vW3FBvIYlx0HWy9kKVI=; b=kZLCwHWTPoCcIznJyyl8a3iAnUuAxn9tfyhdI58/vJAMgIn2vpqTrUBu47TY/hQQiR uO8AIaZfajAooZ2UPa2HYMgSzPac/4wNu15g55rd6AmgzoRYAqWQP0SN6ltG8tS7Gpwe cvPGEkrS1UMHNyhdZ+s044cm6zwkzta+5syQ6XEj2rELova0e++K5/jzAHe/VykL04CF /opVpyk75zqh1BIJF5FWsGs3U7NgrYgjM73UOXodkhPxMj3vLlVsxppmRuP4x5HwNxwX H4D8STjwwLUU5/ZhLRKwZuQaDG9FjrxYMTmHwD6NsHiy0dUrxEnjNfsH+JSYQyb4a+c0 dbtw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to:user-agent; bh=e2srYZCezWerrvydsSZOThr9vW3FBvIYlx0HWy9kKVI=; b=ZPEHQlKhAYdjhZHcDkPJSleJg6KChRTSY1aZzWthaMk59OFRW2/K1eiu63Iop7Js39 LNlQ9qk5J93tiqSXtb30MUllqq2zyUvOF3mzKmJc6I08wL+K72zuZ39CSSONs6e1v94c DyqhAZxvnT1dpNnvTlZVbX1joVlUP6RChB56ZKvr2GZKbo0KyVm/qlh+ZXZnoGqVolyB Yy03IGoamqjTnwNWJg8yNjByvt4rHolurOxuRqwwdMA+EA+nc7V8GjjiHoEP9s+QQRbO j87tXeHVJBkS9jwGLeCAqtlIr6SKjr6iItcAIAOho4JvCdwEYNh6Shg4gjODoELwaDqC 73bA== X-Gm-Message-State: AOAM531b0qP9ThvO5clrAg4TAz4/fSAeQI5A3yaodwaUozgJk022O1Vn O5LwDZKdlQO2h3w3U6LCeyI= X-Google-Smtp-Source: ABdhPJw9DQu6UlUVdcz2g1TBfNWipkC8oX5M3o9tQoOEORwU6A+PwH/pqnvpJVU+kZH5F1gVYOKZGg== X-Received: by 2002:a9d:1d6a:: with SMTP id m97mr16357004otm.322.1619476784287; Mon, 26 Apr 2021 15:39:44 -0700 (PDT) Received: from localhost ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id v81sm3322047oie.13.2021.04.26.15.39.42 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Mon, 26 Apr 2021 15:39:43 -0700 (PDT) Date: Mon, 26 Apr 2021 15:39:42 -0700 From: Guenter Roeck To: Marc Zyngier Subject: Re: [PATCH 2/9] ARM: PXA: Kill use of irq_create_strict_mappings() Message-ID: <20210426223942.GA213931@roeck-us.net> References: <20210406093557.1073423-1-maz@kernel.org> <20210406093557.1073423-3-maz@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210406093557.1073423-3-maz@kernel.org> User-Agent: Mutt/1.9.4 (2018-02-28) X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rich Felker , Yoshinori Sato , linux-sh@vger.kernel.org, Robert Jarzmik , linux-kernel@vger.kernel.org, Haojian Zhuang , linux-mips@vger.kernel.org, Thomas Bogendoerfer , linux-arm-kernel@lists.infradead.org, Thomas Gleixner , linuxppc-dev@lists.ozlabs.org, Daniel Mack Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Tue, Apr 06, 2021 at 10:35:50AM +0100, Marc Zyngier wrote: > irq_create_strict_mappings() is a poor way to allow the use of > a linear IRQ domain as a legacy one. Let's be upfront about > it and use a legacy domain when appropriate. > > Signed-off-by: Marc Zyngier > --- When running the "mainstone" qemu emulation, this patch results in many (32, actually) runtime warnings such as the following. [ 0.528272] ------------[ cut here ]------------ [ 0.528285] WARNING: CPU: 0 PID: 1 at kernel/irq/irqdomain.c:550 irq_domain_associate+0x194/0x1f0 [ 0.528315] error: virq335 is not allocated [ 0.528325] Modules linked in: [ 0.528351] CPU: 0 PID: 1 Comm: swapper Tainted: G W 5.12.0-rc8-next-20210423 #1 [ 0.528372] Hardware name: Intel HCDDBBVA0 Development Platform (aka Mainstone) [ 0.528387] Backtrace: [ 0.528406] [] (dump_backtrace) from [] (show_stack+0x20/0x24) [ 0.528441] r7:00000226 r6:c00796e8 r5:00000009 r4:c088d2a0 [ 0.528454] [] (show_stack) from [] (dump_stack+0x28/0x30) [ 0.528479] [] (dump_stack) from [] (__warn+0xe8/0x110) [ 0.528507] r5:00000009 r4:c0872698 [ 0.528520] [] (__warn) from [] (warn_slowpath_fmt+0xa0/0xe0) [ 0.528551] r7:c00796e8 r6:00000226 r5:c0872698 r4:c0872700 [ 0.528564] [] (warn_slowpath_fmt) from [] (irq_domain_associate+0x194/0x1f0) [ 0.528597] r8:00000130 r7:0000014f r6:0000001f r5:00000000 r4:c11bd780 [ 0.528610] [] (irq_domain_associate) from [] (irq_domain_associate_many+0x60/0xa4) [ 0.528642] r8:00000130 r7:c11bd780 r6:fffffed0 r5:00000150 r4:00000150 [ 0.528655] [] (irq_domain_associate_many) from [] (irq_domain_create_legacy+0x5c/0x68) [ 0.528687] r8:00000130 r7:00000130 r6:00000020 r5:00000000 r4:c11bd780 [ 0.528699] [] (irq_domain_create_legacy) from [] (irq_domain_add_legacy+0x34/0x3c) [ 0.528730] r7:c09b1370 r6:c09b1360 r5:c11bd3a0 r4:00000000 [ 0.528743] [] (irq_domain_add_legacy) from [] (cplds_probe+0x170/0x1ac) [ 0.528768] [] (cplds_probe) from [] (platform_probe+0x50/0xb0) [ 0.528800] r8:c09d2c94 r7:c0aa4f88 r6:c09d2c94 r5:c09b1370 r4:00000000 [ 0.528814] [] (platform_probe) from [] (really_probe+0x100/0x4d4) [ 0.528844] r7:c0aa4f88 r6:00000000 r5:00000000 r4:c09b1370 [ 0.528858] [] (really_probe) from [] (driver_probe_device+0x88/0x20c) [ 0.528892] r10:c0974830 r9:c0a70000 r8:c093b224 r7:c0a31de8 r6:c09d2c94 r5:c09d2c94 [ 0.528907] r4:c09b1370 [ 0.528919] [] (driver_probe_device) from [] (device_driver_attach+0x68/0x70) [ 0.528953] r9:c0a70000 r8:c093b224 r7:c0a31de8 r6:c09d2c94 r5:00000000 r4:c09b1370 [ 0.528969] [] (device_driver_attach) from [] (__driver_attach+0xc0/0x164) [ 0.528997] r7:c0a31de8 r6:c09b1370 r5:c09d2c94 r4:00000000 [ 0.529009] [] (__driver_attach) from [] (bus_for_each_dev+0x84/0xcc) [ 0.529039] r7:c0a31de8 r6:c04306d4 r5:c09d2c94 r4:00000000 [ 0.529052] [] (bus_for_each_dev) from [] (driver_attach+0x28/0x30) [ 0.529082] r6:00000000 r5:c11bd200 r4:c09d2c94 [ 0.529095] [] (driver_attach) from [] (bus_add_driver+0x168/0x210) [ 0.529122] [] (bus_add_driver) from [] (driver_register+0x88/0x120) [ 0.529152] r7:c0a5c7e0 r6:00000000 r5:ffffe000 r4:c09d2c94 [ 0.529165] [] (driver_register) from [] (__platform_driver_register+0x2c/0x34) [ 0.529191] r5:ffffe000 r4:c094ba64 [ 0.529204] [] (__platform_driver_register) from [] (cplds_driver_init+0x20/0x28) [ 0.529230] [] (cplds_driver_init) from [] (do_one_initcall+0x60/0x27c) [ 0.529255] [] (do_one_initcall) from [] (kernel_init_freeable+0x158/0x1e4) [ 0.529284] r7:c0974850 r6:00000007 r5:c0c0f720 r4:c09a02fc [ 0.529297] [] (kernel_init_freeable) from [] (kernel_init+0x18/0x110) [ 0.529328] r10:00000000 r9:00000000 r8:00000000 r7:00000000 r6:00000000 r5:c06c525c [ 0.529343] r4:00000000 [ 0.529354] [] (kernel_init) from [] (ret_from_fork+0x14/0x2c) [ 0.529387] Exception stack(0xc0bdffb0 to 0xc0bdfff8) [ 0.529467] ffa0: 00000000 00000000 00000000 00000000 [ 0.529587] ffc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 0.529684] ffe0: 00000000 00000000 00000000 00000000 00000013 00000000 [ 0.529726] r5:c06c525c r4:00000000 [ 0.529752] ---[ end trace d199929d2b87e077 ]--- Bisect log attached. Guenter --- # bad: [e3d35712f85ac84fb06234848f6c043ab418cf8b] Add linux-next specific files for 20210423 # good: [bf05bf16c76bb44ab5156223e1e58e26dfe30a88] Linux 5.12-rc8 git bisect start 'HEAD' 'v5.12-rc8' # good: [d4b5d9d94679a18bfa4ccdafd19876d58777911e] Merge remote-tracking branch 'crypto/master' git bisect good d4b5d9d94679a18bfa4ccdafd19876d58777911e # good: [27628e42fe59a698e66b671bf1e1f01f6a3fe765] Merge remote-tracking branch 'tip/auto-latest' git bisect good 27628e42fe59a698e66b671bf1e1f01f6a3fe765 # bad: [bc6c3ae4f662fc719d0bf144f150f72cab8912d4] Merge remote-tracking branch 'vfio/next' git bisect bad bc6c3ae4f662fc719d0bf144f150f72cab8912d4 # bad: [5ff5b00609c64a043ccd5bc92273c132b33f7f9a] Merge remote-tracking branch 'driver-core/driver-core-next' git bisect bad 5ff5b00609c64a043ccd5bc92273c132b33f7f9a # bad: [c878be9c883153797d5749620e58f180cc429e88] Merge remote-tracking branch 'kvm/next' git bisect bad c878be9c883153797d5749620e58f180cc429e88 # good: [52acd22faa1af8a0514ccd075a6978ac97986425] KVM: Boost vCPU candidate in user mode which is delivering interrupt git bisect good 52acd22faa1af8a0514ccd075a6978ac97986425 # good: [988aab640a6c46ab9552e65c2c3a8d577a4e30f3] rcu: Make rcu_gp_cleanup() be noinline for tracing git bisect good 988aab640a6c46ab9552e65c2c3a8d577a4e30f3 # bad: [6603c2d8bd6cc7fa591fd3b4232bf25b65a0ea8f] Merge remote-tracking branch 'ftrace/for-next' git bisect bad 6603c2d8bd6cc7fa591fd3b4232bf25b65a0ea8f # good: [c658797f1a70561205a224be0c8be64977ed64e8] tracing: Add method for recording "func_repeats" events git bisect good c658797f1a70561205a224be0c8be64977ed64e8 # good: [46135d6f878ab00261d4a2082d620bfb41019aab] irqchip/gic-v4.1: Disable vSGI upon (GIC CPUIF < v4.1) detection git bisect good 46135d6f878ab00261d4a2082d620bfb41019aab # bad: [05d7bf817019890e4d049e0b851940c596adbd9b] dt-bindings: interrupt-controller: Add IDT 79RC3243x Interrupt Controller git bisect bad 05d7bf817019890e4d049e0b851940c596adbd9b # bad: [1a0b05e435544cd53cd3936bdab425d88784b71a] irqdomain: Get rid of irq_create_strict_mappings() git bisect bad 1a0b05e435544cd53cd3936bdab425d88784b71a # bad: [5f8b938bd790cff6542c7fe3c1495c71f89fef1b] irqchip/jcore-aic: Kill use of irq_create_strict_mappings() git bisect bad 5f8b938bd790cff6542c7fe3c1495c71f89fef1b # bad: [b68761da01114a64b9c521975c3bca6d10eeb950] ARM: PXA: Kill use of irq_create_strict_mappings() git bisect bad b68761da01114a64b9c521975c3bca6d10eeb950 # first bad commit: [b68761da01114a64b9c521975c3bca6d10eeb950] ARM: PXA: Kill use of irq_create_strict_mappings()