On Fri, Mar 08, 2019 at 12:14:15PM +0100, Daniel Kiper wrote:
> On Fri, Mar 08, 2019 at 01:26:37AM +0100, Jesús Diéguez Fernández wrote:
> > In order to be able to read from and write to model-specific registers,
> > two new modules are added. They are i386 specific, as the cpuid module.
> >
> > rdmsr module registers the command rdmsr that allows reading from a MSR.
> > wrmsr module registers the command wrmsr that allows writing to a MSR.
> >
> > wrmsr module is disabled if UEFI secure boot is enabled.
> >
> > Please note that on SMP systems, interacting with a MSR that has a
> > scope per hardware thread, implies that the value only applies to
> > the particular cpu/core/thread that ran the command.
> >
> > Also, if you specify a reserved or unimplemented MSR address, it will
> > cause a general protection exception (which is not currently being handled)
> > and the system will reboot.
> >
> > Signed-off-by: Jesús Diéguez Fernández <jesu...@gmail.com>
>
> LGTM. Reviewed-by: Daniel Kiper <daniel.ki...@oracle.com>
>
> There are a few comments nitpicks which I fix before committing the patches.
>
> If there are no objections I will commit the patches next week.
>
> Thank you for doing the work.

I had to tweak your patches because they broke build at least for coreboot
and x86-64 UEFI. Now they are in.

Daniel

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to