Re: [PATCH 12/17] arm: user_regset: iWMMXt in core dumps

From: Roland McGrath
Date: Mon Apr 27 2009 - 22:55:25 EST


> Are these note definitions centrally defined anywhere? I looked at the
> include/elf/common.h in gdb, but it seemed fairly similar to linux/elf.h.

linux/elf.h constitutes the central place. (And basically, de facto
everyone now seems to check with me for new ones.) New user_regset types
all get the "LINUX" note name, so there is no real authority beyond the
kernel maintainers to harmonize with.

> Both sh and mips would benefit from a note type for DSP registers. Would
> something like an NT_PRDSPREG be accetable for this? Also, the note
> definitions seem to have hopped from 4 to 6, is there a definition for 5,
> or can that be used for a DSP register note type?

There is no particular benefit to packing the space. It's just a 32-bit
identifier, with existing values assigned that are both large and small.
Please do not use any more small values. They are used on other systems
that some tools already deal with, and it just keeps things simpler not to
add any new n_type values that are reused with different names on different
systems. (I'll NAK any new use of values <= 20, which already have names
in glibc's elf.h.)

For a new purpose, it is my inclination to use a new NT_<machine>_<type>
name and pick a new value range, e.g. NT_SH_* at 0x300+n. The
interpretation of these type codes is machine-specific, but it just seems
simpler to assign a new block of numbers to each machine and not have any
overlaps.

If you really mean a new particular note format that would actually match
across different machines, then a new generically-named type for that is
appropriate. For that, I'd start with another new value range like 0x10000+n
(leaving the 0x?00+n blocks to be assigned for particular arch's).


Thanks,
Roland
--
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/