Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp4201818pxa; Mon, 10 Aug 2020 03:36:05 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzMi3XI3yR332F7BY/IFOH2uhIDE/+6G2o+9gYxuNA1hsUPeiR0f6VnL6S6KfPHT+d/JBqP X-Received: by 2002:a17:907:42cd:: with SMTP id nz21mr21150740ejb.395.1597055765487; Mon, 10 Aug 2020 03:36:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597055765; cv=none; d=google.com; s=arc-20160816; b=odSi6cr+ze+KhPBTPPU12mO3e8tEcNGn4mp3w8J7E1LaH3LKf7CN6xMkgaBIGNyoku 4AZ6g7sc4KHwd1XEIwlTxLozLBeiIJiozBJjWheil0I6FHLeDXlgQpYlxjVgSPdj8ZVT chq0+vwCn/AxCcKUMf0VrjbPk0TRLaRWT3iFOBLXdWr5FwBJI9w1rBj3ExqQpYtN91YG ag4E7X0CnWFmXPe02VL6FagfFNrpPTCtuJ/302dnGKp5Ryt5yL7nyAIu+f1Ypm4I5cwn wgrniyKB/DBnRBcm6/fHjVMKs+FlKe6Oi+NrO+pib5w2T2Y6h0Ur9pZ5W3bgDWY7kOEk vKjw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from :dkim-signature:dkim-signature:date; bh=OjjK56vi+OiP5R8kvP7R0V1sr3ZxerDdKSGBt8tWSbQ=; b=odWt2/mRyFFSNZF4pCvMHl0ph7y1m7wVpL6CmYN/0CcUnSMbMISXgPY4mzPLeGVjV1 YxS+jGEXcRSSlPhpLXTED2JWLSxPhhqD/zBT9mIRo4Z3rUSzd7nqRTWbpA2t2DfVm2yn THo+iOU5oJ8VMS9rogje/MMJmDJJiCfrMraxvt6FKPgZDQggwMTK6lVXLfxAgU+dnr4y VhO8nuBzC5Od1accmQgotEpNIdEp7FfKk+RiFLDGo4uihjhn/HUDlFoy5RNIrBfpDvTE 0KgqUT9vf7Rl2M+1wQiBASKizjftlVeK8PM9fo83jpfN55fqwLrkDR3lGPRH7HfYUVpw ZSVQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=OZO9eCCz; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=PPxvtWc+; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i12si844110ejy.492.2020.08.10.03.35.41; Mon, 10 Aug 2020 03:36:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=OZO9eCCz; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=PPxvtWc+; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726402AbgHJKfJ (ORCPT + 99 others); Mon, 10 Aug 2020 06:35:09 -0400 Received: from Galois.linutronix.de ([193.142.43.55]:50350 "EHLO galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726304AbgHJKfI (ORCPT ); Mon, 10 Aug 2020 06:35:08 -0400 Date: Mon, 10 Aug 2020 12:35:08 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1597055706; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=OjjK56vi+OiP5R8kvP7R0V1sr3ZxerDdKSGBt8tWSbQ=; b=OZO9eCCzf/3FQy2RL+5yiI9OGswKwcIhDTU8S2Ozq/N2JddZQYpKPBJk9frCsslIWY9a3x RQd5Nsobt+Wi8DuFCgXteXUBNX5EwquY9WL1eyHEmhTWxI8ByIxt/hGVNH8nh4l8c2mGVv XU8bcwZDUd7crDKUFtUnE8cExP08f50PVKWu5SHSpcuUS8SZJKCBE2FrZ9noktnTvrc4zp S8yF10T4rwXnJFEd8ahpRAFuKGylLNlm/MkhxSfmGUl4i2xKz0KTASQCvFxB/1p4+aNE58 4u7HCWu7tJjfXzJ5eueKNxyYG/sxsWJVMaB2XdS9Ve9Q3rV+hOsEJsQZ/XU0EA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1597055706; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=OjjK56vi+OiP5R8kvP7R0V1sr3ZxerDdKSGBt8tWSbQ=; b=PPxvtWc+bZcQfLzOZptxtrftfrk8+YBuhoUU0Wte37IcC0x5NTrK40eW/1Pjn4+ftQYMo3 1lss+rc6rVql0ABg== From: "Ahmed S. Darwish" To: Greg KH Cc: bigeasy@linutronix.de, linux-kernel@vger.kernel.org, linux@roeck-us.net, mingo@redhat.com, paulmck@kernel.org, peterz@infradead.org, rostedt@goodmis.org, tglx@linutronix.de, will@kernel.org Subject: Re: [PATCH] Revert "seqlock: lockdep assert non-preemptibility on seqcount_t write" Message-ID: <20200810103508.GA142779@lx-t490> References: <20200810085954.GA1591892@kroah.com> <20200810095428.2602276-1-a.darwish@linutronix.de> <20200810100502.GA2406768@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200810100502.GA2406768@kroah.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 10, 2020 at 12:05:02PM +0200, Greg KH wrote: > On Mon, Aug 10, 2020 at 11:54:28AM +0200, Ahmed S. Darwish wrote: > > This reverts commit 859247d39fb008ea812e8f0c398a58a20c12899e. > > > > Current implementation of lockdep_assert_preemption_disabled() uses > > per-CPU variables, which was done to untangle the existing > > seqlock.h<=>sched.h 'current->' task_struct circular dependency. > > > > Using per-CPU variables did not fully untangle the dependency for > > various non-x86 architectures though, resulting in multiple broken > > builds. For the affected architectures, raw_smp_processor_id() led > > back to 'current->', thus having the original seqlock.h<=>sched.h > > dependency in full-effect. > > > > For now, revert adding lockdep_assert_preemption_disabled() to > > seqlock.h. > > > > Reported-by: Guenter Roeck > > Link: https://lkml.kernel.org/r/20200808232122.GA176509@roeck-us.net > > Link: https://lkml.kernel.org/r/20200810085954.GA1591892@kroah.com > > References: Commit a21ee6055c30 ("lockdep: Change hardirq{s_enabled,_context} to per-cpu variables") > > Signed-off-by: Ahmed S. Darwish > > Reviewed-by: Greg Kroah-Hartman > > Even after this, there are still some build errors on arm32, but I don't > think they are due to this change: > > ERROR: modpost: "__aeabi_uldivmod" [drivers/net/ethernet/sfc/sfc.ko] undefined! > ERROR: modpost: "__bad_udelay" [drivers/net/ethernet/aquantia/atlantic/atlantic.ko] undefined! > Yes, they are unrelated to the seqlock.h changes. (I've locally reverted the whole series just to be sure, and the same linking errors as above were still there for an ARM allyesconfig.) Thanks, -- Ahmed S. Darwish Linutronix GmbH