|
From: | Cédric Le Goater |
Subject: | Re: [PATCH v9 01/10] s390x/cpus: Make absence of multithreading clear |
Date: | Tue, 11 Oct 2022 09:28:17 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.3.1 |
On 10/11/22 09:21, Pierre Morel wrote:
On 9/28/22 18:28, Cédric Le Goater wrote:On 9/28/22 18:16, Pierre Morel wrote:More thinking about this I will drop this patch for backward compatibility and in topology masks treat CPUs as being cores*threadsyes. You never know, people might have set threads=2 in their domain file (like me). You could give the user a warning though, with warn_report().More thinking, I come back to the first idea after Daniel comment and protect the change with a new machine type version.
yes. That would be another machine class attribute to set in the new machine, may be 'max_threads' to compare with the user provided value. C.
Thanks, C.
[Prev in Thread] | Current Thread | [Next in Thread] |