From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kjetil Torgrim Homme Subject: pygrub patch to allow explicit offset to fs Date: Wed, 19 Jun 2013 01:40:21 +0200 Message-ID: <51C0EFE5.10109@redpill-linpro.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------070006030107080806040502" Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xensource.com Cc: Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org This is a multi-part message in MIME format. --------------070006030107080806040502 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit I recently needed an old VM to work even though it was created on a SAN LUN with no partition table, just LVM straight onto the raw device. pygrub didn't like this, so I added a simple hack to allow the user to override pygrub's probing when necessary. please consider applying this patch. btw, I think most LVM will have first filesystem at offset 196608. commit 80a3f7b48da235695f8560deb41c19b23e7799e3 Author: Kjetil Torgrim Homme Date: Wed Jun 19 00:54:43 2013 +0200 allow user to specify offset parameter which overrides partition table parsing Signed-off-by: Kjetil Torgrim Homme -- Kjetil T. Homme Redpill Linpro - Changing the game --------------070006030107080806040502 Content-Type: text/x-patch; name="pygrub-offset.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="pygrub-offset.patch" diff --git a/tools/pygrub/src/pygrub b/tools/pygrub/src/pygrub index eedfdb2..d46ee8c 100644 --- a/tools/pygrub/src/pygrub +++ b/tools/pygrub/src/pygrub @@ -712,7 +712,7 @@ if __name__ == "__main__": sel = None def usage(): - print >> sys.stderr, "Usage: %s [-q|--quiet] [-i|--interactive] [-l|--list-entries] [-n|--not-really] [--output=] [--kernel=] [--ramdisk=] [--args=] [--entry=] [--output-directory=] [--output-format=sxp|simple|simple0] " %(sys.argv[0],) + print >> sys.stderr, "Usage: %s [-q|--quiet] [-i|--interactive] [-l|--list-entries] [-n|--not-really] [--output=] [--kernel=] [--ramdisk=] [--args=] [--entry=] [--output-directory=] [--output-format=sxp|simple|simple0] [--offset=] " %(sys.argv[0],) def copy_from_image(fs, file_to_read, file_type, output_directory, not_really): @@ -748,7 +748,7 @@ if __name__ == "__main__": try: opts, args = getopt.gnu_getopt(sys.argv[1:], 'qilnh::', ["quiet", "interactive", "list-entries", "not-really", "help", - "output=", "output-format=", "output-directory=", + "output=", "output-format=", "output-directory=", "offset=", "entry=", "kernel=", "ramdisk=", "args=", "isconfig", "debug"]) except getopt.GetoptError: @@ -765,6 +765,7 @@ if __name__ == "__main__": interactive = True list_entries = False isconfig = False + user_provided_offset = None debug = False not_really = False output_format = "sxp" @@ -797,6 +798,8 @@ if __name__ == "__main__": incfg["ramdisk"] = a elif o in ("--args",): incfg["args"] = a + elif o in ("--offset",): + user_provided_offset = a elif o in ("--entry",): entry = a # specifying the entry to boot implies non-interactive @@ -840,7 +843,10 @@ if __name__ == "__main__": bootfsoptions = "" # get list of offsets into file which start partitions - part_offs = get_partition_offsets(file) + if user_provided_offset is None: + part_offs = get_partition_offsets(file) + else: + part_offs = [ int(user_provided_offset) ] for offset in part_offs: try: --------------070006030107080806040502 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --------------070006030107080806040502--