From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1VgawT-0008OD-Tt for mharc-grub-devel@gnu.org; Wed, 13 Nov 2013 08:57:13 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41843) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VgawL-0008NQ-IK for grub-devel@gnu.org; Wed, 13 Nov 2013 08:57:12 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VgawD-0002jV-LN for grub-devel@gnu.org; Wed, 13 Nov 2013 08:57:05 -0500 Received: from userp1040.oracle.com ([156.151.31.81]:37134) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VgawD-0002jD-E7 for grub-devel@gnu.org; Wed, 13 Nov 2013 08:56:57 -0500 Received: from ucsinet21.oracle.com (ucsinet21.oracle.com [156.151.31.93]) by userp1040.oracle.com (Sentrion-MTA-4.3.1/Sentrion-MTA-4.3.1) with ESMTP id rADDur4f016407 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Wed, 13 Nov 2013 13:56:54 GMT Received: from aserz7022.oracle.com (aserz7022.oracle.com [141.146.126.231]) by ucsinet21.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id rADDuqUT008552 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 13 Nov 2013 13:56:53 GMT Received: from abhmp0004.oracle.com (abhmp0004.oracle.com [141.146.116.10]) by aserz7022.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id rADDuqjb025743; Wed, 13 Nov 2013 13:56:52 GMT Received: from phenom.dumpdata.com (/50.195.21.189) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Wed, 13 Nov 2013 05:56:51 -0800 Received: by phenom.dumpdata.com (Postfix, from userid 1000) id E2C571BF4A4; Wed, 13 Nov 2013 08:56:50 -0500 (EST) Date: Wed, 13 Nov 2013 08:56:50 -0500 From: Konrad Rzeszutek Wilk To: "Vladimir =?utf-8?Q?'=CF=86-coder=2Fphcoder'?= Serbinenko" Subject: Re: [PATCH] remove dependency on /boot/config-* in grub.d/20_linux_xen Message-ID: <20131113135650.GA2647@phenom.dumpdata.com> References: <20130715180006.GA2433@phenom.dumpdata.com> <527FEF3D.1050906@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <527FEF3D.1050906@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Content-Transfer-Encoding: quoted-printable X-Source-IP: ucsinet21.oracle.com [156.151.31.93] X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.4.x-2.6.x [generic] X-Received-From: 156.151.31.81 Cc: The development of GNU GRUB , xen-devel@lists.xensource.com X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Nov 2013 13:57:12 -0000 On Sun, Nov 10, 2013 at 09:40:29PM +0100, Vladimir '=CF=86-coder/phcoder'= Serbinenko wrote: > On 15.07.2013 20:00, Konrad Rzeszutek Wilk wrote: > > Hey, > >=20 > > There is a discussion on the linux-kernel mailing list in which the > > Linus states that "if you depend on any config file, you're broken > > by definition" (https://lkml.org/lkml/2013/7/15/368). > >=20 > Please try my grub_file branch: > http://git.savannah.gnu.org/cgit/grub.git/log/?h=3Dphcoder/grub_file > It should do what you want. >=20 Oddly I get this after I installed it. /sbin/grub-probe: error: failed to get canonical path of `'. And I kind of fixed it by removing the 'set -e' at the start of the file. After that, and with me removing the /boot/config-3.12.0 file it generates an proper entry in grub.cfg for the Xen payload! Yeey! The other thing I saw was that the 'root' option for Linux ended up being 'root=3D' instead of the 'root=3DUUID=3Dblabla'. See here: if [ x"$grub_platform" !=3D xxen \( x"$grub_cpu" =3D xi386 -o x"$grub_cp= u" =3D xx86_64 -o x"$grub_platform" =3D x \) ]; then menuentry 'Fedora GNU/Linux, with Xen hypervisor' --class fedora --class = gnu-linux --class gnu --class os --class xen $menuentry_id_option 'xen-gn= ulinux-simple-' { insmod part_msdos=20 insmod ext2 set root=3D'hd0,msdos1' if [ x$feature_platform_search_hint =3D xy ]; then search --no-floppy --fs-uuid --set=3Droot --hint-bios=3Dhd0,msdos1 --h= int-efi=3Dhd0,msdos1 --hint-baremetal=3Dahci0,msdos1 760e0266-1a60-4621-= b332-0788cf687e33 else search --no-floppy --fs-uuid --set=3Droot 760e0266-1a60-4621-b332-0788= cf687e33 fi echo 'Loading Xen xen ...' if [ "$grub_platform" =3D "pc" -o "$grub_platform" =3D "" ]; then xen_rm_opts=3D else xen_rm_opts=3D"no-real-mode edd=3Doff" fi multiboot /xen.gz placeholder guest_loglvl=3Dall com1=3D115200,8n1 conso= le=3Dcom1,vga loglvl=3Dall iommu=3Dno-amd-iommu-perdev-intremap,verbose,d= ebug,no-intremap ${xen_rm_opts} echo 'Loading Linux 3.12.0 ...' module /vmlinuz-3.12.0 placeholder root=3D ro rd.md=3D0 rd.lvm=3D0 rd.dm= =3D0 rd.luks=3D0 loglevel=3D8 radeon.modeset=3D0 xen-pciback.hide=3D(05:= 00.*) console=3Dhvc0=20 echo 'Loading initial ramdisk ...' module /initramfs-3.12.0.img } From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH] remove dependency on /boot/config-* in grub.d/20_linux_xen Date: Wed, 13 Nov 2013 08:56:50 -0500 Message-ID: <20131113135650.GA2647@phenom.dumpdata.com> References: <20130715180006.GA2433@phenom.dumpdata.com> <527FEF3D.1050906@gmail.com> Reply-To: The development of GNU GRUB Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <527FEF3D.1050906@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: grub-devel-bounces+gcbgd-grub-devel=m.gmane.org@gnu.org Sender: grub-devel-bounces+gcbgd-grub-devel=m.gmane.org@gnu.org To: Vladimir =?utf-8?Q?'=CF=86-coder=2Fphcoder'?= Serbinenko Cc: The development of GNU GRUB , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org T24gU3VuLCBOb3YgMTAsIDIwMTMgYXQgMDk6NDA6MjlQTSArMDEwMCwgVmxhZGltaXIgJ8+GLWNv ZGVyL3BoY29kZXInIFNlcmJpbmVua28gd3JvdGU6Cj4gT24gMTUuMDcuMjAxMyAyMDowMCwgS29u cmFkIFJ6ZXN6dXRlayBXaWxrIHdyb3RlOgo+ID4gSGV5LAo+ID4gCj4gPiBUaGVyZSBpcyBhIGRp c2N1c3Npb24gb24gdGhlIGxpbnV4LWtlcm5lbCBtYWlsaW5nIGxpc3QgaW4gd2hpY2ggdGhlCj4g PiBMaW51cyBzdGF0ZXMgdGhhdCAiaWYgeW91IGRlcGVuZCBvbiBhbnkgY29uZmlnIGZpbGUsIHlv dSdyZSBicm9rZW4KPiA+IGJ5IGRlZmluaXRpb24iIChodHRwczovL2xrbWwub3JnL2xrbWwvMjAx My83LzE1LzM2OCkuCj4gPiAKPiBQbGVhc2UgdHJ5IG15IGdydWJfZmlsZSBicmFuY2g6Cj4gaHR0 cDovL2dpdC5zYXZhbm5haC5nbnUub3JnL2NnaXQvZ3J1Yi5naXQvbG9nLz9oPXBoY29kZXIvZ3J1 Yl9maWxlCj4gSXQgc2hvdWxkIGRvIHdoYXQgeW91IHdhbnQuCj4gCgpPZGRseSBJIGdldCB0aGlz IGFmdGVyIEkgaW5zdGFsbGVkIGl0LgoKL3NiaW4vZ3J1Yi1wcm9iZTogZXJyb3I6IGZhaWxlZCB0 byBnZXQgY2Fub25pY2FsIHBhdGggb2YgYCcuCgpBbmQgSSBraW5kIG9mIGZpeGVkIGl0IGJ5IHJl bW92aW5nIHRoZSAnc2V0IC1lJyBhdCB0aGUgc3RhcnQKb2YgdGhlIGZpbGUuCgpBZnRlciB0aGF0 LCBhbmQgd2l0aCBtZSByZW1vdmluZyB0aGUgL2Jvb3QvY29uZmlnLTMuMTIuMCBmaWxlCml0IGdl bmVyYXRlcyBhbiBwcm9wZXIgZW50cnkgaW4gZ3J1Yi5jZmcgZm9yIHRoZSBYZW4gcGF5bG9hZCEK WWVleSEKClRoZSBvdGhlciB0aGluZyBJIHNhdyB3YXMgdGhhdCB0aGUgJ3Jvb3QnIG9wdGlvbiBm b3IgTGludXgKZW5kZWQgdXAgYmVpbmcgJ3Jvb3Q9JyBpbnN0ZWFkIG9mIHRoZSAncm9vdD1VVUlE PWJsYWJsYScuCgpTZWUgaGVyZToKCmlmIFsgeCIkZ3J1Yl9wbGF0Zm9ybSIgIT0geHhlbiBcKCAg eCIkZ3J1Yl9jcHUiID0geGkzODYgLW8geCIkZ3J1Yl9jcHUiID0geHg4Nl82NCAtbyB4IiRncnVi X3BsYXRmb3JtIiA9IHggXCkgXTsgdGhlbgptZW51ZW50cnkgJ0ZlZG9yYSBHTlUvTGludXgsIHdp dGggWGVuIGh5cGVydmlzb3InIC0tY2xhc3MgZmVkb3JhIC0tY2xhc3MgZ251LWxpbnV4IC0tY2xh c3MgZ251IC0tY2xhc3Mgb3MgLS1jbGFzcyB4ZW4gJG1lbnVlbnRyeV9pZF9vcHRpb24gJ3hlbi1n bnVsaW51eC1zaW1wbGUtJyB7CglpbnNtb2QgcGFydF9tc2RvcyAKCWluc21vZCBleHQyCglzZXQg cm9vdD0naGQwLG1zZG9zMScKCWlmIFsgeCRmZWF0dXJlX3BsYXRmb3JtX3NlYXJjaF9oaW50ID0g eHkgXTsgdGhlbgoJICBzZWFyY2ggLS1uby1mbG9wcHkgLS1mcy11dWlkIC0tc2V0PXJvb3QgLS1o aW50LWJpb3M9aGQwLG1zZG9zMSAtLWhpbnQtZWZpPWhkMCxtc2RvczEgLS1oaW50LWJhcmVtZXRh bD1haGNpMCxtc2RvczEgIDc2MGUwMjY2LTFhNjAtNDYyMS1iMzMyLTA3ODhjZjY4N2UzMwoJZWxz ZQoJICBzZWFyY2ggLS1uby1mbG9wcHkgLS1mcy11dWlkIC0tc2V0PXJvb3QgNzYwZTAyNjYtMWE2 MC00NjIxLWIzMzItMDc4OGNmNjg3ZTMzCglmaQoJZWNobwknTG9hZGluZyBYZW4geGVuIC4uLicK ICAgICAgICBpZiBbICIkZ3J1Yl9wbGF0Zm9ybSIgPSAicGMiIC1vICIkZ3J1Yl9wbGF0Zm9ybSIg PSAiIiBdOyB0aGVuCiAgICAgICAgICAgIHhlbl9ybV9vcHRzPQogICAgICAgIGVsc2UKICAgICAg ICAgICAgeGVuX3JtX29wdHM9Im5vLXJlYWwtbW9kZSBlZGQ9b2ZmIgogICAgICAgIGZpCgltdWx0 aWJvb3QJL3hlbi5neiBwbGFjZWhvbGRlciBndWVzdF9sb2dsdmw9YWxsIGNvbTE9MTE1MjAwLDhu MSBjb25zb2xlPWNvbTEsdmdhIGxvZ2x2bD1hbGwgaW9tbXU9bm8tYW1kLWlvbW11LXBlcmRldi1p bnRyZW1hcCx2ZXJib3NlLGRlYnVnLG5vLWludHJlbWFwICAke3hlbl9ybV9vcHRzfQoJZWNobwkn TG9hZGluZyBMaW51eCAzLjEyLjAgLi4uJwoJbW9kdWxlCS92bWxpbnV6LTMuMTIuMCBwbGFjZWhv bGRlciByb290PSBybyByZC5tZD0wIHJkLmx2bT0wIHJkLmRtPTAgcmQubHVrcz0wICBsb2dsZXZl bD04IHJhZGVvbi5tb2Rlc2V0PTAgeGVuLXBjaWJhY2suaGlkZT0oMDU6MDAuKikgY29uc29sZT1o dmMwIAoJZWNobwknTG9hZGluZyBpbml0aWFsIHJhbWRpc2sgLi4uJwoJbW9kdWxlCS9pbml0cmFt ZnMtMy4xMi4wLmltZwp9CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpHcnViLWRldmVsIG1haWxpbmcgbGlzdApHcnViLWRldmVsQGdudS5vcmcKaHR0cHM6 Ly9saXN0cy5nbnUub3JnL21haWxtYW4vbGlzdGluZm8vZ3J1Yi1kZXZlbAo=