Hi,

The attached patch addresses a problem in grof in which references to
the function which has the highest VMA are erroneously attributed to
the sentinal function <hicore>.

The function core_create_function_syms creates a table of function
symbols and their addresses. This is used to determine which address
ranges belong to which function. In order to classify addresses which
are before the first function and after the last one, two "sentinal"
entriies are created - one with address zero (<locore>)and one with an
address intended to be beyond the end of the function with the highest
VMA (<hicore>). The problem is that only the starting address of each
function is available and so, <hicore> is assigned an address wehich
is one byte higher than the start of the last function. This is
incorrect, since legitemate addresses in this last function are then
attributed to <hicore>.

This patch examines the section containing each function, if
available, and computes the VMA address of the end of the section. The
highest of these addresses is then used as the start of <hicore>. In
this way addresses within the function with the highest VMA are
correctly classified.

Tested against the internal port which exposed the problem.

Seeking approval to commit.

Dave