Re: [PATCH] kexec: Include kernel-end even without crashkernel

2025-09-07 Thread Sourabh Jain
t might help others discover this fix if they run into the same issue. Rest look good to me. Reviewed-by: Sourabh Jain

powerpc/kexec_file: print configured kernel command line

2025-07-25 Thread Sourabh Jain
elfcorehdr=0x22e3 test1 test2 <--- New kexec_elf: Loaded initrd at 0x22eb Cc: Madhavan Srinivasan Cc: Michael Ellerman Signed-off-by: Sourabh Jain --- arch/powerpc/kexec/elf_64.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/powerpc/kexec/elf_64.c b/arch/powerpc/kexec/e

Re: [PATCH v3 3/6] powerpc: fadump: use lock guard for mutex

2025-05-07 Thread Sourabh Jain
scenarios to release the reserved memory: 1. After dump collection 2. When fadump fails to process the dump So even if the compiler messes up something here, there is no impact on dump collection as such. So changes looks good to me: Reviewed-by: Sourabh Jain

kexec is not working in 6.15-rc1

2025-04-10 Thread Sourabh Jain
a226b2de587e22c17f88b72cede3be] crypto: scompress - Fix incorrect stream freeing git bisect good 5a06ef1f8da226b2de587e22c17f88b72cede3be # first bad commit: [27b13425349e94ad77b174b032674097cab241c8] crypto: api - Call crypto_alg_put in crypto_unregister_alg Thanks, Sourabh Jain

Re: [PATCH] crypto: scomp - Fix null-pointer deref when freeing streams

2025-04-09 Thread Sourabh Jain
Hi Herbert, On 08/04/25 09:50, Herbert Xu wrote: On Mon, Apr 07, 2025 at 11:49:27PM +0530, Sourabh Jain wrote: [   90.892796] NIP [c0845eb0] scomp_free_streams+0x6c/0xe8 [   90.892803] LR [c0845ee0] scomp_free_streams+0x9c/0xe8 Looks like I never tested 842 which curiously

Re: [PATCH] crypto: scomp - Fix null-pointer deref when freeing streams

2025-04-07 Thread Sourabh Jain
Hello Herbert, On 08/04/25 09:50, Herbert Xu wrote: On Mon, Apr 07, 2025 at 11:49:27PM +0530, Sourabh Jain wrote: [   90.892796] NIP [c0845eb0] scomp_free_streams+0x6c/0xe8 [   90.892803] LR [c0845ee0] scomp_free_streams+0x9c/0xe8 Looks like I never tested 842 which curiously

