mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* /pub/linux/kernel/people/hedrick/ide-2.5.32
@ 2002-08-29 19:46 Andre Hedrick
  2002-08-29 19:57 ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andrew Morton
  0 siblings, 1 reply; 4+ messages in thread
From: Andre Hedrick @ 2002-08-29 19:46 UTC (permalink / raw)
  To: linux-kernel


Stuff has arrived.


ide-viro-2.5.32.patch
ide-lad-2.5.32.patch

	ide-all-2.5.32.patch == ide-viro-2.5.32.patch + ide-lad-2.5.32.patch

ide-taskfile-2.5.32.patch
scsi-st-2.5.32.patch

There is one more thing to fix.

./fs/mpage.c

/*
 * The largest-sized BIO which this code will assemble, in bytes.  Set this
 * to PAGE_CACHE_SIZE if your drivers are broken.
 */
#define MPAGE_BIO_MAX_SIZE 32768        //BIO_MAX_SIZE

This is confirmed with Al Viro and was required to make things sane!

We are back.
We is a development team being composed to reduce my load and import fresh
ideas.  If you wnat to help please join in, we can make the halloween
party.

Cheers,

Andre Hedrick
LAD Storage Consulting Group


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: /pub/linux/kernel/people/hedrick/ide-2.5.32
  2002-08-29 19:46 /pub/linux/kernel/people/hedrick/ide-2.5.32 Andre Hedrick
@ 2002-08-29 19:57 ` Andrew Morton
  2002-08-29 20:02   ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andre Hedrick
  0 siblings, 1 reply; 4+ messages in thread
From: Andrew Morton @ 2002-08-29 19:57 UTC (permalink / raw)
  To: Andre Hedrick; +Cc: linux-kernel

Andre Hedrick wrote:
> 
> ...
> There is one more thing to fix.
> 
> ./fs/mpage.c
> 
> /*
>  * The largest-sized BIO which this code will assemble, in bytes.  Set this
>  * to PAGE_CACHE_SIZE if your drivers are broken.
>  */
> #define MPAGE_BIO_MAX_SIZE 32768        //BIO_MAX_SIZE
> 
> This is confirmed with Al Viro and was required to make things sane!

You'll need to do the same thing to fs/direct-io.c:DIO_BIO_MAX_SIZE
in that case.

I'd suggest that you just go in and change BIO_MAX_SECTORS
to 64.   Or 32 if you happen to be using a qlogic controller :(

So everything's broken in there - a hardwired constant doesn't
cut it.   Jens is cooking up an `add_page_to_bio()' API which
will do the right thing based upon q->max_sectors.  But that
is not yet available.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: /pub/linux/kernel/people/hedrick/ide-2.5.32
  2002-08-29 19:57 ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andrew Morton
@ 2002-08-29 20:02   ` Andre Hedrick
  2002-08-29 20:35     ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andrew Morton
  0 siblings, 1 reply; 4+ messages in thread
From: Andre Hedrick @ 2002-08-29 20:02 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linux-kernel


Andrew,

I am just now getting back to crawling speeds in the 2.5 tree.
I can only comment on what works and what Viro tells me.
But if you think it needs to be globally set, until the updates from Jens
arrive, please send to Linus.  I am running my stuff by Viro and company.

Cheers,

On Thu, 29 Aug 2002, Andrew Morton wrote:

> Andre Hedrick wrote:
> > 
> > ...
> > There is one more thing to fix.
> > 
> > ./fs/mpage.c
> > 
> > /*
> >  * The largest-sized BIO which this code will assemble, in bytes.  Set this
> >  * to PAGE_CACHE_SIZE if your drivers are broken.
> >  */
> > #define MPAGE_BIO_MAX_SIZE 32768        //BIO_MAX_SIZE
> > 
> > This is confirmed with Al Viro and was required to make things sane!
> 
> You'll need to do the same thing to fs/direct-io.c:DIO_BIO_MAX_SIZE
> in that case.
> 
> I'd suggest that you just go in and change BIO_MAX_SECTORS
> to 64.   Or 32 if you happen to be using a qlogic controller :(
> 
> So everything's broken in there - a hardwired constant doesn't
> cut it.   Jens is cooking up an `add_page_to_bio()' API which
> will do the right thing based upon q->max_sectors.  But that
> is not yet available.
> 

Andre Hedrick
LAD Storage Consulting Group


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: /pub/linux/kernel/people/hedrick/ide-2.5.32
  2002-08-29 20:02   ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andre Hedrick
@ 2002-08-29 20:35     ` Andrew Morton
  0 siblings, 0 replies; 4+ messages in thread
From: Andrew Morton @ 2002-08-29 20:35 UTC (permalink / raw)
  To: Andre Hedrick; +Cc: linux-kernel, Jens Axboe

I'm a wee bit reluctant to shrink these settings globally,
because it will have a small impact on people's ongoing
evaluation of 2.5 on high-end SCSI hardware.

Could you please set BIO_MAX_SECTORS to 64 within the IDE
patch, and add a BIG FAT COMMENT?


Andre Hedrick wrote:
> 
> Andrew,
> 
> I am just now getting back to crawling speeds in the 2.5 tree.
> I can only comment on what works and what Viro tells me.
> But if you think it needs to be globally set, until the updates from Jens
> arrive, please send to Linus.  I am running my stuff by Viro and company.
> 
> Cheers,
> 
> On Thu, 29 Aug 2002, Andrew Morton wrote:
> 
> > Andre Hedrick wrote:
> > >
> > > ...
> > > There is one more thing to fix.
> > >
> > > ./fs/mpage.c
> > >
> > > /*
> > >  * The largest-sized BIO which this code will assemble, in bytes.  Set this
> > >  * to PAGE_CACHE_SIZE if your drivers are broken.
> > >  */
> > > #define MPAGE_BIO_MAX_SIZE 32768        //BIO_MAX_SIZE
> > >
> > > This is confirmed with Al Viro and was required to make things sane!
> >
> > You'll need to do the same thing to fs/direct-io.c:DIO_BIO_MAX_SIZE
> > in that case.
> >
> > I'd suggest that you just go in and change BIO_MAX_SECTORS
> > to 64.   Or 32 if you happen to be using a qlogic controller :(
> >
> > So everything's broken in there - a hardwired constant doesn't
> > cut it.   Jens is cooking up an `add_page_to_bio()' API which
> > will do the right thing based upon q->max_sectors.  But that
> > is not yet available.
> >
> 
> Andre Hedrick
> LAD Storage Consulting Group

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2002-08-29 20:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-08-29 19:46 /pub/linux/kernel/people/hedrick/ide-2.5.32 Andre Hedrick
2002-08-29 19:57 ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andrew Morton
2002-08-29 20:02   ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andre Hedrick
2002-08-29 20:35     ` /pub/linux/kernel/people/hedrick/ide-2.5.32 Andrew Morton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®