[2.5 patch] ide-tape.c must include buffer_head.h

Adrian Bunk (bunk@fs.tum.de)
Sun, 26 May 2002 23:06:14 +0200 (CEST)


Hi Christoph,

after your removal of #include <linux/buffer_head.h> from fs.h ide-tape.c
no longer compiles:

<-- snip -->

...
gcc -D__KERNEL__
-I/home/bunk/linux/kernel-2.5/linux-2.5.18-modular/include -Wall
-Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
-pipe -mpreferred-stack-boundary=2 -march=k6 -DMODULE -DMODVERSIONS
-include /home/bunk/linux/kernel-2.5/linux-2.5.18-modular/include/linux/modversions.h
-DKBUILD_BASENAME=ide_tape -c -o ide-tape.o ide-tape.c
ide-tape.c: In function `__idetape_kmalloc_stage':
ide-tape.c:2810: `BH_Lock' undeclared (first use in this function)
ide-tape.c:2810: (Each undeclared identifier is reported only once
ide-tape.c:2810: for each function it appears in.)
...

<-- snip -->

The following patch (made against 2.5.18-dj1 but it should apply cleanly
against 2.5.18) fixes it (I wasn't able to find an obvious ordering of the
"#include <linux/*>" so I placed it at the end):

--- drivers/ide/ide-tape.c.old Sun May 26 22:55:56 2002
+++ drivers/ide/ide-tape.c Sun May 26 22:56:42 2002
@@ -417,6 +417,7 @@
#include <linux/completion.h>
#include <linux/ide.h>
#include <linux/atapi.h>
+#include <linux/buffer_head.h>

#include <asm/byteorder.h>
#include <asm/irq.h>

cu
Adrian

-- 

You only think this is a free country. Like the US the UK spends a lot of time explaining its a free country because its a police state. Alan Cox

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