From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752071AbdLFPyk (ORCPT ); Wed, 6 Dec 2017 10:54:40 -0500 Received: from mail-pf0-f195.google.com ([209.85.192.195]:32995 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751453AbdLFPyi (ORCPT ); Wed, 6 Dec 2017 10:54:38 -0500 X-Google-Smtp-Source: AGs4zMY6ya9NGGzvlBRpwapt+Bd+Ve6628ZLHFr//457yRdVN1OSsIGFIKWlzVXRnxFRYuUabhn0E4XuUKBjW2tQIqE= MIME-Version: 1.0 In-Reply-To: References: <001a114498f6446843055d744c3a@google.com> <20171108084031.GX3165@worktop.lehotels.local> From: Dmitry Vyukov Date: Wed, 6 Dec 2017 16:54:17 +0100 Message-ID: Subject: Re: general protection fault in lockdep_invariant_state (2) To: Peter Zijlstra Cc: syzbot , LKML , Ingo Molnar , syzkaller-bugs@googlegroups.com Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 29, 2017 at 1:00 PM, Dmitry Vyukov wrote: > On Mon, Nov 20, 2017 at 2:28 PM, Dmitry Vyukov wrote: >>>> > R13: 0000000000402260 R14: 00000000004022f0 R15: 0000000000000000 >>>> > kasan: CONFIG_KASAN_INLINE enabled >>>> > kasan: GPF could be caused by NULL-ptr deref or user memory access >>>> > general protection fault: 0000 [#1] SMP KASAN >>>> > Dumping ftrace buffer: >>>> > (ftrace buffer empty) >>>> > Modules linked in: >>>> > CPU: 3 PID: 2982 Comm: syzkaller521009 Not tainted 4.14.0-rc7-next-20171103+ >>>> > #10 >>>> > Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 >>>> > task: ffff88003d6b25c0 task.stack: ffff88003abd0000 >>>> > RIP: 0010:invalidate_xhlock kernel/locking/lockdep.c:4719 [inline] >>>> > RIP: 0010:lockdep_invariant_state+0xd6/0x120 kernel/locking/lockdep.c:4793 >>> >>> >>> Does this help? >>> >>> diff --git a/kernel/locking/lockdep.c b/kernel/locking/lockdep.c >>> index db933d063bfc..da2904418599 100644 >>> --- a/kernel/locking/lockdep.c >>> +++ b/kernel/locking/lockdep.c >>> @@ -4793,7 +4793,8 @@ void lockdep_invariant_state(bool force) >>> * Verify the former, enforce the latter. >>> */ >>> WARN_ON_ONCE(!force && current->lockdep_depth); >>> - invalidate_xhlock(&xhlock(current->xhlock_idx)); >>> + if (current->xhlocks) >>> + invalidate_xhlock(&xhlock(current->xhlock_idx)); >>> } >>> >>> static int cross_lock(struct lockdep_map *lock) >> >> >> Hi Peter, >> >> We are not able to test all fixes for all bugs, but we are rolling out >> a patch testing feature for syzbot: >> >> https://github.com/google/syzkaller/blob/master/docs/syzbot.md#communication-with-syzbot >> >> Please give it a try! > > > Peter, ping. > This still happens. Please either submit the fix if you already tested > it, or test using syzbot and submit. Peter, do you want me to mail your patch?