All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: "David S. Miller" <davem@redhat.com>
Cc: frival@zk3.dec.com, paulus@samba.org, Martin.Bligh@us.ibm.com,
	alan@lxorguk.ukuu.org.uk, torvalds@transmeta.com,
	linux-kernel@vger.kernel.org, jay.estabrook@compaq.com,
	rth@twiddle.net
Subject: Re: [PATCH] change name of rep_nop
Date: Tue, 09 Oct 2001 00:06:28 +0100	[thread overview]
Message-ID: <14658.1002582388@redhat.com> (raw)
In-Reply-To: <20011008.154650.48796051.davem@redhat.com>
In-Reply-To: <20011008.154650.48796051.davem@redhat.com>  <1573466920.1002300846@mbligh.des.sequent.com> <15294.24873.866942.423260@cargo.ozlabs.ibm.com> <13962.1002580586@redhat.com>


davem@redhat.com said:
>  Yes, I know the text isn't there, but that is the implication. Add
> the text, don't add a stupid "simon_says.." interface.

> The mtrr stuff, if it really does need the flush, should probably make
> it's own macro/inline with a huge comment about it explaining why the
> flush is actually needed.

It's not just mtrr stuff, and it's not just arch-specific code either. In
some cases, there is a need for a function which actually does flush the
cache.

Feel free to suggest an alternative name for it, as long as it's not
'wbinvd' - I would have picked flush_cache_all(), but that is already taken
by a function which generally doesn't actually flush the cache :)

--
dwmw2



  parent reply	other threads:[~2001-10-08 23:06 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-10-05 10:46 [PATCH] change name of rep_nop Paul Mackerras
2001-10-05 14:37 ` Alan Cox
2001-10-05 18:06   ` Peter Rival
2001-10-05 23:28     ` Paul Mackerras
2001-10-05 23:54       ` Martin J. Bligh
2001-10-06  1:40         ` Paul Mackerras
2001-10-08 19:27           ` Peter Rival
2001-10-08 22:36           ` David Woodhouse
2001-10-08 22:46             ` David S. Miller
2001-10-08 23:16               ` Alan Cox
2001-10-08 23:24                 ` Dave Jones
2001-10-08 23:30                   ` David S. Miller
2001-10-08 23:33                   ` Alan Cox
2001-10-09  5:01                   ` George Greer
2001-10-09 10:33                 ` Benjamin Herrenschmidt
2001-10-09 11:30                   ` Keith Owens
2001-10-09 12:13                     ` Benjamin Herrenschmidt
2001-10-09 12:15                     ` Alan Cox
2001-10-08 22:49             ` Alan Cox
2001-10-08 23:06             ` David Woodhouse [this message]
2001-10-08 23:08               ` David S. Miller
2001-10-08 23:42               ` David Woodhouse
2001-10-08 23:46                 ` David S. Miller
2001-10-08 23:46             ` Linus Torvalds
2001-10-09  0:03               ` David Woodhouse
  -- strict thread matches above, loose matches on Subject: below --
2001-10-09  8:51 Etienne Lorrain
2001-10-09 11:30 ` Alan Cox

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=14658.1002582388@redhat.com \
    --to=dwmw2@infradead.org \
    --cc=Martin.Bligh@us.ibm.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=davem@redhat.com \
    --cc=frival@zk3.dec.com \
    --cc=jay.estabrook@compaq.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paulus@samba.org \
    --cc=rth@twiddle.net \
    --cc=torvalds@transmeta.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.