Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1653200

RE: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold blocks outside the list

Show all headers | View raw

> -----Original Message-----
> From: Borislav Petkov [mailto:bp@alien8.de]
> Sent: Sunday, May 28, 2017 1:22 PM
> To: Ghannam, Yazen <Yazen.Ghannam@amd.com>
> Cc: linux-edac@vger.kernel.org; Borislav Petkov <bp@suse.de>; Tony Luck
> <tony.luck@intel.com>; x86@kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold
> blocks outside the list
> 
> On Wed, May 24, 2017 at 03:41:46PM -0500, Yazen Ghannam wrote:
> > From: Yazen Ghannam <yazen.ghannam@amd.com>
> >
> > There needs to be a list_head outside of a linked list in order to
> > iterate over it and have access to all its elements. This is because
> > the
> > list_for_each* macros iterate starting from head->next rather than head.
> >
> > Define a list_head for the threshold blocks list in struct
> > threshold_bank
> 
> struct threshold_block {
> 
> 	...
> 
>         struct list_head miscj;                 /*
>                                                  * List of threshold blocks
>                                                  * within a bank.
>                                                  */
> 
> There's your list_head right there.
> 

Like I said in the commit message, the list_head needs to be outside the
list to access all the elements using list_for_each*. Otherwise, we won't
get a reference to the "head" element since we iterate starting from
head->next and break when !head.

For example here, we use block 0 as the head and I find that I don't get a
reference to it when using list_for_each* as the code is currently. Am I
doing something wrong?

Thanks,
Yazen

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold blocks  outside the list Borislav Petkov <bp@alien8.de> - 2017-05-28 19:30 +0200
  RE: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold blocks  outside the list "Ghannam, Yazen" <Yazen.Ghannam@amd.com> - 2017-05-30 14:50 +0200
    Re: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold blocks  outside the list Borislav Petkov <bp@alien8.de> - 2017-05-30 16:00 +0200
      RE: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold blocks  outside the list "Ghannam, Yazen" <Yazen.Ghannam@amd.com> - 2017-05-30 16:10 +0200
        Re: [PATCH 2/3] x86/mce/AMD: Define a list_head for threshold blocks  outside the list Borislav Petkov <bp@alien8.de> - 2017-05-30 16:20 +0200

csiph-web