Messages in this thread | | | Subject | Re: [PATCH 5/7] x86: Use IBRS for firmware update path | From | Tim Chen <> | Date | Thu, 4 Jan 2018 13:13:44 -0800 |
| |
On 01/04/2018 12:51 PM, Yves-Alexis Perez wrote: > On Thu, 2018-01-04 at 09:56 -0800, Tim Chen wrote: >> @@ -44,6 +47,7 @@ static inline void apm_bios_call_asm(u32 func, u32 ebx_in, >> u32 ecx_in, >> "=S" (*esi) >> : "a" (func), "b" (ebx_in), "c" (ecx_in) >> : "memory", "cc"); >> + unprotected_formware_end(); > > Just an obvious typo (form/firm) but I thought I'd report it. >> } >> >> static inline bool apm_bios_call_simple_asm(u32 func, u32 ebx_in, >> @@ -52,6 +56,7 @@ static inline bool apm_bios_call_simple_asm(u32 func, u32 >> ebx_in, >> int cx, dx, si; >> bool error; >> >> + unprotected_firmware_begin(); >> /* >> * N.B. We do NOT need a cld after the BIOS call >> * because we always save and restore the flags. >> @@ -68,6 +73,7 @@ static inline bool apm_bios_call_simple_asm(u32 func, u32 >> ebx_in, >> "=S" (si) >> : "a" (func), "b" (ebx_in), "c" (ecx_in) >> : "memory", "cc"); >> + unprotected_formware_end(); >> return error; >> } > > Same here. >
Thanks. Noted.
Tim
| |