<div dir="ltr">Hi, Pingfan<div><br></div><div>Thank you for the update.</div><div><br></div><div><pre style="color:rgb(0,0,0)"><span style="font-family:Arial,Helvetica,sans-serif;color:rgb(34,34,34)">></span><span style="font-family:Arial,Helvetica,sans-serif;color:rgb(34,34,34)"> </span>This bug connects with kernel commit 7bc1a0f9e176 ("arm64: mm: use
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>single quantity to represent the PA to VA translation"), memstart_addr
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>can be negative, which makes it different from real phys_offset. If
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>using memstart_addr to calculate the real paddr, the unreasonable paddr
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>will be got.

<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>Furthermore, in crash utility, PTOV() needs memstart_addr to calculate
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>VA from PA, while getting PFN offset in a dumpfile, phys_offset is
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>required.
<br></pre><pre style="color:rgb(0,0,0)">As you mentioned above, the calculation formula has been changed, how to</pre><pre style="color:rgb(0,0,0)">deal with the backward compatibility issue? Should we use kernel version</pre><pre style="color:rgb(0,0,0)">to determine which code branch it should execute? Please correct me if I</pre><pre style="color:rgb(0,0,0)">was wrong.</pre><pre style="color:rgb(0,0,0)"><br></pre><pre style="color:rgb(0,0,0)">Thanks.</pre><pre style="color:rgb(0,0,0)">Lianbo</pre><pre style="color:rgb(0,0,0)"> 
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>To serve the different purpose, using phys_offset_nominal and
<span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">> </span>phys_offset to store them.
</pre><br class="gmail-Apple-interchange-newline"></div></div>