| Message ID | 20220126191711.4917-2-kristen@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Fixups for overcommit limit pages | expand |
On Wed, Jan 26, 2022 at 11:17:10AM -0800, Kristen Carlson Accardi wrote: > Remove improper parentheses from calculation for available > backing bytes. Wtihout this fix, the result will be incorrect > due to rounding. > > Signed-off-by: Kristen Carlson Accardi <kristen@linux.intel.com> > Acked-by: Jarkko Sakkinen <jarkko@kernel.org> > --- > arch/x86/kernel/cpu/sgx/main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/kernel/cpu/sgx/main.c b/arch/x86/kernel/cpu/sgx/main.c > index 990f341bbd30..c4030fb608c6 100644 > --- a/arch/x86/kernel/cpu/sgx/main.c > +++ b/arch/x86/kernel/cpu/sgx/main.c > @@ -881,7 +881,7 @@ static bool __init sgx_page_cache_init(void) > return false; > } > > - available_backing_bytes = total_epc_bytes * (sgx_overcommit_percent / 100); > + available_backing_bytes = total_epc_bytes * sgx_overcommit_percent / 100; > atomic_long_set(&sgx_nr_available_backing_pages, available_backing_bytes >> PAGE_SHIFT); > > return true; > -- > 2.20.1 > You can turn it as: Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> /Jarkko
diff --git a/arch/x86/kernel/cpu/sgx/main.c b/arch/x86/kernel/cpu/sgx/main.c index 990f341bbd30..c4030fb608c6 100644 --- a/arch/x86/kernel/cpu/sgx/main.c +++ b/arch/x86/kernel/cpu/sgx/main.c @@ -881,7 +881,7 @@ static bool __init sgx_page_cache_init(void) return false; } - available_backing_bytes = total_epc_bytes * (sgx_overcommit_percent / 100); + available_backing_bytes = total_epc_bytes * sgx_overcommit_percent / 100; atomic_long_set(&sgx_nr_available_backing_pages, available_backing_bytes >> PAGE_SHIFT); return true;