<blockquote>
<pre><code>* First major one: code size, or rather binary size, you don't want it on a pcbios build with limited memory, unless you really need to. And there are still targets (ROMs) that are limited to 64Kb binaries, or in some cases even less. Virtual box for example is one such thing.
</code></pre>
</blockquote>
<p>Which is precisely the reason why the change I am proposing <em>only</em> enables IPv6 on EFI builds. PCBIOS builds are <em>not</em> changed.</p>
<blockquote>
<pre><code>* Another reason is compability, unless your network is "properly" configured for IPv6, and you expect IPv4 to work, then you might see issues. feel free to read the mailing list
</code></pre>
</blockquote>
<p>As mentioned in the initial PR comment, IPv6 PXE support is <em>mandated</em> by the UEFI specification. This has been the case for over nine years at the point of writing.</p>
<p>If you have a network with some kind of defective IPv6 (non-)connectivity configured, then sure, you «might see issues» (most likely occurring before iPXE is even loaded). On networks with some kind of defective IPv4 (non-)connectivity, the statement is equally true.</p>
<p>It is however not the bootloader's role to fix broken networks, that's a task better left to the network operator. Especially when the attempt to fix the broken networks inevitably results in iPXE being 100% defective on <em>correctly</em> configured IPv6 network.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/ipxe/ipxe/pull/84?email_source=notifications&email_token=AAFNGVETBKVBYB76BQGR2GDQXFOWJA5CNFSM4GFAQL52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGCAGDY#issuecomment-562299663">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAFNGVDO322RTS233IUXZHDQXFOWJANCNFSM4GFAQL5Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAFNGVABCCJYZUAVZLFOBSDQXFOWJA5CNFSM4GFAQL52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGCAGDY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/ipxe/ipxe/pull/84?email_source=notifications\u0026email_token=AAFNGVETBKVBYB76BQGR2GDQXFOWJA5CNFSM4GFAQL52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGCAGDY#issuecomment-562299663",
"url": "https://github.com/ipxe/ipxe/pull/84?email_source=notifications\u0026email_token=AAFNGVETBKVBYB76BQGR2GDQXFOWJA5CNFSM4GFAQL52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGCAGDY#issuecomment-562299663",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>