Received-SPF: pass (google.com: domain of linux-kernel+bounces-174966-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <fbc3f489-f8e2-4e1c-ad68-7a72b1bf3e70@nvidia.com>
Date: Thu, 9 May 2024 14:08:24 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 0/1] arch/fault: don't print logs for simulated poison
 errors
To: Axel Rasmussen <axelrasmussen@google.com>,
 Andrew Morton <akpm@linux-foundation.org>, Andy Lutomirski
 <luto@kernel.org>, "Aneesh Kumar K.V" <aneesh.kumar@kernel.org>,
 Borislav Petkov <bp@alien8.de>,
 Christophe Leroy <christophe.leroy@csgroup.eu>,
 Dave Hansen <dave.hansen@linux.intel.com>,
 David Hildenbrand <david@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
 Helge Deller <deller@gmx.de>, Ingo Molnar <mingo@redhat.com>,
 "James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>,
 Liu Shixin <liushixin2@huawei.com>,
 "Matthew Wilcox (Oracle)" <willy@infradead.org>,
 Michael Ellerman <mpe@ellerman.id.au>, Muchun Song <muchun.song@linux.dev>,
 "Naveen N. Rao" <naveen.n.rao@linux.ibm.com>,
 Nicholas Piggin <npiggin@gmail.com>, Oscar Salvador <osalvador@suse.de>,
 Peter Xu <peterx@redhat.com>, Peter Zijlstra <peterz@infradead.org>,
 Suren Baghdasaryan <surenb@google.com>, Thomas Gleixner <tglx@linutronix.de>
Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org,
 linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, x86@kernel.org
References: <20240509203907.504891-1-axelrasmussen@google.com>
Content-Language: en-US
From: John Hubbard <jhubbard@nvidia.com>
In-Reply-To: <20240509203907.504891-1-axelrasmussen@google.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?RXp6L0U4NDlYN1I2VFpFcEFzaFowVWRES2IwRHlYU1hodnl1ZGpHMkZFRXhD?=
 =?utf-8?B?VzY3OW53M3NmMDB6cjIwdnp2VVA3SngvTlpzb3VmQVlyTzFQNmZiK284TGxj?=
 =?utf-8?B?bnhnR2hCdlBPNnFaSlZZMjJtSHpmSjdvRlROeURleCtYbEtKbnEyL0szanB2?=
 =?utf-8?B?b1ZTcDlyd3RmL0JzS3ZyL0toM0F1SlJuSVNTaXZyNTU0U2ptc0o3djZsL05S?=
 =?utf-8?B?WjQ1aXU4WEQ0aGU2ZStaRmI2MUh6V3VRUUdoRDVFSUpvVEZ3Mk9qNWRjVHVE?=
 =?utf-8?B?Q2hWaVVYbEpRTnArakc1VlBtcVRUdWtSZ3RDMVk3VFlPTjVnNHdqaG81YjNC?=
 =?utf-8?B?MGduWFRtcllTekVSNTFobWhNaWpvVXpxT08wMzA2U0h6cUVDTTRqb2hhQUNa?=
 =?utf-8?B?azl0Q1pZczkzWjVwakxVZkRqYmJPYzAzTUZFclIzNlhVOWF2enZtT2JJZTMr?=
 =?utf-8?B?enM1MkYyR0FEOXhFNmQ4dStKTVg2RzBwTE5lRzA2ZW1CK1BhcE5NVWpaUXBD?=
 =?utf-8?B?VVpvSmdDRFExQmZzdnY2UHZpMVhLWUh2T2ltS2ppYVI3THM1dGJMS1RQSk5V?=
 =?utf-8?B?Z0dBL01mWXEvVC9oWnFVN253KzJtYkg4eDgrT3oxQVFzcnphekhPM2x5cEd5?=
 =?utf-8?B?OTdZOEUyVmVua3YzSHlvUmNwbE9kcW1ic3NuVnkvYVFIc09SSVdFMnhDUmlj?=
 =?utf-8?B?VG5GMTZ1Q1QzbVpaU2N1STBCcFp4U0ZBL1M3MVh4cG9JZTN5d205em1WbFhY?=
 =?utf-8?B?U21UQW91dlloZmtrcXRjbzBoVGdTSHRXN3VZNVNBcEhPRENhcVFndU1ReU1W?=
 =?utf-8?B?WEtsejZaT1FPRHpueFpZYkhHVWErdHZRdFJWUHFzVVRmb2dCaW1teHZyN3Jm?=
 =?utf-8?B?OUEvK0ZSTTIxaTBwSWdScUtvY0JMU25veE1FbXRJVWtyZmxRQ0o3Nk0xaThX?=
 =?utf-8?B?TG40c3pjS2ZmV012bW9sN3ArRlA0NW5KcmFrcHgxVDlOV3FmYzZWUy9leFQw?=
 =?utf-8?B?T3h6WFMxUEIyYm5jaEU4M0dJMU9paTRXWkU5NGpmeUZFVXlsQkZIUkZSZjRD?=
 =?utf-8?B?YlVYLzFGTVk1SGE5VlIySXJPbmJML2N1djh6YkRoaG92WjUyalI2WmU4R1Ex?=
 =?utf-8?B?Z2U4aVpyZ3p5WDNRbzNGdEJreXlYeHFCU2NpSFdIWmtnNjRjbDR5emxXakJN?=
 =?utf-8?B?U0x6VVZYdmNpWWRVWjV4OG9IbWFDNTBRbmhRMUYxYW90Vm5XK1Y0U0llbzZT?=
 =?utf-8?B?WW0yaHIxbFo0RXJXL0wzWWNaZ2p1YVVrNXhUUWNZY0s3WjJlZGhXRkN6UDFK?=
 =?utf-8?B?NG8zeTQxV3RwNU00eXROSE9ETE83Z1dJa3k4b01qUE41bG1IS0pwM1o0NVVu?=
 =?utf-8?B?MExDNy8rWTAzSWFLek42OVFGbHJGVWtkbzQ0K3grRkdjQzFJTERCWEdDMVlC?=
 =?utf-8?B?TW8yWjludjVIS3FQQ1diTUpYZGx1Mzg4SGRNTVJWbTVTSHNFRkJ3TSticEhF?=
 =?utf-8?B?NVVXUHZQR1N5V2NQSno4RjlTdkRsZ1hWWlBHdkljL0FhSERUTTF5MTdhcSt0?=
 =?utf-8?B?ZjgzNXkvQjYyeUYzSXVTRG1pNUxZYnpZWVFnWS9wVGZGeWc3M2J0UjBJZGVz?=
 =?utf-8?B?N3JqaXpVNklNdW5mbW9yTkhhaXZFWHNlUThrQ2wwTzVuakt1Z0VXOVFIWGdo?=
 =?utf-8?B?ZTJwTXIvRW5UTllFMElHTUNxRU5GQVJua1hFUEl6Sllka015TDFTa1BRRUdh?=
 =?utf-8?B?eGxtY2c2bnhESzNUUXU2TVpMM2tDVGZGZ1dvdGdaQW9lRmg0NnE4eGJ1Ujcr?=
 =?utf-8?B?QVhmMHVSeVVGa0UxaS82eHJwckErNEg3UUJPTmxYa0VRZlhPeUlSdmswbnlE?=
 =?utf-8?B?bi9MSFlwMzZ3Tk9nVEF6d3A4QTJzbUtuWUpPMnFEMWhvcC9zdURYb0JqY2Nw?=
 =?utf-8?B?c0k3ckRvSUFNdUwvaEx4L1dkeHcvcUpIdlNubFl2aUtyaE4wWm03OEFUSVhT?=
 =?utf-8?B?MVg2ZkhTbVRqMS9TVHpnaHZwcithdERGNEE3emlEZ2RmWEgvaGF3VWcyMXdv?=
 =?utf-8?B?cUt6YVJ4TFNLRWxWWDVkd0xWTmJtUStnOEJCbkp1am5PaHdRd1Q3TzgzWGd5?=
 =?utf-8?B?TXJmZExvZ3FNOVNXS3RtV0xyQ241MG1PN29rbzJLZ05HQlU4anFxaXpOdUlD?=
 =?utf-8?B?Mnc9PQ==?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 650a29bb-8fe3-4d76-8c2f-08dc706c30fe
