All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: lkml <linux-kernel@vger.kernel.org>,
	linux-kbuild <linux-kbuild@vger.kernel.org>,
	linux arch <linux-arch@vger.kernel.org>,
	Guan Xuetao <guanxuetao@mprc.pku.edu.cn>,
	Michal Marek <mmarek@suse.cz>
Subject: Re: [RFC] kbuild: generic support for asm-generic
Date: Sun, 9 Jan 2011 01:15:58 +0100	[thread overview]
Message-ID: <201101090115.59234.arnd@arndb.de> (raw)
In-Reply-To: <20110108213353.GA24666@merkur.ravnborg.org>

On Saturday 08 January 2011, Sam Ravnborg wrote:
> We need a variables to express:
> In include/asm-generic/Kbuild.asm
> - This file exists in a generic variant and it is exported.
>   The arch may use it verbatim or may have a local variant

Actually, this is not how we traditionally use Kbuild.asm:
This file is only included from the other asm/Kbuild files to
get the list of common files that should be exported from the
architectures. Originally, this included even files that had
no asm-generic counterpart, but that has probably changed now.

The naming of that file may have been a bit confusing.

We already have the list of exported files in
include/asm-generic/Kbuild that sort of does what you describe.

> In arch/$(ARCH)/include/Kbuild.asm-generic
> - This file is provided verbatim by asm-generic

I would just put this information into the 
arch/$(ARCH)/include/Kbuild file, with a different variable
name.

The other complication is that not all the files that an
architecture may take verbatim from asm-generic are also
exported.

> Something like:
> 
> export-y - for files listed in Kbuild.asm
> generic-y - for files listed in Kbuild.asm-generic
> 
> I will try to come up with a patch that uses this - or something
> similar.

Ok.

Would it be enough to have this?:

"generic-y" in arch/$(ARCH)/include/asm/Kbuild:
  The architecture does not provide this header, the kernel should use
  the asm-generic version through a generated file.

"header-y" in arch/$(ARCH)/include/asm/Kbuild and include/asm-generic/Kbuild.asm:
  This file gets exported to user space (as before), and may come from either
  arch/$(ARCH)/include/asm or include/asm-generic (if listed as generic-y).

"header-y" in include/asm-generic/Kbuild: The asm-generic version of the header
 gets installed. We always do this, because there might be reasons to install
 headers for multiple architectures (with a symlink or similar).

We could either install the generated file or a copy of the asm-generic file
for any exported header that the architecture does not have itself.

	Arnd

  reply	other threads:[~2011-01-09  0:16 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-08 13:03 [RFC] kbuild: generic support for asm-generic Sam Ravnborg
2011-01-08 13:03 ` Sam Ravnborg
2011-01-08 13:04 ` [PATCH 1/2] kbuild: asm-generic support Sam Ravnborg
2011-01-08 13:04   ` Sam Ravnborg
2011-01-08 13:04   ` Sam Ravnborg
2011-01-08 13:45   ` Stephen Rothwell
2011-01-08 14:03     ` Sam Ravnborg
2011-01-08 14:10   ` Guan Xuetao
2011-01-08 14:10     ` Guan Xuetao
2011-01-08 13:05 ` [EXAMPLE PATCH 2/2] x86: start to utilize kbuild " Sam Ravnborg
2011-01-08 13:05   ` Sam Ravnborg
2011-01-08 13:05   ` Sam Ravnborg
2011-01-08 20:53 ` [RFC] kbuild: generic support for asm-generic Arnd Bergmann
2011-01-08 21:33   ` Sam Ravnborg
2011-01-09  0:15     ` Arnd Bergmann [this message]
2011-01-09  8:28 ` [RFC v2] " Sam Ravnborg
2011-01-09  8:28   ` Sam Ravnborg
2011-01-09  8:31   ` [PATCH 1/2] kbuild: asm-generic support Sam Ravnborg
2011-01-09  8:31     ` Sam Ravnborg
2011-01-09  8:31     ` Sam Ravnborg
2011-01-09 14:03     ` Arnd Bergmann
2011-01-09 15:10       ` Sam Ravnborg
2011-01-09 16:13         ` Arnd Bergmann
2011-01-09  8:32   ` [EXAMPLE PATCH 2/2] x86: start to utilize kbuild " Sam Ravnborg
2011-01-09  8:32     ` Sam Ravnborg
2011-01-09  8:32     ` Sam Ravnborg
2011-01-09 19:27 ` [RFC v3] kbuild: generic support for asm-generic Sam Ravnborg
2011-01-09 19:27   ` Sam Ravnborg
2011-01-09 19:29   ` [PATCH 1/2] kbuild: asm-generic support Sam Ravnborg
2011-01-09 19:29     ` Sam Ravnborg
2011-01-09 19:29     ` Sam Ravnborg
2011-01-09 20:31     ` Arnd Bergmann
2011-01-09 21:24       ` Sam Ravnborg
2011-01-10 13:14     ` Guan Xuetao
2011-01-10 13:14       ` Guan Xuetao
2011-01-10 16:26       ` Sam Ravnborg
2011-01-11  1:39         ` Guan Xuetao
2011-01-10 13:31     ` Guan Xuetao
2011-01-10 13:31       ` Guan Xuetao
2011-01-13 16:14     ` Michal Marek
2011-01-13 17:01       ` Sam Ravnborg
2011-01-14 14:43         ` Michal Marek
2011-04-22 15:53           ` [PATCH v3] " Sam Ravnborg
2011-04-22 15:53             ` Sam Ravnborg
2011-04-25  1:29             ` Guan Xuetao
2011-04-27 19:42               ` Sam Ravnborg
2011-04-27 19:46                 ` [PATCH v4] " Sam Ravnborg
2011-04-27 19:46                   ` Sam Ravnborg
2011-04-27 19:46                   ` Sam Ravnborg
2011-04-27 19:52                   ` Arnd Bergmann
2011-04-27 20:29                     ` [PATCH v5] " Sam Ravnborg
2011-04-27 20:29                       ` Sam Ravnborg
2011-04-27 20:29                       ` Sam Ravnborg
2011-04-28 16:16                       ` Michal Marek
2011-01-09 19:29   ` [EXAMPLE PATCH 2/2] x86: start to utilize kbuild " Sam Ravnborg
2011-01-09 19:29     ` Sam Ravnborg
2011-01-09 19:29     ` Sam Ravnborg

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=201101090115.59234.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=guanxuetao@mprc.pku.edu.cn \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=sam@ravnborg.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.