[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [External] [PATCH] lib/vfscore: Correct the macro name of UK_LIBC_SYSCALLS


  • To: Jia He <justin.he@xxxxxxx>
  • From: "Jung, Alexander (Student)" <a.jung@xxxxxxxxxxxxxxx>
  • Date: Thu, 13 May 2021 14:33:22 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=lancaster.ac.uk; dmarc=pass action=none header.from=lancaster.ac.uk; dkim=pass header.d=lancaster.ac.uk; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0o4Y21kwONzbFm9qDFlRhgXzAzk405uDU+NmRb/1vMc=; b=Fdv4mRozOn1uvYgI9Y+4hEH88VPWOxbSLFobFg2L+k3FBYHJVNGVQgAJaSyHPPBh/jrwdp+lvHgtyVEet8RRZDQ6Tn6jwoyJDTg07w1t1Zlpb4+39ibJ9+GH1LfJun5chrOppZhfCpdw3AEZ0zw//cWM8SUkvhyNQzwG552VFNqOJ7M4P6ZuQM1RiFoUxZxXlLoXIM4io1gYTLMS4c70BPfC6wF24WGrc2Nj4af8lyyPCQmyftjJHSAK9HpQnL/iydcn97LIwBF9rm9Ig906EpbN4dm6bAldhUVSk2Bty4MRdlx3WJqFoReReOeQZ2MbRp7PCZjsJjwkhvGy0CDEWw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mSmd3nFel7QzKX8UUcc1cUBhnFQLA3Wd919aF4jLTX7fuqFGvADeRLUaEej9iDVaJWtjLBlcc5ryOWKQR6qOm8qMVOfBterGWPE5X/GGmvdKjYH1N4zkFB3WG0+hFjr+9dkh5eosJREHoxi7JH/OXBou9dUt923vN0NC3gAR503xSpNi0WJhnsNQ9Pd3Au3jPE99Y5Srd8CfWc64AlnB0ppqU/P/cg/ag2tfBngSqJ2qYgOqeaz9nForJrvWGwEYFsVjqorFX3zfwbBT7YVbHMgcu8NlbCKGt7asjSAPr3q+AY0S/N7Ry+4dUiIT3FcBhb5rv4EpSaRPmORpHtOBjQ==
  • Authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=lancaster.ac.uk;
  • Cc: "minios-devel@xxxxxxxxxxxxxxxxxxxx" <minios-devel@xxxxxxxxxxxxxxxxxxxx>, Simon Kuenzer <simon.kuenzer@xxxxxxxxx>, "Sharan.Santhanam@xxxxxxxxx" <Sharan.Santhanam@xxxxxxxxx>, Felipe Huici <felipe.huici@xxxxxxxxx>, "sergiu.moga@xxxxxxxxxxxxxx" <sergiu.moga@xxxxxxxxxxxxxx>
  • Delivery-date: Thu, 13 May 2021 14:33:31 +0000
  • List-id: Mini-os development list <minios-devel.lists.xenproject.org>
  • Thread-index: AQHXR9IcqfDXr+hozE+50Oc+ZcM796rhejIA
  • Thread-topic: [External] [PATCH] lib/vfscore: Correct the macro name of UK_LIBC_SYSCALLS

Hi Justin,

Thanks for this fix! :D  Turns out this “spelling mistake” occurs in several 
other places too.  For this reason we’ve created a PR on Github[0] with general 
fix and included your patch in the same series.  

Also, I couldn’t see your email turn up on patchwork for some reason — not sure 
why it wasn’t caught.  Anyway, with the migration to Github currently on-going, 
can you please submit new features/fixes as PRs to the Unikraft repo? :)

Thanks and all the best,

Alexander Jung,

Ph.D Student,
Networking & Systems Research Group, 
Lancaster University
net.scc.lancs.ac.uk

F4D5 C6B6 9139 4942 9407  7F54 ED30 1A8A 1763 1B62

[0]: https://github.com/unikraft/unikraft/pull/204

> On 13. May 2021, at 09:28, Jia He <justin.he@xxxxxxx> wrote:
> 
> This email originated outside the University. Check before clicking links or 
> attachments.
> 
> Correct the name from UK_LIBC_SYSCALL to UK_LIBC_SYSCALLS
> 
> Otherwise the linking will report:
> libs/newlib/musl-imported/src/termios/tcsetattr.c:35:
> undefined reference to `ioctl'
> /bin/ld:
> apps/sqlite/build/sqlite_kvm-x86_64.o: in
> function `tcgetattr':
> libs/newlib/musl-imported/src/termios/tcgetattr.c:30:
> undefined reference to `ioctl'
> 
> Reported-by: Alexander Jung <a.jung@xxxxxxxxxxx>
> Signed-off-by: Jia He <justin.he@xxxxxxx>
> ---
> lib/vfscore/main.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/vfscore/main.c b/lib/vfscore/main.c
> index 11bfa93..0ecad74 100644
> --- a/lib/vfscore/main.c
> +++ b/lib/vfscore/main.c
> @@ -673,7 +673,7 @@ out_errno:
>        return -error;
> }
> 
> -#if UK_LIBC_SYSCALL
> +#if UK_LIBC_SYSCALLS
> int ioctl(int fd, unsigned long int request, ...)
> {
>        va_list ap;
> --
> 2.17.1
> 
> 


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.