Re: [PATCH 2/2] x86/MCE: Add command line option to extend MCE Records pool

From: Yazen Ghannam
Date: Mon Feb 12 2024 - 14:43:42 EST

On 2/12/2024 1:58 PM, Luck, Tony wrote:
I need to look more at the genpool code, but I thought I'd ask too.


gen_pool_add_owner() is the code that adds an extra chunk to an existing genpool.

This bit doesn't look obviously safe against a #MC at the wrong moment in the middle of
the list_add_rcu()

list_add_rcu(&chunk->next_chunk, &pool->chunks);

Thanks Tony.

So the concern is not about traversal, but rather that the #MC can break the
list_add_rcu(). Is this correct?