<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
p
{mso-style-priority:99;
margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style></head><body lang=EN-US link=blue vlink="#954F72"><div class=WordSection1><p>Hi *,</p><p><o:p> </o:p></p><p>I had a few questions about the linker I was hoping someone can help me with,</p><p>I'm pretty new to it so any insights would be appreciated.</p><p><o:p> </o:p></p><p>1) Has to do with checkProddableBlock and #10672 and #10563</p><p><o:p> </o:p></p><p>static void checkProddableBlock (ObjectCode *oc, void *addr, size_t size ) </p><p>{ </p><p> ProddableBlock* pb; </p><p><o:p> </o:p></p><p> for (pb = oc->proddables; pb != NULL; pb = pb->next) { </p><p> char* s = (char*)(pb->start); </p><p> char* e = s + pb->size; </p><p> char* a = (char*)addr; </p><p> if (a >= s && (a+size) <= e) return; </p><p> } </p><p> barf("checkProddableBlock: invalid fixup in runtime linker: %p", addr); </p><p>} </p><p><o:p> </o:p></p><p>From what I have found, these errors seem to happen because oc->proddables is initially NULL, </p><p>the for loop is skipped. From what I can tell, this function is checking if there's a "proddable"</p><p>that fits within the supplied address and size. So if there is no proddables to begin with, should this</p><p>check just not be skipped and the callee of this call not use this ObjectCode instead of erroring out?</p><p><o:p> </o:p></p><p>2) The second question is about static int ocGetNames_PEi386 ( ObjectCode* oc ) </p><p>I am getting a test failure because it is claiming that .eh_frame section is misaligned.</p><p>This comes from this code:</p><p><o:p> </o:p></p><p> if (kind != SECTIONKIND_OTHER && end >= start) { </p><p> if ((((size_t)(start)) % 4) != 0) { </p><p> errorBelch("Misaligned section %s: %p", (char*)secname, start); </p><p> stgFree(secname); </p><p> return 0; </p><p> } </p><p><o:p> </o:p></p><p>Where start is defined as:</p><p><o:p> </o:p></p><p>start = ((UChar*)(oc->image)) + sectab_i->PointerToRawData;</p><p>and oc->image is a memory location received by allocateImageAndTrampolines.</p><p><o:p> </o:p></p><p>In the case of my test failure it is because the .eh_frame section seems to begin at 0x30F</p><p>since oc->image will always be 4 aligned (so it doesn't really matter in the check) it gives that error because PointerToRawData isn't aligned by 4.</p><p><o:p> </o:p></p><p>So my question is would it not be better just to check the alignment flag in the PE section header instead of checking the load address (which is always going to aligned to 4?) and The file pointer to </p><p>the first page of the section within the COFF file to determine the alignment? Like objdump and dumpbin do?</p><p><o:p> </o:p></p><p>e.g.</p><p><o:p> </o:p></p><p>9 .eh_frame 00000038 00000000 00000000 0000030f 2**2</p><p> CONTENTS, ALLOC, LOAD, RELOC, READONLY, DATA</p><p> </p><p>Is the output from objdump which correctly determines the alignment from the section. From what I understand from the PE specification</p><p>the on disk address doesn't have to be aligned by 4:</p><p><o:p> </o:p></p><p>"For object files, the value *should* be aligned on a 4-byte boundary for best performance."</p><p><o:p> </o:p></p><p>I'm wondering if we are incorrectly erroring out here, instead of using the section and making sure we pad it to the alignment boundary.</p><p><o:p> </o:p></p><p>Regards,</p><p>Tamar</p></div></body></html>