Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Thu, 2015-07-09 at 01:05 -0500, Zhao Qiang-B45475 wrote: On Wed, 2015-07-09 at 11:51 -0500, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 11:51 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ That doesn't involve a different alignment for each allocation. It uses the same alignment for all of them, and the alignment that cpm_common.c provides to rh_init() is 1 byte. ...but sigh, cpm_muram_alloc() is changing cpm_muram_info.alignment behind the rheap code's back. Despite the existence of rh_alloc_align(). So yes, add aligned allocation functionality to genalloc, but don't duplicate gen_pool_alloc() to do so. Instead, rename gen_pool_alloc() to gen_pool_alloc_align() with an alignment parameter (also modifying the algo function to take an alignment arg, which gen_pool_first_fit_order_align() would ignore), and provide a Here, I don’t understand how to handle the algo In your mind. Can you explain more detailly? The algorithms would be unchanged except that they would receive a new alignment (or alignment mask) parameter. gen_pool_first_fit_order_align() would ignore it, but the other algorithms would pass it through to the bitmap allocator. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Wed, 2015-07-09 at 11:51 -0500, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 11:51 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Wed, 2015-07-08 at 22:18 -0500, Zhao Qiang-B45475 wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 2:59 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Wed, 2015-07-08 at 02:25 -0500, Zhao Qiang-B45475 wrote: So I will add two func for my use, do you think it is ok? I need to align the address of allocated muram. And I will set algo = gen_pool_first_fit_align. +unsigned long gen_pool_alloc_align(struct gen_pool *pool, size_t size, + unsigned long align) Again, please explain why you need this for CPM/QE. I don't see rh_alloc_align() currently being used by either. Also, please stop top-posting. unsigned long rh_alloc(struct _rh_info *info, int size, const char *owner) { return rh_alloc_align(info, size, info-alignment, owner); } EXPORT_SYMBOL_GPL(rh_alloc); That doesn't involve a different alignment for each allocation. It uses the same alignment for all of them, and the alignment that cpm_common.c provides to rh_init() is 1 byte. ...but sigh, cpm_muram_alloc() is changing cpm_muram_info.alignment behind the rheap code's back. Despite the existence of rh_alloc_align(). So yes, add aligned allocation functionality to genalloc, but don't duplicate gen_pool_alloc() to do so. Instead, rename gen_pool_alloc() to gen_pool_alloc_align() with an alignment parameter (also modifying the algo function to take an alignment arg, which gen_pool_first_fit_order_align() would ignore), and provide a Here, I don’t understand how to handle the algo In your mind. Can you explain more detailly? gen_pool_alloc() wrapper that specifies 1 as the required alignment. Also be sure to CC lkml on the patchset since you're touching core code that doesn't have its own maintainer or list. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Wed, 2015-07-09 at 02:09PM -0500, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 2:09 PM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Thu, 2015-07-09 at 01:05 -0500, Zhao Qiang-B45475 wrote: On Wed, 2015-07-09 at 11:51 -0500, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 11:51 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ That doesn't involve a different alignment for each allocation. It uses the same alignment for all of them, and the alignment that cpm_common.c provides to rh_init() is 1 byte. ...but sigh, cpm_muram_alloc() is changing cpm_muram_info.alignment behind the rheap code's back. Despite the existence of rh_alloc_align(). So yes, add aligned allocation functionality to genalloc, but don't duplicate gen_pool_alloc() to do so. Instead, rename gen_pool_alloc() to gen_pool_alloc_align() with an alignment parameter (also modifying the algo function to take an alignment arg, which gen_pool_first_fit_order_align() would ignore), and provide a Here, I don’t understand how to handle the algo In your mind. Can you explain more detailly? The algorithms would be unchanged except that they would receive a new alignment (or alignment mask) parameter. gen_pool_first_fit_order_align() would ignore it, but the other algorithms would pass it through to the bitmap allocator. How about to add an align_mask parameter to gen_pool_first_fit? -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Thu, 2015-07-09 at 01:14 -0500, Zhao Qiang-B45475 wrote: On Wed, 2015-07-09 at 02:09PM -0500, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 2:09 PM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Thu, 2015-07-09 at 01:05 -0500, Zhao Qiang-B45475 wrote: On Wed, 2015-07-09 at 11:51 -0500, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 11:51 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ That doesn't involve a different alignment for each allocation. It uses the same alignment for all of them, and the alignment that cpm_common.c provides to rh_init() is 1 byte. ...but sigh, cpm_muram_alloc() is changing cpm_muram_info.alignment behind the rheap code's back. Despite the existence of rh_alloc_align(). So yes, add aligned allocation functionality to genalloc, but don't duplicate gen_pool_alloc() to do so. Instead, rename gen_pool_alloc() to gen_pool_alloc_align() with an alignment parameter (also modifying the algo function to take an alignment arg, which gen_pool_first_fit_order_align() would ignore), and provide a Here, I don’t understand how to handle the algo In your mind. Can you explain more detailly? The algorithms would be unchanged except that they would receive a new alignment (or alignment mask) parameter. gen_pool_first_fit_order_align() would ignore it, but the other algorithms would pass it through to the bitmap allocator. How about to add an align_mask parameter to gen_pool_first_fit? All of the algorithms have to have the same function signature. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
So I will add two func for my use, do you think it is ok? I need to align the address of allocated muram. And I will set algo = gen_pool_first_fit_align. +unsigned long gen_pool_alloc_align(struct gen_pool *pool, size_t size, + unsigned long align) +{ + struct gen_pool_chunk *chunk; + unsigned long addr = 0; + unsigned long align_mask; + int order = pool-min_alloc_order; + int nbits, start_bit = 0, end_bit, remain; + +#ifndef CONFIG_ARCH_HAVE_NMI_SAFE_CMPXCHG + BUG_ON(in_nmi()); +#endif + + if (size == 0) + return 0; + + align_mask = ((align + (1UL order) - 1) order) - 1; + nbits = (size + (1UL order) - 1) order; + rcu_read_lock(); + list_for_each_entry_rcu(chunk, pool-chunks, next_chunk) { + if (size atomic_read(chunk-avail)) + continue; + + end_bit = chunk_size(chunk) order; +retry: + start_bit = pool-algo(chunk-bits, end_bit, start_bit, nbits, + pool-data, align_mask); + if (start_bit = end_bit) + continue; + remain = bitmap_set_ll(chunk-bits, start_bit, nbits); + if (remain) { + remain = bitmap_clear_ll(chunk-bits, start_bit, +nbits - remain); + BUG_ON(remain); + goto retry; + } + + addr = chunk-start_addr + ((unsigned long)start_bit order); + size = nbits order; + atomic_sub(size, chunk-avail); + break; + } + rcu_read_unlock(); + return addr; +} +EXPORT_SYMBOL(gen_pool_alloc); + * gen_pool_first_fit_align - find the first available region + * of memory matching the size requirement. The region will be aligned + * to the order of the size specified. + * @map: The address to base the search on + * @size: The bitmap size in bits + * @start: The bitnumber to start searching at + * @nr: The number of zeroed bits we're looking for + * @data: additional data - unused + */ +unsigned long gen_pool_first_fit_align(unsigned long *map, + unsigned long size, unsigned long start, + unsigned int nr, void *data, unsigned long align_mask) +{ + return bitmap_find_next_zero_area(map, size, start, nr, align_mask); +} +EXPORT_SYMBOL(gen_pool_first_fit_align); + Best Regards Zhao Qiang -Original Message- From: Wood Scott-B07421 Sent: Wednesday, July 08, 2015 11:28 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Tue, 2015-07-07 at 22:26 -0500, Zhao Qiang-B45475 wrote: Now the point is, genalloc is not so proper to qe muram while rheap is written to manage muram, rheap is not specific to muram. if use genalloc instead of rheap, there will be amounts of work to do. Not much. I think I've spent more time responding to continual e-mails on this topic than it would have taken to convert it. :-P I have a suggestion, how about to put rheap under drivers/soc/qe, because rheap is To manage muram when it is added to sdk. No. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Wed, 2015-07-08 at 22:18 -0500, Zhao Qiang-B45475 wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 2:59 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Wed, 2015-07-08 at 02:25 -0500, Zhao Qiang-B45475 wrote: So I will add two func for my use, do you think it is ok? I need to align the address of allocated muram. And I will set algo = gen_pool_first_fit_align. +unsigned long gen_pool_alloc_align(struct gen_pool *pool, size_t size, + unsigned long align) Again, please explain why you need this for CPM/QE. I don't see rh_alloc_align() currently being used by either. Also, please stop top-posting. unsigned long rh_alloc(struct _rh_info *info, int size, const char *owner) { return rh_alloc_align(info, size, info-alignment, owner); } EXPORT_SYMBOL_GPL(rh_alloc); That doesn't involve a different alignment for each allocation. It uses the same alignment for all of them, and the alignment that cpm_common.c provides to rh_init() is 1 byte. ...but sigh, cpm_muram_alloc() is changing cpm_muram_info.alignment behind the rheap code's back. Despite the existence of rh_alloc_align(). So yes, add aligned allocation functionality to genalloc, but don't duplicate gen_pool_alloc() to do so. Instead, rename gen_pool_alloc() to gen_pool_alloc_align() with an alignment parameter (also modifying the algo function to take an alignment arg, which gen_pool_first_fit_order_align() would ignore), and provide a gen_pool_alloc() wrapper that specifies 1 as the required alignment. Also be sure to CC lkml on the patchset since you're touching core code that doesn't have its own maintainer or list. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
-Original Message- From: Wood Scott-B07421 Sent: Thursday, July 09, 2015 2:59 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Wed, 2015-07-08 at 02:25 -0500, Zhao Qiang-B45475 wrote: So I will add two func for my use, do you think it is ok? I need to align the address of allocated muram. And I will set algo = gen_pool_first_fit_align. +unsigned long gen_pool_alloc_align(struct gen_pool *pool, size_t size, + unsigned long align) Again, please explain why you need this for CPM/QE. I don't see rh_alloc_align() currently being used by either. Also, please stop top-posting. unsigned long rh_alloc(struct _rh_info *info, int size, const char *owner) { return rh_alloc_align(info, size, info-alignment, owner); } EXPORT_SYMBOL_GPL(rh_alloc); -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Wed, 2015-07-08 at 02:25 -0500, Zhao Qiang-B45475 wrote: So I will add two func for my use, do you think it is ok? I need to align the address of allocated muram. And I will set algo = gen_pool_first_fit_align. +unsigned long gen_pool_alloc_align(struct gen_pool *pool, size_t size, + unsigned long align) Again, please explain why you need this for CPM/QE. I don't see rh_alloc_align() currently being used by either. Also, please stop top-posting. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
I need to ensure one thing, In your point, you want me to use lib/genalloc.c instead of rheap.c. Or just think rheap.c is not proper to put into lib? Best Regards Zhao Qiang -Original Message- From: Wood Scott-B07421 Sent: Friday, June 05, 2015 6:41 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Thu, 2015-06-04 at 04:27 -0500, Zhao Qiang-B45475 wrote: On Thu, 2015-06-04 at 2:14PM, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, June 04, 2015 2:14 PM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ 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. I need to use the func rh_alloc_align, what it the similar func? I just find a func gen_pool_first_fit_order_align. I don't see anywhere the QE code currently calls rh_alloc_align() -- and cpm_muram_init() calls rh_init() with an alignment of 1. If you do need some functionality that genalloc doesn't offer, add the functionality to genalloc. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Tue, 2015-07-07 at 21:54 -0500, Zhao Qiang-B45475 wrote: I need to ensure one thing, In your point, you want me to use lib/genalloc.c instead of rheap.c. Or just think rheap.c is not proper to put into lib? Best Regards Zhao Qiang I want you to use lib/genalloc.c. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
Now the point is, genalloc is not so proper to qe muram while rheap is written to manage muram, if use genalloc instead of rheap, there will be amounts of work to do. I have a suggestion, how about to put rheap under drivers/soc/qe, because rheap is To manage muram when it is added to sdk. Best Regards Zhao Qiang -Original Message- From: Wood Scott-B07421 Sent: Wednesday, July 08, 2015 10:55 AM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ On Tue, 2015-07-07 at 21:54 -0500, Zhao Qiang-B45475 wrote: I need to ensure one thing, In your point, you want me to use lib/genalloc.c instead of rheap.c. Or just think rheap.c is not proper to put into lib? Best Regards Zhao Qiang I want you to use lib/genalloc.c. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Tue, 2015-07-07 at 22:26 -0500, Zhao Qiang-B45475 wrote: Now the point is, genalloc is not so proper to qe muram while rheap is written to manage muram, rheap is not specific to muram. if use genalloc instead of rheap, there will be amounts of work to do. Not much. I think I've spent more time responding to continual e-mails on this topic than it would have taken to convert it. :-P I have a suggestion, how about to put rheap under drivers/soc/qe, because rheap is To manage muram when it is added to sdk. No. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Thu, 2015-06-04 at 04:27 -0500, Zhao Qiang-B45475 wrote: On Thu, 2015-06-04 at 2:14PM, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, June 04, 2015 2:14 PM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ 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. I need to use the func rh_alloc_align, what it the similar func? I just find a func gen_pool_first_fit_order_align. I don't see anywhere the QE code currently calls rh_alloc_align() -- and cpm_muram_init() calls rh_init() with an alignment of 1. If you do need some functionality that genalloc doesn't offer, add the functionality to genalloc. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
On Thu, 2015-06-04 at 2:14PM, Wood Scott wrote: -Original Message- From: Wood Scott-B07421 Sent: Thursday, June 04, 2015 2:14 PM To: Zhao Qiang-B45475 Cc: linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 Subject: Re: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/ 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. I need to use the func rh_alloc_align, what it the similar func? I just find a func gen_pool_first_fit_order_align. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 2/2] rheap: move rheap.c from arch/powerpc/lib/ to lib/
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. Also, please don't use coreid-based e-mail addresses with no real names associated, which makes it hard to tell who has been CCed. -Scott Best Regards Zhao Qiang ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
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. Also, please don't use coreid-based e-mail addresses with no real names associated, which makes it hard to tell who has been CCed. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev