All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Travis <travis@sgi.com>
To: Paul Jackson <pj@sgi.com>
Cc: mingo@elte.hu, tglx@linutronix.de, hpa@zytor.com,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3] x86: add cpuset_scnprintf function
Date: Wed, 02 Apr 2008 00:47:41 -0700	[thread overview]
Message-ID: <47F33A1D.3050301@sgi.com> (raw)
In-Reply-To: <20080402012006.1722c2bd.pj@sgi.com>


> However doing this is worse in my view than simply breaking the format
> outright, unilaterally and irrevocably.  If you just flat out stick a
> fork in an API and break it hard on some release, then at least user
> space knows that it must adapt or die at that version.  If you hand
> user space the means to break that API, then any properly and
> defensively written user code has to be prepared to deal with both API
> flavors, and the majority of user space code is broken half the time,
> when run on a system with the API variant it wasn't expecting.  More
> over, you end up with apps having "toilet seat wars" with each other:
> you left it up and it should be down; no you left it down and it should
> be up.  Not a pretty sight.
> 
> Perhaps I totally misunderstand this patchset ?
> 

Hi,

I wanted to not break current apps unmercifully, but perhaps I should
default it to the "non-compatible" mode (and adjust the schedstat version
to indicate this)?  [It's the only output that I found that seemed to care.]

And if users have apps that they can't convert, they can revert to the
"old" (compatible) method of outputs.  I know if I'm a user and I'm really
interested in understanding the outputs when there's hundreds and hundreds
of cpus, then the more compact format is much more useful.

I can't believe there hasn't been many changes in all of these outputs.
Like what happened before Hyperthreading, or 3rd level caches, or ?
Even the new Intel announcements for Nehalem may introduce more changes
in what's important in the output information.  Plus I was under the
impression that one of the basic tenets of Linux was that API's can and
will change?

Thanks,
Mike

  reply	other threads:[~2008-04-02  7:47 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-01 22:54 [PATCH 0/3] x86: add cpuset_scnprintf function Mike Travis
2008-04-01 22:54 ` [PATCH 1/3] " Mike Travis
2008-04-01 22:54 ` [PATCH 2/3] x86: modify show_shared_cpu_map in intel_cacheinfo Mike Travis
2008-04-01 22:54 ` [PATCH 3/3] cpumask: use new cpuset_scnprintf function Mike Travis
2008-04-02  6:20 ` [PATCH 0/3] x86: add " Paul Jackson
2008-04-02  7:47   ` Mike Travis [this message]
2008-04-02 10:39     ` Paul Jackson
2008-04-02 14:36   ` Mike Travis
2008-04-02 15:28     ` Paul Jackson

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=47F33A1D.3050301@sgi.com \
    --to=travis@sgi.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=pj@sgi.com \
    --cc=tglx@linutronix.de \
    /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.