From: Gerd Hoffmann <kraxel@redhat.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: "Pekka Enberg" <penberg@kernel.org>,
"Theodore Tso" <tytso@mit.edu>,
"Peter Zijlstra" <a.p.zijlstra@chello.nl>,
"kvm@vger.kernel.org list" <kvm@vger.kernel.org>,
"qemu-devel Developers" <qemu-devel@nongnu.org>,
"Vince Weaver" <vince@deater.net>,
"linux-kernel@vger.kernel.org List"
<linux-kernel@vger.kernel.org>,
"Arnaldo Carvalho de Melo" <acme@redhat.com>,
"Blue Swirl" <blauwirbel@gmail.com>,
"Alexander Graf" <agraf@suse.de>, "Avi Kivity" <avi@redhat.com>,
"Américo Wang" <xiyou.wangcong@gmail.com>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Linus Torvalds" <torvalds@linux-foundation.org>
Subject: Re: [Qemu-devel] [F.A.Q.] the advantages of a shared tool/kernel Git repository, tools/perf/ and tools/kvm/
Date: Wed, 09 Nov 2011 11:40:01 +0100 [thread overview]
Message-ID: <4EBA5881.7080409@redhat.com> (raw)
In-Reply-To: <20111109085120.GD11473@elte.hu>
Hi,
> What we want to have is to have a set of distinctive colors - just
> two (background, foreground) colors are not enough - we also need
> colors to highlight certain information - we need 5-6 colors for the
> output to be maximally expressive. Is there a canonical way to handle
> that while still adapting to user preferences automatically by taking
> background/foreground color scheme of the xterm into account?
> I suspect to fix the worst of the fallout we could add some logic to
> detect low contrast combinations (too low color distance) and fall
> back to the foreground/background colors in that case.
As far I know it is pretty much impossible to figure the
foreground/background colors of the terminal you are running on. You
can try some guesswork based on $TERM (linux console usually has black
background, xterm is white by default), but there will always be cases
where it fails.
You can run without colors. You can use bold to highlight things and
reverse for the cursor. Surely a bit limited and not as pretty as
colored, but works for sure everywhere.
You can go for a linux-console style black background. Pretty much any
color is readable here, so you should have no problems at all to find
the 5-6 colors you want.
You can go for a xterm-like light background, for example the lightgray
used by older perf versions. I like that background color, problem is
with most colors the contrast is pretty low. IMHO only red, blue and
violet are readable on lightgray. And black of course.
> Plus allowing full .perfconfig configurability of all the relevant
> colors, for those with special taste.
Sure. Maybe also allow multiple color sections and pick them by $TERM
or --colors switch, i.e. [colors "xterm"].
cheers,
Gerd
next prev parent reply other threads:[~2011-11-09 11:03 UTC|newest]
Thread overview: 158+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-06 1:35 [Qemu-devel] [PATCH] KVM: Add wrapper script around QEMU to test kernels Alexander Graf
2011-11-06 1:14 ` Andreas Färber
2011-11-06 10:04 ` Pekka Enberg
2011-11-06 10:07 ` Avi Kivity
2011-11-06 10:12 ` Pekka Enberg
2011-11-06 10:23 ` Avi Kivity
2011-11-06 11:08 ` Pekka Enberg
2011-11-06 11:50 ` Avi Kivity
2011-11-06 12:14 ` Pekka Enberg
2011-11-06 12:27 ` Avi Kivity
2011-11-06 12:32 ` Pekka Enberg
2011-11-06 12:43 ` Avi Kivity
2011-11-06 13:06 ` Pekka Enberg
2011-11-06 15:56 ` Avi Kivity
2011-11-06 16:35 ` Pekka Enberg
2011-11-06 16:50 ` Avi Kivity
2011-11-06 17:08 ` Anthony Liguori
2011-11-06 18:09 ` Pekka Enberg
2011-11-07 1:38 ` Anthony Liguori
2011-11-07 6:45 ` Pekka Enberg
2011-11-06 18:31 ` Ted Ts'o
2011-11-06 18:54 ` Pekka Enberg
2011-11-06 18:58 ` Pekka Enberg
2011-11-06 23:19 ` Ted Ts'o
2011-11-07 6:42 ` Pekka Enberg
2011-11-07 17:03 ` Vince Weaver
2011-11-07 17:59 ` Ingo Molnar
2011-11-07 20:03 ` Frank Ch. Eigler
2011-11-07 20:09 ` Pekka Enberg
2011-11-07 20:35 ` Ted Ts'o
2011-11-08 10:22 ` [Qemu-devel] [F.A.Q.] perf ABI backwards and forwards compatibility Ingo Molnar
2011-11-08 10:32 ` Peter Zijlstra
2011-11-08 11:34 ` Ingo Molnar
2011-11-08 10:41 ` Theodore Tso
2011-11-08 11:20 ` Pekka Enberg
2011-11-08 11:25 ` Theodore Tso
2011-11-08 11:29 ` Pekka Enberg
2011-11-08 11:31 ` Frank Ch. Eigler
2011-11-08 11:39 ` Pekka Enberg
2011-11-08 12:15 ` Ingo Molnar
2011-11-08 12:20 ` Peter Zijlstra
2011-11-08 12:59 ` Ingo Molnar
2011-11-09 10:05 ` Peter Zijlstra
2011-11-08 5:29 ` [Qemu-devel] [PATCH] KVM: Add wrapper script around QEMU to test kernels Vince Weaver
2011-11-08 12:07 ` Ingo Molnar
2011-11-08 13:08 ` Arnaldo Carvalho de Melo
2011-11-09 6:04 ` Vince Weaver
2011-11-07 19:53 ` Pekka Enberg
2011-11-07 20:32 ` Ted Ts'o
2011-11-07 21:36 ` Pekka Enberg
2011-11-07 22:19 ` Anthony Liguori
2011-11-07 23:42 ` Theodore Tso
2011-11-08 9:32 ` [Qemu-devel] [F.A.Q.] the advantages of a shared tool/kernel Git repository, tools/perf/ and tools/kvm/ Ingo Molnar
2011-11-08 10:21 ` Theodore Tso
2011-11-08 12:55 ` Ingo Molnar
2011-11-08 16:33 ` Ted Ts'o
2011-11-08 17:14 ` Anca Emanuel
2011-11-08 19:24 ` Ted Ts'o
2011-11-09 8:28 ` Ingo Molnar
2011-11-08 21:15 ` John Kacur
2011-11-09 8:38 ` Ingo Molnar
2011-11-09 8:23 ` Ingo Molnar
2011-11-10 1:41 ` Alexander Graf
2011-11-10 8:14 ` Ingo Molnar
2011-11-08 12:56 ` Arnaldo Carvalho de Melo
2011-11-08 13:40 ` Gerd Hoffmann
2011-11-08 14:32 ` Arnaldo Carvalho de Melo
2011-11-08 15:38 ` Gerd Hoffmann
2011-11-08 16:13 ` Arnaldo Carvalho de Melo
2011-11-09 8:55 ` Ingo Molnar
2011-11-09 8:51 ` Ingo Molnar
2011-11-09 10:40 ` Gerd Hoffmann [this message]
2011-11-09 10:50 ` Hagen Paul Pfeifer
2011-11-09 11:55 ` Arnaldo Carvalho de Melo
2011-11-09 12:26 ` Gerd Hoffmann
2011-11-09 12:30 ` Arnaldo Carvalho de Melo
2011-11-09 12:33 ` Arnaldo Carvalho de Melo
2011-11-09 12:46 ` Peter Zijlstra
2011-11-09 12:51 ` Arnaldo Carvalho de Melo
2011-11-09 13:17 ` Ingo Molnar
2011-11-09 19:25 ` Jim Paris
2011-11-09 20:13 ` Arnaldo Carvalho de Melo
2011-11-09 22:32 ` Anca Emanuel
2011-11-10 8:00 ` Ingo Molnar
2011-11-10 8:12 ` Anca Emanuel
2011-11-10 8:39 ` Gerd Hoffmann
2011-11-08 15:43 ` Steven Rostedt
2011-11-09 9:21 ` Ingo Molnar
2011-11-09 12:03 ` Arnaldo Carvalho de Melo
2011-11-09 13:40 ` Américo Wang
2011-11-10 7:47 ` Ingo Molnar
2011-11-07 10:31 ` [Qemu-devel] [PATCH] KVM: Add wrapper script around QEMU to test kernels Kevin Wolf
2011-11-07 11:38 ` Pekka Enberg
2011-11-07 11:59 ` Kevin Wolf
2011-11-06 16:19 ` Jan Kiszka
2011-11-06 16:30 ` Pekka Enberg
2011-11-06 16:39 ` Jan Kiszka
2011-11-06 17:11 ` Pekka Enberg
2011-11-06 17:23 ` Jan Kiszka
2011-11-06 17:55 ` Pekka Enberg
2011-11-06 16:39 ` Pekka Enberg
2011-11-07 10:11 ` Gerd Hoffmann
2011-11-07 10:18 ` Pekka Enberg
2011-11-06 17:10 ` Anthony Liguori
2011-11-06 17:15 ` Alexander Graf
2011-11-06 17:28 ` Pekka Enberg
2011-11-06 17:30 ` Alexander Graf
2011-11-06 18:05 ` Pekka Enberg
2011-11-06 19:14 ` Paolo Bonzini
2011-11-06 19:19 ` Pekka Enberg
2011-11-06 22:08 ` Frank Ch. Eigler
2011-11-07 6:58 ` Pekka Enberg
2011-11-06 19:11 ` Paolo Bonzini
2011-11-06 19:17 ` Pekka Enberg
2011-11-06 20:01 ` Paolo Bonzini
2011-11-06 20:17 ` Pekka Enberg
2011-11-07 8:00 ` Paolo Bonzini
2011-11-07 8:09 ` Pekka Enberg
2011-11-07 8:20 ` Paolo Bonzini
2011-11-07 8:45 ` Pekka Enberg
2011-11-07 8:52 ` Paolo Bonzini
2011-11-07 8:57 ` Pekka Enberg
2011-11-07 8:13 ` Pekka Enberg
2011-11-06 20:31 ` Pekka Enberg
2011-11-07 10:23 ` Gerd Hoffmann
2011-11-07 10:30 ` Sasha Levin
2011-11-07 11:02 ` Paolo Bonzini
2011-11-07 11:44 ` Pekka Enberg
2011-11-07 12:18 ` Gerd Hoffmann
2011-11-07 12:21 ` Pekka Enberg
2011-11-07 12:26 ` Avi Kivity
2011-11-07 12:29 ` Pekka Enberg
2011-11-07 12:43 ` Ted Ts'o
2011-11-07 12:44 ` Avi Kivity
2011-11-07 11:34 ` Pekka Enberg
2011-11-07 11:57 ` Ingo Molnar
2011-11-07 13:17 ` Anthony Liguori
2011-11-07 12:08 ` Gerd Hoffmann
2011-11-07 12:29 ` Ted Ts'o
2011-11-07 12:42 ` Pekka Enberg
2011-11-07 12:47 ` Ted Ts'o
2011-11-07 12:59 ` Pekka Enberg
2011-11-07 13:12 ` Pekka Enberg
2011-11-08 13:29 ` Karel Zak
2011-11-08 14:30 ` Pekka Enberg
2011-11-06 13:11 ` Pekka Enberg
2011-11-06 17:09 ` Alexander Graf
2011-11-06 12:27 ` Pekka Enberg
2011-11-08 14:41 ` Avi Kivity
2011-11-08 14:52 ` Christoph Hellwig
2011-11-08 14:55 ` Sasha Levin
2011-11-08 14:57 ` Avi Kivity
2011-11-08 14:59 ` Christoph Hellwig
2011-11-08 17:34 ` Alexander Graf
2011-11-08 17:36 ` Avi Kivity
2011-11-08 15:04 ` Jan Kiszka
2011-11-08 15:26 ` Pekka Enberg
2011-11-08 15:28 ` Christoph Hellwig
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=4EBA5881.7080409@redhat.com \
--to=kraxel@redhat.com \
--cc=a.p.zijlstra@chello.nl \
--cc=acme@redhat.com \
--cc=agraf@suse.de \
--cc=avi@redhat.com \
--cc=blauwirbel@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=penberg@kernel.org \
--cc=qemu-devel@nongnu.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=tytso@mit.edu \
--cc=vince@deater.net \
--cc=xiyou.wangcong@gmail.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 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).