| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <353238.3571.qm@web50101.mail.re2.yahoo.com>
Date: Wed, 2 Dec 2009 14:12:06 -0800 (PST)
From: Doug Thompson <norsk5@...oo.com>
To: Borislav Petkov <borislav.petkov@....com>,
Randy Dunlap <randy.dunlap@...cle.com>
Cc: Borislav Petkov <petkovbb@...glemail.com>,
LKML <linux-kernel@...r.kernel.org>,
Doug Thompson <dougthompson@...ssion.com>
Subject: Re: 2.6.32-rc8: amd64_edac slub error
--- On Wed, 12/2/09, Randy Dunlap <randy.dunlap@...cle.com> wrote:
> From: Randy Dunlap <randy.dunlap@...cle.com>
> Subject: Re: 2.6.32-rc8: amd64_edac slub error
> To: "Borislav Petkov" <borislav.petkov@....com>
> Cc: "Borislav Petkov" <petkovbb@...glemail.com>, "LKML" <linux-kernel@...r.kernel.org>, "Doug Thompson" <dougthompson@...ssion.com>
> Date: Wednesday, December 2, 2009, 11:11 AM
> On Wed, 2 Dec 2009 11:58:38 +0100
> Borislav Petkov wrote:
>
> > On Tue, Dec 01, 2009 at 09:19:31AM -0800, Randy Dunlap
> wrote:
> > > Here's the new log file (attached).
> >
> > Thanks for testing. Meanwhile, I noticed that the
> other places where
> > rdmsr_on_cpus() gets called with non-contigious
> cpumasks need fixing
> > too. Here's a version that takes care of that, I'd be
> nice if you could
> > give it a run too (patch against today's upstream).
> You could also
> > enforce the module loading by setting
> 'ecc_enable_override=1' to verify
> > the other rdmsr_on_cpus calls.
> >
> > Thanks.
>
> This patch also works for me. Thanks.
>
> Acked-by: Randy Dunlap <randy.dunlap@...cle.com>
Acked-by: Doug Thompson <dougthompson@...ssion.com>
>
>
> boot log attached.
>
> > ---
> > diff --git a/drivers/edac/amd64_edac.c
> b/drivers/edac/amd64_edac.c
> > index a38831c..da2428b 100644
> > --- a/drivers/edac/amd64_edac.c
> > +++ b/drivers/edac/amd64_edac.c
> > @@ -2618,6 +2618,9 @@ static int
> amd64_init_csrows(struct mem_ctl_info *mci)
> > return empty;
> > }
> >
> > +static struct msr *alloc_msrs(const cpumask_t
> *mask);
> > +static void free_msrs(struct msr *msrs);
> > +
> > /*
> > * Only if 'ecc_enable_override' is
> set AND BIOS had ECC disabled, do "we"
> > * enable it.
> > @@ -2627,14 +2630,16 @@ static void
> amd64_enable_ecc_error_reporting(struct mem_ctl_info *mci)
> > struct amd64_pvt *pvt =
> mci->pvt_info;
> > const cpumask_t *cpumask =
> cpumask_of_node(pvt->mc_node_id);
> > int cpu, idx = 0, err = 0;
> > - struct msr
> msrs[cpumask_weight(cpumask)];
> > + struct msr *msrs;
> > u32 value;
> > u32 mask = K8_NBCTL_CECCEn |
> K8_NBCTL_UECCEn;
> >
> > if (!ecc_enable_override)
> > return;
> >
> > - memset(msrs, 0, sizeof(msrs));
> > + msrs = alloc_msrs(cpumask);
> > + if (!msrs)
> > + return;
> >
> > amd64_printk(KERN_WARNING,
> >
> "'ecc_enable_override' parameter is active, "
> > @@ -2697,20 +2702,24 @@ static void
> amd64_enable_ecc_error_reporting(struct mem_ctl_info *mci)
> > (value
> & K8_NBCFG_ECC_ENABLE) ? "Enabled" : "Disabled");
> >
> > pvt->ctl_error_info.nbcfg
> = value;
> > +
> > + free_msrs(msrs);
> > }
> >
> > static void
> amd64_restore_ecc_error_reporting(struct amd64_pvt *pvt)
> > {
> > const cpumask_t *cpumask =
> cpumask_of_node(pvt->mc_node_id);
> > int cpu, idx = 0, err = 0;
> > - struct msr
> msrs[cpumask_weight(cpumask)];
> > + struct msr *msrs;
> > u32 value;
> > u32 mask = K8_NBCTL_CECCEn |
> K8_NBCTL_UECCEn;
> >
> > if
> (!pvt->nbctl_mcgctl_saved)
> > return;
> >
> > - memset(msrs, 0, sizeof(msrs));
> > + msrs = alloc_msrs(cpumask);
> > + if (!msrs)
> > + return;
> >
> > err =
> pci_read_config_dword(pvt->misc_f3_ctl, K8_NBCTL,
> &value);
> > if (err)
> > @@ -2731,6 +2740,8 @@ static void
> amd64_restore_ecc_error_reporting(struct amd64_pvt *pvt)
> > }
> >
> > wrmsr_on_cpus(cpumask,
> K8_MSR_MCGCTL, msrs);
> > +
> > + free_msrs(msrs);
> > }
> >
> > /* get all cores on this DCT */
> > @@ -2743,6 +2754,40 @@ static void
> get_cpus_on_this_dct_cpumask(cpumask_t *mask, int nid)
> >
> cpumask_set_cpu(cpu, mask);
> > }
> >
> > +/*
> > + * Allocate enough msr structs for the supplied
> cpumask. Also, take care of
> > + * non-contigious bitmasks.
> > + */
> > +static struct msr *alloc_msrs(const cpumask_t *mask)
> > +{
> > + struct msr *msrs;
> > + int i, first_cpu, last_cpu = 0;
> > +
> > + if (cpumask_empty(mask)) {
> > +
> amd64_printk(KERN_WARNING, "%s: Empty cpumask!\n",
> __func__);
> > + return NULL;
> > + }
> > +
> > + first_cpu = cpumask_first(mask);
> > + for (i = first_cpu; i <
> nr_cpu_ids; i++)
> > + if
> (cpumask_test_cpu(i, mask))
> > +
> last_cpu = i;
> > +
> > + msrs = kzalloc(sizeof(*msrs) *
> (last_cpu - first_cpu + 1), GFP_KERNEL);
> > + if (!msrs) {
> > +
> amd64_printk(KERN_WARNING, "%s: error allocating msrs\n",
> > +
> __func__);
> > +
> return NULL;
> > + }
> > +
> > + return msrs;
> > +}
> > +
> > +static void free_msrs(struct msr *msrs)
> > +{
> > + kfree(msrs);
> > +}
> > +
> > /* check MCG_CTL on all the cpus on this node
> */
> > static bool
> amd64_nb_mce_bank_enabled_on_node(int nid)
> > {
> > @@ -2755,12 +2800,9 @@ static bool
> amd64_nb_mce_bank_enabled_on_node(int nid)
> >
> >
> get_cpus_on_this_dct_cpumask(&mask, nid);
> >
> > - msrs = kzalloc(sizeof(struct msr)
> * cpumask_weight(&mask), GFP_KERNEL);
> > - if (!msrs) {
> > -
> amd64_printk(KERN_WARNING, "%s: error allocating msrs\n",
> > -
> __func__);
> > -
> return false;
> > - }
> > + msrs = alloc_msrs(&mask);
> > + if (!msrs)
> > + goto out_err;
> >
> > rdmsr_on_cpus(&mask,
> MSR_IA32_MCG_CTL, msrs);
> >
> > @@ -2779,7 +2821,9 @@ static bool
> amd64_nb_mce_bank_enabled_on_node(int nid)
> > ret = true;
> >
> > out:
> > - kfree(msrs);
> > + free_msrs(msrs);
> > +
> > +out_err:
> > return ret;
> > }
> >
> >
> > --
> > Regards/Gruss,
> > Boris.
> >
> > Operating | Advanced Micro Devices GmbH
> > System |
> Karl-Hammerschmidt-Str. 34, 85609 Dornach b. München,
> Germany
> > Research | Geschäftsführer: Andrew Bowd,
> Thomas M. McCoy, Giuliano Meroni
> > Center | Sitz: Dornach,
> Gemeinde Aschheim, Landkreis München
> > (OSRC) | Registergericht
> München, HRB Nr. 43632
> >
>
>
> ---
> ~Randy
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists