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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6C047C4332F for ; Wed, 23 Nov 2022 23:18:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229749AbiKWXST (ORCPT ); Wed, 23 Nov 2022 18:18:19 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48936 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229698AbiKWXSS (ORCPT ); Wed, 23 Nov 2022 18:18:18 -0500 Received: from mail-pf1-x436.google.com (mail-pf1-x436.google.com [IPv6:2607:f8b0:4864:20::436]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 40EF2114BAE for ; Wed, 23 Nov 2022 15:18:17 -0800 (PST) Received: by mail-pf1-x436.google.com with SMTP id y203so87192pfb.4 for ; Wed, 23 Nov 2022 15:18:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=FsfR9yi785BtGGSVwV+zwwFDYAOlG2z5wKIfOUjvejk=; b=eJo7pE5mpeh9IGoBDw/Ld4X77ZhvDqBSGJFGeiKlXjFaOMQqKwb6A9ZHA2TiC7fIhA dCmnvavKmtwGZSOR2BFCb2Du8muvmjnjA69CefzsP+Lc9ul8ppdETcWXZQ8lEVaY1PAA J1n9VdtU9SCxNX65O14tyT6aYADTFAVI2S/Y9/2mHuq6XNW+nctXUlrenppJiHVW1yG9 U0RBjVCB9oFsuZpUCc8HSXsXWaAMsZlYXHaLH7DCY3zAfYPdE3ehFQLykdnZ2c8/HwLh Uu356puteWG8Ww8AsRGKaVVSRnofUV5LpYGg3fj4WgdN034AcwwCQq/pytDyM1H5ugHi 3K1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=FsfR9yi785BtGGSVwV+zwwFDYAOlG2z5wKIfOUjvejk=; b=Ll8/riPzRty9yoTb2VSshNxjOZi7TPlTPwamNqFjFE3iJPnbcaBXYJqp3Dz0ktdoZt asvLy34LBbIdQkaOujpK6D4rrcMDImxTYcJkfjwOKRipzwbT05X9ZjtOZars93tm2ODC juTXGG8Fo6j1QMyzmS41lRP6S+ydTJZhkYRQEN+4knrC1IANKsWQz03jLH3ezXSuIm1g kHivQjvzt+oRiMU28NWvTb797MOxA+QmhcPPWrP40w/3ov2rx0AfUjX6NbOxR/ZusVhR QTwUh0eTVB4MS/kxUpJPohVgskRL2/dMrTJ/2rvWdUabF6K62Gpwa04gW5wj9G9/a25M /6LA== X-Gm-Message-State: ANoB5pnapAoug7Chs3IUhbR1GvyDn8zFLzigLUIvpDGyl+PBUh+j0avm cr+MpTbIUJVqXEPs1ml6bHw2Jg== X-Google-Smtp-Source: AA0mqf7FM0xg/oQ5G6HHhO0sFJT90RFNI09VskgLnQ8XoxQbAahGHweiwMRxN1TNy4NjiISv+9jGTA== X-Received: by 2002:a05:6a00:a88:b0:574:2c5e:b18c with SMTP id b8-20020a056a000a8800b005742c5eb18cmr7950686pfl.10.1669245496670; Wed, 23 Nov 2022 15:18:16 -0800 (PST) Received: from google.com (7.104.168.34.bc.googleusercontent.com. [34.168.104.7]) by smtp.gmail.com with ESMTPSA id e10-20020a17090301ca00b00188b63f0782sm13367878plh.288.2022.11.23.15.18.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 23 Nov 2022 15:18:16 -0800 (PST) Date: Wed, 23 Nov 2022 23:18:12 +0000 From: Sean Christopherson To: Mingwei Zhang Cc: Paolo Bonzini , "H. Peter Anvin" , kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Nagareddy Reddy , Jim Mattson , David Matlack Subject: Re: [RFC PATCH] KVM: x86/mmu: replace BUG() with KVM_BUG() in shadow mmu Message-ID: References: <20221123231206.274392-1-mizhang@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221123231206.274392-1-mizhang@google.com> Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org On Wed, Nov 23, 2022, Mingwei Zhang wrote: > diff --git a/arch/x86/kvm/mmu/mmu.c b/arch/x86/kvm/mmu/mmu.c > index 4736d7849c60..075d31b0db9c 100644 > --- a/arch/x86/kvm/mmu/mmu.c > +++ b/arch/x86/kvm/mmu/mmu.c > @@ -955,12 +955,12 @@ static void pte_list_remove(u64 *spte, struct kvm_rmap_head *rmap_head) > > if (!rmap_head->val) { > pr_err("%s: %p 0->BUG\n", __func__, spte); > - BUG(); > + KVM_BUG(); This won't compile. KVM_BUG() isn't a direct replacement for BUG(), it's more akin to WARN(). And that's why I suggested this be RFC: @kvm needs to be plumbed down here in order to use KVM_BUG(). I don't mind that too much, it's just a little unfortunate.