* [PATCH] pyGrub: Use proper bootloader class when entering command manually
@ 2010-05-25 14:39 Michal Novotny
2010-05-25 15:13 ` Michal Novotny
0 siblings, 1 reply; 5+ messages in thread
From: Michal Novotny @ 2010-05-25 14:39 UTC (permalink / raw)
To: 'xen-devel@lists.xensource.com'
[-- Attachment #1: Type: text/plain, Size: 822 bytes --]
Hi,
this is the patch to use the proper bootloader class when entering the
boot commands manually (i.e. using the 'c' option). Before this patch
the bootloader was always treated to be Grub but when user is using
Grub2/ExtLinux or Lilo it's rather confusing. After applying this patch
the proper bootloader image class is being used, e.g. Grub2Image for
Grub2 etc. when you define the boot commands manually using the 'c'
command in pyGrub.
Also, fix for using isconfig has been applied since if there is not fs
set in the run_grub() method the read_config() would fail since it's
trying to access undefined self.cf which is now being set to parser()
from cfg_list.
Signed-off-by: Michal Novotny <minovotn@redhat.com>
--
Michal Novotny<minovotn@redhat.com>, RHCE
Virtualization Team (xen userspace), Red Hat
[-- Attachment #2: xen-pygrub-command-use-proper-bootloader-class.patch --]
[-- Type: text/x-patch, Size: 1857 bytes --]
diff -r 10ad9b50b4ca tools/pygrub/src/pygrub
--- a/tools/pygrub/src/pygrub Tue May 25 11:28:58 2010 +0100
+++ b/tools/pygrub/src/pygrub Tue May 25 16:31:06 2010 +0200
@@ -356,7 +356,7 @@ class Grub:
continue
# if we got boot, then we want to boot the entered image
- img = grub.GrubConf.GrubImage(lines)
+ img = self.imgcl(lines)
self.cf.add_image(img)
self.selected_image = len(self.cf.images) - 1
self.isdone = True
@@ -392,15 +392,28 @@ class Grub:
if not fs:
# set the config file and parse it
- self.cf.filename = fn
- self.cf.parse()
- return
+ for f,parser in cfg_list:
+ self.cf = parser()
+ self.cf.filename = fn
+ self.cf.parse()
+ return
for f,parser in cfg_list:
if fs.file_exists(f):
print >>sys.stderr, "Using %s to parse %s" % (parser,f)
self.cf = parser()
self.cf.filename = f
+
+ # Get the bootloader image file constructor to imgcl
+ if type(self.cf) == type(grub.LiloConf.LiloConfigFile):
+ self.imgcl = grub.LiloConf.LiloImage
+ elif type(self.cf) == type(grub.GrubConf.Grub2ConfigFile):
+ self.imgcl = grub.GrubConf.Grub2Image
+ elif type(self.cf) == type(grub.ExtLinuxConf.ExtLinuxConfigFile):
+ self.imgcl = grub.ExtLinuxConf.ExtLinuxImage
+ else:
+ self.imgcl = grub.GrubConf.GrubImage
+
break
if self.__dict__.get('cf', None) is None:
raise RuntimeError, "couldn't find bootloader config file in the image provided."
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pyGrub: Use proper bootloader class when entering command manually
2010-05-25 14:39 [PATCH] pyGrub: Use proper bootloader class when entering command manually Michal Novotny
@ 2010-05-25 15:13 ` Michal Novotny
2010-05-25 16:22 ` Ian Campbell
0 siblings, 1 reply; 5+ messages in thread
From: Michal Novotny @ 2010-05-25 15:13 UTC (permalink / raw)
To: xen-devel
[-- Attachment #1: Type: text/plain, Size: 1445 bytes --]
Ok, I found that the infrastructure of ExtLinuxImage and LiloImage is
different so I rewrote it a little (but according to the code the old
behaviour should be preserved) and also fixed the isconfig bug (since no
img.initrd is accessible that time).
So please ignore the previous version of my patch and use this one.
Thanks,
Michal
Signed-off-by: Michal Novotny <minovotn@redhat.com>
On 05/25/2010 04:39 PM, Michal Novotny wrote:
> Hi,
> this is the patch to use the proper bootloader class when entering the
> boot commands manually (i.e. using the 'c' option). Before this patch
> the bootloader was always treated to be Grub but when user is using
> Grub2/ExtLinux or Lilo it's rather confusing. After applying this
> patch the proper bootloader image class is being used, e.g. Grub2Image
> for Grub2 etc. when you define the boot commands manually using the
> 'c' command in pyGrub.
>
> Also, fix for using isconfig has been applied since if there is not fs
> set in the run_grub() method the read_config() would fail since it's
> trying to access undefined self.cf which is now being set to parser()
> from cfg_list.
>
> Signed-off-by: Michal Novotny <minovotn@redhat.com>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>
--
Michal Novotny<minovotn@redhat.com>, RHCE
Virtualization Team (xen userspace), Red Hat
[-- Attachment #2: xen-pygrub-command-use-proper-bootloader-class.patch --]
[-- Type: text/x-patch, Size: 5469 bytes --]
diff -r 10ad9b50b4ca tools/pygrub/src/ExtLinuxConf.py
--- a/tools/pygrub/src/ExtLinuxConf.py Tue May 25 11:28:58 2010 +0100
+++ b/tools/pygrub/src/ExtLinuxConf.py Tue May 25 17:10:32 2010 +0200
@@ -16,8 +16,8 @@ import GrubConf
import GrubConf
class ExtLinuxImage(object):
- def __init__(self, lines, path):
- self.reset(lines, path)
+ def __init__(self, title, lines, path = ""):
+ self.reset(lines, title, path)
def __repr__(self):
return ("title: %s\n"
@@ -26,10 +26,10 @@ class ExtLinuxImage(object):
" args: %s\n"
" initrd: %s\n" %(self.title, self.root, self.kernel,
self.args, self.initrd))
- def reset(self, lines, path):
+ def reset(self, lines, title, path):
self._initrd = self._kernel = self._readonly = None
self._args = ""
- self.title = ""
+ self.title = title
self.lines = []
self.path = path
self.root = ""
@@ -143,7 +143,7 @@ class ExtLinuxConfigFile(object):
# new image
if l.lower().startswith("label"):
if len(img) > 0:
- self.add_image(ExtLinuxImage(img, path))
+ self.add_image(ExtLinuxImage("", img, path))
img = [l]
continue
@@ -162,7 +162,7 @@ class ExtLinuxConfigFile(object):
logging.warning("Unknown directive %s" %(com,))
if len(img) > 0:
- self.add_image(ExtLinuxImage(img, path))
+ self.add_image(ExtLinuxImage("", img, path))
def hasPassword(self):
return False
diff -r 10ad9b50b4ca tools/pygrub/src/LiloConf.py
--- a/tools/pygrub/src/LiloConf.py Tue May 25 11:28:58 2010 +0100
+++ b/tools/pygrub/src/LiloConf.py Tue May 25 17:10:32 2010 +0200
@@ -7,8 +7,8 @@ import GrubConf
import GrubConf
class LiloImage(object):
- def __init__(self, lines, path):
- self.reset(lines, path)
+ def __init__(self, title, lines, path = ""):
+ self.reset(lines, title, path)
def __repr__(self):
return ("title: %s\n"
@@ -17,10 +17,10 @@ class LiloImage(object):
" args: %s\n"
" initrd: %s\n" %(self.title, self.root, self.kernel,
self.args, self.initrd))
- def reset(self, lines, path):
+ def reset(self, lines, title, path):
self._initrd = self._kernel = self._readonly = None
self._args = ""
- self.title = ""
+ self.title = title
self.lines = []
self.path = path
self.root = ""
@@ -118,7 +118,7 @@ class LiloConfigFile(object):
# new image
if l.startswith("image"):
if len(img) > 0:
- self.add_image(LiloImage(img, path))
+ self.add_image(LiloImage("", img, path))
img = [l]
continue
@@ -136,7 +136,7 @@ class LiloConfigFile(object):
logging.warning("Unknown directive %s" %(com,))
if len(img) > 0:
- self.add_image(LiloImage(img, path))
+ self.add_image(LiloImage("", img, path))
def hasPassword(self):
return False
diff -r 10ad9b50b4ca tools/pygrub/src/pygrub
--- a/tools/pygrub/src/pygrub Tue May 25 11:28:58 2010 +0100
+++ b/tools/pygrub/src/pygrub Tue May 25 17:10:32 2010 +0200
@@ -356,7 +356,7 @@ class Grub:
continue
# if we got boot, then we want to boot the entered image
- img = grub.GrubConf.GrubImage(lines)
+ img = self.imgcl("entered", lines)
self.cf.add_image(img)
self.selected_image = len(self.cf.images) - 1
self.isdone = True
@@ -392,15 +392,28 @@ class Grub:
if not fs:
# set the config file and parse it
- self.cf.filename = fn
- self.cf.parse()
- return
+ for f,parser in cfg_list:
+ self.cf = parser()
+ self.cf.filename = fn
+ self.cf.parse()
+ return
for f,parser in cfg_list:
if fs.file_exists(f):
print >>sys.stderr, "Using %s to parse %s" % (parser,f)
self.cf = parser()
self.cf.filename = f
+
+ # Get the bootloader image file constructor to imgcl
+ if type(self.cf) == grub.LiloConf.LiloConfigFile:
+ self.imgcl = grub.LiloConf.LiloImage
+ elif type(self.cf) == grub.GrubConf.Grub2ConfigFile:
+ self.imgcl = grub.GrubConf.Grub2Image
+ elif type(self.cf) == grub.ExtLinuxConf.ExtLinuxConfigFile:
+ self.imgcl = grub.ExtLinuxConf.ExtLinuxImage
+ else:
+ self.imgcl = grub.GrubConf.GrubImage
+
break
if self.__dict__.get('cf', None) is None:
raise RuntimeError, "couldn't find bootloader config file in the image provided."
@@ -689,7 +702,7 @@ if __name__ == "__main__":
if isconfig:
chosencfg = run_grub(file, entry, fs, incfg["args"])
print " kernel: %s" % chosencfg["kernel"]
- if img.initrd:
+ if chosencfg["ramdisk"]:
print " initrd: %s" % chosencfg["ramdisk"]
print " args: %s" % chosencfg["args"]
sys.exit(0)
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pyGrub: Use proper bootloader class when entering command manually
2010-05-25 15:13 ` Michal Novotny
@ 2010-05-25 16:22 ` Ian Campbell
2010-05-26 9:27 ` Michal Novotny
0 siblings, 1 reply; 5+ messages in thread
From: Ian Campbell @ 2010-05-25 16:22 UTC (permalink / raw)
To: Michal Novotny; +Cc: xen-devel@lists.xensource.com
On Tue, 2010-05-25 at 16:13 +0100, Michal Novotny wrote:
> Ok, I found that the infrastructure of ExtLinuxImage and LiloImage is
> different so I rewrote it a little (but according to the code the old
> behaviour should be preserved) and also fixed the isconfig bug (since no
> img.initrd is accessible that time).
>
> So please ignore the previous version of my patch and use this one.
Perhaps instead of these two hunks:
--- a/tools/pygrub/src/pygrub Tue May 25 11:28:58 2010 +0100
+++ b/tools/pygrub/src/pygrub Tue May 25 17:10:32 2010 +0200
@@ -356,7 +356,7 @@ class Grub:
continue
# if we got boot, then we want to boot the entered image
- img = grub.GrubConf.GrubImage(lines)
+ img = self.imgcl("entered", lines)
self.cf.add_image(img)
self.selected_image = len(self.cf.images) - 1
self.isdone = True
[...]
self.cf = parser()
self.cf.filename = f
+
+ # Get the bootloader image file constructor to imgcl
+ if type(self.cf) == grub.LiloConf.LiloConfigFile:
+ self.imgcl = grub.LiloConf.LiloImage
+ elif type(self.cf) == grub.GrubConf.Grub2ConfigFile:
+ self.imgcl = grub.GrubConf.Grub2Image
+ elif type(self.cf) ==
grub.ExtLinuxConf.ExtLinuxConfigFile:
+ self.imgcl = grub.ExtLinuxConf.ExtLinuxImage
+ else:
+ self.imgcl = grub.GrubConf.GrubImage
+
break
if self.__dict__.get('cf', None) is None:
raise RuntimeError, "couldn't find bootloader config file
in the image provided."
We could add a method to each of the self.cf classes which returns a new
image from the title+lines given. Then the first hunk becomes something
like:
- img = grub.GrubConf.GrubImage(lines)
+ img = self.cf.new_image("entered", lines)
and the second bit goes away. This would be nice since it avoids
hardcoding another list of bootloaders.
Ian.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pyGrub: Use proper bootloader class when entering command manually
2010-05-25 16:22 ` Ian Campbell
@ 2010-05-26 9:27 ` Michal Novotny
2010-05-26 9:29 ` Ian Campbell
0 siblings, 1 reply; 5+ messages in thread
From: Michal Novotny @ 2010-05-26 9:27 UTC (permalink / raw)
To: Ian Campbell; +Cc: xen-devel@lists.xensource.com, Keir Fraser
[-- Attachment #1: Type: text/plain, Size: 2880 bytes --]
Yeah, that's right Ian. However adding a new method called new_image()
to all of the boot loader configuration classes (*ConfigFile classes)
will still be necessary. I wrote a new version of my patch to add
support for new_image() method all the existing classes and did the
modifications to pygrub.
As in previous version of my patch, the isconfig debug option has been
fixed too.
Keir, please use this version to be committed.
Thanks,
Michal
Signed-off-by: Michal Novotny <minovotn@redhat.com>
On 05/25/2010 06:22 PM, Ian Campbell wrote:
> On Tue, 2010-05-25 at 16:13 +0100, Michal Novotny wrote:
>
>> Ok, I found that the infrastructure of ExtLinuxImage and LiloImage is
>> different so I rewrote it a little (but according to the code the old
>> behaviour should be preserved) and also fixed the isconfig bug (since no
>> img.initrd is accessible that time).
>>
>> So please ignore the previous version of my patch and use this one.
>>
> Perhaps instead of these two hunks:
>
> --- a/tools/pygrub/src/pygrub Tue May 25 11:28:58 2010 +0100
> +++ b/tools/pygrub/src/pygrub Tue May 25 17:10:32 2010 +0200
> @@ -356,7 +356,7 @@ class Grub:
> continue
>
> # if we got boot, then we want to boot the entered image
> - img = grub.GrubConf.GrubImage(lines)
> + img = self.imgcl("entered", lines)
> self.cf.add_image(img)
> self.selected_image = len(self.cf.images) - 1
> self.isdone = True
>
> [...]
> self.cf = parser()
> self.cf.filename = f
> +
> + # Get the bootloader image file constructor to imgcl
> + if type(self.cf) == grub.LiloConf.LiloConfigFile:
> + self.imgcl = grub.LiloConf.LiloImage
> + elif type(self.cf) == grub.GrubConf.Grub2ConfigFile:
> + self.imgcl = grub.GrubConf.Grub2Image
> + elif type(self.cf) ==
> grub.ExtLinuxConf.ExtLinuxConfigFile:
> + self.imgcl = grub.ExtLinuxConf.ExtLinuxImage
> + else:
> + self.imgcl = grub.GrubConf.GrubImage
> +
> break
> if self.__dict__.get('cf', None) is None:
> raise RuntimeError, "couldn't find bootloader config file
> in the image provided."
>
> We could add a method to each of the self.cf classes which returns a new
> image from the title+lines given. Then the first hunk becomes something
> like:
>
> - img = grub.GrubConf.GrubImage(lines)
> + img = self.cf.new_image("entered", lines)
>
> and the second bit goes away. This would be nice since it avoids
> hardcoding another list of bootloaders.
>
> Ian.
>
>
--
Michal Novotny<minovotn@redhat.com>, RHCE
Virtualization Team (xen userspace), Red Hat
[-- Attachment #2: xen-pygrub-command-use-proper-bootloader-class.patch --]
[-- Type: text/x-patch, Size: 3590 bytes --]
diff -r 15f4ead9e686 tools/pygrub/src/ExtLinuxConf.py
--- a/tools/pygrub/src/ExtLinuxConf.py Wed May 26 08:15:31 2010 +0100
+++ b/tools/pygrub/src/ExtLinuxConf.py Wed May 26 11:22:43 2010 +0200
@@ -118,6 +118,12 @@ class ExtLinuxConfigFile(object):
if fn is not None:
self.parse()
+
+ def new_image(self, title, lines):
+ # ExtLinuxImage constructor doesn't have title but since path
+ # is being used by get_{kernel|initrd} functions we pass
+ # empty string rather than None (see lines above)
+ return ExtLinuxImage(lines, "")
def parse(self, buf = None):
if buf is None:
diff -r 15f4ead9e686 tools/pygrub/src/GrubConf.py
--- a/tools/pygrub/src/GrubConf.py Wed May 26 08:15:31 2010 +0100
+++ b/tools/pygrub/src/GrubConf.py Wed May 26 11:22:43 2010 +0200
@@ -252,6 +252,9 @@ class GrubConfigFile(_GrubConfigFile):
def __init__(self, fn = None):
_GrubConfigFile.__init__(self,fn)
+ def new_image(self, title, lines):
+ return GrubImage(title, lines)
+
def parse(self, buf = None):
if buf is None:
if self.filename is None:
@@ -345,7 +348,10 @@ class Grub2ConfigFile(_GrubConfigFile):
class Grub2ConfigFile(_GrubConfigFile):
def __init__(self, fn = None):
_GrubConfigFile.__init__(self, fn)
-
+
+ def new_image(self, title, lines):
+ return Grub2Image(title, lines)
+
def parse(self, buf = None):
if buf is None:
if self.filename is None:
diff -r 15f4ead9e686 tools/pygrub/src/LiloConf.py
--- a/tools/pygrub/src/LiloConf.py Wed May 26 08:15:31 2010 +0100
+++ b/tools/pygrub/src/LiloConf.py Wed May 26 11:22:43 2010 +0200
@@ -147,6 +147,12 @@ class LiloConfigFile(object):
def add_image(self, image):
self.images.append(image)
+ def new_image(self, title, lines):
+ # LiloImage constructor doesn't have title but since path
+ # is being used by get_{kernel|initrd} functions we pass
+ # empty string rather than None (see lines above)
+ return LiloImage(lines, "")
+
def _get_default(self):
for i in range(len(self.images)):
if self.images[i].title == self._default:
diff -r 15f4ead9e686 tools/pygrub/src/pygrub
--- a/tools/pygrub/src/pygrub Wed May 26 08:15:31 2010 +0100
+++ b/tools/pygrub/src/pygrub Wed May 26 11:22:43 2010 +0200
@@ -356,7 +356,7 @@ class Grub:
continue
# if we got boot, then we want to boot the entered image
- img = grub.GrubConf.GrubImage(lines)
+ img = self.cf.new_image("entered", lines)
self.cf.add_image(img)
self.selected_image = len(self.cf.images) - 1
self.isdone = True
@@ -392,9 +392,11 @@ class Grub:
if not fs:
# set the config file and parse it
- self.cf.filename = fn
- self.cf.parse()
- return
+ for f,parser in cfg_list:
+ self.cf = parser()
+ self.cf.filename = fn
+ self.cf.parse()
+ return
for f,parser in cfg_list:
if fs.file_exists(f):
@@ -689,7 +691,7 @@ if __name__ == "__main__":
if isconfig:
chosencfg = run_grub(file, entry, fs, incfg["args"])
print " kernel: %s" % chosencfg["kernel"]
- if img.initrd:
+ if chosencfg["ramdisk"]:
print " initrd: %s" % chosencfg["ramdisk"]
print " args: %s" % chosencfg["args"]
sys.exit(0)
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] pyGrub: Use proper bootloader class when entering command manually
2010-05-26 9:27 ` Michal Novotny
@ 2010-05-26 9:29 ` Ian Campbell
0 siblings, 0 replies; 5+ messages in thread
From: Ian Campbell @ 2010-05-26 9:29 UTC (permalink / raw)
To: Michal Novotny; +Cc: Keir, xen-devel@lists.xensource.com, Fraser
On Wed, 2010-05-26 at 10:27 +0100, Michal Novotny wrote:
> Yeah, that's right Ian. However adding a new method called new_image()
> to all of the boot loader configuration classes (*ConfigFile classes)
> will still be necessary. I wrote a new version of my patch to add
> support for new_image() method all the existing classes and did the
> modifications to pygrub.
>
> As in previous version of my patch, the isconfig debug option has been
> fixed too.
>
> Keir, please use this version to be committed.
>
> Thanks,
> Michal
>
> Signed-off-by: Michal Novotny <minovotn@redhat.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
>
> On 05/25/2010 06:22 PM, Ian Campbell wrote:
> > On Tue, 2010-05-25 at 16:13 +0100, Michal Novotny wrote:
> >
> >> Ok, I found that the infrastructure of ExtLinuxImage and LiloImage is
> >> different so I rewrote it a little (but according to the code the old
> >> behaviour should be preserved) and also fixed the isconfig bug (since no
> >> img.initrd is accessible that time).
> >>
> >> So please ignore the previous version of my patch and use this one.
> >>
> > Perhaps instead of these two hunks:
> >
> > --- a/tools/pygrub/src/pygrub Tue May 25 11:28:58 2010 +0100
> > +++ b/tools/pygrub/src/pygrub Tue May 25 17:10:32 2010 +0200
> > @@ -356,7 +356,7 @@ class Grub:
> > continue
> >
> > # if we got boot, then we want to boot the entered image
> > - img = grub.GrubConf.GrubImage(lines)
> > + img = self.imgcl("entered", lines)
> > self.cf.add_image(img)
> > self.selected_image = len(self.cf.images) - 1
> > self.isdone = True
> >
> > [...]
> > self.cf = parser()
> > self.cf.filename = f
> > +
> > + # Get the bootloader image file constructor to imgcl
> > + if type(self.cf) == grub.LiloConf.LiloConfigFile:
> > + self.imgcl = grub.LiloConf.LiloImage
> > + elif type(self.cf) == grub.GrubConf.Grub2ConfigFile:
> > + self.imgcl = grub.GrubConf.Grub2Image
> > + elif type(self.cf) ==
> > grub.ExtLinuxConf.ExtLinuxConfigFile:
> > + self.imgcl = grub.ExtLinuxConf.ExtLinuxImage
> > + else:
> > + self.imgcl = grub.GrubConf.GrubImage
> > +
> > break
> > if self.__dict__.get('cf', None) is None:
> > raise RuntimeError, "couldn't find bootloader config file
> > in the image provided."
> >
> > We could add a method to each of the self.cf classes which returns a new
> > image from the title+lines given. Then the first hunk becomes something
> > like:
> >
> > - img = grub.GrubConf.GrubImage(lines)
> > + img = self.cf.new_image("entered", lines)
> >
> > and the second bit goes away. This would be nice since it avoids
> > hardcoding another list of bootloaders.
> >
> > Ian.
> >
> >
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-05-26 9:29 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-25 14:39 [PATCH] pyGrub: Use proper bootloader class when entering command manually Michal Novotny
2010-05-25 15:13 ` Michal Novotny
2010-05-25 16:22 ` Ian Campbell
2010-05-26 9:27 ` Michal Novotny
2010-05-26 9:29 ` Ian Campbell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.