diff options
author | Michal Marek <mmarek@suse.cz> | 2011-02-03 23:57:09 +0100 |
---|---|---|
committer | Michal Marek <mmarek@suse.cz> | 2011-03-17 15:13:56 +0100 |
commit | e37ddb82500393cb417c3ab0fe0726d9a8652372 (patch) | |
tree | 6ecc94992cb5affad4fe438d9b586a61b803f928 /Documentation/video-output.txt | |
parent | 01762c4ec5f6f62c550304b9c70e824293cefdd0 (diff) | |
download | op-kernel-dev-e37ddb82500393cb417c3ab0fe0726d9a8652372.zip op-kernel-dev-e37ddb82500393cb417c3ab0fe0726d9a8652372.tar.gz |
genksyms: Track changes to enum constants
Enum constants can be used as array sizes; if the enum itself does not
appear in the symbol expansion, a change in the enum constant will go
unnoticed. Example patch that changes the ABI but does not change the
checksum with current genksyms:
| enum e {
| E1,
| E2,
|+ E3,
| E_MAX
| };
|
| struct s {
| int a[E_MAX];
| }
|
| int f(struct s *s) { ... }
| EXPORT_SYMBOL(f)
Therefore, remember the value of each enum constant and
expand each occurence to <constant> <value>. The value is not actually
computed, but instead an expression in the form
(last explicitly assigned value) + N
is used. This avoids having to parse and semantically understand whole
of C.
Note: The changes won't take effect until the lexer and parser are
rebuilt by the next patch.
Signed-off-by: Michal Marek <mmarek@suse.cz>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Diffstat (limited to 'Documentation/video-output.txt')
0 files changed, 0 insertions, 0 deletions