Messages in this thread | | | Subject | Re: [PATCH 1 of 3] Introduce __raw_memcpy_toio32 | From | Bryan O'Sullivan <> | Date | Tue, 10 Jan 2006 08:07:56 -0800 |
| |
On Tue, 2006-01-10 at 06:55 -0800, Roland Dreier wrote:
> That is a good question, especially since the optimized > x86_64-specific version is out-of-line. I suspect the answer is > mainly that that's the easiest way to stick it in a header in > include/asm-generic.
Yes, this is correct.
> I think it would be worth working a little > harder and making the generic version out-of-line.
I'm fine with doing that, but I wonder what an appropriate way to do it would be.
Really, we'd like the generic implementation to be declared in asm-generic and defined in lib. Each arch that needed the generic version could then have its arch/XXX/lib/Makefile modified to pull in the generic version from lib, while arches that had special versions could remain unencumbered.
The only problem with this is that it's an unusual approach, so I don't have any obvious examples to copy. The closest I can think of is arch/x86_64/kernel/Makefile, which pulls in routines from the i386 tree like this:
bootflag-y += ../../i386/kernel/bootflag.o
So say arch/ia64/lib/Makefile, for example, could have a line like this:
obj-y += ../../../lib/raw_memcpy_toio32.o
Sam, do you think this is safe to do in generalwith respect to kbuild?
Additionally, does it meet everyone's needs in terms of being generic, safe, in good style, and keeping bloat to a minimum?
<b
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |