Re: splice/tee bugs?

From: Luiz Fernando N. Capitulino
Date: Sun Jul 09 2006 - 12:42:05 EST


On Sun, 9 Jul 2006 13:16:29 +0200
Jens Axboe <axboe@xxxxxxx> wrote:

| On Sun, Jul 09 2006, Jens Axboe wrote:
| > On Sat, Jul 08 2006, Luiz Fernando N. Capitulino wrote:
| > >
| > > Hi Jens,
| > >
| > > On Sat, 8 Jul 2006 08:41:32 +0200
| > > Jens Axboe <axboe@xxxxxxx> wrote:
| > >
| > > | On Fri, Jul 07 2006, Luiz Fernando N. Capitulino wrote:
| > > | > On Fri, 7 Jul 2006 04:07:49 -0700
| > > | > Andrew Morton <akpm@xxxxxxxx> wrote:
| > > | >
| > > | > | On Fri, 07 Jul 2006 09:07:03 +0200
| > > | > | "Michael Kerrisk" <mtk-manpages@xxxxxxx> wrote:
| > > | > |
| > > | > | > c) Occasionally the command line just hangs, producing no output.
| > > | > | > In this case I can't kill it with ^C or ^\. This is a
| > > | > | > hard-to-reproduce behaviour on my (x86) system, but I have
| > > | > | > seen it several times by now.
| > > | > |
| > > | > | aka local DoS. Please capture sysrq-T output next time.
| > > | >
| > > | > If I run lots of them in parallel, I get the following OOPs in a few
| > > | > seconds:
| > > |
| > > | With the patch posted? You need the i vs nrbufs fix.
| > >
| > > Yes, it fixes the problem. I didn't try it before because I thought
| > > you were going to double check it [1].
| >
| > Yeah the patch needs reworking, however the isolated i vs nrbufs fix is
| > safe enough on its own. I'll post a full patch for inclusion, I'm afraid
| > I wont be able to fully test it enough for submitting it until tomorrow
| > though.
|
| Something like this, testing would be appreciated! Michael, can you
| repeat your testing as well? Thanks.

Yeah, it fixes the problem for 2.6.18-rc1.

But doesn't compile for 2.6.17.4:

CC fs/splice.o
fs/splice.c: In function `link_pipe':
fs/splice.c:1378: warning: implicit declaration of function `mutex_lock_nested'
fs/splice.c:1378: error: `I_MUTEX_PARENT' undeclared (first use in this function)
fs/splice.c:1378: error: (Each undeclared identifier is reported only once
fs/splice.c:1378: error: for each function it appears in.)
fs/splice.c:1379: error: `I_MUTEX_CHILD' undeclared (first use in this function)
make[1]: ** [fs/splice.o] Erro 1
make: ** [fs] Erro 2

Should we use the first patch for it? It does work too.

--
Luiz Fernando N. Capitulino
-
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/