EDAC/mc: Do not BUG_ON() in edac_mc_alloc()
No need to crash the system in case edac_mc_alloc() is called with invalid arguments, just warn and return. This would cause a checkpatch warning when touching the code later, so just fix it. Signed-off-by: Robert Richter <rrichter@marvell.com> Signed-off-by: Borislav Petkov <bp@suse.de> Reviewed-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org> Cc: "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org> Cc: James Morse <james.morse@arm.com> Cc: Tony Luck <tony.luck@intel.com> Link: https://lkml.kernel.org/r/20191106093239.25517-5-rrichter@marvell.com
This commit is contained in:
parent
c498afaf7d
commit
d260e8ff51
|
@ -323,7 +323,8 @@ struct mem_ctl_info *edac_mc_alloc(unsigned int mc_num,
|
||||||
int i, j, row, chn, n, len;
|
int i, j, row, chn, n, len;
|
||||||
bool per_rank = false;
|
bool per_rank = false;
|
||||||
|
|
||||||
BUG_ON(n_layers > EDAC_MAX_LAYERS || n_layers == 0);
|
if (WARN_ON(n_layers > EDAC_MAX_LAYERS || n_layers == 0))
|
||||||
|
return NULL;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Calculate the total amount of dimms and csrows/cschannels while
|
* Calculate the total amount of dimms and csrows/cschannels while
|
||||||
|
|
Loading…
Reference in New Issue