Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-20 Thread Milan Broz
Chuck Ebbert wrote: >> [ 126.754204] BUG: unable to handle kernel NULL pointer dereference at >> virtual address >> ... > mempool_free() was called with a NULL pool. That can't be good. Yes, it is really not good :) Bug http://bugzilla.kernel.org/show_bug.cgi?id=7388 Attached

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-20 Thread Milan Broz
Chuck Ebbert wrote: [ 126.754204] BUG: unable to handle kernel NULL pointer dereference at virtual address ... mempool_free() was called with a NULL pool. That can't be good. Yes, it is really not good :) Bug http://bugzilla.kernel.org/show_bug.cgi?id=7388 Attached patch

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-18 Thread Chuck Ebbert
On 07/17/2007 09:16 AM, Patrick McHardy wrote: > Alasdair G Kergon wrote: >> From: "Jun'ichi Nomura" <[EMAIL PROTECTED]> >> >> bio_alloc_bioset() will return NULL if 'num_vecs' is too large. >> Use bio_get_nr_vecs() to get estimation of maximum number. >> >> Signed-off-by: "Jun'ichi Nomura"

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-18 Thread Patrick McHardy
Jun'ichi Nomura wrote: > Patrick McHardy wrote: > >>Jun'ichi Nomura wrote: >> >>>Are you using other dm modules such as dm-multipath, dm-mirror >>>or dm-snapshot? >>>If so, can you take the output of 'dmsetup table' and 'dmsetup ls'? >> >>No other modules. >> >> >>>Do you have a reliable way to

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-18 Thread Patrick McHardy
Jun'ichi Nomura wrote: Patrick McHardy wrote: Jun'ichi Nomura wrote: Are you using other dm modules such as dm-multipath, dm-mirror or dm-snapshot? If so, can you take the output of 'dmsetup table' and 'dmsetup ls'? No other modules. Do you have a reliable way to reproduce the oops which I

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-18 Thread Chuck Ebbert
On 07/17/2007 09:16 AM, Patrick McHardy wrote: Alasdair G Kergon wrote: From: Jun'ichi Nomura [EMAIL PROTECTED] bio_alloc_bioset() will return NULL if 'num_vecs' is too large. Use bio_get_nr_vecs() to get estimation of maximum number. Signed-off-by: Jun'ichi Nomura [EMAIL PROTECTED]

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Jun'ichi Nomura
Patrick McHardy wrote: > Jun'ichi Nomura wrote: >> Are you using other dm modules such as dm-multipath, dm-mirror >> or dm-snapshot? >> If so, can you take the output of 'dmsetup table' and 'dmsetup ls'? > > No other modules. > >> Do you have a reliable way to reproduce the oops which I can try?

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Patrick McHardy
Jun'ichi Nomura wrote: >>>From: "Jun'ichi Nomura" <[EMAIL PROTECTED]> >>> >>>bio_alloc_bioset() will return NULL if 'num_vecs' is too large. >>>Use bio_get_nr_vecs() to get estimation of maximum number. >>> >>>Signed-off-by: "Jun'ichi Nomura" <[EMAIL PROTECTED]> >>>Signed-off-by: Alasdair G Kergon

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Jun'ichi Nomura
Hi, Patrick McHardy wrote: > Alasdair G Kergon wrote: >> From: "Jun'ichi Nomura" <[EMAIL PROTECTED]> >> >> bio_alloc_bioset() will return NULL if 'num_vecs' is too large. >> Use bio_get_nr_vecs() to get estimation of maximum number. >> >> Signed-off-by: "Jun'ichi Nomura" <[EMAIL PROTECTED]> >>

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Patrick McHardy
Alasdair G Kergon wrote: > From: "Jun'ichi Nomura" <[EMAIL PROTECTED]> > > bio_alloc_bioset() will return NULL if 'num_vecs' is too large. > Use bio_get_nr_vecs() to get estimation of maximum number. > > Signed-off-by: "Jun'ichi Nomura" <[EMAIL PROTECTED]> > Signed-off-by: Alasdair G Kergon

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Patrick McHardy
Alasdair G Kergon wrote: From: Jun'ichi Nomura [EMAIL PROTECTED] bio_alloc_bioset() will return NULL if 'num_vecs' is too large. Use bio_get_nr_vecs() to get estimation of maximum number. Signed-off-by: Jun'ichi Nomura [EMAIL PROTECTED] Signed-off-by: Alasdair G Kergon [EMAIL PROTECTED]

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Jun'ichi Nomura
Hi, Patrick McHardy wrote: Alasdair G Kergon wrote: From: Jun'ichi Nomura [EMAIL PROTECTED] bio_alloc_bioset() will return NULL if 'num_vecs' is too large. Use bio_get_nr_vecs() to get estimation of maximum number. Signed-off-by: Jun'ichi Nomura [EMAIL PROTECTED] Signed-off-by: Alasdair G

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Patrick McHardy
Jun'ichi Nomura wrote: From: Jun'ichi Nomura [EMAIL PROTECTED] bio_alloc_bioset() will return NULL if 'num_vecs' is too large. Use bio_get_nr_vecs() to get estimation of maximum number. Signed-off-by: Jun'ichi Nomura [EMAIL PROTECTED] Signed-off-by: Alasdair G Kergon [EMAIL PROTECTED] This

Re: [2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-17 Thread Jun'ichi Nomura
Patrick McHardy wrote: Jun'ichi Nomura wrote: Are you using other dm modules such as dm-multipath, dm-mirror or dm-snapshot? If so, can you take the output of 'dmsetup table' and 'dmsetup ls'? No other modules. Do you have a reliable way to reproduce the oops which I can try?

[2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-11 Thread Alasdair G Kergon
From: "Jun'ichi Nomura" <[EMAIL PROTECTED]> bio_alloc_bioset() will return NULL if 'num_vecs' is too large. Use bio_get_nr_vecs() to get estimation of maximum number. Signed-off-by: "Jun'ichi Nomura" <[EMAIL PROTECTED]> Signed-off-by: Alasdair G Kergon <[EMAIL PROTECTED]> ---

[2.6.23 PATCH 07/18] dm io: fix panic on large request

2007-07-11 Thread Alasdair G Kergon
From: Jun'ichi Nomura [EMAIL PROTECTED] bio_alloc_bioset() will return NULL if 'num_vecs' is too large. Use bio_get_nr_vecs() to get estimation of maximum number. Signed-off-by: Jun'ichi Nomura [EMAIL PROTECTED] Signed-off-by: Alasdair G Kergon [EMAIL PROTECTED] --- drivers/md/dm-io.c |5