On Mon, Dec 04, 2023 at 04:20:03PM +0800, Lianbo Jiang wrote:
Hi, Alexander
Hi Lianbo!
Thank you for the patch set.
...
I did the test, and the patches can work on old kernel versions.
However,
since VA and PA are the same on S390x, it's hard to determine if all changes
have been made.
In addition, I can not test the patch 3, seems that the current kernel does
not support this feature(I tried it on the kernel 6.7, did not see the
different VA and PA).
That is correct. There is no version that supports this feature yet.
Anyway, If you still want to have these patches in crash-utility for
now, I
can help ack this one.
Yes, I would prefer to pick it now.
...
For patch 3, I saw a warning:
s390x.c: In function ‘is_read_proc_kcore’:
s390x.c:671:13: warning: unused variable ‘kcore_fd’ [-Wunused-variable]
671 | int kcore_fd, fd;
| ^~~~~~~~
The variable kcore_fd is not used in the patch 3, it would be good to remove
it.
I will send v2 in reply to this mail.
Thank you, Lianbo!
> Thanks
>
> Lianbo