Re: [PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-03-06 Thread Sourabh Jain
On 06/03/25 00:47, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: Hello Ritesh, On 04/03/25 10:27, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: Hello Ritesh, Thanks for the review. On 02/03/25 12:05, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The fadump kernel

Re: [PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-03-05 Thread Sourabh Jain
Hello Ritesh, On 04/03/25 10:27, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: Hello Ritesh, Thanks for the review. On 02/03/25 12:05, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The fadump kernel boots with limited memory solely to collect the kernel core dump. Having

Re: [PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-03-04 Thread Sourabh Jain
Hello Ritesh, On 04/03/25 10:27, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: Hello Ritesh, Thanks for the review. On 02/03/25 12:05, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The fadump kernel boots with limited memory solely to collect the kernel core dump. Having

Re: [PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-03-02 Thread Sourabh Jain
Hello Ritesh, Thanks for the review. On 02/03/25 12:05, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The fadump kernel boots with limited memory solely to collect the kernel core dump. Having gigantic hugepages in the fadump kernel is of no use. Sure got it. Many times, the fadump

Re: [PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-03-02 Thread Sourabh Jain
Hello Ritesh, Thanks for the review. On 02/03/25 12:05, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The fadump kernel boots with limited memory solely to collect the kernel core dump. Having gigantic hugepages in the fadump kernel is of no use. Sure got it. Many times, the fadump

Re: [PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-02-23 Thread Sourabh Jain
Hello Maddy, Any feedback on this patch? Thanks, Sourabh Jain On 28/01/25 10:03, Sourabh Jain wrote: The fadump kernel boots with limited memory solely to collect the kernel core dump. Having gigantic hugepages in the fadump kernel is of no use. Many times, the fadump kernel encounters OOM

Re: [linux-next-20250212] syscall kexec_file_load not available

2025-02-14 Thread Sourabh Jain
Hello, On 14/02/25 12:15, Sourabh Jain wrote: Hello Hari, On 14/02/25 12:02, Hari Bathini wrote: On 13/02/25 8:34 pm, Venkat Rao Bagalkote wrote: Greetings!!!  From kernel next-20250210, I am observing syscall kexec_file_load not available, there by kdump service is failing to start

Re: [linux-next-20250212] syscall kexec_file_load not available

2025-02-13 Thread Sourabh Jain
On 14/02/25 12:38, Hari Bathini wrote: On 14/02/25 12:15 pm, Sourabh Jain wrote: Hello Hari, On 14/02/25 12:02, Hari Bathini wrote: On 13/02/25 8:34 pm, Venkat Rao Bagalkote wrote: Greetings!!!  From kernel next-20250210, I am observing syscall kexec_file_load not available, there

Re: [linux-next-20250212] syscall kexec_file_load not available

2025-02-13 Thread Sourabh Jain
alue. I am suspecting kexec_file_load returned early form sanity_check_segment_list() because the segment is 0x0. I am investigating further to find how segment.mem for all segment is 0x0. Thanks, Sourabh Jain

Re: [linux-next-20250212] syscall kexec_file_load not available

2025-02-13 Thread Sourabh Jain
. Thanks for reporting the issue. I will debug and find what went wrong with next-20250210 or the kexec you used to load the kdump kernel. Thanks, Sourabh Jain Logs: [root@ltc-zzci-1 ~]# kexec -p --initrd=/boot/initramfs-6.14.0-rc2-next-20250212kdump.img /boot/vmlinuz-6.14.0-rc2-next-20250212 -c

Re: [next-20250212] FStests generic/451 on EXT4 FS resulting in kernel OOPs

2025-02-12 Thread Sourabh Jain
Hello Venkat, Thanks for reporting the issue. I noticed that system rebooted after kernel panic. Just wondering did you configured kdump/fadump? Thanks, Sourabh Jain On 13/02/25 11:21, Venkat Rao Bagalkote wrote: Greetings!!! I am observing kernel OOPs, while running FStests generic/451

Re: [PATCH v3 2/4] powerpc/fadump: fix additional param memory reservation for HASH MMU

2025-02-10 Thread Sourabh Jain
On 10/02/25 12:48, Hari Bathini wrote: Hi Sourabh, On 23/01/25 5:12 pm, Sourabh Jain wrote: Commit 683eab94da75bc ("powerpc/fadump: setup additional parameters for dump capture kernel") introduced the additional parameter feature in fadump for HASH MMU with the understanding that

Re: [PATCH v3 3/7] crash: Let arch decide usable memory range in reserved area

2025-01-31 Thread Sourabh Jain
Hello Baoquan, On 01/02/25 09:52, Baoquan he wrote: On 01/31/25 at 05:08pm, Sourabh Jain wrote: Although the crashkernel area is reserved, on architectures like PowerPC, it is possible for the crashkernel reserved area to contain components like RTAS, TCE, OPAL, etc. To avoid placing kexec

[PATCH v3 5/7] powerpc/crash: preserve user-specified memory limit

2025-01-31 Thread Sourabh Jain
: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Acked-by: Hari Bathini Reviewed-by: Mahesh Salgaonkar Signed-off-by: Sourabh Jain --- arch/powerpc/kexec/core.c | 8 1 file changed, 8 deletions(-) diff --git a/arch/powerpc/kexec/co

[PATCH v3 0/7] powerpc/crash: use generic crashkernel reservation

2025-01-31 Thread Sourabh Jain
: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Cc: sourabhj...@linux.ibm.com Sourabh Jain (7): kexec: Initialize ELF lowest address to ULONG_MAX crash: remove an unuse

[PATCH v3 6/7] powerpc: insert System RAM resource to prevent crashkernel conflict

2025-01-31 Thread Sourabh Jain
le of handling `System RAM` and `Crash Kernel` in the above format. The same format is used in other architectures. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Soura

[PATCH v3 7/7] powerpc/crash: use generic crashkernel reservation

2025-01-31 Thread Sourabh Jain
ts own function name get_crash_base() for better readability and maintainability. Cc: Andrew Morton Cc: Baoquan he CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Acked-by: Hari Bathini Reviewed-by: Mahesh Salgaonkar Signed-off-b

[PATCH v3 4/7] powerpc/crash: Use generic APIs to locate memory hole for kdump

2025-01-31 Thread Sourabh Jain
an Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/powerpc/include/asm/kexec.h | 6 +- arch/powerpc/kexec/file_load_64.c | 259 ++ 2 files changed, 13 insertions(+), 252 deletions(-) diff --git a/arch/powerpc/include/asm/kexec.h b/ar

[PATCH v3 3/7] crash: Let arch decide usable memory range in reserved area

2025-01-31 Thread Sourabh Jain
ocate memory hole for kdump. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- include/linux/kexec.h | 9 + kernel/kexec_f

[PATCH v3 1/7] kexec: Initialize ELF lowest address to ULONG_MAX

2025-01-31 Thread Sourabh Jain
el Ellerman Cc: ke...@lists.infradead.org Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-ker...@vger.kernel.org Acked-by: Hari Bathini Acked-by: Baoquan He Signed-off-by: Sourabh Jain --- kernel/kexec_elf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/kexec_elf.c b/ker

[PATCH v3 2/7] crash: remove an unused argument from reserve_crashkernel_generic()

2025-01-31 Thread Sourabh Jain
-ker...@vger.kernel.org Acked-by: Hari Bathini Acked-by: Baoquan He Signed-off-by: Sourabh Jain --- arch/arm64/mm/init.c | 6 ++ arch/loongarch/kernel/setup.c | 5 ++--- arch/riscv/mm/init.c | 6 ++ arch/x86/kernel/setup.c | 6 ++ include/linux

[PATCH v4] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-01-27 Thread Sourabh Jain
$ cat /proc/meminfo | grep -i "hugetlb" -- Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Cc: Ritesh Harjani (IBM)" Reviewed-by: Christophe Leroy Signed-off-by: Sourabh Jain --- Changelog: v1: https:/

Re: [PATCH v3] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-01-27 Thread Sourabh Jain
Hello Christophe, On 27/01/25 13:23, Christophe Leroy wrote: Le 25/01/2025 à 11:49, Sourabh Jain a écrit : The fadump kernel boots with limited memory solely to collect the kernel core dump. Having gigantic hugepages in the fadump kernel is of no use. Many times, the fadump kernel

Re: [PATCH v2] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-01-27 Thread Sourabh Jain
On 27/01/25 13:18, Christophe Leroy wrote: Le 25/01/2025 à 10:32, Sourabh Jain a écrit : Hello Christophe On 24/01/25 19:44, Christophe Leroy wrote: Le 24/01/2025 à 11:32, Sourabh Jain a écrit : The fadump kernel boots with limited memory solely to collect the kernel core dump

Re: [PATCH v3 4/4] Documentation/powerpc/fadump: add additional parameter feature details

2025-01-26 Thread Sourabh Jain
Hello Mahesh, On 26/01/25 17:31, Mahesh J Salgaonkar wrote: On 2025-01-23 17:12:54 Thu, Sourabh Jain wrote: Update the fadump document to include details about the fadump additional parameter feature. The document includes the following: - Significance of the feature - How to use it

[PATCH v3] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-01-25 Thread Sourabh Jain
ini Cc: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Cc: Ritesh Harjani (IBM)" Signed-off-by: Sourabh Jain --- Changelog: v1: https://lore.kernel.org/all/20250121150419.1342794-1-sourabhj...@linux.ibm.com/ v2: https://lore.kernel.org/all/20250124103220.111303-1-s

Re: [PATCH v2] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-01-25 Thread Sourabh Jain
Hello Christophe On 24/01/25 19:44, Christophe Leroy wrote: Le 24/01/2025 à 11:32, Sourabh Jain a écrit : The fadump kernel boots with limited memory solely to collect the kernel core dump. Having gigantic hugepages in the fadump kernel is of no use. Many times, the fadump kernel encounters

[PATCH v2] powerpc/hugetlb: Disable gigantic hugepages if fadump is active

2025-01-24 Thread Sourabh Jain
an Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Cc: Ritesh Harjani (IBM)" Signed-off-by: Sourabh Jain --- Changelog: v1: https://lore.kernel.org/all/20250121150419.1342794-1-sourabhj...@linux.ibm.com/ v2: - disable gigantic hugepage in arch code, arch_hugetlb_valid_size() --- arch/

Re: [PATCH v2 6/6] crash: option to let arch decide mem range is usable

2025-01-24 Thread Sourabh Jain
Hello Hari, On 24/01/25 15:22, Hari Bathini wrote: Hi Sourabh, On 21/01/25 5:24 pm, Sourabh Jain wrote: On PowerPC, the memory reserved for the crashkernel can contain components like RTAS, TCE, OPAL, etc., which should be avoided when loading kexec segments into crashkernel memory. Due to

Re: [PATCH v2] powerpc: increase MIN RMA size for CAS negotiation

2025-01-23 Thread Sourabh Jain
above patch series and a changelog explaining additional details in the cover letter. Please let me know if you have queries regarding this. Thanks, Sourabh Jain

Re: [PATCH 2/2] powerpc/fadump: fix additional param memory reservation for HASH MMU

2025-01-23 Thread Sourabh Jain
Hello Mahesh, On 23/01/25 12:28, Mahesh J Salgaonkar wrote: On 2025-01-20 23:05:00 Mon, Sourabh Jain wrote: Commit 683eab94da75bc ("powerpc/fadump: setup additional parameters for dump capture kernel") introduced the additional parameter feature in fadump for HASH MMU with the und

Re: [PATCH v2 5/6] powerpc/crash: use generic crashkernel reservation

2025-01-23 Thread Sourabh Jain
Hello Hari, On 23/01/25 16:15, Hari Bathini wrote: Hi Sourabh, On 21/01/25 5:24 pm, Sourabh Jain wrote: Commit 0ab97169aa05 ("crash_core: add generic function to do reservation") added a generic function to reserve crashkernel memory. So let's use the same function on powerpc

Re: [PATCH v2 3/6] crash: let arch decide crash memory export to iomem_resource

2025-01-23 Thread Sourabh Jain
Hello Hari, On 23/01/25 15:56, Hari Bathini wrote: Hi Sourabh, On 21/01/25 5:24 pm, Sourabh Jain wrote: insert_crashkernel_resources() adds crash memory to iomem_resource if generic crashkernel reservation is enabled on an architecture. On PowerPC, system RAM is added to iomem_resource

[PATCH v3 4/4] Documentation/powerpc/fadump: add additional parameter feature details

2025-01-23 Thread Sourabh Jain
: Madhavan Srinivasan Cc: Michael Ellerman Cc: Mahesh Salgaonkar Signed-off-by: Sourabh Jain --- Documentation/ABI/testing/sysfs-kernel-fadump | 3 ++- .../arch/powerpc/firmware-assisted-dump.rst | 22 +++ 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a

[PATCH v3 3/4] powerpc: increase MIN RMA size for CAS negotiation

2025-01-23 Thread Sourabh Jain
user_thread+0x14/0x1c [0.68] --- interrupt: 0 at 0x0 [0.784238] pstore: backend (nvram) writing error (-1) [0.790447] Rebooting in 10 seconds.. Cc: Brian King Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Meghana Prakash Cc: Michael Ellerman Cc: Mahesh Salgaonkar Cc: Sourabh Jain Signe

[PATCH v3 2/4] powerpc/fadump: fix additional param memory reservation for HASH MMU

2025-01-23 Thread Sourabh Jain
rman Reviewed-by: Mahesh Salgaonkar Signed-off-by: Sourabh Jain --- arch/powerpc/kernel/fadump.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c index 4b371c738213..26e3d151e048 100644 --- a/arc

[PATCH v3 1/4] powerpc: export MIN RMA size

2025-01-23 Thread Sourabh Jain
Ellerman Reviewed-by: Mahesh Salgaonkar Signed-off-by: Sourabh Jain --- arch/powerpc/include/asm/prom.h | 2 ++ arch/powerpc/kernel/prom_init.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/include/asm/prom.h b/arch/powerpc/include/asm/prom.h index c0107d8ddd8c..

[PATCH v3 0/4] powerpc: increase MIN RMA size for CAS negotiation

2025-01-23 Thread Sourabh Jain
changing for HASH MMU, the corresponding FADump documentation is also updated to reflect this change in patch 04/04. - Add Reviewed-by tag Cc: Avnish Chouhan Cc: Brian King Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Michael Ellerman Cc: Mahesh Salgaonkar Cc: Sourabh Jain Avnish Chouhan (1

Re: [PATCH v2 1/6] kexec: Initialize ELF lowest address to ULONG_MAX

2025-01-23 Thread Sourabh Jain
Hello Hari, On 23/01/25 15:34, Hari Bathini wrote: On 21/01/25 5:24 pm, Sourabh Jain wrote: kexec_elf_load() loads an ELF executable and sets the address of the lowest PT_LOAD section to the address held by the lowest_load_addr function argument. To determine the lowest PT_LOAD address, a

Re: [PATCH v2 4/6] powerpc/kdump: preserve user-specified memory limit

2025-01-23 Thread Sourabh Jain
Hello Hari, On 23/01/25 16:00, Hari Bathini wrote: On 21/01/25 5:24 pm, Sourabh Jain wrote: Commit 59d58189f3d9 ("crash: fix crash memory reserve exceed system memory bug") fails crashkernel parsing if the crash size is found to be higher than system RAM, which makes the me

Re: [PATCH] mm/hugetlb: bring gigantic page allocation under hugepages_supported()

2025-01-22 Thread Sourabh Jain
Hello Gerald, On 22/01/25 19:36, Gerald Schaefer wrote: On Tue, 21 Jan 2025 20:34:19 +0530 Sourabh Jain wrote: Despite having kernel arguments to enable gigantic hugepages, this provides a way for the architecture to disable gigantic hugepages on the fly, similar to what we do for hugepages

Re: [PATCH v2] powerpc: increase MIN RMA size for CAS negotiation

2025-01-22 Thread Sourabh Jain
. Thanks, Sourabh Jain

[PATCH] mm/hugetlb: bring gigantic page allocation under hugepages_supported()

2025-01-21 Thread Sourabh Jain
Signed-off-by: Sourabh Jain --- To evaluate the impact of this change on architectures other than PowerPC, I did the following analysis: For architectures where hugepages_supported() is not redefined, it depends on HPAGE_SHIFT, which is found to be a constant. It is mostly initialized to

[PATCH v2 6/6] crash: option to let arch decide mem range is usable

2025-01-21 Thread Sourabh Jain
, and remove the PowerPC functions to locate holes. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/powerpc/include/asm/ke

[PATCH v2 5/6] powerpc/crash: use generic crashkernel reservation

2025-01-21 Thread Sourabh Jain
onkar Signed-off-by: Sourabh Jain --- arch/powerpc/Kconfig | 3 + arch/powerpc/include/asm/crash_reserve.h | 18 + arch/powerpc/include/asm/kexec.h | 4 +- arch/powerpc/kernel/prom.c | 2 +- arch/powerpc/kexec/core.c| 90 ++---

[PATCH v2 4/6] powerpc/kdump: preserve user-specified memory limit

2025-01-21 Thread Sourabh Jain
ari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Reviewed-by: Mahesh Salgaonkar Signed-off-by: Sourabh Jain --- arch/powerpc/kexec/core.c | 8 1 file changed, 8 deletions(-) diff --git a/arch/powerpc/kexec/co

[PATCH v2 3/6] crash: let arch decide crash memory export to iomem_resource

2025-01-21 Thread Sourabh Jain
nel memory should be added to iomem_resource. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- include/linux/

[PATCH v2 2/6] crash: remove an unused argument from reserve_crashkernel_generic()

2025-01-21 Thread Sourabh Jain
...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Acked-by: Baoquan He Signed-off-by: Sourabh Jain --- arch/arm64/mm/init.c | 6 ++ arch/loongarch/kernel/setup.c | 5 ++--- arch/riscv/mm/init.c | 6 ++ arch/x86/kernel/setup.c | 6 ++ include/linux/crash_reserve.h

[PATCH v2 1/6] kexec: Initialize ELF lowest address to ULONG_MAX

2025-01-21 Thread Sourabh Jain
Salgaonkar Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-ker...@vger.kernel.org Acked-by: Baoquan He Signed-off-by: Sourabh Jain --- kernel/kexec_elf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/kexec_elf.c b/kernel/ke

[PATCH v2 0/6] powerpc/crash: use generic crashkernel reservation

2025-01-21 Thread Sourabh Jain
is not removed from patch series. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Cc: sourabhj...@linux.ibm.com Sourabh Jain (6): kexec: Initialize ELF lowest address to ULONG_MAX

[PATCH 2/2] powerpc/fadump: fix additional param memory reservation for HASH MMU

2025-01-20 Thread Sourabh Jain
as the bootloader can use memory up to the MIN_RMA size. The range should be between MIN_RMA and the RMA size (ppc64_rma_size), and it must not overlap with the fadump reserved area. Cc: Avnish Chouhan Cc: Brian King Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Michael Ellerman Cc: Mahesh Salgaonka

[PATCH 1/2] powerpc: export MIN RMA size

2025-01-20 Thread Sourabh Jain
"powerpc/fadump: fix additional param memory reservation for HASH MMU") can identify the correct memory range for the additional parameter feature in fadump for HASH MMU. Cc: Avnish Chouhan Cc: Brian King Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Michael Ellerman Cc: Mahesh Salgaonka

[PATCH 0/2] powerpc/fadump: fix additional parameter for HASH MMU

2025-01-20 Thread Sourabh Jain
scribed above. Note: This patch series is dependent on the below patch: https://lore.kernel.org/all/20241206065545.14815-1-avn...@linux.ibm.com/ Cc: Avnish Chouhan Cc: Brian King Cc: Hari Bathini Cc: Madhavan Srinivasan Cc: Michael Ellerman Cc: Mahesh Salgaonkar Cc: Sourabh Jain Sourabh

Re: [PATCH v3 1/2] powerpc/fadump: allocate memory for additional parameters early

2025-01-17 Thread Sourabh Jain
Hello Christophe, On 17/01/25 17:43, Christophe Leroy wrote: Le 13/11/2024 à 08:06, Sourabh Jain a écrit : From: Hari Bathini Memory for passing additional parameters to fadump capture kernel is allocated during subsys_initcall level, using memblock. But as slab is already available by

Re: [PATCH] fadump: Use str_yes_no() helper in fadump_show_config()

2025-01-13 Thread Sourabh Jain
("HPTE region size : %lx\n", fw_dump.hpte_region_size); Yes, it is better to use `str_yes_no()` instead of hard-coded strings. I have also tested your patch, and everything is working fine. Reviewed-by: Sourabh Jain Thanks, Sourabh Jain

Re: [PATCH v2] powerpc: increase MIN RMA size for CAS negotiation

2025-01-09 Thread Sourabh Jain
and boots the fadump kernel to capture the dump. If RMA is set to 1GB, it will allow GRUB to use memory up to 1GB. In such a case, there is a possibility that GRUB could corrupt the fadump reserved area. Thanks, Sourabh Jain

Re: [PATCH] kexec: Initialize ELF lowest address to ULONG_MAX

2025-01-08 Thread Sourabh Jain
Hello Andrew, On 09/01/25 10:58, Andrew Morton wrote: On Thu, 9 Jan 2025 09:42:14 +0530 Sourabh Jain wrote: Hello Baoquan and Eric, On 12/12/24 08:25, Baoquan he wrote: On 12/10/24 at 02:43pm, Sourabh Jain wrote: kexec_elf_load() loads an ELF executable and sets the address of the

Re: [PATCH] kexec: Initialize ELF lowest address to ULONG_MAX

2025-01-08 Thread Sourabh Jain
Hello Baoquan and Eric, On 12/12/24 08:25, Baoquan he wrote: On 12/10/24 at 02:43pm, Sourabh Jain wrote: kexec_elf_load() loads an ELF executable and sets the address of the lowest PT_LOAD section to the address held by the lowest_load_addr function argument. To determine the lowest PT_LOAD

Re: [PATCH RESEND v1 1/5] crash: remove an unused argument from reserve_crashkernel_generic()

2025-01-08 Thread Sourabh Jain
Hello Baoquan, On 08/01/25 16:46, Baoquan he wrote: On 01/08/25 at 03:44pm, Sourabh Jain wrote: cmdline argument is not used in reserve_crashkernel_generic() so remove it. Correspondingly, all the callers have been updated as well. No functional change intended. Cc: Andrew Morton Cc

Re: [PATCH RESEND v1 4/5] powerpc/crash: use generic crashkernel reservation

2025-01-08 Thread Sourabh Jain
Hello Mahesh, On 08/01/25 22:35, Mahesh J Salgaonkar wrote: On 2025-01-08 15:44:57 Wed, Sourabh Jain wrote: Commit 0ab97169aa05 ("crash_core: add generic function to do reservation") added a generic function to reserve crashkernel memory. So let's use the same function on pow

Re: [PATCH RESEND v1 3/5] powerpc/kdump: preserve user-specified memory limit

2025-01-08 Thread Sourabh Jain
Hello Mahesh, On 08/01/25 22:36, Mahesh J Salgaonkar wrote: On 2025-01-08 15:44:56 Wed, Sourabh Jain wrote: Commit 59d58189f3d9 ("crash: fix crash memory reserve exceed system memory bug") fails crashkernel parsing if the crash size is found to be higher than system RAM, which

Re: [PATCH RESEND v1 5/5] crash: option to let arch decide mem range is usable

2025-01-08 Thread Sourabh Jain
On 08/01/25 17:08, Baoquan he wrote: On 01/08/25 at 03:44pm, Sourabh Jain wrote: ...snip... diff --git a/include/linux/kexec.h b/include/linux/kexec.h index f0e9f8eda7a3..407f8b0346aa 100644 --- a/include/linux/kexec.h +++ b/include/linux/kexec.h @@ -205,6 +205,15 @@ static inline int

Re: [PATCH RESEND v1 2/5] crash: let arch decide crash memory export to iomem_resource

2025-01-08 Thread Sourabh Jain
Hello Baoquan On 08/01/25 16:55, Baoquan he wrote: Hi, On 01/08/25 at 03:44pm, Sourabh Jain wrote: insert_crashkernel_resources() adds crash memory to iomem_resource if generic crashkernel reservation is enabled on an architecture. On PowerPC, system RAM is added to iomem_resource. See

[PATCH RESEND v1 5/5] crash: option to let arch decide mem range is usable

2025-01-08 Thread Sourabh Jain
, and remove the PowerPC functions to locate holes. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/powerpc/include/asm/kexec.h | 6 +- arch/po

[PATCH RESEND v1 4/5] powerpc/crash: use generic crashkernel reservation

2025-01-08 Thread Sourabh Jain
se refer to the previous commit titled "crash: let arch decide crash memory export to iomem_resource. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jai

[PATCH RESEND v1 3/5] powerpc/kdump: preserve user-specified memory limit

2025-01-08 Thread Sourabh Jain
ari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/powerpc/kexec/core.c | 8 1 file changed, 8 deletions(-) diff --git a/arch/powerpc/kexec/core.c b/arch/powerpc/kexec/

[PATCH RESEND v1 1/5] crash: remove an unused argument from reserve_crashkernel_generic()

2025-01-08 Thread Sourabh Jain
Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/arm64/mm/init.c | 6 ++ arch/loongarch/kernel/setup.c | 5 ++--- arch/riscv/mm/init.c | 6 ++ arch/x86/kernel/setup.c | 6 ++ include/linux/crash_reserve.h | 11 +-- kernel

[PATCH RESEND v1 0/5] powerpc/crash: use generic crashkernel reservation

2025-01-08 Thread Sourabh Jain
Changelog: v1 Resend: - Rebased on top of 6.13-rc6 Sourabh Jain (5): crash: remove an unused argument from reserve_crashkernel_generic() crash: let arch decide crash memory export to iomem_resource powerpc/kdump: preserve user-specified memory limit powerpc/crash: use generic crashker

[PATCH RESEND v1 2/5] crash: let arch decide crash memory export to iomem_resource

2025-01-08 Thread Sourabh Jain
nel memory should be added to iomem_resource. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- include/linux/crash_reserve.h | 11 +++ ke

Re: [PATCH] powerpc/book3s64/hugetlb: Fix disabling hugetlb when fadump is active

2024-12-16 Thread Sourabh Jain
Hello Ritesh, On 17/12/24 02:19, Ritesh Harjani (IBM) wrote: Ritesh Harjani (IBM) writes: Sourabh Jain writes: Commit 8597538712eb ("powerpc/fadump: Do not use hugepages when fadump is active") disabled hugetlb support when fadump is active by returning early from hugetlbpage_i

[PATCH v2] powerpc/book3s64/hugetlb: Fix disabling hugetlb when fadump is active

2024-12-16 Thread Sourabh Jain
el is independent of hpage_shift/HPAGE_SHIFT. Fixes: 2354ad252b66 ("powerpc/mm: Update default hugetlb size early") CC: Aneesh Kumar K.V CC: Hari Bathini CC: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman Reviewed-by: Ritesh Harjani (IBM) Signed-off-by: Sourabh Jain ---

[PATCH 5/5] crash: option to let arch decide mem range is usable

2024-12-16 Thread Sourabh Jain
, and remove the PowerPC functions to locate holes. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/powerpc/include/asm/kexec.h | 6 +- arch/po

[PATCH 1/5] crash: remove an unused argument from reserve_crashkernel_generic()

2024-12-16 Thread Sourabh Jain
Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/arm64/mm/init.c | 6 ++ arch/loongarch/kernel/setup.c | 5 ++--- arch/riscv/mm/init.c | 6 ++ arch/x86/kernel/setup.c | 6 ++ include/linux/crash_reserve.h | 11 +-- kernel

[PATCH 4/5] powerpc/crash: use generic crashkernel reservation

2024-12-16 Thread Sourabh Jain
se refer to the previous commit titled "crash: let arch decide crash memory export to iomem_resource. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jai

[PATCH 3/5] powerpc/kdump: preserve user-specified memory limit

2024-12-16 Thread Sourabh Jain
ari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- arch/powerpc/kexec/core.c | 8 1 file changed, 8 deletions(-) diff --git a/arch/powerpc/kexec/core.c b/arch/powerpc/kexec/

[PATCH 2/5] crash: let arch decide crash memory export to iomem_resource

2024-12-16 Thread Sourabh Jain
nel memory should be added to iomem_resource. Cc: Andrew Morton Cc: Baoquan he Cc: Hari Bathini CC: Madhavan Srinivasan Cc: Michael Ellerman Cc: ke...@lists.infradead.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- include/linux/crash_reserve.h | 11 +++ ke

[PATCH 0/5] powerpc/crash: use generic crashkernel reservation

2024-12-16 Thread Sourabh Jain
series summary: = Patch 1-2: generic changes Patch 3-4: powerpc changes Patch 5: generic + powerpc changes Sourabh Jain (5): crash: remove an unused argument from reserve_crashkernel_generic() crash: let arch decide crash memory export to iomem_resource powerpc/kdump: preserve user-specifi

Re: [PATCH] powerpc/book3s64/hugetlb: Fix disabling hugetlb when fadump is active

2024-12-16 Thread Sourabh Jain
Hello Ritesh, On 17/12/24 00:07, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: Commit 8597538712eb ("powerpc/fadump: Do not use hugepages when fadump is active") disabled hugetlb support when fadump is active by returning early from hugetlbpage_init():arch/powerpc/mm/hugetlbpage

Re: [PATCH] kexec: Initialize ELF lowest address to ULONG_MAX

2024-12-11 Thread Sourabh Jain
On 12/12/24 08:25, Baoquan he wrote: On 12/10/24 at 02:43pm, Sourabh Jain wrote: kexec_elf_load() loads an ELF executable and sets the address of the lowest PT_LOAD section to the address held by the lowest_load_addr function argument. To determine the lowest PT_LOAD address, a local

[PATCH] kexec: Initialize ELF lowest address to ULONG_MAX

2024-12-10 Thread Sourabh Jain
llerman Cc: ke...@lists.infradead.org Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-ker...@vger.kernel.org Signed-off-by: Sourabh Jain --- kernel/kexec_elf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/kexec_elf.c b/kernel/kexec_elf.c index d3689632e8b9..3a5c25b2adc9 1006

[PATCH] powerpc/book3s64/hugetlb: Fix disabling hugetlb when fadump is active

2024-12-01 Thread Sourabh Jain
el is independent of hpage_shift/HPAGE_SHIFT. Fixes: 2354ad252b66 ("powerpc/mm: Update default hugetlb size early") CC: Aneesh Kumar K.V CC: Hari Bathini CC: Madhavan Srinivasan Cc: Mahesh Salgaonkar Cc: Michael Ellerman CC: Ritesh Harjani (IBM) Signed-off-by: Sourabh Jain --- Note: E

[PATCH v3 2/2] fadump: reserve param area if below boot_mem_top

2024-11-12 Thread Sourabh Jain
. Fixes: 3416c9daa6b1 ("powerpc/fadump: pass additional parameters when fadump is active") Cc: Mahesh Salgaonkar Cc: Michael Ellerman Reviewed-by: Ritesh Harjani (IBM) Acked-by: Hari Bathini Signed-off-by: Sourabh Jain --- Changelog: Since v1: https://lore.kernel.org/all/202411040835

[PATCH v3 1/2] powerpc/fadump: allocate memory for additional parameters early

2024-11-12 Thread Sourabh Jain
s for dump capture kernel") Reported-by: Venkat Rao Bagalkote Closes: https://lore.kernel.org/lkml/a70e4064-a040-447b-8556-1fd02f193...@linux.vnet.ibm.com/T/#u Cc: Mahesh Salgaonkar Cc: Michael Ellerman Reviewed-by: Ritesh Harjani (IBM) Signed-off-by: Hari Bathini Signed-off-by: So

Re: [PATCH v2 2/2] fadump: reserve param area if below boot_mem_top

2024-11-12 Thread Sourabh Jain
Hello Ritesh, On 12/11/24 18:40, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: Hello Ritesh, On 12/11/24 17:23, Ritesh Harjani (IBM) wrote: Ritesh Harjani (IBM) writes: Sourabh Jain writes: Hello Ritesh, On 12/11/24 11:51, Ritesh Harjani (IBM) wrote: Sourabh Jain writes

Re: [PATCH v2 2/2] fadump: reserve param area if below boot_mem_top

2024-11-12 Thread Sourabh Jain
Hello Ritesh, On 12/11/24 17:23, Ritesh Harjani (IBM) wrote: Ritesh Harjani (IBM) writes: Sourabh Jain writes: Hello Ritesh, On 12/11/24 11:51, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The param area is a memory region where the kernel places additional command-line

Re: [PATCH v2 2/2] fadump: reserve param area if below boot_mem_top

2024-11-12 Thread Sourabh Jain
Hello Ritesh, On 12/11/24 11:51, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: The param area is a memory region where the kernel places additional command-line arguments for fadump kernel. Currently, the param memory area is reserved in fadump kernel if it is above boot_mem_top. However

Re: [PATCH v2 1/2] powerpc/fadump: allocate memory for additional parameters early

2024-11-12 Thread Sourabh Jain
Hello Ritesh On 12/11/24 12:33, Ritesh Harjani (IBM) wrote: Sourabh Jain writes: From: Hari Bathini Memory for passing additional parameters to fadump capture kernel is allocated during subsys_initcall level, using memblock. But as slab is already available by this time, allocation happens

Re: [PATCH v2 1/2] powerpc/fadump: allocate memory for additional parameters early

2024-11-07 Thread Sourabh Jain
Hello Venkat, Thanks for testing the patch series. - Sourabh Jain On 07/11/24 19:25, Venkat Rao Bagalkote wrote: Applied the below patch to 6.12.0-rc6-next20241106 and issue is not seen. Results look good to me. [root@ltcden8-lp5 ~]# uname -r 6.12.0-rc6-next-20241106-2-gf374fbb4ee1a

[PATCH v2 2/2] fadump: reserve param area if below boot_mem_top

2024-11-06 Thread Sourabh Jain
. Fixes: 3416c9daa6b1 ("powerpc/fadump: pass additional parameters when fadump is active") Cc: Mahesh Salgaonkar Cc: Michael Ellerman Acked-by: Hari Bathini Signed-off-by: Sourabh Jain --- Changelog: Since v1: https://lore.kernel.org/all/20241104083528.99520-1-sourabhj...@lin

[PATCH v2 1/2] powerpc/fadump: allocate memory for additional parameters early

2024-11-06 Thread Sourabh Jain
s for dump capture kernel") Reported-by: Venkat Rao Bagalkote Closes: https://lore.kernel.org/lkml/a70e4064-a040-447b-8556-1fd02f193...@linux.vnet.ibm.com/T/#u Cc: Mahesh Salgaonkar Cc: Michael Ellerman Signed-off-by: Hari Bathini Signed-off-by: Sourabh Jain --- Changelog: Since

Re: [PATCH 1/2] powerpc/fadump: allocate memory for additional parameters early

2024-11-06 Thread Sourabh Jain
Hello Michael and Hari, On 05/11/24 13:46, Michael Ellerman wrote: Hi Sourabh, Sourabh Jain writes: From: Hari Bathini Memory for passing additional parameters to fadump capture kernel is allocated during subsys_initcall level, using memblock. But as slab is already available by this time

Re: [PATCH 2/2] fadump: reserve param area if below boot_mem_top

2024-11-04 Thread Sourabh Jain
Hello Hari, On 04/11/24 15:50, Hari Bathini wrote: On 04/11/24 2:05 pm, Sourabh Jain wrote: The param area is a memory region where the kernel places additional command-line arguments for fadump kernel. Currently, the param memory area is reserved in fadump kernel if it is above

[PATCH 2/2] fadump: reserve param area if below boot_mem_top

2024-11-04 Thread Sourabh Jain
: Mahesh Salgaonkar Cc: Hari Bathini Cc: Michael Ellerman Signed-off-by: Sourabh Jain --- arch/powerpc/kernel/fadump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c index 4a3f80f42118..35a8a107e16b 100644 --- a/arch

[PATCH 1/2] powerpc/fadump: allocate memory for additional parameters early

2024-11-04 Thread Sourabh Jain
s for dump capture kernel") Reported-by: Venkat Rao Bagalkote Closes: https://lore.kernel.org/lkml/a70e4064-a040-447b-8556-1fd02f193...@linux.vnet.ibm.com/T/#u Cc: Mahesh Salgaonkar Cc: Michael Ellerman Signed-off-by: Hari Bathini Signed-off-by: Sourabh Jain --- arch/powerpc/include/as

  1   2   3   4   5   >