On 3/20/23 06:49, Juergen Gross wrote:
@@ -750,6 +750,7 @@ static int __init mtrr_init_finialize(void)
* TBD: is there any system with such CPU which supports
* suspend/resume? If no, we should remove the code.
*/
+ mtrr_value = kcalloc(num_var_ranges, sizeof(*mtrr_value),
GFP_KERNEL);
Theoretically dynamic allocation can fail, although it should not
happen as this
happens during kernel boot and the size is small. Maybe a WARN()?
Fine with me.
What *actually* happens if the system is running out of memory and this
is the _first_ failure? Does a WARN_ON() here help someone debug what
is going on?
Attachment:
OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key
Attachment:
OpenPGP_signature
Description: OpenPGP digital signature