github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-07-27 01:32:21 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

EDAC/i7300: disable error reporting if init fails and refactor helper

Browse Source 
If error reporting is enabled during initialization but initialization
fails immediately after, or during normal driver exit, error reporting
is left enabled in the mask register even after exit.

Replace i7300_enable_error_reporting() with i7300_set_error_reporting()
to combine enabling/disabling. Disable reporting at initialization
failure and driver exit, before call to i7300_put_devices() for cleanup.
Add enabled reporting flag to i7300_pvt.

This ensures clean hardware handling by disabling any unused error
reporting bits before exiting.

Signed-off-by: Tushar Tibude <tushar.tibude1000@gmail.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Reviewed-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Link: https://patch.msgid.link/20260429094806.25097-1-tushar.tibude1000@gmail.com
This commit is contained in:
Tushar Tibude 2026-04-29 15:18:06 +05:30 committed by Tony Luck
parent e360a6d65b
commit 7944f44098
1 changed files with 24 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
drivers/edac/i7300_edac.c
View File

@ -111,6 +111,9 @@ struct i7300_pvt {
/* Temporary buffer for use when preparing error messages */
char *tmp_prt_buffer;
/* Hardware error reporting status */
bool enabled_error_reporting;
};
/* FIXME: Why do we need to have this static? */
@ -550,11 +553,12 @@ static void i7300_clear_error(struct mem_ctl_info *mci)
}
/**
* i7300_enable_error_reporting() - Enable the memory reporting logic at the
* i7300_set_error_reporting() - Enable or disable the memory reporting logic at the
* hardware
* @mci: struct mem_ctl_info pointer
* @enable: enables if 'true', disables if 'false'
*/
static void i7300_enable_error_reporting(struct mem_ctl_info *mci)
static void i7300_set_error_reporting(struct mem_ctl_info *mci, bool enable)
{
struct i7300_pvt *pvt = mci->pvt_info;
u32 fbd_error_mask;
@ -563,8 +567,11 @@ static void i7300_enable_error_reporting(struct mem_ctl_info *mci)
pci_read_config_dword(pvt->pci_dev_16_1_fsb_addr_map,
EMASK_FBD, &fbd_error_mask);
/* Enable with a '0' */
fbd_error_mask &= ~(EMASK_FBD_ERR_MASK);
/* Enable with 0, disable with 1 */
if (enable)
fbd_error_mask &= ~(EMASK_FBD_ERR_MASK);
else
fbd_error_mask |= EMASK_FBD_ERR_MASK;
pci_write_config_dword(pvt->pci_dev_16_1_fsb_addr_map,
EMASK_FBD, fbd_error_mask);
@ -1087,17 +1094,19 @@ static int i7300_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
if (i7300_get_mc_regs(mci)) {
edac_dbg(0, "MC: Setting mci->edac_cap to EDAC_FLAG_NONE because i7300_init_csrows() returned nonzero value\n");
mci->edac_cap = EDAC_FLAG_NONE; /* no csrows found */
pvt->enabled_error_reporting = false;
} else {
edac_dbg(1, "MC: Enable error reporting now\n");
i7300_enable_error_reporting(mci);
i7300_set_error_reporting(mci, true);
pvt->enabled_error_reporting = true;
}
/* add this new MC control structure to EDAC's list of MCs */
if (edac_mc_add_mc(mci)) {
edac_dbg(0, "MC: failed edac_mc_add_mc()\n");
/* FIXME: perhaps some code should go here that disables error
* reporting if we just enabled it
*/
/* Disable error reporting if we just enabled it */
if (pvt->enabled_error_reporting)
i7300_set_error_reporting(mci, false);
goto fail1;
}
@ -1134,6 +1143,7 @@ static int i7300_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
static void i7300_remove_one(struct pci_dev *pdev)
{
struct mem_ctl_info *mci;
struct i7300_pvt *pvt;
char *tmp;
edac_dbg(0, "\n");
@ -1145,7 +1155,12 @@ static void i7300_remove_one(struct pci_dev *pdev)
if (!mci)
return;
tmp = ((struct i7300_pvt *)mci->pvt_info)->tmp_prt_buffer;
pvt = (struct i7300_pvt *)mci->pvt_info;
tmp = pvt->tmp_prt_buffer;
/* Disable error reporting before unregistering device */
if (pvt->enabled_error_reporting)
i7300_set_error_reporting(mci, false);
/* retrieve references to resources, and free those resources */
i7300_put_devices(mci);