| To: | David Daney <david.daney@cavium.com> |
|---|---|
| Subject: | Re: [PATCH] MIPS: Refactor 'clear_page' and 'copy_page' functions. |
| From: | Ralf Baechle <ralf@linux-mips.org> |
| Date: | Wed, 30 May 2012 22:01:41 +0100 |
| Cc: | "Steven J. Hill" <sjhill@mips.com>, linux-mips@linux-mips.org |
| In-reply-to: | <4FC6628F.9060807@cavium.com> |
| List-archive: | <http://www.linux-mips.org/archives/linux-mips/> |
| List-help: | <mailto:ecartis@linux-mips.org?Subject=help> |
| List-id: | linux-mips <linux-mips.eddie.linux-mips.org> |
| List-owner: | <mailto:ralf@linux-mips.org> |
| List-post: | <mailto:linux-mips@linux-mips.org> |
| List-software: | Ecartis version 1.0.0 |
| List-subscribe: | <mailto:ecartis@linux-mips.org?subject=subscribe%20linux-mips> |
| List-unsubscribe: | <mailto:ecartis@linux-mips.org?subject=unsubscribe%20linux-mips> |
| References: | <1337891904-24093-1-git-send-email-sjhill@mips.com> <4FC6628F.9060807@cavium.com> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Wed, May 30, 2012 at 11:10:23AM -0700, David Daney wrote:
> On 05/24/2012 01:38 PM, Steven J. Hill wrote:
> >From: "Steven J. Hill"<sjhill@mips.com>
> >
> >Remove usage of the '__attribute__((alias("...")))' hack that aliased
> >to integer arrays containing micro-assembled instructions. This hack
> >breaks when building a microMIPS kernel. It also makes the code much
> >easier to understand.
> >
> >Signed-off-by: Steven J. Hill<sjhill@mips.com>
>
> Looks good to (and even works for) me:
>
> Acked-by: David Daney <david.daney@cavium.com>
I have to admit that the attribute hack was a desperate attempt at
avoiding the last bit of assembler code for the page functions. But somehow
the patch isn't quite ripe yet. Building malta_defconfig from
7a3434a78b36be2d398a46fb505a3196a9df4a60 with this patch applied on top
I'm getting:
MODPOST 413 modules
ERROR: "copy_page" [fs/fuse/fuse.ko] undefined!
ERROR: "clear_page" [fs/fuse/fuse.ko] undefined!
ERROR: "clear_page" [drivers/net/ethernet/toshiba/tc35815.ko] undefined!
Ralf
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] MIPS: Refactor 'clear_page' and 'copy_page' functions., David Daney |
|---|---|
| Next by Date: | Re: [PATCH] MIPS: Refactor 'clear_page' and 'copy_page' functions., Ralf Baechle |
| Previous by Thread: | Re: [PATCH] MIPS: Refactor 'clear_page' and 'copy_page' functions., David Daney |
| Next by Thread: | Re: [PATCH] MIPS: Refactor 'clear_page' and 'copy_page' functions., Ralf Baechle |
| Indexes: | [Date] [Thread] [Top] [All Lists] |