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


Groups > linux.kernel > #1204972 > unrolled thread

Re: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path

Started byNikunj A Dadhania <nikunj@linux.vnet.ibm.com>
First post2015-08-11 12:00 +0200
Last post2015-08-18 05:20 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> - 2015-08-11 12:00 +0200
    Re: [PATCH RESEND] powerpc/numa: initialize distance lookup table  from drconf path Michael Ellerman <mpe@ellerman.id.au> - 2015-08-18 05:20 +0200

#1204972 — Re: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path

FromNikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Date2015-08-11 12:00 +0200
SubjectRe: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path
Message-ID<pWevE-1vq-5@gated-at.bofh.it>
Hi Michael,

Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> writes:
> In some situations, a NUMA guest that supports
> ibm,dynamic-memory-reconfiguration node will end up having flat NUMA
> distances between nodes. This is because of two problems in the
> current code.
>
> 1) Different representations of associativity lists.
>
>    There is an assumption about the associativity list in
>    initialize_distance_lookup_table(). Associativity list has two forms:
>
>    a) [cpu,memory]@x/ibm,associativity has following
>       format:
>            <N> <N integers>
>
>    b) ibm,dynamic-reconfiguration-memory/ibm,associativity-lookup-arrays
>
>            <M> <N> <M associativity lists each having N integers>
>            M = the number of associativity lists
>            N = the number of entries per associativity list
>
>    Fix initialize_distance_lookup_table() so that it does not assume
>    "case a". And update the caller to skip the length field before
>    sending the associativity list.
>
> 2) Distance table not getting updated from drconf path.
>
>    Node distance table will not get initialized in certain cases as
>    ibm,dynamic-reconfiguration-memory path does not initialize the
>    lookup table.
>
>    Call initialize_distance_lookup_table() from drconf path with
>    appropriate associativity list.
>
> Reported-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
> Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
> Acked-by: Anton Blanchard <anton@samba.org>

Have you pulled this?

Regards,
Nikunj

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1209021 — Re: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path

FromMichael Ellerman <mpe@ellerman.id.au>
Date2015-08-18 05:20 +0200
SubjectRe: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path
Message-ID<pYFBn-2aI-1@gated-at.bofh.it>
In reply to#1204972
On Tue, 2015-08-11 at 15:25 +0530, Nikunj A Dadhania wrote:
> Hi Michael,
> 
> Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> writes:
> > In some situations, a NUMA guest that supports
> > ibm,dynamic-memory-reconfiguration node will end up having flat NUMA
> > distances between nodes. This is because of two problems in the
> > current code.
> >
> > 1) Different representations of associativity lists.
> >
> >    There is an assumption about the associativity list in
> >    initialize_distance_lookup_table(). Associativity list has two forms:
> >
> >    a) [cpu,memory]@x/ibm,associativity has following
> >       format:
> >            <N> <N integers>
> >
> >    b) ibm,dynamic-reconfiguration-memory/ibm,associativity-lookup-arrays
> >
> >            <M> <N> <M associativity lists each having N integers>
> >            M = the number of associativity lists
> >            N = the number of entries per associativity list
> >
> >    Fix initialize_distance_lookup_table() so that it does not assume
> >    "case a". And update the caller to skip the length field before
> >    sending the associativity list.
> >
> > 2) Distance table not getting updated from drconf path.
> >
> >    Node distance table will not get initialized in certain cases as
> >    ibm,dynamic-reconfiguration-memory path does not initialize the
> >    lookup table.
> >
> >    Call initialize_distance_lookup_table() from drconf path with
> >    appropriate associativity list.
> >
> > Reported-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
> > Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
> > Acked-by: Anton Blanchard <anton@samba.org>
> 
> Have you pulled this?

Nope.

Will add it now.

cheers


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web