Attachment 292990 Details for Bug 496811

[patch] Only dump RSDT if XSDT must not be used or is not available or if --rsdt param is provided

acpidump_rsdt_param.patch (text/plain), 1.98 KB, created by Thomas Renninger on 2009-05-19 11:10:40 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Thomas Renninger

Created: 2009-05-19 11:10:40 UTC

Size: 1.98 KB

patch

obsolete

>---
> acpidump.c |   16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
>Index: acpidump/acpidump.c
>===================================================================
>--- a/acpidump/acpidump.c
>+++ b/acpidump/acpidump.c
>@@ -59,7 +59,7 @@ static inline u8 checksum(u8 * buffer, u
> }
> 
> static unsigned long psz, addr, length;
>-static int print, connect, skip;
>+static int print, connect, skip, dump_rsdt;
> static u8 select_sig[4];
> 
> static unsigned long read_efi_systab( void )
>@@ -401,6 +401,7 @@ static void usage(const char *progname)
> 	puts("\t--length 0x456 or -l 0x456 -- works only with --addr, dump physical memory"
> 		"\n\t\tregion without trying to understand it's contents");
> 	puts("\t--skip 2 or -s 2 -- skip 2 tables of the given name and output only 3rd one");
>+	puts("\t--rsdt or -r -- Try to access RSDT and related tables even if XSDT is available");
> 	puts("\t--help or -h -- this help message");
> 	exit(0);
> }
>@@ -412,6 +413,7 @@ static struct option long_options[] = {
> 	{"binary", 0, 0, 'b'},
> 	{"length", 1, 0, 'l'},
> 	{"skip", 1, 0, 's'},
>+	{"rsdt", 0, 0, 'r'},
> 	{"help", 0, 0, 'h'},
> 	{0, 0, 0, 0}
> };
>@@ -428,7 +430,7 @@ int main(int argc, char **argv)
> 	addr = length = 0;
> 	skip = 0;
> 	while (1) {
>-		c = getopt_long(argc, argv, "a:t:o:bl:s:h",
>+		c = getopt_long(argc, argv, "a:t:o:brl:s:h",
> 				    long_options, &option_index);
> 		if (c == -1)
> 			break;
>@@ -437,6 +439,9 @@ int main(int argc, char **argv)
> 		case 'a':
> 			addr = strtoul(optarg, (char **)NULL, 16);
> 			break;
>+		case 'r':
>+			dump_rsdt = 1;
>+			break;
> 		case 't':
> 			memcpy(select_sig, optarg, 4);
> 			break;
>@@ -506,9 +511,12 @@ int main(int argc, char **argv)
> 		/* ACPIDUMP uses xsdt table */
> 		if (!acpi_dump_XSDT(fd, &rsdpx))
> 			goto not_found;
>+		if (dump_rsdt)
>+			acpi_dump_RSDT(fd, &rsdpx);
>+	} else {
>+		if (!acpi_dump_RSDT(fd, &rsdpx))
>+			goto not_found;
> 	}
>-	if (!acpi_dump_RSDT(fd, &rsdpx))
>-		goto not_found;
> 	if (connect) {
> 		lseek(fd, 0, SEEK_SET);
> 		write(fd, x, (rsdpx.revision < 2) ?

Actions: View | Diff

Attachments on bug 496811: 287053 | 292990 | 293024