Re: [PATCH 5/6] FDPIC: Add coredump capability for the ELF-FDPIC binfmt [try #3]

From: David Howells
Date: Thu Jul 06 2006 - 14:37:30 EST


Andrew Morton <akpm@xxxxxxxx> wrote:

> llseek takes a loff_t and file->f_pos is loff_t. I guess it's a bit moot
> on such a CPU. Was it deliberate?

It compiles with no error and no warning, so I haven't noticed. This is as
binfmt_elf.c is, I believe, so that is probably wrong too.

> (how come the kernel doesn't have a SEEK_SET #define?)

I don't know. It probably should.

> Three callsites - seems too large to inline.

Again taken from binfmt_elf.c, although I added the debugging stuff. It
shouldn't matter as the compiler will make its own decision (or does "inline"
get #defined to always-inline nowadays?).

> Which seems reasonable to me. I'll steal it from them.

Okay.

> Embedding returns and gotos in macros is evil. For new code it's worth
> doing it vaguely tastefully.

Again, stolen verbatim from binfmt_elf.c. I'd prefer to keep it comparable by
the blink-comparator method if possible.

> Does this need locking?

It shouldn't do; we own our own vma chain, and because we're part of exec, we
have a fresh mm_struct to play with. The VMAs themselves aren't allowed to
change, not even on NOMMU.

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