* /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®