On 2023/06/20 11:27, HAGIO KAZUHITO(萩尾 一仁) wrote:
On 2023/06/16 20:55, Likhitha Korrapati wrote:
> Patch removes redundant checks for PTE (Page Table Entry) because those
> conditions are already covered.
>
> if (!(pte & _PAGE_PRESENT)) {
> ...
> return FALSE;
> }
>
> if (!pte)
> return FALSE;
>
> The second pte check is redundant because it holds true only when pte is
> 0. if pte is 0 then (!(pte & _PAGE_PRESENT)) is true and it will return
> false. so there is no need for one more pte check.
>
> Signed-off-by: Likhitha Korrapati <likhitha(a)linux.ibm.com>
thanks for the cleanup.
Acked-by: Kazuhito Hagio <k-hagio-ab(a)nec.com>
with Lianbo's ack, applied.
https://github.com/crash-utility/crash/commit/8b24b2025fb4ae9bd6102bb054b...
Thanks,
Kazu