From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f41.google.com (mail-ej1-f41.google.com [209.85.218.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6A097211475 for ; Thu, 23 Jan 2025 16:01:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737648065; cv=none; b=jhaqQCFXKL93BDOiqHCrthwpmEcXRagm6+mwLz0yvR369icRNRgbec4vtdw9VWwKCfnaGrsjiUQmD2De6rbYiprwLyOaeVvt1TM5iHdZZSgNwpE/fv1C01tv+fKLhPhc1vWRxq2yFPxayaP4/V0u/Oe9SAY3fA8+rVOl9+KAh4o= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737648065; c=relaxed/simple; bh=1kUo76AdbdInNUYiaAbGGk6BEc0K5sUJ9fGGb2RLCDI=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=N31+QmSdit3HYLGdnAcipygBnEqrbqfl1JlKf90lwzigZ34stW/r0ArBVBt8miw+FjP+sg058kJyZlnvQaCOQlERrk7aw9wapdMpI3agm2n28q9cdc/5QOpASneKUTAD1dotcp0iZ7uQ+1yuAQgJicPQtibb2APwSHtM1G5XfdY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=Q5V1w2GO; arc=none smtp.client-ip=209.85.218.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="Q5V1w2GO" Received: by mail-ej1-f41.google.com with SMTP id a640c23a62f3a-aab6fa3e20eso192386366b.2 for ; Thu, 23 Jan 2025 08:01:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1737648062; x=1738252862; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Xh3+8eFQABhtF13QI7U7fE3d8Hf8FSiqNKcHdOqMxak=; b=Q5V1w2GO91aKFkTRgXxEWP+GB4iwf2pnUDvpeTlE9I7zzVN7RP78R9ZbQEvgd8aSnG Y1kOZb/6XBOBEiDlsAKAN9GzeMk6i9Dpk1wNyQt35i19hKfZF7Q/oPqv7pe0bn2CO+Ug bB236QfAv+ivslVo6AQ6GW0J+1tlZ4vDJr2YUzBKpov0RnMxkZmDpINGKvcEgQb5JI72 svnBdkNvoRTvct7HKZevU7yCb7DxS3cRttSrd1C0iH90nuGp4txp+Lo0TAbhLXJ80alV ARMSahUJAGx00wZWkXy6qdnr0n4TWXHCoSREwScf1CZhgTIiupA+OOcpfWCELBAg0QP+ KhMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737648062; x=1738252862; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Xh3+8eFQABhtF13QI7U7fE3d8Hf8FSiqNKcHdOqMxak=; b=sRCvBn96eSLu2AjzntnauDVhon6FAWAIXzfQGjW2LQuSnQxvV51A24tNTIQcUouzJu U2WqfOjIda4IChxynx9J6d2SJw08+2uwm3apqQIFJh2N5JROj22RWrzeDnOBo/03KKNs NXlZClzMZ8aslgUoqu0B+lMoFClywDNW2KQf2R6EqhdRmDdp9g7Lbm3ZIjKibo8cisAU BGqiz6UnAsRlzrPDRHDM8GDzxn2CcWYpFm8ndOJCfV08ar4CcruFJzYovsQt7yBYM5H0 PFPOz6CVHCK+Dze7Odd1F/2cQjKTqodVlV5B1DOFu3c6iYkHA8ed85wfAlSCUISPzb0z kPqg== X-Forwarded-Encrypted: i=1; AJvYcCW871Pz33ohiBIHn67sbx1kWsoNRVaVDYVDZkRtXiHt6hbiRy0Seyz/kDqbcjhNjhZS8w8LRFdTZnJx2gk=@vger.kernel.org X-Gm-Message-State: AOJu0YyrwHLKaiLvqCcctIDL1l2Gmt47byBBbWYJNHRp8w2neBZaJ2Vw gIYijBZQ9rFFkpqmvMPCg46OVn37zW0rdOR6iYeFefkmqwh2LTbVqB7Q/SS4M90= X-Gm-Gg: ASbGncseSKvc+ixu1bmKje9vGo6omsgsB8RsOrl5GnCaFbaNeX0H9GvTcxHL3ErkYFk gdl/X/2ktDqXrdBU8UWZQ07yaik7B2AKoa6b0IrB6E+nlUujoMNqlIifJr0HXJ30VWYzhk9K5ay M7cvvLKnweOHux4GzZN+yE37YqwNOuCdQaRgb3yU7m0xijyLf2cv1oZup059xjzv0ThcYQKzgBf vL3vq6zNpOYXlcU9vUMOqM3sbr3tgKXgGMsH00Wdzdc0Rb4yO1GCmACSkyHprvfT98LOasK4gtc KU/qf+cwXw== X-Google-Smtp-Source: AGHT+IFHEnqCVvZ4lgttQ4MJzcAW1C2/hsGPw7wH4081/1BgH39bZ2tm2jus/cyF3UabeFRY00xy6Q== X-Received: by 2002:a17:907:7da1:b0:ab2:c1da:b725 with SMTP id a640c23a62f3a-ab38b298c5bmr2677492466b.30.1737648061540; Thu, 23 Jan 2025 08:01:01 -0800 (PST) Received: from [192.168.0.20] ([212.21.159.131]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-ab384c5c47bsm1095940266b.16.2025.01.23.08.01.00 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 23 Jan 2025 08:01:01 -0800 (PST) Message-ID: <0f0ca15c-7d6e-4274-ae4c-20f55340aad6@suse.com> Date: Thu, 23 Jan 2025 18:00:59 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [RESEND PATCH 1/3] x86/mce/inject: Remova call to mce_notify_irq() To: Yazen Ghannam Cc: linux-edac@vger.kernel.org, x86@kernel.org, linux-kernel@vger.kernel.org, bp@alien8.de References: <20250114163722.34850-1-nik.borisov@suse.com> <20250115073640.77099-1-nik.borisov@suse.com> <20250115073640.77099-2-nik.borisov@suse.com> <20250122182438.GA464145@yaz-khff2.amd.com> Content-Language: en-US From: Nikolay Borisov In-Reply-To: <20250122182438.GA464145@yaz-khff2.amd.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 22.01.25 г. 20:24 ч., Yazen Ghannam wrote: > On Wed, Jan 15, 2025 at 09:36:38AM +0200, Nikolay Borisov wrote: > > Hi Nikolay, > > There is a typo in the $SUBJECT. > >> The call is actually a noop because when the MCE is raised the early >> notifier is the only call site that correctly calls mce_notify_irq() >> because it also sets mce_need_notify. So let's just remove this call, >> which allows to unexport mce_notify_irq. >> > > The commit message should be in passive and imperative voice. > > "So let's just remove this..." -> "Remove this..." > >> Signed-off-by: Nikolay Borisov >> --- >> arch/x86/kernel/cpu/mce/core.c | 1 - >> arch/x86/kernel/cpu/mce/inject.c | 1 - >> 2 files changed, 2 deletions(-) >> >> diff --git a/arch/x86/kernel/cpu/mce/core.c b/arch/x86/kernel/cpu/mce/core.c >> index 0dc00c9894c7..23e5e7f7c554 100644 >> --- a/arch/x86/kernel/cpu/mce/core.c >> +++ b/arch/x86/kernel/cpu/mce/core.c >> @@ -1793,7 +1793,6 @@ bool mce_notify_irq(void) >> } >> return false; >> } >> -EXPORT_SYMBOL_GPL(mce_notify_irq); >> >> static void __mcheck_cpu_mce_banks_init(void) >> { >> diff --git a/arch/x86/kernel/cpu/mce/inject.c b/arch/x86/kernel/cpu/mce/inject.c >> index 313fe682db33..06e3cf7229ce 100644 >> --- a/arch/x86/kernel/cpu/mce/inject.c >> +++ b/arch/x86/kernel/cpu/mce/inject.c >> @@ -229,7 +229,6 @@ static int raise_local(void) >> } else if (m->status) { >> pr_info("Starting machine check poll CPU %d\n", cpu); >> raise_poll(m); >> - mce_notify_irq(); > > With this change, there are no users of mce_notify_irq() outside of > mce/core.c. So you could go further and make the function static to > core.c. > > In other words, you could squash the second patch into this one. Thanks, I've incorporated those in V2 that I sent. > > Thanks, > Yazen