public inbox for linux-s390@vger.kernel.org
 help / color / mirror / Atom feed
From: Claudio Imbrenda <imbrenda@linux.ibm.com>
To: "Christoph Schlameuss" <schlameuss@linux.ibm.com>
Cc: <kvm@vger.kernel.org>,
	"Christian Borntraeger" <borntraeger@linux.ibm.com>,
	"Janosch Frank" <frankja@linux.ibm.com>,
	"David Hildenbrand" <david@redhat.com>,
	"Heiko Carstens" <hca@linux.ibm.com>,
	"Vasily Gorbik" <gor@linux.ibm.com>,
	"Alexander Gordeev" <agordeev@linux.ibm.com>,
	"Sven Schnelle" <svens@linux.ibm.com>,
	<linux-s390@vger.kernel.org>
Subject: Re: [PATCH 2/3] KVM: s390: Always allocate esca_block
Date: Thu, 15 May 2025 15:15:11 +0200	[thread overview]
Message-ID: <20250515151511.101fd385@p-imbrenda> (raw)
In-Reply-To: <D9WR9VL90Z44.34FLLVUM55PGQ@linux.ibm.com>

On Thu, 15 May 2025 15:07:18 +0200
"Christoph Schlameuss" <schlameuss@linux.ibm.com> wrote:

> On Thu May 15, 2025 at 1:24 PM CEST, Claudio Imbrenda wrote:
> > On Wed, 14 May 2025 18:34:50 +0200
> > Christoph Schlameuss <schlameuss@linux.ibm.com> wrote:
> >  
> >> Instead of allocating a BSCA and upgrading it for PV or when adding the
> >> 65th cpu we can always use the ESCA.
> >> 
> >> The only downside of the change is that we will always allocate 4 pages
> >> for a 248 cpu ESCA instead of a single page for the BSCA per VM.
> >> In return we can delete a bunch of checks and special handling depending
> >> on the SCA type as well as the whole BSCA to ESCA conversion.
> >> 
> >> As a fallback we can still run without SCA entries when the SIGP
> >> interpretation facility is not available.  
> >
> > s/is/or BSCA are/
> >  
> 
> With this merged we do no longer care for the BSCA. So I will change this to
> s/is/or ESCA are/

that was a typo, I meant ESCA, of course :)

> 
> I will also apply the other changes for the next version. But I will wait for
> more feedback before sending that. And I will run checkpatch with the strict
> option, which I clearly did not do.
> 
> [...]


  reply	other threads:[~2025-05-15 13:15 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-14 16:34 [PATCH 0/3] KVM: s390: Use ESCA instead of BSCA at VM init Christoph Schlameuss
2025-05-14 16:34 ` [PATCH 1/3] KVM: s390: Set KVM_MAX_VCPUS to 256 Christoph Schlameuss
2025-05-15 11:14   ` Claudio Imbrenda
2025-05-14 16:34 ` [PATCH 2/3] KVM: s390: Always allocate esca_block Christoph Schlameuss
2025-05-15 11:24   ` Claudio Imbrenda
2025-05-15 13:07     ` Christoph Schlameuss
2025-05-15 13:15       ` Claudio Imbrenda [this message]
2025-05-14 16:34 ` [PATCH 3/3] KVM: s390: Specify kvm->arch.sca as esca_block Christoph Schlameuss
2025-05-15 11:25   ` Claudio Imbrenda
2025-05-14 17:34 ` [PATCH 0/3] KVM: s390: Use ESCA instead of BSCA at VM init David Hildenbrand
2025-05-15  7:38   ` Christoph Schlameuss
2025-05-15 11:26 ` Claudio Imbrenda

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250515151511.101fd385@p-imbrenda \
    --to=imbrenda@linux.ibm.com \
    --cc=agordeev@linux.ibm.com \
    --cc=borntraeger@linux.ibm.com \
    --cc=david@redhat.com \
    --cc=frankja@linux.ibm.com \
    --cc=gor@linux.ibm.com \
    --cc=hca@linux.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=schlameuss@linux.ibm.com \
    --cc=svens@linux.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox