Re: [PATCH v1 09/13] KVM: s390: stop using lists to keep track of used dat tables

Janosch Frank
2025-01-15T09:01:13Z

On 1/8/25 7:14 PM, Claudio Imbrenda wrote:
> Until now, every dat table allocated to map a guest was put in a
> linked list. The page->lru field of struct page was used to keep track
> of which pages were being used, and when the gmap is torn down, the
> list was walked and all pages freed.
>
> This patch gets rid of the usage of page->lru. Page table are now freed
> by recursively walking the dat table tree.
>
> Since s390_unlist_old_asce() becomes useless now, remove it.
>
> Signed-off-by: Claudio Imbrenda <imbrenda@linux.ibm.com>

The lists and page indexes are likely a huge part of the vsie problems
we're currently experiencing. I'm hopeful that this series resolves
those issues and makes new additions less painful.

Nice work and thanks for tackling this!