All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Wilcox <matthew@wil.cx>
To: Kyle McMartin <kyle@mcmartin.ca>
Cc: parisc-linux@lists.parisc-linux.org
Subject: Re: [parisc-linux] __LP64__ / CONFIG_64BIT?
Date: Mon, 16 Jan 2006 09:54:50 -0700	[thread overview]
Message-ID: <20060116165450.GD19769@parisc-linux.org> (raw)
In-Reply-To: <20060116164054.GB4049@quicksilver.road.mcmartin.ca>

On Mon, Jan 16, 2006 at 11:40:54AM -0500, Kyle McMartin wrote:
> Er, why? One of them is shorter to write, and makes the
> same amount of sense... I have no particularly preference at any
> rate. In fact, using __LP64__ might be preferable, as CONFIG_64BIT
> requires us including <linux/config.h> which means any change
> to the config will cause this file to be rebuilt whether or not
> it is actually required. Then again, it's entirely possible
> my understanding of this config crap is antiquated.

Doubly antiquated, I'm afraid.  Most recently, changes to the build
system mean that including <linux/config.h> is a no-op.  It's
automatically included for the files that need it.

The earlier antiquation happened many, many years ago, and it was
that the dependency writer (back in the days of 'make dep') treated
<linux/config.h> specially.  It scanned the file looking for CONFIG_*
symbols and added dependencies on the autobuilt include/config/ directory
(one file per config symbol).

I don't think we need to change from __LP64__ to CONFIG_64BIT overnight.
I think CONFIG_64IT has meaning to people from a non-HP environment, so
has slight clarity advantages.  If you're touching some code that uses
__LP64__, by all means do the conversion to CONFIG_64BIT, but don't do
it unnecessarily.
_______________________________________________
parisc-linux mailing list
parisc-linux@lists.parisc-linux.org
http://lists.parisc-linux.org/mailman/listinfo/parisc-linux

  reply	other threads:[~2006-01-16 16:54 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-15 19:18 [parisc-linux] __LP64__ / CONFIG_64BIT? Joel Soete
2006-01-16 16:40 ` Kyle McMartin
2006-01-16 16:54   ` Matthew Wilcox [this message]
2006-01-16 17:04     ` Kyle McMartin

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=20060116165450.GD19769@parisc-linux.org \
    --to=matthew@wil.cx \
    --cc=kyle@mcmartin.ca \
    --cc=parisc-linux@lists.parisc-linux.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.