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 /arch/x86/kernel/cpu/microcode | |
| 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>
Diffstat (limited to 'arch/x86/kernel/cpu/microcode')
| -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; |
