Hi, Hi,Hatayama
Since zram page not a existing page,so we can't use vtop find exactly physical address,so gcore have to make a little change for this.gcore patch i've already sent in previous mail Please don't remove uvtop_quiet(), which is necessary. The reason is to avoid some bug in crash utility in the past as written in the comment in uvtop_quiet(). Looking at the following stanza, the necessary change is to add a call of try_zram_decompress() when uvtop_quiet() returns false but paddr is non-zero. https://github.com/crash-utility/crash/commit/b12bdd36cf7caad24957c0b8c030001321ab2df4#diff-997c8aaa6bf9a3d9f77e5cab843cd9a6R2290 defs.h belongs to crash utility. Please don't post change of defs.h to crash gcore command. I've answered your question in the previous email about exactly kernel commit in aarch64 stack,please refer to below change The latest two changes are attached,please review. Thanks. I'll review this later. commit 34be98f4944f99076f049a6806fc5f5207a755d3 Author: Ard Biesheuvel <ard.biesheu...@linaro.org> Date: Thu Jul 20 17:15:45 2017 +0100 arm64: kernel: remove {THREAD,IRQ_STACK}_START_SP For historical reasons, we leave the top 16 bytes of our task and IRQ stacks unused, a practice used to ensure that the SP can always be masked to find the base of the current stack (historically, where thread_info could be found). However, this is not necessary, as: * When an exception is taken from a task stack, we decrement the SP by S_FRAME_SIZE and stash the exception registers before we compare the SP against the task stack. In such cases, the SP must be at least S_FRAME_SIZE below the limit, and can be safely masked to determine whether the task stack is in use. * When transitioning to an IRQ stack, we'll place a dummy frame onto the IRQ stack before enabling asynchronous exceptions, or executing code we expect to trigger faults. Thus, if an exception is taken from the IRQ stack, the SP must be at least 16 bytes below the limit. * We no longer mask the SP to find the thread_info, which is now found via sp_el0. Note that historically, the offset was critical to ensure that cpu_switch_to() found the correct stack for new threads that hadn't yet executed ret_from_fork(). Given that, this initial offset serves no purpose, and can be removed. This brings us in-line with other architectures (e.g. x86) which do not rely on this masking. Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> [Mark: rebase, kill THREAD_START_SP, commit msg additions] Signed-off-by: Mark Rutland <mark.rutl...@arm.com> Reviewed-by: Will Deacon <will.dea...@arm.com> Tested-by: Laura Abbott <labb...@redhat.com> Cc: Catalin Marinas <catalin.mari...@arm.com> Cc: James Morse <james.mo...@arm.com>
-- Crash-utility mailing list Crash-utility@redhat.com https://www.redhat.com/mailman/listinfo/crash-utility
