From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from jpms-ob01.noc.sony.co.jp (jpms-ob01.noc.sony.co.jp [211.125.140.164]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 69FA1333730; Fri, 6 Feb 2026 03:45:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=211.125.140.164 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770349504; cv=none; b=HIbZd5rPUKwuaReS9h3xXin6Tq+0PiVm33KSXHv96EdX8YxHVlgl7CKQKV9eI6i5JFiYeRAEyvJeGOhKgZYRs7yNSXeBmxLSJkrYYdAPm6J4rYP9zSPvvoUKNarnwhjLXX8W8n9yitUbX+ED/fog+XfQ4BMMkp/P+65BxL8Jb+E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770349504; c=relaxed/simple; bh=hnE5I4/wZ5Znpn5z3YA2kW21UHAlwuw/qzxayvYooEY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=aa5nF73T+8POQy1jc8L8SCighRKp2Rbf/BraY+A7WcZoPSYg0ruUhnUdCJKgRPzP0iMmHFVEn/6nXscXaUUaTDFUoRiWSn5YxGEqH1/Gn3vVX1buYZixxdbKmPoYhFqKHEL7qXMoqsmF6Sdb87Mo0ws9HZf813g+opQmFO5z6mg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=sony.com; spf=pass smtp.mailfrom=sony.com; dkim=pass (2048-bit key) header.d=sony.com header.i=@sony.com header.b=BitVipMj; arc=none smtp.client-ip=211.125.140.164 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=sony.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sony.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=sony.com header.i=@sony.com header.b="BitVipMj" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sony.com; s=s1jp; t=1770349504; x=1801885504; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=IuPnlqREwJB3pbqSqs+4wA7Rlfs308sbPiaPxE9gRow=; b=BitVipMjANx8XmpHpr7NpGZhZrLBn7nvnAhMDgtFMiwmANVf0j9wOcDb kvvWQ7barMOJpb5QdynBIitZOq7E5V39XRwcYJaUAMfBuzDyo6NksNjdQ 6ydQO72THnbp14AGDTmPXUKzwCP3B4V7bxUpWj/7MP7kNfXNq+9wEJBU2 22GztKGo8xeqqL5FUUYDNpUGdD6TB/g12VDhK8zPbB8rjN4N8Xq+C3Vqs DMnlBjP2sizxuDjiJ0MFlZ9sLwIR429iQ/6QNEDdGjM25bu4ocEbm5/us alc9gEYxL8V/sCX8CMSj+686hm7I+2NCIlGjCTME9hdZ2RQ+oe5s/+Awe g==; Received: from unknown (HELO jpmta-ob02.noc.sony.co.jp) ([IPv6:2001:cf8:0:6e7::7]) by jpms-ob01.noc.sony.co.jp with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Feb 2026 12:45:02 +0900 X-IronPort-AV: E=Sophos;i="6.21,275,1763391600"; d="scan'208";a="578775205" Received: from unknown (HELO JPC00244420) ([IPv6:2001:cf8:1:573:0:dddd:6b3e:119e]) by jpmta-ob02.noc.sony.co.jp with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Feb 2026 12:45:01 +0900 Date: Fri, 6 Feb 2026 12:44:57 +0900 From: Shashank Balaji To: Sohil Mehta Cc: Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Suresh Siddha , "K. Y. Srinivasan" , Haiyang Zhang , Wei Liu , Dexuan Cui , Long Li , Ajay Kaher , Alexey Makhalov , Broadcom internal kernel review list , Jan Kiszka , Paolo Bonzini , Vitaly Kuznetsov , Juergen Gross , Boris Ostrovsky , Ingo Molnar , linux-kernel@vger.kernel.org, linux-hyperv@vger.kernel.org, virtualization@lists.linux.dev, jailhouse-dev@googlegroups.com, kvm@vger.kernel.org, xen-devel@lists.xenproject.org, Rahul Bukte , Daniel Palmer , Tim Bird , stable@vger.kernel.org Subject: Re: [PATCH 1/3] x86/x2apic: disable x2apic on resume if the kernel expects so Message-ID: References: <20260202-x2apic-fix-v1-0-71c8f488a88b@sony.com> <20260202-x2apic-fix-v1-1-71c8f488a88b@sony.com> <0149c37d-7065-4c72-ab56-4cea1a6c15d0@intel.com> <722b53a7-7560-4a1b-ab26-73eeed3dffa5@intel.com> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, Feb 05, 2026 at 03:18:58PM -0800, Sohil Mehta wrote: > Maybe a warning would be useful to encourage firmware to fix this going > forward. I don't have a strong preference on the wording, but how about? > > pr_warn_once("x2apic unexpectedly re-enabled by the firmware during > resume.\n"); That works > A few nits: > > For the code comments, you can use more of the line width. Generally, 72 > (perhaps even 80) chars is okay for comments dependent on the code in > the vicinity. > > The tip tree has slightly unique preferences, such as capitalizing the > first word of the patch title. > > Please refer: > https://www.kernel.org/doc/html/latest/process/maintainer-tip.html#patch-submission-notes Thanks! I noticed that I also didn't use '()' for function names in the commit message. I'll fix all these and add the pr_warn_once in v2.