Re: 2.6.21-mm1: many processes end up in D state

2007-05-04 Thread Jiri Slaby
Cc: Mikael Pettersson <[EMAIL PROTECTED]> Cc: linux-ide@vger.kernel.org Cc: Jeff Garzik <[EMAIL PROTECTED]> Jiri Slaby napsal(a): > Cc: [EMAIL PROTECTED] > > Jiri Slaby napsal(a): >> Hi, >> >> I have a problem with higher disk loads (e.g. running git-log or yum update). >> Many processes end up i

Re: 2.6.21-mm1: many processes end up in D state

2007-05-04 Thread Jiri Slaby
Jiri Slaby napsal(a): > Cc: Mikael Pettersson <[EMAIL PROTECTED]> > Cc: linux-ide@vger.kernel.org > Cc: Jeff Garzik <[EMAIL PROTECTED]> > > Jiri Slaby napsal(a): >> Cc: [EMAIL PROTECTED] >> >> Jiri Slaby napsal(a): >>> Hi, >>> >>> I have a problem with higher disk loads (e.g. running git-log or

Re: 2.6.21-mm1: many processes end up in D state

2007-05-04 Thread Mikael Pettersson
On Fri, 04 May 2007 17:02:10 +0200, Jiri Slaby wrote: I have a problem with higher disk loads (e.g. running git-log or yum update). Many processes end up in D state and system is unusable -- I'm not able to run anything but smooth mouse moving when this happens. ...(borin

Re: 2.6.21-mm1: many processes end up in D state

2007-05-05 Thread Tejun Heo
Mikael Pettersson wrote: >> I.e. no freezing of ports... > > Your patch to delete the 'return 1;' on error is correct, > and makes the code match exactly the behaviour of previous > versions of sata_promise, except for the additional error > decoding. > > ahci and sata_sil24 do the return in this

Re: 2.6.21-mm1: many processes end up in D state

2007-05-05 Thread Mikael Pettersson
On Sat, 05 May 2007 17:30:51 +0200, Tejun Heo wrote: > Mikael Pettersson wrote: > >> I.e. no freezing of ports... > > > > Your patch to delete the 'return 1;' on error is correct, > > and makes the code match exactly the behaviour of previous > > versions of sata_promise, except for the additional

Re: 2.6.21-mm1: many processes end up in D state

2007-05-07 Thread Jiri Slaby
Mikael Pettersson napsal(a): > Jiri: please test the patch below instead. That is, revert to > the original code _with_ the 'return 1;', and then add this > patch to it. It should have pretty much the same effect as > removing the 'return 1;', however calling ata_port_abort() > is more in line with