mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Vince Weaver <vincent.weaver@maine.edu>
To: "Naveen N. Rao" <naveen.n.rao@linux.vnet.ibm.com>
Cc: Jiri Olsa <jolsa@redhat.com>,
	Vince Weaver <vincent.weaver@maine.edu>,
	Peter Zijlstra <peterz@infradead.org>,
	Arnaldo Carvalho de Melo <acme@kernel.org>,
	Ingo Molnar <mingo@kernel.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] Notifications for perf sideband events
Date: Thu, 13 Jul 2017 11:35:08 -0400 (EDT)	[thread overview]
Message-ID: <alpine.DEB.2.20.1707131132350.23692@macbook-air> (raw)
In-Reply-To: <20170713142002.udgtzxdv6n7y3w3z@naverao1-tp.localdomain>

On Thu, 13 Jul 2017, Naveen N. Rao wrote:

> > could you please check on this? thanks
> > 
> > Naveen,
> > have you run Vince's test suite on this?
> >   http://github.com/deater/perf_event_tests.git
> 
> I just tried this and I see quite a few failures even without these 
> patches.
> 
> The behavior is similar with/without these patches and all the ioctl 
> tests pass, but I see some failures with the overflow tests. I'll look 
> into those tests in detail tomorrow. I may be missing something.

the signal/overflow interface has always been a troublesome one :(
It's part of why I haven't had much to say about the whole kernel 
addresses leaking into samples mess.

I had noticed some of the related overflow tests have been failing 
recently (both in perf_event_tests and in PAPI), but I haven't had
a chance to see if it was an actual regression or just due to the way the 
tests are designed.  I'll see if I can figure out what's going on.

Vince

      reply	other threads:[~2017-07-13 15:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-19 14:31 Naveen N. Rao
2017-06-19 14:31 ` [PATCH 1/2] kernel/events: Add option to notify through signals on wakeup Naveen N. Rao
2017-07-12 11:46   ` Peter Zijlstra
2017-07-13 14:37     ` Naveen N. Rao
2017-06-19 14:31 ` [PATCH 2/2] kernel/events: Add option to enable counting sideband events in wakeup_events Naveen N. Rao
2017-07-12 10:48   ` Jiri Olsa
2017-07-13 14:34     ` Naveen N. Rao
2017-07-12 11:48   ` Peter Zijlstra
2017-07-13 14:55     ` Naveen N. Rao
2017-06-27  9:04 ` [PATCH 0/2] Notifications for perf sideband events Naveen N. Rao
2017-07-12 10:48 ` Jiri Olsa
2017-07-13 14:20   ` Naveen N. Rao
2017-07-13 15:35     ` Vince Weaver [this message]

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=alpine.DEB.2.20.1707131132350.23692@macbook-air \
    --to=vincent.weaver@maine.edu \
    --cc=acme@kernel.org \
    --cc=jolsa@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=naveen.n.rao@linux.vnet.ibm.com \
    --cc=peterz@infradead.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®