<html>
  <head>
    <meta content="text/html; charset=ISO-8859-15"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 03/07/13 08:02, Don Slutz wrote:<br>
    </div>
    <blockquote cite="mid:51388FD1.2000201@terremark.com" type="cite">
      <meta content="text/html; charset=ISO-8859-15"
        http-equiv="Content-Type">
      <div class="moz-cite-prefix">On 03/07/13 03:43, Aurelien Jarno
        wrote:<br>
      </div>
      <blockquote cite="mid:20130307084304.GA17400@ohm.aurel32.net"
        type="cite">
        <div class="moz-text-plain" wrap="true" graphical-quote="true"
          style="font-family: -moz-fixed; font-size: 12px;"
          lang="x-western">
          <pre wrap="">On Wed, Mar 06, 2013 at 07:53:51PM -0500, Kevin O'Connor wrote:
</pre>
          <blockquote type="cite" style="color: #000000;">
            <pre wrap=""><span class="moz-txt-citetags">> </span>On Thu, Mar 07, 2013 at 12:12:08AM +0100, Aurelien Jarno wrote:
</pre>
            <blockquote type="cite" style="color: #000000;">
              <pre wrap=""><span class="moz-txt-citetags">> > </span>On Wed, Mar 06, 2013 at 08:21:11AM +0000, Dietmar Maurer wrote:
</pre>
            </blockquote>
          </blockquote>
        </div>
      </blockquote>
      [snip]<br>
      <blockquote cite="mid:20130307084304.GA17400@ohm.aurel32.net"
        type="cite">
        <div class="moz-text-plain" wrap="true" graphical-quote="true"
          style="font-family: -moz-fixed; font-size: 12px;"
          lang="x-western">
          <blockquote type="cite" style="color: #000000;">
            <blockquote type="cite" style="color: #000000;"> Maybe I am
              doing something wrong or there is a bug in my toolchain
              (Debian Sid). It would be nice if someone could try to
              reproduce that on another distro/system. </blockquote>
          </blockquote>
          <pre wrap=""><div class="moz-txt-sig">-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
<a moz-do-not-send="true" class="moz-txt-link-abbreviated" href="mailto:aurelien@aurel32.net">aurelien@aurel32.net</a>                 <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://www.aurel32.net">http://www.aurel32.net</a>

