From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:33974)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <marc.mari.barcelo@gmail.com>) id 1aQagK-0006B6-NX
	for qemu-devel@nongnu.org; Tue, 02 Feb 2016 08:07:45 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <marc.mari.barcelo@gmail.com>) id 1aQagC-0007kD-Uv
	for qemu-devel@nongnu.org; Tue, 02 Feb 2016 08:07:44 -0500
Received: from mail-wm0-x244.google.com ([2a00:1450:400c:c09::244]:32850)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <marc.mari.barcelo@gmail.com>) id 1aQagC-0007ip-NN
	for qemu-devel@nongnu.org; Tue, 02 Feb 2016 08:07:36 -0500
Received: by mail-wm0-x244.google.com with SMTP id r129so2351793wmr.0
	for <qemu-devel@nongnu.org>; Tue, 02 Feb 2016 05:07:36 -0800 (PST)
Date: Tue, 2 Feb 2016 14:07:33 +0100
From: Marc =?UTF-8?B?TWFyw60=?= <marc.mari.barcelo@gmail.com>
Message-ID: <20160202140733.6cef6568@crunchbang>
In-Reply-To: <1454414695.9300.57.camel@redhat.com>
References: <1454065944-15887-1-git-send-email-markmb@redhat.com>
	<1454411187.9300.54.camel@redhat.com>
	<1454414695.9300.57.camel@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH v4] Add optionrom compatible with fw_cfg
	DMA version
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: "Gabriel L. Somlo" <somlo@cmu.edu>, qemu-devel@nongnu.org, Kevin O'Connor <kevin@koconnor.net>, Stefan Hajnoczi <stefanha@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Marc =?UTF-8?B?TWFyw60=?= <markmb@redhat.com>, Laszlo <lersek@redhat.com>

El Tue, 02 Feb 2016 13:04:55 +0100
Gerd Hoffmann <kraxel@redhat.com> escribi=C3=B3:
> On Di, 2016-02-02 at 12:06 +0100, Gerd Hoffmann wrote:
> >   Hi,
> >=20
> > >  %.img: %.o
> > > -	$(call quiet-command,$(LD) $(LDFLAGS_NOPIE) -Ttext 0 -e
> > > _start -s -o $@ $<,"  Building $(TARGET_DIR)$@")
> > > +	$(call quiet-command,$(LD) $(LDFLAGS_NOPIE) -m elf_i386
> > > -Ttext 0 -e _start -s -o $@ $<,"  Building $(TARGET_DIR)$@")
> >=20
> > Hmm, that breaks the windows cross build:
> >=20
> > make: Entering directory `/home/kraxel/projects/qemu/build-win32'
> >   Building optionrom/linuxboot_dma.img
> > i686-w64-mingw32-ld: unrecognised emulation mode: elf_i386
> > Supported emulations: i386pe
> > make[1]: *** [linuxboot_dma.img] Error 1
>=20
> Testing shows two more problems:
>=20
> (1) initrd loading is broken, kernel complains it finds only
> gibberish:
>=20
> [    0.934582] Unpacking initramfs...
> [    1.166983] Initramfs unpacking failed: junk in compressed archive
> [    1.168458] Freeing initrd memory: 32812k freed

Will look at it. It is probably being copied to a wrong location.

> (2) going back to non-dma boot via -M pc-$old doesn't work,
> appearently fw_cfg dma is enabled even for old machine types.

I don't remember discussing the topic of machine types when touching
fw_cfg DMA. Which means, it probably slipped amongst the other details.
But it is now merged and in stable, so it should probably be left as it
is now.

Should this optionrom be enabled only with the latest machine type?

Thanks
Marc