Hi Tao,

Thank you for helping improve the patch and explain the purpose of the 
vmcoreinfo branch.

> I re-drafted the patch, which re-ordered the if block in
> arm64_set_irq_stack_size(). I guess what Lianbo mean, is to let the
> normal vmcores, aka the ones with vmcoreinfo, go into the default path;
> only the ones with no vmcoreinfo, go into the tbnz disassembly path. In
> this way the patch change have minimal influence for the normal
> vmcores(this is the majority case).

After reviewing the recent emails, I get lianbo's point.
(Previously, I can't understand the purpose of introducing vmcoreinfo.)

> In addition, this patch is not tested, please have a review and
> test against it.

I have tested this patch and it works well.


Thanks,
Yeping
--
Crash-utility mailing list -- devel@lists.crash-utility.osci.io
To unsubscribe send an email to devel-le...@lists.crash-utility.osci.io
https://${domain_name}/admin/lists/devel.lists.crash-utility.osci.io/
Contribution Guidelines: https://github.com/crash-utility/crash/wiki

Reply via email to