lkml.org 
[lkml]   [2012]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3 07/31] edac: Prepare to push down to drivers the filling of the dimm_info
    Date
    Several data that it is currently stored per csrow will be
    pushed into the dimm themselves. Due to that, the mci->dimms
    filling will need to happen inside the drivers.

    Prepare for that, by initializing the dimm fields during mci
    alloc. With this change, the changes at the drivers will be
    smaller, as they won't need to touch at the fields they don't
    currently initialize.

    No functional changes.

    Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
    ---
    drivers/edac/edac_mc.c | 43 ++++++++++++++++++++-----------------------
    drivers/edac/i5100_edac.c | 1 +
    drivers/edac/i7core_edac.c | 1 +
    drivers/edac/i82975x_edac.c | 1 +
    drivers/edac/sb_edac.c | 1 +
    5 files changed, 24 insertions(+), 23 deletions(-)

    diff --git a/drivers/edac/edac_mc.c b/drivers/edac/edac_mc.c
    index 93ef044..37dca79 100644
    --- a/drivers/edac/edac_mc.c
    +++ b/drivers/edac/edac_mc.c
    @@ -214,6 +214,24 @@ struct mem_ctl_info *edac_mc_alloc(unsigned sz_pvt, unsigned nr_csrows,
    }
    }

    + /*
    + * By default, assumes that a per-csrow arrangement will be used,
    + * as most drivers are based on such assumption.
    + */
    + if (!mci->nr_dimms) {
    + mci->dimm_loc_type = DIMM_LOC_CSROW;
    + dimm = mci->dimms;
    + for (row = 0; row < mci->nr_csrows; row++) {
    + for (chn = 0; chn < mci->csrows[row].nr_channels; chn++) {
    + mci->csrows[row].channels[chn].dimm = dimm;
    + dimm->location.csrow = row;
    + dimm->location.csrow_channel = chn;
    + dimm++;
    + mci->nr_dimms++;
    + }
    + }
    + }
    +
    mci->op_state = OP_ALLOC;
    INIT_LIST_HEAD(&mci->grp_kobj_list);

    @@ -512,37 +530,16 @@ EXPORT_SYMBOL(edac_mc_find);
    /* FIXME - should a warning be printed if no error detection? correction? */
    int edac_mc_add_mc(struct mem_ctl_info *mci)
    {
    - int i, j;
    - struct dimm_info *dimm;
    -
    debugf0("%s()\n", __func__);

    - /*
    - * If nr_dimms is not filled, that means that the driver itself
    - * were not converted to use the new struct, or that the driver
    - * is for a csrow-based device.
    - * Fill the dimms accordingly.
    - */
    - if (!mci->nr_dimms) {
    - mci->dimm_loc_type = DIMM_LOC_CSROW;
    - dimm = mci->dimms;
    - for (i = 0; i < mci->nr_csrows; i++) {
    - for (j = 0; j < mci->csrows[i].nr_channels; j++) {
    - mci->csrows[i].channels[j].dimm = dimm;
    - dimm->location.csrow = i;
    - dimm->location.csrow_channel = j;
    - dimm++;
    - mci->nr_dimms++;
    - }
    - }
    - }
    #ifdef CONFIG_EDAC_DEBUG
    if (edac_debug_level >= 3)
    edac_mc_dump_mci(mci);

    if (edac_debug_level >= 4) {
    -
    + int i;
    for (i = 0; i < mci->nr_csrows; i++) {
    + int j;
    edac_mc_dump_csrow(&mci->csrows[i]);
    for (j = 0; j < mci->csrows[i].nr_channels; j++)
    edac_mc_dump_channel(&mci->csrows[i].
    diff --git a/drivers/edac/i5100_edac.c b/drivers/edac/i5100_edac.c
    index 302e43b..52939ca 100644
    --- a/drivers/edac/i5100_edac.c
    +++ b/drivers/edac/i5100_edac.c
    @@ -852,6 +852,7 @@ static void __devinit i5100_init_csrows(struct mem_ctl_info *mci)

    dimm = mci->dimms;
    mci->dimm_loc_type = DIMM_LOC_MC_CHANNEL;
    + mci->nr_dimms = 0;
    for (i = 0; i < mci->nr_csrows; i++) {
    const unsigned long npages = i5100_npages(mci, i);
    const unsigned chan = i5100_csrow_to_chan(mci, i);
    diff --git a/drivers/edac/i7core_edac.c b/drivers/edac/i7core_edac.c
    index 4819df8..d6dd9bf 100644
    --- a/drivers/edac/i7core_edac.c
    +++ b/drivers/edac/i7core_edac.c
    @@ -641,6 +641,7 @@ static int get_dimm_config(struct mem_ctl_info *mci)

    dimm = mci->dimms;
    mci->dimm_loc_type = DIMM_LOC_MC_CHANNEL;
    + mci->nr_dimms = 0;
    for (i = 0; i < NUM_CHANS; i++) {
    u32 data, dimm_dod[3], value[8];

    diff --git a/drivers/edac/i82975x_edac.c b/drivers/edac/i82975x_edac.c
    index 4a4026e..47f023e 100644
    --- a/drivers/edac/i82975x_edac.c
    +++ b/drivers/edac/i82975x_edac.c
    @@ -379,6 +379,7 @@ static void i82975x_init_csrows(struct mem_ctl_info *mci,

    mci->dimm_loc_type = DIMM_LOC_CSROW;
    dimm = mci->dimms;
    + mci->nr_dimms = 0;
    for (index = 0; index < mci->nr_csrows; index++) {
    csrow = &mci->csrows[index];

    diff --git a/drivers/edac/sb_edac.c b/drivers/edac/sb_edac.c
    index 34fa898..43fc65e 100644
    --- a/drivers/edac/sb_edac.c
    +++ b/drivers/edac/sb_edac.c
    @@ -614,6 +614,7 @@ static int get_dimm_config(struct mem_ctl_info *mci)

    dimm = mci->dimms;
    mci->dimm_loc_type = DIMM_LOC_MC_CHANNEL;
    + mci->nr_dimms = 0;
    for (i = 0; i < NUM_CHANNELS; i++) {
    u32 mtr;

    --
    1.7.8


    \
     
     \ /
      Last update: 2012-02-10 01:09    [W:4.312 / U:0.164 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site