From: Scott Wood <scottwood@freescale•com>
To: Zhao Qiang-B45475 <qiang.zhao@freescale•com>
Cc: "linuxppc-dev@lists•ozlabs.org" <linuxppc-dev@lists•ozlabs.org>,
"Xie Xiaobo-R63061" <X.Xie@freescale•com>
Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
Date: Thu, 4 Jun 2015 01:13:43 -0500 [thread overview]
Message-ID: <1433398423.3415.34.camel@freescale.com> (raw)
In-Reply-To: <SN1PR0301MB15500729CC3A65394FED67389BB30@SN1PR0301MB1550.namprd03.prod.outlook.com>
On Thu, 2015-06-04 at 00:56 -0500, Zhao Qiang-B45475 wrote:
> On Thu, 2015-05-28 at 1:37AM +0800, Wood Scott wrote:
>
>
> > -----Original Message-----
> > From: Wood Scott-B07421
> > Sent: Thursday, May 28, 2015 1:37 AM
> > To: Zhao Qiang-B45475
> > Cc: linuxppc-dev@lists•ozlabs.org; Wood Scott-B07421; Xie Xiaobo-
> > R63061
> > Subject: Re: [PATCH 2/2] rheap: move rheap.c from
> > arch/powerpc/lib/ to
> > lib/
> >
> > On Wed, 2015-05-27 at 17:12 +0800, Zhao Qiang wrote:
> > > qe need to use the rheap, so move it to public directory.
> >
> > You've been previously asked to use lib/genalloc.c rather than
> > introduce
> > duplicate functionality into /lib. NACK.
>
> Can't use lib/genalloc.c instead of rheap.c.
> Qe need to alloc muram of qe, not DIMM.
lib/genalloc.h is not for allocating main memory. It is for
allocating special regions. It is serving the same purpose as rheap.
-Scott
next prev parent reply other threads:[~2015-06-04 6:13 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-27 9:11 [PATCH 1/2] qe_common: add qe common functions into qe_common.c Zhao Qiang
2015-05-27 9:12 ` [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ Zhao Qiang
2015-05-27 17:36 ` Scott Wood
2015-06-04 5:56 ` Zhao Qiang
2015-06-04 6:13 ` Scott Wood [this message]
2015-06-04 9:27 ` Zhao Qiang
2015-06-04 22:40 ` Scott Wood
2015-07-08 2:54 ` Zhao Qiang
2015-07-08 2:55 ` Scott Wood
2015-07-08 3:26 ` Zhao Qiang
2015-07-08 3:27 ` Scott Wood
2015-07-08 7:25 ` Zhao Qiang
2015-07-08 18:58 ` Scott Wood
2015-07-09 3:18 ` Zhao Qiang
2015-07-09 3:50 ` Scott Wood
2015-07-09 6:05 ` Zhao Qiang
2015-07-09 6:09 ` Scott Wood
2015-07-09 6:14 ` Zhao Qiang
2015-07-09 6:47 ` Scott Wood
2015-05-27 22:51 ` [PATCH 1/2] qe_common: add qe common functions into qe_common.c Scott Wood
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1433398423.3415.34.camel@freescale.com \
--to=scottwood@freescale$(echo .)com \
--cc=X.Xie@freescale$(echo .)com \
--cc=linuxppc-dev@lists$(echo .)ozlabs.org \
--cc=qiang.zhao@freescale$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox