Re: Cleanup possibility in asm-i386/string.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Feb 09, 2006, at 19:23, Andi Kleen wrote:
On Friday 10 February 2006 01:05, Adrian Bunk wrote:
On Tue, Feb 07, 2006 at 01:39:50PM +0100, Roman Zippel wrote:
Hi,

On Tue, 7 Feb 2006, Andi Kleen wrote:

This means you define a prototype for the builtin function and not for the normal function. I'm not sure this is really intended.

What good would be a prototype for a symbol that is defined to a different symbol?

The point is you define a prototype for a builtin function, I'm not sure that's a good thing to do. Actually I'd prefer to remove -ffreestanding again, especially because it disables builtin functions, which we have to painfully enable all again one by one, instead of leaving it just to gcc.

I remember playing with using more gcc builtins in the kernel some time ago, and some gcc builtin used a different library function, which was a function the kernel did not supply.

It works fine on x86-64. If something is missing it can be also supplied.

I don't remember exactly, but I think the problem was something like this (even if not this exact case, it was similarly obscure): If - ffreestanding was not specified, then the following code would generate an implicit call to memcpy() or some other library function.

struct a {
	[... large struct with lots of fields ...]
}

struct a first = { ..... };
[... more code ...];
struct a second = first; /* <== This line would generate implicit memcpy */

Cheers,
Kyle Moffett

--
Diplomacy involves walking softly and _carrying_ a big stick. Actually using the big stick means the diplomacy part failed.
  -- Rob Landley



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux