RFR: 8261098: Add clhsdb "findsym" command [v3]

[Serguei Spitsyn]

On Wed, 17 Feb 2021 02:12:56 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Add "findsym" to clhsdb. See the CR and CSR for details. The [CSR](https://bugs.openjdk.java.net/browse/JDK-8261101) still needs a reviewer.
>> 
>> There is a fix in LinuxDebuggerLocal_lookupByName0() to allow passing in NULL for the dso name. This is allowed, and in fact even not null it gets ignored. It is needed by the new findsym support in order for the following to work, which passes in null for the dso name:
>> 
>> `    Address addr = VM.getVM().getDebugger().lookup(null, symbol);`
>> 
>> There is one other somewhat unrelated fix in the test:
>> 
>>  -            String value = parts[1];
>>  +           String value = parts[1].split(linesep)[0];
>> 
>> This is suppose to capture just the value at the specified address, but it also captures the newline and some text after. The result if that when `findpc <value>` is executed, it also executes another command or two of garbage commands after that, which produce errors. They were not impacting the test, but were noticeable in the log. I first noticed it when similar code in the new part of the test had the same issue.
>
> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Improve handling of NULL objectName

It looks good to me.

-------------

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2567