Re: [PATCH v2 4/7] s390/physmem_info: query diag500(STORAGE LIMIT) to support QEMU/KVM memory devices

2024-10-17 Thread Alexander Gordeev
On Mon, Oct 14, 2024 at 04:46:16PM +0200, David Hildenbrand wrote: Hi David! > @@ -157,7 +189,9 @@ unsigned long detect_max_physmem_end(void) > { > unsigned long max_physmem_end = 0; > > - if (!sclp_early_get_memsize(&max_physmem_end)) { > + if (!diag500_storage_limit(&max_physme

Re: [PATCH v2 4/7] s390/physmem_info: query diag500(STORAGE LIMIT) to support QEMU/KVM memory devices

2024-10-17 Thread Alexander Gordeev
> > Why search_mem_end() is not tried in case sclp_early_get_memsize() failed? > > Patch #3 documents that: > > +The storage limit does not indicate currently usable storage, it may > +include holes, standby storage and areas reserved for other means, such > +as memory hotplug or virt

Re: [PATCH v3 3/7] s390/physmem_info: query diag500(STORAGE LIMIT) to support QEMU/KVM memory devices

2024-10-30 Thread Alexander Gordeev
/physmem_info.c| 47 +++- > arch/s390/boot/startup.c | 7 +++-- > arch/s390/include/asm/physmem_info.h | 3 ++ > 3 files changed, 54 insertions(+), 3 deletions(-) Reviewed-by: Alexander Gordeev

Re: [PATCH v2 4/7] s390/physmem_info: query diag500(STORAGE LIMIT) to support QEMU/KVM memory devices

2024-10-30 Thread Alexander Gordeev
> > arch/s390/boot/physmem_info.c| 46 ++-- > > arch/s390/include/asm/physmem_info.h | 3 ++ > > 2 files changed, 46 insertions(+), 3 deletions(-) > > Reviewed-by: Alexander Gordeev Sorry, it supposed to be for v3, so please dismiss this one. > Thanks!

Re: [PATCH v2 4/7] s390/physmem_info: query diag500(STORAGE LIMIT) to support QEMU/KVM memory devices

2024-10-30 Thread Alexander Gordeev
proc/vmcore access so we don't end up reading from unplugged > device blocks). > > Tested-by: Mario Casquero > Signed-off-by: David Hildenbrand > --- > arch/s390/boot/physmem_info.c| 46 ++-- > arch/s390/include/asm/physmem_info.h | 3 ++ > 2 files changed, 46 insertions(+), 3 deletions(-) Reviewed-by: Alexander Gordeev Thanks!

Re: [PATCH 11/12] mm: Remove pXX_devmap callers

2024-09-27 Thread Alexander Gordeev
On Tue, Sep 10, 2024 at 02:14:36PM +1000, Alistair Popple wrote: Hi Alistair, > diff --git a/arch/powerpc/mm/book3s64/pgtable.c > b/arch/powerpc/mm/book3s64/pgtable.c > index 5a4a753..4537a29 100644 > --- a/arch/powerpc/mm/book3s64/pgtable.c > +++ b/arch/powerpc/mm/book3s64/pgtable.c > @@ -193,7

Re: [PATCH v2 4/7] s390/physmem_info: query diag500(STORAGE LIMIT) to support QEMU/KVM memory devices

2024-10-17 Thread Alexander Gordeev
On Thu, Oct 17, 2024 at 02:07:12PM +0200, David Hildenbrand wrote: > On 17.10.24 12:00, David Hildenbrand wrote: > > Well, DIAGNOSE 260 is z/VM only and DIAG 500 is KVM only. So there are > > currently not really any other reasonable ways besides SCLP. > > Correction: Staring at the code again, in