linux-kernel - Re: [OP] v2.6.22.22-op1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6101e8c40805010817p33cc9bb3v8d80ab857af967b9@mail.gmail.com>
Date:	Thu, 1 May 2008 17:17:03 +0200
From:	"Oliver Pinter" <oliver.pntr@...il.com>
To:	"Jeff Moyer" <jmoyer@...hat.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [OP] v2.6.22.22-op1

sure, bad branch see I

On 5/1/08, Oliver Pinter <oliver.pntr@...il.com> wrote:
> On 5/1/08, Jeff Moyer <jmoyer@...hat.com> wrote:
> > <oliver@...ernowsky.hu> writes:
> >
> > > I announced the v2.6.22.22-op1 "stable" kernel.
> >
> > >  fs/aio.c                    |   23 ++++++++++++-----------
> >
> > > Jeff Moyer (1):
> > >       aio: account I/O wait time properly
> >
> > This version of the patch introduced a regression for UML.  It was
> > seeing 100% iowait time when it wasn't waiting on any I/O.  There was a
> > follow-up patch which did the io_schedule conditionally, see commit id:
> >
> >   e00ba3dae077f54cfd2af42e939a618caa7a3bca
> >
> > Cheers,
> >
> > Jeff
> >
> > > diff --git a/fs/aio.c b/fs/aio.c
> > > index dbe699e..b3419c5 100644
> > > --- a/fs/aio.c
> > > +++ b/fs/aio.c
> > > @@ -1170,7 +1171,7 @@ retry:
> > >  			ret = 0;
> > >  			if (to.timed_out)	/* Only check after read evt */
> > >  				break;
> > > -			schedule();
> > > +			io_schedule();
> > >  			if (signal_pending(tsk)) {
> > >  				ret = -EINTR;
> > >  				break;
> >
>
>
> hmm, this is the relevant code in 2.6.22:
> ----8<-------------->8-----
>                         if (min_nr <= i)
>                                 break;
>                         ret = 0;
>                         if (to.timed_out)       /* Only check after read evt
> */
>                                 break;
>                         schedule(); <-----------
>                         if (signal_pending(tsk)) {
>                                 ret = -EINTR;
>                                 break;
>                         }
>                         /*ret = aio_read_evt(ctx, &ent);*/
> ----8<-------------->8-----
>
> --
> Thanks,
> Oliver
>


-- 
Thanks,
Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/