All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@kernel.org>
To: David Ahern <dsahern@gmail.com>
Cc: linux-kernel@vger.kernel.org, acme@ghostprotocols.net,
	fweisbec@gmail.com, peterz@infradead.org, jolsa@redhat.com,
	namhyung@kernel.org, xiaoguangrong@linux.vnet.ibm.com,
	runzhen@linux.vnet.ibm.com, Runzhen Wang <icycoder@gmail.com>
Subject: Re: [PATCH 14/16] perf kvm: reuse some code of perf_kvm__timerfd_create()
Date: Tue, 29 Oct 2013 14:23:04 +0100	[thread overview]
Message-ID: <20131029132304.GA22052@gmail.com> (raw)
In-Reply-To: <20130912134058.GF23826@gmail.com>


* Ingo Molnar <mingo@kernel.org> wrote:

> * David Ahern <dsahern@gmail.com> wrote:
> 
> > On 9/12/13 6:08 AM, Ingo Molnar wrote:
> > >
> > >btw., I tried to build perf on a really old distro that has no timerfd.h,
> > >and got:
> > 
> > Right. I hit that last week or so with RHEL5 as a build server.
> > 
> > timerfd was introduced before perf so any system with perf capability 
> > will also have timerfd. So the problem is strictly building perf on an 
> > older system. [...]
> 
> I'm booting new kernels on an old system and I'm building it there.
> 
> So it's new kernel, new perf source and binary, old userspace.

Btw., this build failure is still there today:

  CC       builtin-kvm.o
builtin-kvm.c:23:25: error: sys/timerfd.h: No such file or directorycc1: warnings being treated as errors
builtin-kvm.c: In function ‘perf_kvm__timerfd_create’:
builtin-kvm.c:975: warning: implicit declaration of function ‘timerfd_create’
builtin-kvm.c:975: warning: nested extern declaration of ‘timerfd_create’
builtin-kvm.c:975: error: ‘TFD_NONBLOCK’ undeclared (first use in this function)
builtin-kvm.c:975: error: (Each undeclared identifier is reported only once
builtin-kvm.c:975: error: for each function it appears in.)
builtin-kvm.c:986: warning: implicit declaration of function ‘timerfd_settime’
builtin-kvm.c:986: warning: nested extern declaration of ‘timerfd_settime’
make[1]: *** [builtin-kvm.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [all] Error 2

So it would be nice to at least make it build somehow.

Thanks,

	Ingo

  reply	other threads:[~2013-10-29 13:23 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-26  0:24 [PATCH 00/16] perf: kvm live mode David Ahern
2013-05-26  0:24 ` [PATCH 01/16] perf evlist: restore methods removed in earlier cleanup David Ahern
2013-05-26  0:24 ` [PATCH 02/16] perf evlist: move tracepoint processing code to evlist.c David Ahern
2013-05-26  0:24 ` [PATCH 03/16 v2] perf evlist: add initialzation function for tracepoints David Ahern
2013-05-26  0:24 ` [PATCH 04/16 v2] perf session: export a few functions for event processing David Ahern
2013-05-26  0:24 ` [PATCH 05/16] perf top: move CONSOLE_CLEAR to header file David Ahern
2013-05-26  0:24 ` [PATCH 06/16] perf kvm: handle realloc failures David Ahern
2013-05-31 11:49   ` [tip:perf/core] perf kvm: Handle " tip-bot for David Ahern
2013-05-26  0:24 ` [PATCH 07/16] perf kvm: split out tracepoints from record args David Ahern
2013-05-26  0:24 ` [PATCH 08/16] perf stats: fix divide by 0 in variance David Ahern
2013-05-31 11:51   ` [tip:perf/core] perf stats: Fix " tip-bot for David Ahern
2013-05-26  0:24 ` [PATCH 09/16] perf stats: add max and min stats David Ahern
2013-05-26  0:24 ` [PATCH 10/16 v2] perf kvm: add live mode David Ahern
2013-05-26  0:24 ` [PATCH 11/16] perf kvm: add min and max stats to display David Ahern
2013-05-26  0:24 ` [PATCH 12/16] perf kvm: option to print events that exceed a threshold David Ahern
2013-05-26  0:24 ` [PATCH 13/16] perf kvm: debug for missing vmexit/vmentry event David Ahern
2013-05-26  0:24 ` [PATCH 14/16] perf kvm: reuse some code of perf_kvm__timerfd_create() David Ahern
2013-09-12 13:08   ` Ingo Molnar
2013-09-12 13:34     ` David Ahern
2013-09-12 13:40       ` Ingo Molnar
2013-10-29 13:23         ` Ingo Molnar [this message]
2013-10-29 13:53           ` David Ahern
2013-10-29 13:58             ` Ingo Molnar
2013-05-26  0:24 ` [PATCH 15/16] perf kvm: move the prompt_integer() to /util/top.c David Ahern
2013-05-26  0:24 ` [PATCH 16/16] perf kvm: set live mode refresh time dynamically David Ahern
2013-05-26  0:29 ` Fwd: [PATCH 00/16] perf: kvm live mode David Ahern

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=20131029132304.GA22052@gmail.com \
    --to=mingo@kernel.org \
    --cc=acme@ghostprotocols.net \
    --cc=dsahern@gmail.com \
    --cc=fweisbec@gmail.com \
    --cc=icycoder@gmail.com \
    --cc=jolsa@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=namhyung@kernel.org \
    --cc=peterz@infradead.org \
    --cc=runzhen@linux.vnet.ibm.com \
    --cc=xiaoguangrong@linux.vnet.ibm.com \
    /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.