Commit a30a6a2c authored by Shaohua Li's avatar Shaohua Li Committed by Linus Torvalds
Browse files

[PATCH] x86 microcode: using request_firmware to pull microcode 



Using request_firmware to pull ucode from userspace, so we don't need the
application 'microcode_ctl' to assist.  We name each ucode file according
to CPU's info as intel-ucode/family-model-stepping.  In this way we could
split ucode file as small one.  This has a lot of advantages such as
selectively update and validate microcode for specific models, better
manage microcode file, easily write tools for administerators and so on.
with the changes, we should put all intel-ucode/xx-xx-xx microcode files
into the firmware dir (I had a tool to split previous big data file into
small one and later we will release new style data file).  The init script
should be changed to just loading the driver without unloading

Signed-off-by: default avatarShaohua Li <shaohua.li@intel.com>
Acked-by: default avatarTigran Aivazian <tigran@veritas.com>
Cc: Greg KH <greg@kroah.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 9a3110bf

arch/i386/kernel/microcode.c

+116 −0
Original line number Diff line number Diff line
@@ -83,6 +83,9 @@ 
#include <linux/spinlock.h>

#include <linux/mm.h>

#include <linux/mutex.h>

#include <linux/cpu.h>

#include <linux/firmware.h>

#include <linux/platform_device.h>



#include <asm/msr.h>

#include <asm/uaccess.h>
@@ -493,6 +496,112 @@ MODULE_ALIAS_MISCDEV(MICROCODE_MINOR); 
#define microcode_dev_exit() do { } while(0)

#endif



static long get_next_ucode_from_buffer(void **mc, void *buf,

	unsigned long size, long offset)

{

	microcode_header_t *mc_header;

	unsigned long total_size;



	/* No more data */

	if (offset >= size)

		return 0;

	mc_header = (microcode_header_t *)(buf + offset);

	total_size = get_totalsize(mc_header);



	if ((offset + total_size > size)

		|| (total_size < DEFAULT_UCODE_TOTALSIZE)) {

		printk(KERN_ERR "microcode: error! Bad data in microcode data file\n");

		return -EINVAL;

	}



	*mc = vmalloc(total_size);

	if (!*mc) {

		printk(KERN_ERR "microcode: error! Can not allocate memory\n");

		return -ENOMEM;

	}

	memcpy(*mc, buf + offset, total_size);

	return offset + total_size;

}



/* fake device for request_firmware */

static struct platform_device *microcode_pdev;



static int cpu_request_microcode(int cpu)

{

	char name[30];

	struct cpuinfo_x86 *c = cpu_data + cpu;

	const struct firmware *firmware;

	void * buf;

	unsigned long size;

	long offset = 0;

	int error;

	void *mc;



	/* We should bind the task to the CPU */

	BUG_ON(cpu != raw_smp_processor_id());

	sprintf(name,"intel-ucode/%02x-%02x-%02x",

		c->x86, c->x86_model, c->x86_mask);

	error = request_firmware(&firmware, name, &microcode_pdev->dev);

	if (error) {

		pr_debug("ucode data file %s load failed\n", name);

		return error;

	}

	buf = (void *)firmware->data;

	size = firmware->size;

	while ((offset = get_next_ucode_from_buffer(&mc, buf, size, offset))

			> 0) {

		error = microcode_sanity_check(mc);

		if (error)

			break;

		error = get_maching_microcode(mc, cpu);

		if (error < 0)

			break;

		/*

		 * It's possible the data file has multiple matching ucode,

		 * lets keep searching till the latest version

		 */

		if (error == 1) {

			apply_microcode(cpu);

			error = 0;

		}

		vfree(mc);

	}

	if (offset > 0)

		vfree(mc);

	if (offset < 0)

		error = offset;

	release_firmware(firmware);



	return error;

}



static void microcode_init_cpu(int cpu)

{

	cpumask_t old;

	struct ucode_cpu_info *uci = ucode_cpu_info + cpu;



	old = current->cpus_allowed;



	set_cpus_allowed(current, cpumask_of_cpu(cpu));

	mutex_lock(&microcode_mutex);

	collect_cpu_info(cpu);

	if (uci->valid)

		cpu_request_microcode(cpu);

	mutex_unlock(&microcode_mutex);

	set_cpus_allowed(current, old);

}



static void microcode_fini_cpu(int cpu)

{

	struct ucode_cpu_info *uci = ucode_cpu_info + cpu;



	mutex_lock(&microcode_mutex);

	uci->valid = 0;

	vfree(uci->mc);

	uci->mc = NULL;

	mutex_unlock(&microcode_mutex);

}



static int __init microcode_init (void)

{

	int error;
@@ -500,6 +609,12 @@ static int __init microcode_init (void) 
	error = microcode_dev_init();

	if (error)

		return error;

	microcode_pdev = platform_device_register_simple("microcode", -1,

							 NULL, 0);

	if (IS_ERR(microcode_pdev)) {

		microcode_dev_exit();

		return PTR_ERR(microcode_pdev);

	}



	printk(KERN_INFO 

		"IA-32 Microcode Update Driver: v" MICROCODE_VERSION " <tigran@veritas.com>\n");
@@ -509,6 +624,7 @@ static int __init microcode_init (void) 
static void __exit microcode_exit (void)

{

	microcode_dev_exit();

	platform_device_unregister(microcode_pdev);

}



module_init(microcode_init)

drivers/base/firmware_class.c

+1 −1
Original line number Diff line number Diff line
@@ -602,7 +602,7 @@ firmware_class_exit(void) 
	class_unregister(&firmware_class);

}



module_init(firmware_class_init);

fs_initcall(firmware_class_init);

module_exit(firmware_class_exit);



EXPORT_SYMBOL(release_firmware);

CRA Git | Maintained and supported by SUSTech CRA and CCSE