Fixed.

-----Original Message-----
From: [email protected] 
[mailto:[email protected]] On Behalf Of Maxim Uvarov
Sent: Tuesday, February 24, 2015 8:07 AM
To: [email protected]
Subject: Re: [lng-odp] [PATCHv3 08/10] linux-generic: shm: add term_global

On 02/23/2015 11:36 PM, Robbie King wrote:
> +int odp_shm_term_global(void)
> +{
> +     int ret = 0;
> +
no need to set it to 0.

+       ret = munmap(odp_shm_tbl, sizeof(odp_shm_table_t));

Please add some debug message:
if (ret)
     ODP_ERR("unable to munmap\n.");

2 errors can be here: 1) area already unmapped 2)odp_shm_tbl memory 
corruption.


Maxim.
> +     return ret;
> +}
> +


_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to