[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 for-2.12 13/15] s390x/tcg: use program_interr
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH v1 for-2.12 13/15] s390x/tcg: use program_interrupt_ra() in STSI |
Date: |
Wed, 29 Nov 2017 16:56:48 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 29.11.2017 16:53, Cornelia Huck wrote:
> On Tue, 28 Nov 2017 21:33:23 +0100
> David Hildenbrand <address@hidden> wrote:
>
>> STSI needs some more love, but let's do one step at a time.
>
> Out of curiousity: What else do you want to do?
>
Will post the patch as soon as this series has been picked up.
Here is a preview:
commit a1a86f4db56e307d22918117515184fde540c14b
Author: David Hildenbrand <address@hidden>
Date: Wed Nov 22 18:18:52 2017 +0100
s390x/tcg: fix size + content of STSI blocks
All blocks are 4k in size, which is only true for one of them right now.
Also some reserved fields were wrong, fix it and convert all reserved
fields to speficy u8.
This also fixes the LPAR part output in /proc/sysinfo under LPAR.
Signed-off-by: David Hildenbrand <address@hidden>
commit 41192e74109532d4c106c7b1a6bcf7f989b2f4fd
Author: David Hildenbrand <address@hidden>
Date: Wed Nov 22 18:21:14 2017 +0100
s390x/tcg: STSI overhaul
Current STSI implementation is a mess, so let's rewrite it.
Problems fixed by this patch:
1) The order of exceptions/when recognized is wrong.
2) We have to store to virtual address space, not absolute.
3) Alignment check of the block is missing.
3) The SMP information is not indicated.
While at it:
a) Make the code look nicer
- get rid of nesting levels
- use struct initialization instead of initializing to zero
- rename a misspelled field and rename function code defines
- use a union and have only one write statement
- use cpu_to_beX()
b) Indicate the VM name/extended name + UUID just like KVM does
c) Indicate that all LPAR CPUs we fake are dedicated
d) Add a comment why we fake being a KVM guest
e) Give our guest as default the name "TCGguest"
f) Fake the same CPU information we have in our Guest for all layers
While at it, get rid of "potential_page_fault()" by forwarding the
retaddr properly.
The result is best verified by looking at "/proc/sysinfo" in the guest
when specifying on the qemu command line
-uuid "74738ff5-5367-5958-9aee-98fffdcd1876" \
-name "extra long guest name"
Signed-off-by: David Hildenbrand <address@hidden>
--
Thanks,
David / dhildenb
- Re: [Qemu-devel] [PATCH v1 for-2.12 10/15] s390x/tcg: use program_interrupt_ra() in DIAG, (continued)
- [Qemu-devel] [PATCH v1 for-2.12 11/15] s390x/tcg: use program_interrupt_ra() in per_check_exception(), David Hildenbrand, 2017/11/28
- [Qemu-devel] [PATCH v1 for-2.12 12/15] s390x/tcg: use program_interrupt_ra() in SACF, David Hildenbrand, 2017/11/28
- [Qemu-devel] [PATCH v1 for-2.12 14/15] s390x/tcg: drop program_interrupt(), David Hildenbrand, 2017/11/28
- [Qemu-devel] [PATCH v1 for-2.12 13/15] s390x/tcg: use program_interrupt_ra() in STSI, David Hildenbrand, 2017/11/28
- [Qemu-devel] [PATCH v1 for-2.12 15/15] s390x/tcg: drop potential_page_fault(), David Hildenbrand, 2017/11/28
- Re: [Qemu-devel] [qemu-s390x] [PATCH v1 for-2.12 00/15] s390x/tcg: cleanup and fix program interrupts, Thomas Huth, 2017/11/29