From: Luiz Capitulino <lcapitulino@redhat.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/3] trace-cmd record: add --cpu-list option
Date: Wed, 16 Nov 2016 09:09:57 -0500 [thread overview]
Message-ID: <20161116090957.2816dba5@redhat.com> (raw)
In-Reply-To: <20161115163819.5291b752@gandalf.local.home>
On Tue, 15 Nov 2016 16:38:19 -0500
Steven Rostedt <rostedt@goodmis.org> wrote:
> On Fri, 28 Oct 2016 17:14:56 -0400
> Luiz Capitulino <lcapitulino@redhat.com> wrote:
>
>
> > >
> > > cpu_set(ret_mask, i) sets bits from begin to end in uint64_t cpumask
> > > from below.
> > >
> > > What happens if this range is greater than 64? We already have boxes
> > > that run this with 80 CPUs. Looks to be out of memory range to me.
> >
> > The best solution is probably to detect the number of CPUs at run-time
> > and use the CPU_SET() API. The lazy and ugly solution is to just fail.
> >
> > Any objections to the CPU_SET() solution?
> >
>
> I thought I replied to this, but I'm guessing the reply is still
> pending.
>
> I have no objecting. I'd like to see a patch though.
I imagined you didn't have objections :) I haven't started working
on this yet because I'm very busy at other stuff. But I'll go
back to this soon.
next prev parent reply other threads:[~2016-11-16 14:09 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-07 16:47 [PATCH 0/3] trace-cmd record: add --cpu-list option Luiz Capitulino
2016-10-07 16:47 ` [PATCH 1/3] cpu.h: use standard types instead of glib's Luiz Capitulino
2016-10-07 16:47 ` [PATCH 2/3] trace-cmd record: refactor set_mask() Luiz Capitulino
2016-10-07 16:47 ` [PATCH 3/3] trace-cmd record: add --cpu-list option Luiz Capitulino
2016-10-28 19:49 ` Steven Rostedt
2016-10-28 19:50 ` Steven Rostedt
2016-10-28 21:15 ` Luiz Capitulino
2016-10-28 21:39 ` Steven Rostedt
2016-10-28 21:14 ` Luiz Capitulino
2016-11-15 21:38 ` Steven Rostedt
2016-11-16 14:09 ` Luiz Capitulino [this message]
2016-10-19 13:30 ` [PATCH 0/3] " Luiz Capitulino
2016-10-19 13:55 ` Steven Rostedt
2016-10-19 13:58 ` Luiz Capitulino
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=20161116090957.2816dba5@redhat.com \
--to=lcapitulino@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rostedt@goodmis.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 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.