All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Mohring <martin.mohring@opensuse.org>
To: Laurent Desnogues <laurent.desnogues@gmail.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
	Riku Voipio <riku.voipio@iki.fi>,
	Mika Westerberg <mika.westerberg@iki.fi>,
	qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [Qemu-commits] [COMMIT 74d753a] linux-user: strace now handles guest stringscorrectly [v2]
Date: Wed, 17 Jun 2009 14:41:31 +0200	[thread overview]
Message-ID: <4A38E47B.3020806@opensuse.org> (raw)
In-Reply-To: <761ea48b0906170032g318457e4p81d028375971f2b2@mail.gmail.com>

Laurent Desnogues wrote:
> On Wed, Jun 17, 2009 at 9:18 AM, Riku Voipio<riku.voipio@iki.fi> wrote:
>   
>> On Wed, Jun 17, 2009 at 02:27:06AM +0400, malc wrote:
>>     
>>> On Tue, 16 Jun 2009, Anthony Liguori wrote:
>>> On my system the above line causes gcc to emit:
>>>       
>>> In file included from /home/malc/x/rcs/git/qemu/linux-user/strace.c:12:
>>> /usr/include/linux/futex.h:48: error: field `__user' has incomplete type
>>> /usr/include/linux/futex.h:48: error: syntax error before '*' token
>>> /usr/include/linux/futex.h:63: error: field `list' has incomplete type
>>> /usr/include/linux/futex.h:83: error: field `__user' has incomplete type
>>> /usr/include/linux/futex.h:83: error: syntax error before '*' token
>>> make[1]: *** [strace.o] Error 1
>>>       
>>> Following fixes it, but i wonder if there's better way:
>>>       
>> It doesn't look like your kernel headers have been sanitized beforing installing.
>> futex.h exposed to userspace shouldn't have __user anywhere.
>>
>> Previously distros did that by hand, nowadays you can do a "make headers_install"
>> in the kernel tree itself.
>>     
>
> I got something different on my CentOS 5.3 x86_64:
>
> /usr/include/linux/futex.h:108: error: expected ‘)’ before ‘*’ token
> /usr/include/linux/futex.h:112: error: expected ‘)’ before ‘*’ token
>
> Apparently it lacks u32 definition.
>
>
> Laurent
>   
I did some bigger compile test after the big user mode patch pull from
yesterday for all the patches Riku and me had collected in the past
weeks. I applied already the "__user patch". Also I had to disable
"-Werror" option, because there are still too much warnings inside now
causing an error.

Results:

CentOS 5 / RHEL 5 (i586/x86_64): - CentOS 5 was already reported

> In file included from
> /usr/src/redhat/BUILD/qemu-0.11git2009.06.16.1935/linux-user/strace.c:12:
> /usr/include/linux/futex.h:96: error: expected ')' before '*' token
> /usr/include/linux/futex.h:100: error: expected ')' before '*' token

Debian Etch/Lenny (i586/x86_64): builds

Fedora 9/10 (i586/x86_64): builds

SLES 11/openSUSE 11.0/openSUSE 11.1 (i586/x86_64): builds

Ubuntu 8.10/9.04 (i586/x86_64): builds


Regards, Martin

       reply	other threads:[~2009-06-17 12:41 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200906161934.n5GJYBvD030723@d01av04.pok.ibm.com>
     [not found] ` <Pine.LNX.4.64.0906170224150.3846@linmac.oyster.ru>
     [not found]   ` <20090617071833.GA5636@kos.to>
     [not found]     ` <761ea48b0906170032g318457e4p81d028375971f2b2@mail.gmail.com>
2009-06-17 12:41       ` Martin Mohring [this message]
     [not found]   ` <4A38449B.5050204@us.ibm.com>
2009-06-18 19:51     ` [Qemu-devel] Re: [Qemu-commits] [COMMIT 74d753a] linux-user: strace now handles guest stringscorrectly [v2] Riku Voipio
2009-06-18 22:53       ` malc

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A38E47B.3020806@opensuse.org \
    --to=martin.mohring@opensuse.org \
    --cc=aliguori@us.ibm.com \
    --cc=laurent.desnogues@gmail.com \
    --cc=mika.westerberg@iki.fi \
    --cc=qemu-devel@nongnu.org \
    --cc=riku.voipio@iki.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.