X-MS-Exchange-CrossTenant-AuthSource: BY5PR12MB4130.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 May 2024 21:08:36.1841
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: nz7/oCRUi8Pk/s0L18OufHAymdPBILZ6BFtkupJ9Xemhc8egwtC8WLhb6/lyRHM3NyhAA3puYtgeEylPUlp0Zw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR12MB7256

On 5/9/24 1:39 PM, Axel Rasmussen wrote:
> This patch expects to be applied on top of both of the following related
> fixes:
> 
> - x86/fault: speed up uffd-unit-test by 10x: rate-limit "MCE: Killing" logs
>    https://lore.kernel.org/r/20240507022939.236896-1-jhubbard@nvidia.com

This got mostly effectively nacked. I disagree with that but do not
intend to "appeal". :)

> - [0/2] Minor fixups for hugetlb fault path
>    https://lore.kernel.org/r/20240509100148.22384-1-osalvador@suse.de
> 
> The latter is in mm-unstable currently, but the former is not (yet?). It
> would need to be taken before this patch for it to apply cleanly.
> 
> Axel Rasmussen (1):
>    arch/fault: don't print logs for simulated poison errors
> 
>   arch/parisc/mm/fault.c   | 7 +++++--
>   arch/powerpc/mm/fault.c  | 6 ++++--
>   arch/x86/mm/fault.c      | 6 ++++--
>   include/linux/mm_types.h | 5 +++++
>   mm/hugetlb.c             | 3 ++-
>   mm/memory.c              | 2 +-
>   6 files changed, 21 insertions(+), 8 deletions(-)
> 
> --
> 2.45.0.118.g7fe29c98d7-goog
> 

thanks,
-- 
John Hubbard
NVIDIA