On 12/16/2014 06:34 PM, Mike Holmes wrote:
nit: commit message does not follow the pattern of "component:
file/fn: description"
Pattern? Did we agree to use that pattern? I reworded commit with text
bellow. If needed I can reword
it one more time before push.
linux-generic: fix return value in pool_create
return correct error result if pool shm cannot be allocated
Fix to Bug #1002
Signed-off-by: Bill Fischofer <[email protected]>
Signed-off-by: Maxim Uvarov <[email protected]>
linux-generic: odp_buffer_pool_create: fix return type
change ODP_BUFFER_INVALID to ODP_BUFFER_POOL_INVALID
address bug #1002
Signed-off-by: Bill Fischofer <[email protected]
<mailto:[email protected]>>
---
On 16 December 2014 at 10:13, Maxim Uvarov <[email protected]
<mailto:[email protected]>> wrote:
Took this to queue will be merged.
Does it have reviewed-by assume you are adding yours ?
Maxim.
On 12/16/2014 01:59 PM, Bill Fischofer wrote:
Fix to Bug #1002
1002 is "Resolved fixed" already but this is not merged yet - confused
how that can be :)
Signed-off-by: Bill Fischofer <[email protected]
<mailto:[email protected]>>
---
v2 corrects commit message
platform/linux-generic/odp_buffer_pool.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/platform/linux-generic/odp_buffer_pool.c
b/platform/linux-generic/odp_buffer_pool.c
index e947dde..4c8b037 100644
--- a/platform/linux-generic/odp_buffer_pool.c
+++ b/platform/linux-generic/odp_buffer_pool.c
@@ -234,7 +234,7 @@ odp_buffer_pool_t
odp_buffer_pool_create(const char *name,
ODP_PAGE_SIZE, 0);
if (shm == ODP_SHM_INVALID) {
POOL_UNLOCK(&pool->s.lock);
- return ODP_BUFFER_INVALID;
+ return ODP_BUFFER_POOL_INVALID;
}
pool->s.pool_base_addr =
odp_shm_addr(shm);
} else {
_______________________________________________
lng-odp mailing list
[email protected] <mailto:[email protected]>
http://lists.linaro.org/mailman/listinfo/lng-odp
--
*Mike Holmes*
Linaro Sr Technical Manager
LNG - ODP
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp
