| [PATCH] vpu-io: fix IOSystemInit failure |
| |
| When using a kernel where user/kernel split is 3G/1G, the address |
| returned by IOGetVirtMem() can appear to be a negative int. |
| |
| IOSystemInit() incorrectly checks the return value of IOGetVirtMem(). |
| IOGetVirtMem() returns -1 on error (and not MAP_FAILED, nor any other |
| negative value.) |
| |
| Fix that by correctly checking against -1 (and not MAP_FAILED!) |
| |
| Signed-off-by: Gary Bisson <bisson.gary@gmail.com> |
| [yann.morin.1998@free.fr: expand the commit log] |
| Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> |
| --- |
| vpu/vpu_io.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/vpu/vpu_io.c b/vpu/vpu_io.c |
| index 8cbb571..14759da 100644 |
| --- a/vpu/vpu_io.c |
| +++ b/vpu/vpu_io.c |
| @@ -265,7 +265,7 @@ int IOSystemInit(void *callback) |
| goto err; |
| } |
| |
| - if (IOGetVirtMem(&bit_work_addr) <= 0) |
| + if (IOGetVirtMem(&bit_work_addr) == -1) |
| goto err; |
| #endif |
| UnlockVpu(vpu_semap); |
| |