[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v3] os-android: Add support to android platf
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [RFC PATCH v3] os-android: Add support to android platform |
Date: |
Mon, 28 Sep 2015 13:40:38 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 24/09/2015 15:21, Houcheng Lin wrote:
> +if [ "$android" = "yes" ] ; then
> + LIBS="-lglib-2.0 -lgthread-2.0 -lz -lpixman-1 -lintl -liconv -lc $LIBS"
> + libs_qga="-lglib-2.0 -lgthread-2.0 -lz -lpixman-1 -lintl -liconv -lc"
> +fi
This change should not be necessary.
> +#define getdtablesize qemu_getdtablesize
Please instead replace all occurrences of getdtablesize with
qemu_getdtablesize.
>
> +#ifdef CONFIG_ANDROID
> +#include "sysemu/os-android.h"
> +#endif
> +
Please replace this with
#include <libgen.h>
#ifndef IOV_MAX
#define IOV_MAX 1024
#endif
and get rid of os-android.h.
>
> +#if defined(CONFIG_ANDROID)
> + char pty_buf[PATH_MAX];
> + #define ptsname(fd) pty_buf
> +#endif
> const char *slave;
> int mfd = -1, sfd = -1;
>
> @@ -67,17 +72,21 @@ static int openpty(int *amaster, int *aslave, char *name,
>
> if (grantpt(mfd) == -1 || unlockpt(mfd) == -1)
> goto err;
> -
> +#if defined(CONFIG_ANDROID)
> + if (ptsname_r(mfd, pty_buf, PATH_MAX) < 0)
> + goto err;
> +#endif
> if ((slave = ptsname(mfd)) == NULL)
> goto err;
>
Better:
#if defined(CONFIG_ANDROID)
char slave[PATH_MAX];
#else
const char *slave;
#endif
...
#if defined(CONFIG_ANDROID)
if (ptsname_r(mfd, slave, PATH_MAX) < 0)
goto err;
#else
if ((slave = ptsname(mfd)) == NULL)
goto err;
#endif