From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59198) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dxtnh-0007n3-0W for qemu-devel@nongnu.org; Fri, 29 Sep 2017 07:49:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dxtnd-0000Zc-4g for qemu-devel@nongnu.org; Fri, 29 Sep 2017 07:49:49 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46418) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dxtnc-0000Yr-Ul for qemu-devel@nongnu.org; Fri, 29 Sep 2017 07:49:45 -0400 Date: Fri, 29 Sep 2017 13:49:38 +0200 From: Cornelia Huck Message-ID: <20170929134938.18fae231.cohuck@redhat.com> In-Reply-To: <20170927170027.8539-1-david@redhat.com> References: <20170927170027.8539-1-david@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC 0/3] s390x/tcg: LAP support using immediate TLB invalidation List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Hildenbrand Cc: qemu-devel@nongnu.org, thuth@redhat.com, Christian Borntraeger , Alexander Graf , Richard Henderson List-ID: On Wed, 27 Sep 2017 19:00:24 +0200 David Hildenbrand wrote: > Details about Low-Address Protection can be found in description of > patch 1 and 2. It is basically a subpage protection of the first two > pages of every address space (for which it is enabled). > > We can achieve this by simply directly invalidating the TLB entry and > therefore forcing every write accesses onto these two pages into the slow > path. > > With this patch, I can boot Linux just fine (which uses LAP). This also > makes all related kvm-unit-tests that we have pass. > > The checks are working that good, that I discovered a STFL bug. STFL > stores into the low addresses but low-address protection does explicitly > not apply. The Linux kernel calls STFL while LAP is active. So without > patch nr 3, booting Linux will fail. (this change is also part of a patch > of my SMP series). I fear I have lost track a bit with all those patches floating around. IIUC, patch 3 fixes a real bug that is only exposed by your LAP changes. It used to be part of the stfl changes in v1 of your smp series but is no longer in v2. So, is this a patch that can be applied to current s390-next? > > Based on: https://github.com/cohuck/qemu.git s390-next > Available on: https://github.com/dhildenb/qemu.git s390x_lap > > > David Hildenbrand (3): > accel/tcg: allow to invalidate a write TLB entry immediately > s390x/tcg: low-address protection support > s390x/tcg: make STFL store into the lowcore > > accel/tcg/cputlb.c | 5 ++- > accel/tcg/softmmu_template.h | 4 +- > include/exec/cpu-all.h | 3 ++ > target/s390x/excp_helper.c | 3 +- > target/s390x/helper.h | 2 +- > target/s390x/mem_helper.c | 8 ---- > target/s390x/misc_helper.c | 7 +++- > target/s390x/mmu_helper.c | 96 ++++++++++++++++++++++++++++---------------- > 8 files changed, 78 insertions(+), 50 deletions(-) >