diff options
author | Borislav Petkov <bp@suse.de> | 2016-06-06 11:10:45 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2016-06-08 05:04:19 -0400 |
commit | 852ad5b94524fd76d49944b9db0a93f7c9ee5814 (patch) | |
tree | 39e1c945bcf25d34100e485853f4f19091b345d2 | |
parent | 7557933e6b99d381c19b196901ed537b00f6d121 (diff) |
x86/microcode: Get rid of find_cpio_data()'s dummy offset arg
The microcode loader doesn't use it and now that that arg has been made
optional in find_cpio_data(), get rid of it here.
No functionality change.
Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Andy Lutomirski <luto@amacapital.net>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Brian Gerst <brgerst@gmail.com>
Cc: Denys Vlasenko <dvlasenk@redhat.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1465225850-7352-5-git-send-email-bp@alien8.de
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | arch/x86/kernel/cpu/microcode/amd.c | 3 | ||||
-rw-r--r-- | arch/x86/kernel/cpu/microcode/intel.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/arch/x86/kernel/cpu/microcode/amd.c b/arch/x86/kernel/cpu/microcode/amd.c index 11dd1cc8e444..b1d1e345f5f5 100644 --- a/arch/x86/kernel/cpu/microcode/amd.c +++ b/arch/x86/kernel/cpu/microcode/amd.c | |||
@@ -64,7 +64,6 @@ static struct cpio_data ucode_cpio; | |||
64 | static struct cpio_data __init find_ucode_in_initrd(void) | 64 | static struct cpio_data __init find_ucode_in_initrd(void) |
65 | { | 65 | { |
66 | #ifdef CONFIG_BLK_DEV_INITRD | 66 | #ifdef CONFIG_BLK_DEV_INITRD |
67 | long offset = 0; | ||
68 | char *path; | 67 | char *path; |
69 | void *start; | 68 | void *start; |
70 | size_t size; | 69 | size_t size; |
@@ -92,7 +91,7 @@ static struct cpio_data __init find_ucode_in_initrd(void) | |||
92 | size = boot_params.hdr.ramdisk_size; | 91 | size = boot_params.hdr.ramdisk_size; |
93 | #endif /* !CONFIG_X86_32 */ | 92 | #endif /* !CONFIG_X86_32 */ |
94 | 93 | ||
95 | return find_cpio_data(path, start, size, &offset); | 94 | return find_cpio_data(path, start, size, NULL); |
96 | #else | 95 | #else |
97 | return (struct cpio_data){ NULL, 0, "" }; | 96 | return (struct cpio_data){ NULL, 0, "" }; |
98 | #endif | 97 | #endif |
diff --git a/arch/x86/kernel/cpu/microcode/intel.c b/arch/x86/kernel/cpu/microcode/intel.c index 5835d5b0db81..2ad40b73c8df 100644 --- a/arch/x86/kernel/cpu/microcode/intel.c +++ b/arch/x86/kernel/cpu/microcode/intel.c | |||
@@ -685,7 +685,6 @@ static __init enum ucode_state | |||
685 | __scan_microcode_initrd(struct cpio_data *cd, struct ucode_blobs *blbp) | 685 | __scan_microcode_initrd(struct cpio_data *cd, struct ucode_blobs *blbp) |
686 | { | 686 | { |
687 | #ifdef CONFIG_BLK_DEV_INITRD | 687 | #ifdef CONFIG_BLK_DEV_INITRD |
688 | long offset = 0; | ||
689 | static __initdata char ucode_name[] = "kernel/x86/microcode/GenuineIntel.bin"; | 688 | static __initdata char ucode_name[] = "kernel/x86/microcode/GenuineIntel.bin"; |
690 | char *p = IS_ENABLED(CONFIG_X86_32) ? (char *)__pa_nodebug(ucode_name) | 689 | char *p = IS_ENABLED(CONFIG_X86_32) ? (char *)__pa_nodebug(ucode_name) |
691 | : ucode_name; | 690 | : ucode_name; |
@@ -716,7 +715,7 @@ __scan_microcode_initrd(struct cpio_data *cd, struct ucode_blobs *blbp) | |||
716 | } | 715 | } |
717 | # endif | 716 | # endif |
718 | 717 | ||
719 | *cd = find_cpio_data(p, (void *)start, size, &offset); | 718 | *cd = find_cpio_data(p, (void *)start, size, NULL); |
720 | if (cd->data) { | 719 | if (cd->data) { |
721 | blbp->start = start; | 720 | blbp->start = start; |
722 | blbp->valid = true; | 721 | blbp->valid = true; |