qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] make error
@ 2018-02-20 15:40 Ronald Delobel
  2018-03-06 11:31 ` Stefan Hajnoczi
  0 siblings, 1 reply; 2+ messages in thread
From: Ronald Delobel @ 2018-02-20 15:40 UTC (permalink / raw)
  To: qemu-devel

Hello,

This is the error I get when building qemu-2.11.[0-1] with
gcc-7.3.0/glibc-2.27, removing the static attribute suffice to finnish the
build using the glibc provided memfd_create and qemu don't crash, but I
miss some knowledge here about what a static function really is, a kind of
private one? Shouldn't it be renamed instead and stay as static?

Just to report it.


  CC      util/qemu-thread-posix.o
  CC      util/memfd.o
util/memfd.c:40:12: error: la déclaration statique de « memfd_create » suit
une déclaration non statique
 static int memfd_create(const char *name, unsigned int flags)
            ^~~~~~~~~~~~
In file included from /usr/include/bits/mman-linux.h:115:0,
                 from /usr/include/bits/mman.h:45,
                 from /usr/include/sys/mman.h:41,
                 from
/usr/local/src/qemu-2.11.1/include/sysemu/os-posix.h:29,
                 from /usr/local/src/qemu-2.11.1/include/qemu/osdep.h:104,
                 from util/memfd.c:28:
/usr/include/bits/mman-shared.h:46:5: note: la déclaration précédente de
« memfd_create » était ici
 int memfd_create (const char *__name, unsigned int __flags) __THROW;
     ^~~~~~~~~~~~
make: *** [/usr/local/src/qemu-2.11.1/rules.mak:66: util/memfd.o] Error 1

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] make error
  2018-02-20 15:40 [Qemu-devel] make error Ronald Delobel
@ 2018-03-06 11:31 ` Stefan Hajnoczi
  0 siblings, 0 replies; 2+ messages in thread
From: Stefan Hajnoczi @ 2018-03-06 11:31 UTC (permalink / raw)
  To: Ronald Delobel; +Cc: qemu-devel

[-- Attachment #1: Type: text/plain, Size: 1674 bytes --]

On Tue, Feb 20, 2018 at 04:40:49PM +0100, Ronald Delobel wrote:
> Hello,
> 
> This is the error I get when building qemu-2.11.[0-1] with
> gcc-7.3.0/glibc-2.27, removing the static attribute suffice to finnish the
> build using the glibc provided memfd_create and qemu don't crash, but I
> miss some knowledge here about what a static function really is, a kind of
> private one? Shouldn't it be renamed instead and stay as static?
> 
> Just to report it.

Thanks for reporting this.  Peter Maydell looked into this and answered
it in another email thread:

  "this looks like a QEMU incompatibility with newer glibc that
  we fixed in QEMU commit 75e5b70e6b5dcc4. That didn't quite make
  the 2.11.0 release, but will be in 2.12.0."

>   CC      util/qemu-thread-posix.o
>   CC      util/memfd.o
> util/memfd.c:40:12: error: la déclaration statique de « memfd_create » suit
> une déclaration non statique
>  static int memfd_create(const char *name, unsigned int flags)
>             ^~~~~~~~~~~~
> In file included from /usr/include/bits/mman-linux.h:115:0,
>                  from /usr/include/bits/mman.h:45,
>                  from /usr/include/sys/mman.h:41,
>                  from
> /usr/local/src/qemu-2.11.1/include/sysemu/os-posix.h:29,
>                  from /usr/local/src/qemu-2.11.1/include/qemu/osdep.h:104,
>                  from util/memfd.c:28:
> /usr/include/bits/mman-shared.h:46:5: note: la déclaration précédente de
> « memfd_create » était ici
>  int memfd_create (const char *__name, unsigned int __flags) __THROW;
>      ^~~~~~~~~~~~
> make: *** [/usr/local/src/qemu-2.11.1/rules.mak:66: util/memfd.o] Error 1

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-03-06 11:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-20 15:40 [Qemu-devel] make error Ronald Delobel
2018-03-06 11:31 ` Stefan Hajnoczi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).