</div></pre>
        </div>
      </blockquote>
      <br>
      I reproduced this on "Fedora release 17 (Beefy Miracle)"
      3.7.3-101.fc17.x86_64 doing the steps provided.<br>
         -Don Slutz<br>
    </blockquote>
    <br>
    Turns out that this is not the normal kind of issue.  Newer seabios
    works, older does not:<br>
    <br>
    good   * 88cb66e (HEAD, tag: rel-1.7.2.1, origin/1.7.2-stable,
    1.7.2-stable) seabios: Add a dummy PCI slot to irq mapping funct<br>
    good   * 985a9d3 seabios q35: Add new PCI slot to irq routing
    function<br>
    good   * 2b57726 seabios q35: Enable all PIRQn IRQs at startup<br>
    good   * e5fe4f9 Verify CC is valid during build tests.<br>
    good   * 4219149 build: Don't require $(OUT) to be a sub-directory
    of the main directory.<br>
    bad    * d75c22f Fix Makefile - don't reference "out/" directly,
    instead use "$(OUT)".<br>
    bad    * 12e8199 USB-EHCI: Fix null pointer assignment<br>
    bad    * f396871 Update tools/acpi_extract.py to handle iasl
    20130117 release.<br>
    bad    * 4bd8aeb (tag: rel-1.7.2) vgabios: implement AX=1120H..1124H
    functions<br>
    <br>
    <br>
    good:<br>
    Version:
    rel-1.7.2-4-g4219149-20130307_085117-don-lt.don.CloudSwitch.com<br>
    Fixed space: 0xe05b-0x10000  total: 8101  slack: 5  Percent slack:
    0.1%<br>
    16bit size:           39856<br>
    32bit segmented size: 1430<br>
    32bit flat size:      18778<br>
    32bit flat init size: 62400<br>
    Lowmem size:          2176<br>
    <br>
    bad:<br>
    Version:
    rel-1.7.2-3-gd75c22f-20130307_085345-don-lt.don.CloudSwitch.com<br>
    Fixed space: 0xe05b-0x10000  total: 8101  slack: 5  Percent slack:
    0.1%<br>
    16bit size:           39808<br>
    32bit segmented size: 1430<br>
    32bit flat size:      18778<br>
    32bit flat init size: 62400<br>
    Lowmem size:          2176<br>
    <br>
    <br>
    The 16bit size change is the only output change.<br>
    <br>
    The changed that fixed it:<br>
    <br>
    commit 4219149ad2b783abfa61e80e9e9f6910db0c76c9<br>
    Author: Kevin O'Connor <a class="moz-txt-link-rfc2396E" href="mailto:kevin@koconnor.net"><kevin@koconnor.net></a><br>
    Date:   Sun Feb 17 10:56:10 2013 -0500<br>
    <br>
        build: Don't require $(OUT) to be a sub-directory of the main
    directory.<br>
        <br>
        Remove references to "../" and "out/" from the build so that
    "make<br>
        OUT=/a/b/c/" will work.<br>
        <br>
        Signed-off-by: Kevin O'Connor <a class="moz-txt-link-rfc2396E" href="mailto:kevin@koconnor.net"><kevin@koconnor.net></a><br>
    <br>
    diff --git a/Makefile b/Makefile<br>
    index a482c94..20da6d0 100644<br>
    --- a/Makefile<br>
    +++ b/Makefile<br>
    @@ -84,7 +84,7 @@ vpath %.S src vgasrc<br>
     ################ Common build rules<br>
     <br>
     # Verify the build environment works.<br>
    -TESTGCC:=$(shell CC="$(CC)" LD="$(LD)" IASL="$(IASL)"
    tools/test-build.sh)<br>
    +TESTGCC:=$(shell OUT="$(OUT)" CC="$(CC)" LD="$(LD)" IASL="$(IASL)"
    tools/test-build.sh)<br>
     ifeq "$(TESTGCC)" "-1"<br>
     $(error "Please upgrade the build environment")<br>
     endif<br>
    @@ -97,7 +97,7 @@ endif<br>
     # Do a whole file compile by textually including all C code.<br>
     define whole-compile<br>
     @echo "  Compiling whole program $3"<br>
    -$(Q)printf '$(foreach i,$2,#include "../$i"\n)' > $3.tmp.c<br>
    +$(Q)printf '$(foreach i,$2,#include "$(CURDIR)/$i"\n)' >
    $3.tmp.c<br>
     $(Q)$(CC) $1 $(CFLAGSWHOLE) -c $3.tmp.c -o $3<br>
     endef<br>
     <br>
    diff --git a/src/Kconfig b/src/Kconfig<br>
    index 0b112ed..2c9100d 100644<br>
    --- a/src/Kconfig<br>
    +++ b/src/Kconfig<br>
    @@ -367,7 +367,7 @@ menu "BIOS Tables"<br>
                 Support generation of ACPI tables.<br>
     endmenu<br>
     <br>
    -source ../vgasrc/Kconfig<br>
    +source vgasrc/Kconfig<br>
     <br>
     menu "Debugging"<br>
         config DEBUG_LEVEL<br>
    diff --git a/tools/test-build.sh b/tools/test-build.sh<br>
    index 7bd6d1f..ce0aca9 100755<br>
    --- a/tools/test-build.sh<br>
    +++ b/tools/test-build.sh<br>
    @@ -14,13 +14,13 @@ if [ $? -ne 0 ]; then<br>
         exit 0<br>
     fi<br>
     <br>
    -mkdir -p out<br>
    -TMPFILE1=out/tmp_testcompile1.c<br>
    -TMPFILE1o=out/tmp_testcompile1.o<br>
    -TMPFILE1_ld=out/tmp_testcompile1.lds<br>
    -TMPFILE2=out/tmp_testcompile2.c<br>
    -TMPFILE2o=out/tmp_testcompile2.o<br>
    -TMPFILE3o=out/tmp_testcompile3.o<br>
    +mkdir -p ${OUT}<br>
    +TMPFILE1=${OUT}/tmp_testcompile1.c<br>
    +TMPFILE1o=${OUT}/tmp_testcompile1.o<br>
    +TMPFILE1_ld=${OUT}/tmp_testcompile1.lds<br>
    +TMPFILE2=${OUT}/tmp_testcompile2.c<br>
    +TMPFILE2o=${OUT}/tmp_testcompile2.o<br>
    +TMPFILE3o=${OUT}/tmp_testcompile3.o<br>
     <br>
     # Test if ld's alignment handling is correct.  This is a known
    problem<br>
     # with the linker that ships with Ubuntu 11.04.<br>
    <br>
      -Don Slutz<br>
  </body>
</html>