<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, May 15, 2013 at 1:24 PM, Malea, Daniel <span dir="ltr"><<a href="mailto:daniel.malea@intel.com" target="_blank">daniel.malea@intel.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div id=":1ze" style="overflow:hidden">Hey
 Mike, looks good, but any reason to not put this implementation in 
ObjectFileELF (or re-use that class if possible)? I notice that 
ELFHeader::Parse() (see ELFHeader.cpp:108) seems to read some similar 
fields as your function below.  If we can improve the ObjectFile plugin 
and re-use it instead of replicating its functionality, I think that 
would be preferable.<br>
<br>
OTOH, if it is not possible to initialize an ObjectFile at the point 
where the CPU type is needed, or that is otherwise undesirable, I'm OK 
with having some minimal ELF functionality in the Linux host plugin.</div></blockquote></div><br>On Wed, May 15, 2013 at 2:03 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>

<blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">The correct fix for this is to fill in the GetModuleSpecifications() in ObjectFileELF:<br></blockquote>

<br></div><div class="gmail_extra">I
 had looked at the ObjectFileELF class, but it seemed a bit heavyweight
 for just getting the file architecture (2 bytes in the elf header). When someone does a 
"platform process list" this can get called on just about every process 
the user is running. It looks like calling ObjectFile::GetModuleSpecifications() reads 512 bytes of the file and then tries to callback all the ObjectFile plug-ins until a match is found.<br><br>I'll implement the 
ObjectFileELF::GetModuleSpecifications() and see how that looks though.<br><br>Thanks.<br></div><div class="gmail_extra"> -Mike<br></div></div>