On Tue, 31 Jul 2007 23:12:26 +0300
Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote:
> The tested Kernels:
>
> 1. Jens's sglist-arch
> I was not able to pass all tests with this Kernel. For some reason when
> bigger than 256 pages commands are queued the Machine will run out
> of memory and will kill the test. After the test is killed the system
> is left with 10M of memory and can hardly reboot.
> I have done some prints at the queuecommand entry in scsi_debug.c
> and I can see that I receive the expected large sg_count and bufflen
> but unlike other tests I get a different pointer at scsi_sglist().
> In other tests since nothing is happening at this machine while in
> the test, the sglist pointer is always the same. commands comes in,
> allocates memory, do nothing in scsi_debug, freed, and returns.
> I suspect sglist leak or allocation bug.
Ok, I found the leak.
>From 011c05c2e514d1db4834147ed83526473711b0a3 Mon Sep 17 00:00:00 2001
From: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>
Date: Mon, 6 Aug 2007 16:16:24 +0900
Subject: [PATCH] fix sg chaining leak
Signed-off-by: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>
---
drivers/scsi/scsi_lib.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
index 5884b1b..25988b9 100644
--- a/drivers/scsi/scsi_lib.c
+++ b/drivers/scsi/scsi_lib.c
@@ -48,7 +48,6 @@ static struct scsi_host_sg_pool scsi_sg_pools[] = {
SP(32),
SP(64),
SP(128),
- SP(256),
};
#undef SP
--
1.5.2.4
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
