linux-next: build failure after merge of the final tree (kgdb tree related)

linux-next: build failure after merge of the final tree (kgdb tree related)

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 950 bytes --]

Hi all,

After merging the final tree, next-20100726 build (i386 allmodconfig)
failed like this:

In file included from arch/x86/include/asm/uaccess.h:571,
                 from include/linux/uaccess.h:5,
                 from include/linux/highmem.h:6,
                 from include/linux/pagemap.h:10,
                 from kernel/trace/trace.c:24:
arch/x86/include/asm/uaccess_32.h: In function 'copy_from_user':
arch/x86/include/asm/uaccess_32.h:212: error: call to 'copy_from_user_overflow' declared with attribute error: copy_from_user() buffer size is not provably correct

Probably caused by commit e5738a3d75254c6d135ccad925e513ddb860967e
("ftrace,kdb: Extend kdb to be able to dump the ftrace buffer") from the
kgdb tree which turned off optimization (and therefore probably inlining)
for kernel/trace/trace.c ...

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]

Re: linux-next: build failure after merge of the final tree (kgdb tree related)

[Jason Wessel]

On 07/31/2011 11:22 PM, Stephen Rothwell wrote:
> Hi Jason,
>
> After merging the final tree, today's linux-next build (sparc32_defconfig)
> failed like this:
>
> kernel/debug/gdbstub.c: In function 'gdbstub_state':
> kernel/debug/gdbstub.c:1098:3: error: 'gdbstub_use_prev_in_buf' undeclared (first use in this function)
> kernel/debug/gdbstub.c:1098:3: note: each undeclared identifier is reported only once for each function it appears in
> kernel/debug/gdbstub.c:1099:3: error: 'gdbstub_prev_in_buf_pos' undeclared (first use in this function)
>
> Caused by commit faed702e3b31 ("kdb,kgdb: Implement switch and pass
> buffer from kdb -> gdb").
>
> I reverted that commit (and 621ebf205b99 "kdb: Remove all references to
> DOING_KGDB2" and "kdb,kgdb: Allow arbitrary kgdb magic knock
> sequences" that depend on it) for today.
>
> BTW, that last commit (29e3c61ef33c) doesn't have a SOB line.

Thanks for the detailed report.   I fixed this by re-basing kgdb-next against the kgdb-dev branch.  These branches are not supposed to get out of sync.  I'll try and figure out what went wrong. :-)

Jason.

linux-next: build failure after merge of the final tree (kgdb tree related)

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 942 bytes --]

Hi Jason,

After merging the final tree, today's linux-next build (sparc32_defconfig)
failed like this:

kernel/debug/gdbstub.c: In function 'gdbstub_state':
kernel/debug/gdbstub.c:1098:3: error: 'gdbstub_use_prev_in_buf' undeclared (first use in this function)
kernel/debug/gdbstub.c:1098:3: note: each undeclared identifier is reported only once for each function it appears in
kernel/debug/gdbstub.c:1099:3: error: 'gdbstub_prev_in_buf_pos' undeclared (first use in this function)

Caused by commit faed702e3b31 ("kdb,kgdb: Implement switch and pass
buffer from kdb -> gdb").

I reverted that commit (and 621ebf205b99 "kdb: Remove all references to
DOING_KGDB2" and "kdb,kgdb: Allow arbitrary kgdb magic knock
sequences" that depend on it) for today.

BTW, that last commit (29e3c61ef33c) doesn't have a SOB line.
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]