diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-20 13:19:49 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-20 13:19:49 -0400 |
commit | ad5cf46b4e3b29f2bc08543e8ef0bf8b13d3f19e (patch) | |
tree | 302266c69730de99546f1f5020e9fe134a0b47bf | |
parent | 93db6294958f62643a917e44d1abc85440e600cf (diff) | |
parent | e34e7dbb35474ffc75f639eca64a18a15f4f8688 (diff) |
Merge git://git.kernel.org/pub/scm/linux/kernel/git/sam/kbuild-fixes
* git://git.kernel.org/pub/scm/linux/kernel/git/sam/kbuild-fixes:
kernel-doc: fix param matching for array params
kernel-doc: ignore kmemcheck_bitfield_begin/end
kallsyms: fix inverted valid symbol checking
kbuild: fix build error during make htmldocs
-rw-r--r-- | scripts/basic/docproc.c | 2 | ||||
-rw-r--r-- | scripts/kallsyms.c | 6 | ||||
-rwxr-xr-x | scripts/kernel-doc | 19 |
3 files changed, 23 insertions, 4 deletions
diff --git a/scripts/basic/docproc.c b/scripts/basic/docproc.c index 4c9523ef9c00..99ca7a698687 100644 --- a/scripts/basic/docproc.c +++ b/scripts/basic/docproc.c | |||
@@ -385,7 +385,7 @@ int main(int argc, char *argv[]) | |||
385 | if (!srctree) | 385 | if (!srctree) |
386 | srctree = getcwd(NULL, 0); | 386 | srctree = getcwd(NULL, 0); |
387 | kernsrctree = getenv("KBUILD_SRC"); | 387 | kernsrctree = getenv("KBUILD_SRC"); |
388 | if (!kernsrctree) | 388 | if (!kernsrctree || !*kernsrctree) |
389 | kernsrctree = srctree; | 389 | kernsrctree = srctree; |
390 | if (argc != 3) { | 390 | if (argc != 3) { |
391 | usage(); | 391 | usage(); |
diff --git a/scripts/kallsyms.c b/scripts/kallsyms.c index 3cb57895c9ea..64343cc084b4 100644 --- a/scripts/kallsyms.c +++ b/scripts/kallsyms.c | |||
@@ -167,11 +167,11 @@ static int symbol_valid_tr(struct sym_entry *s) | |||
167 | for (i = 0; i < ARRAY_SIZE(text_ranges); ++i) { | 167 | for (i = 0; i < ARRAY_SIZE(text_ranges); ++i) { |
168 | tr = &text_ranges[i]; | 168 | tr = &text_ranges[i]; |
169 | 169 | ||
170 | if (s->addr >= tr->start && s->addr < tr->end) | 170 | if (s->addr >= tr->start && s->addr <= tr->end) |
171 | return 0; | 171 | return 1; |
172 | } | 172 | } |
173 | 173 | ||
174 | return 1; | 174 | return 0; |
175 | } | 175 | } |
176 | 176 | ||
177 | static int symbol_valid(struct sym_entry *s) | 177 | static int symbol_valid(struct sym_entry *s) |
diff --git a/scripts/kernel-doc b/scripts/kernel-doc index a193fa3f5272..ed591e9b7d1d 100755 --- a/scripts/kernel-doc +++ b/scripts/kernel-doc | |||
@@ -1468,6 +1468,8 @@ sub dump_enum($$) { | |||
1468 | } | 1468 | } |
1469 | 1469 | ||
1470 | } | 1470 | } |
1471 | # strip kmemcheck_bitfield_{begin,end}.*; | ||
1472 | $members =~ s/kmemcheck_bitfield_.*?;//gos; | ||
1471 | 1473 | ||
1472 | output_declaration($declaration_name, | 1474 | output_declaration($declaration_name, |
1473 | 'enum', | 1475 | 'enum', |
@@ -1651,6 +1653,15 @@ sub push_parameter($$$) { | |||
1651 | } | 1653 | } |
1652 | } | 1654 | } |
1653 | 1655 | ||
1656 | # strip spaces from $param so that it is one continous string | ||
1657 | # on @parameterlist; | ||
1658 | # this fixes a problem where check_sections() cannot find | ||
1659 | # a parameter like "addr[6 + 2]" because it actually appears | ||
1660 | # as "addr[6", "+", "2]" on the parameter list; | ||
1661 | # but it's better to maintain the param string unchanged for output, | ||
1662 | # so just weaken the string compare in check_sections() to ignore | ||
1663 | # "[blah" in a parameter string; | ||
1664 | ###$param =~ s/\s*//g; | ||
1654 | push @parameterlist, $param; | 1665 | push @parameterlist, $param; |
1655 | $parametertypes{$param} = $type; | 1666 | $parametertypes{$param} = $type; |
1656 | } | 1667 | } |
@@ -1669,6 +1680,14 @@ sub check_sections($$$$$$) { | |||
1669 | $prm_clean = $prms[$px]; | 1680 | $prm_clean = $prms[$px]; |
1670 | $prm_clean =~ s/\[.*\]//; | 1681 | $prm_clean =~ s/\[.*\]//; |
1671 | $prm_clean =~ s/__attribute__\s*\(\([a-z,_\*\s\(\)]*\)\)//; | 1682 | $prm_clean =~ s/__attribute__\s*\(\([a-z,_\*\s\(\)]*\)\)//; |
1683 | # ignore array size in a parameter string; | ||
1684 | # however, the original param string may contain | ||
1685 | # spaces, e.g.: addr[6 + 2] | ||
1686 | # and this appears in @prms as "addr[6" since the | ||
1687 | # parameter list is split at spaces; | ||
1688 | # hence just ignore "[..." for the sections check; | ||
1689 | $prm_clean =~ s/\[.*//; | ||
1690 | |||
1672 | ##$prm_clean =~ s/^\**//; | 1691 | ##$prm_clean =~ s/^\**//; |
1673 | if ($prm_clean eq $sects[$sx]) { | 1692 | if ($prm_clean eq $sects[$sx]) { |
1674 | $err = 0; | 1693 | $err = 0; |