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.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham 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 712E9C4360F for ; Wed, 3 Apr 2019 04:54:52 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 0014B20882 for ; Wed, 3 Apr 2019 04:54:51 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=tobin.cc header.i=@tobin.cc header.b="RV90u1gU"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="UnzHKVfN" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0014B20882 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=tobin.cc Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 6CB286B0292; Wed, 3 Apr 2019 00:54:51 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 67B3D6B0294; Wed, 3 Apr 2019 00:54:51 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5436D6B0295; Wed, 3 Apr 2019 00:54:51 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from mail-qk1-f199.google.com (mail-qk1-f199.google.com [209.85.222.199]) by kanga.kvack.org (Postfix) with ESMTP id 2E8BA6B0292 for ; Wed, 3 Apr 2019 00:54:51 -0400 (EDT) Received: by mail-qk1-f199.google.com with SMTP id x18so13565530qkf.8 for ; Tue, 02 Apr 2019 21:54:51 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:dkim-signature:dkim-signature:date:from:to:cc :subject:message-id:references:mime-version:content-disposition :in-reply-to:user-agent; bh=dRD1L04hChZhTlTZdre1MhwWkHZl8qy1+Gyjem9CXC0=; b=nVB6lOyR5T1ZlITYDr5T+cLtlgsewN69ZSW19XDOncReWu4GWbQIBfqmkdhR6IBS7M lYbzCkiVd6/2brh92v6qdamVk+7CqLsM750EHocCFKbSyAH4c2BGMSTeyaPOWsiVuEWa 1Dw4QGOP9LgCHy5fV7X+WHqVe9zPwwX7x0n9UqPzswtCjJVZ61vADP33PMJZpmkBfuW6 DNSlfs04jOiU+Eeg6UjkCscQT77kHK+EAL3JciDLq4UxluHamYQioUeCibciNWuQmWlD TKDPCBdnjcvjklJVs6L/V847linQnli4EkQNimhV8EiJvbUGbpKorZs2QlkBpJzf6e1z DXwg== X-Gm-Message-State: APjAAAVPDQEIzC8uwpXTHja8B9FnVPn52ynfkiHP1i9BS2Wmxde8YafQ orjhbm90Kbh6bJD51tNiZFJwf/LPqJ+/CGxgn0Oaog3EqRF2sZZ3xVgQhDntE+rj1p3Tji2maWJ 4H0ZrgZcqBqTlYNGJPQuZyJBcp3W7l2OjxbP9Ly/7chaYjLkjts0UWyjJU2j2N1yoYA== X-Received: by 2002:ae9:e005:: with SMTP id m5mr59830114qkk.313.1554267290840; Tue, 02 Apr 2019 21:54:50 -0700 (PDT) X-Google-Smtp-Source: APXvYqyEmp+8ObF83qNwv+bpKRKHOgfzAAqUU4NJq41OSHkSSIpTTRUzzHoSlrGkV8Kqz0Y0kYn0 X-Received: by 2002:ae9:e005:: with SMTP id m5mr59830084qkk.313.1554267290072; Tue, 02 Apr 2019 21:54:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1554267290; cv=none; d=google.com; s=arc-20160816; b=N8Nelb9CxxsGHgb9nOCNiCGrqe9kfp+h+xfWkIMqjs7fwIenYsq+h4dvu0CswOB8LW Q4kG80ak1GAIV7TEXAEqmlVwP9bA8jreeIY6vZtJF05BthE4Mya6JaiSkCuUfpyFvnxc MSbY5/h2SZlSby1Dk4nlzPfA7QIxFT7dK8h7kuL3XjeFIbt5Y/pEkszO2QUKB1t/idfW Z7dSzWRTrEJamWIVR/h+gEE6Pst1tfR/L3f6StExvb7l9L+dxleGS3/COQMCbJS5k9fK 2JXQ6uEcFdo4o1sfmToYQrU7QGVdvMlscnCRpwhPJlHuY+kHFzzIOoGLWVpzHNLMP02f IB7w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:dkim-signature:dkim-signature; bh=dRD1L04hChZhTlTZdre1MhwWkHZl8qy1+Gyjem9CXC0=; b=GuXlQs0NeJIOHt4lKfA5iK+LY0RhBfiHuK5WZMSQ081mEJyMPmCidKmNKzIvs4nKKG zgCHT1GPw2xtHpUNiH1LDwhJ8dWfvRiVGwhodL+Wom5E6hGaOPoW0zbyoorxbDPaTe2T t7qvT4VMB5o+4Qhus566MbdstcMjevBtGRObkIpFmlgr9iCZ6fThwdVvMKA38dewgIBw 9iLgr5KVbPSO5W3t2jgHtklXovIrKX0zYnYiyenc3TNaAFpasr11Pvo23rfu+q02KO0F RBX/F6O1RXTdBt+fOvdAEbSOWnPlCGbh6qQzIGrnQJ7vQ0UXHxjJvH1VQtTJOaVNh4St ZO1Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@tobin.cc header.s=fm2 header.b=RV90u1gU; dkim=pass header.i=@messagingengine.com header.s=fm2 header.b=UnzHKVfN; spf=neutral (google.com: 66.111.4.28 is neither permitted nor denied by best guess record for domain of me@tobin.cc) smtp.mailfrom=me@tobin.cc Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com. [66.111.4.28]) by mx.google.com with ESMTPS id u49si4771409qtk.310.2019.04.02.21.54.49 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 02 Apr 2019 21:54:49 -0700 (PDT) Received-SPF: neutral (google.com: 66.111.4.28 is neither permitted nor denied by best guess record for domain of me@tobin.cc) client-ip=66.111.4.28; Authentication-Results: mx.google.com; dkim=pass header.i=@tobin.cc header.s=fm2 header.b=RV90u1gU; dkim=pass header.i=@messagingengine.com header.s=fm2 header.b=UnzHKVfN; spf=neutral (google.com: 66.111.4.28 is neither permitted nor denied by best guess record for domain of me@tobin.cc) smtp.mailfrom=me@tobin.cc Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id 7605421EDB; Wed, 3 Apr 2019 00:54:49 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute5.internal (MEProxy); Wed, 03 Apr 2019 00:54:49 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tobin.cc; h=date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm2; bh=dRD1L04hChZhTlTZdre1MhwWkHZ l8qy1+Gyjem9CXC0=; b=RV90u1gUiM8BnetFp56ONnHB6PkfQUeM6zA3ajYkOAu j7qPVKdAzKP361VnZqrHif1EO6QE9rDFnnqRG707YJIcHZPRqTu+Ynzkqx/OUUFs gvbFQsLHFCkOrEyY6dlcS79lix0+BuES34/mCu9XJcqtxN9b6b41+Da56359Kiid Wicp2A+hfkSQrllKtqhZ0yChA8uAJGc32xPfWlNgI8b7OsuiQN+/GUZmgLu7KODm VwgktTK9cnhYG86LQyTT4iUZlLYC0Ozk8oPSuV2UJ9gA5fBIEV91oTvJwqpLSqDJ u1lfubXw2Ov9NOx3mgLUM0z96xA+ubueiVC/w7HdWag== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=dRD1L0 4hChZhTlTZdre1MhwWkHZl8qy1+Gyjem9CXC0=; b=UnzHKVfNQTDvpsPZJ8iE3B IkzERqxjl+PXhHXjFKKSxK9HlTAtsSPVgxZvEftuaCsvfx+r6gKzkAVWyu1pQBlU dd5ADMfbNYeoN120YEk4WaEe+ksgeQVuYW2q4pVHLaS5wcgQhIIth/iL0UeFq6f+ LoRTycJaxeJecqX/0+UGwBYQgeeUxqH4r4hItaCfhmLi7lgHTzbE5NkwPwLDJVYh NaQi/gytoVYcoTgIGyFKpS+Bb9kbDTk+JCmpX9x/cG2LzftCf69aaY6XSQdjYPUm aNB6t6hFLqA63sMDDOGuVvsKhfXEVnORtpHdhkKHYooxmsaihhplOf3UagA1AERw == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduuddrtddugdekieculddtuddrgedutddrtddtmd cutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdp uffrtefokffrpgfnqfghnecuuegrihhlohhuthemuceftddtnecufghrlhcuvffnffculd ejmdenucfjughrpeffhffvuffkfhggtggujgfofgesthdtredtofervdenucfhrhhomhep fdfvohgsihhnucevrdcujfgrrhguihhnghdfuceomhgvsehtohgsihhnrdgttgeqnecuff homhgrihhnpehkvghrnhgvlhdrohhrghenucfkphepuddvgedrudeiledrvdejrddvtdek necurfgrrhgrmhepmhgrihhlfhhrohhmpehmvgesthhosghinhdrtggtnecuvehluhhsth gvrhfuihiivgeptd X-ME-Proxy: Received: from localhost (124-169-27-208.dyn.iinet.net.au [124.169.27.208]) by mail.messagingengine.com (Postfix) with ESMTPA id B5111E4841; Wed, 3 Apr 2019 00:54:47 -0400 (EDT) Date: Wed, 3 Apr 2019 15:54:17 +1100 From: "Tobin C. Harding" To: kernel test robot Cc: "Tobin C. Harding" , LKP , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Andrew Morton Subject: Re: 15c8410c67 ("mm/slob.c: respect list_head abstraction layer"): WARNING: CPU: 0 PID: 1 at lib/list_debug.c:28 __list_add_valid Message-ID: <20190403045417.GA19313@eros.localdomain> References: <5ca413c6.9TM84kwWw8lLhnmK%lkp@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5ca413c6.9TM84kwWw8lLhnmK%lkp@intel.com> X-Mailer: Mutt 1.11.4 (2019-03-13) User-Agent: Mutt/1.11.4 (2019-03-13) X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Wed, Apr 03, 2019 at 10:00:38AM +0800, kernel test robot wrote: > Greetings, > > 0day kernel testing robot got the below dmesg and the first bad commit is > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > > commit 15c8410c67adefd26ea0df1f1b86e1836051784b > Author: Tobin C. Harding > AuthorDate: Fri Mar 29 10:01:23 2019 +1100 > Commit: Stephen Rothwell > CommitDate: Sat Mar 30 16:09:41 2019 +1100 > > mm/slob.c: respect list_head abstraction layer > > Currently we reach inside the list_head. This is a violation of the layer > of abstraction provided by the list_head. It makes the code fragile. > More importantly it makes the code wicked hard to understand. > > The code logic is based on the page in which an allocation was made, we > want to modify the slob_list we are working on to have this page at the > front. We already have a function to check if an entry is at the front of > the list. Recently a function was added to list.h to do the list > rotation. We can use these two functions to reduce line count, reduce > code fragility, and reduce cognitive load required to read the code. > > Use list_head functions to interact with lists thereby maintaining the > abstraction provided by the list_head structure. > > Link: http://lkml.kernel.org/r/20190318000234.22049-3-tobin@kernel.org > Signed-off-by: Tobin C. Harding > Cc: Christoph Lameter > Cc: David Rientjes > Cc: Joonsoo Kim > Cc: Pekka Enberg > Cc: Roman Gushchin > Signed-off-by: Andrew Morton > Signed-off-by: Stephen Rothwell > > 2e1f88301e include/linux/list.h: add list_rotate_to_front() > 15c8410c67 mm/slob.c: respect list_head abstraction layer > 05d08e2995 Add linux-next specific files for 20190402 > +-------------------------------------------------------+------------+------------+---------------+ > | | 2e1f88301e | 15c8410c67 | next-20190402 | > +-------------------------------------------------------+------------+------------+---------------+ > | boot_successes | 1009 | 198 | 299 | > | boot_failures | 0 | 2 | 44 | > | WARNING:at_lib/list_debug.c:#__list_add_valid | 0 | 2 | 44 | > | RIP:__list_add_valid | 0 | 2 | 44 | > | WARNING:at_lib/list_debug.c:#__list_del_entry_valid | 0 | 2 | 25 | > | RIP:__list_del_entry_valid | 0 | 2 | 25 | > | WARNING:possible_circular_locking_dependency_detected | 0 | 2 | 44 | > | RIP:_raw_spin_unlock_irqrestore | 0 | 2 | 2 | > | BUG:kernel_hang_in_test_stage | 0 | 0 | 6 | > | BUG:unable_to_handle_kernel | 0 | 0 | 1 | > | Oops:#[##] | 0 | 0 | 1 | > | RIP:slob_page_alloc | 0 | 0 | 1 | > | Kernel_panic-not_syncing:Fatal_exception | 0 | 0 | 1 | > | RIP:delay_tsc | 0 | 0 | 2 | > +-------------------------------------------------------+------------+------------+---------------+ > > [ 2.618737] db_root: cannot open: /etc/target > [ 2.620114] mtdoops: mtd device (mtddev=name/number) must be supplied > [ 2.620967] slram: not enough parameters. > [ 2.621614] ------------[ cut here ]------------ > [ 2.622254] list_add corruption. prev->next should be next (ffffffffaeeb71b0), but was ffffcee1406d3f70. (prev=ffffcee140422508). Is this perhaps a false positive because we hackishly move the list_head 'head' and insert it back into the list. Perhaps this is confusing the validation functions? Tobin