Re: [PATCH 1/1] kernel/crash_core.c - Add crashkernel=auto for x86 and ARM

2021-01-22 Thread Dave Young
Hi Saeed, On 01/22/21 at 05:14pm, Saeed Mirzamohammadi wrote: > Hi, > > > On Jan 21, 2021, at 7:12 PM, Dave Young wrote: > > > > On 01/22/21 at 09:22am, Dave Young wrote: > >> Hi John, > >> > >> On 01/21/21 at 09:32am, john.p.donne...@oracle.com wrote: > >>> On 11/22/20 9:47 PM, Dave Young

Re: [PATCH v9 08/18] arm64: kexec: move relocation function setup

2021-01-22 Thread Pavel Tatashin
On Wed, Apr 29, 2020 at 1:01 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > Currently, kernel relocation function is configured in machine_kexec() > > at the time of kexec reboot by using control_code_page. > > > > This operation, however, is more logical

Re: [PATCH v9 02/18] arm64: hibernate: move page handling function to new trans_pgd.c

2021-01-22 Thread Pavel Tatashin
On Wed, Apr 29, 2020 at 1:00 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > Now, that we abstracted the required functions move them to a new home. > > Later, we will generalize these function in order to be useful outside > > of hibernation. > >

Re: [PATCH v9 10/18] arm64: kexec: cpu_soft_restart change argument types

2021-01-22 Thread Pavel Tatashin
On Wed, Apr 29, 2020 at 1:01 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > Change argument types from unsigned long to a more descriptive > > phys_addr_t. > > For 'entry', which is a physical addresses, sure... > > > diff --git

Re: [PATCH v9 05/18] arm64: trans_pgd: pass NULL instead of init_mm to *_populate functions

2021-01-22 Thread Pavel Tatashin
> Acked-by: James Morse Thank you for the review. Pasha > > > Thanks, > > James ___ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec

Re: [PATCH v9 04/18] arm64: trans_pgd: pass allocator trans_pgd_create_copy

2021-01-22 Thread Pavel Tatashin
On Wed, Apr 29, 2020 at 1:01 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > Make trans_pgd_create_copy and its subroutines to use allocator that is > > passed as an argument > > Reviewed-by: James Morse Thank you, Pasha

Re: [PATCH v9 07/18] arm64: trans_pgd: hibernate: idmap the single page that holds the copy page routines

2021-01-22 Thread Pavel Tatashin
On Wed, Apr 29, 2020 at 1:01 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > From: James Morse > > > > To resume from hibernate, the contents of memory are restored from > > the swap image. This may overwrite any page, including the running > > kernel and

Re: [PATCH 1/1] kernel/crash_core.c - Add crashkernel=auto for x86 and ARM

2021-01-22 Thread Dave Young
On 01/23/21 at 11:51am, Dave Young wrote: > Hi Saeed, > On 01/22/21 at 05:14pm, Saeed Mirzamohammadi wrote: > > Hi, > > > > > On Jan 21, 2021, at 7:12 PM, Dave Young wrote: > > > > > > On 01/22/21 at 09:22am, Dave Young wrote: > > >> Hi John, > > >> > > >> On 01/21/21 at 09:32am,

Issue in dmesg time with lockless ring buffer

2021-01-22 Thread J. Avila
Hello, When doing some internal testing on a 5.10.4 kernel, we found that the time taken for dmesg seemed to increase from the order of milliseconds to the order of seconds when the dmesg size approached the ~1.2MB limit. After doing some digging, we found that by reverting all of the patches in

Re: [PATCH v9 01/18] arm64: kexec: make dtb_mem always enabled

2021-01-22 Thread Pavel Tatashin
On Wed, Apr 29, 2020 at 1:00 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > Currently, dtb_mem is enabled only when CONFIG_KEXEC_FILE is > > enabled. This adds ugly ifdefs to c files. > > ~s/dtb_mem/ARCH_HAS_KIMAGE_ARCH/ ? > dtb_mem is just one member of

Re: [PATCH v9 13/18] arm64: kexec: add expandable argument to relocation function

2021-01-22 Thread Pavel Tatashin
On Thu, May 7, 2020 at 12:22 PM James Morse wrote: > > Hi Pavel, > > On 26/03/2020 03:24, Pavel Tatashin wrote: > > Currently, kexec relocation function (arm64_relocate_new_kernel) accepts > > the following arguments: > > > > head: start of array that contains relocation information. > >

Re: [PATCH v9 03/18] arm64: trans_pgd: make trans_pgd_map_page generic

2021-01-22 Thread Pavel Tatashin
Hi James, I am working on an updated version of this patch series. We had back and forth discussion on the list and off the list about MMU-enabled series. So, I decided to sync the last series I had with the current mainline. Address your last comments (those that I can address), and send it