lkml.org 
[lkml]   [2005]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [8/many] acrypto: crypto_dev.c
On Mon, 7 Mar 2005 14:51:21 -0800
Nish Aravamudan <nish.aravamudan@gmail.com> wrote:

> On Tue, 8 Mar 2005 02:14:31 +0300, Evgeniy Polyakov <johnpol@2ka.mipt.ru> wrote:
> > On Mon, 7 Mar 2005 14:40:52 -0800
> > Nish Aravamudan <nish.aravamudan@gmail.com> wrote:
> >
> > > On Mon, 7 Mar 2005 23:37:34 +0300, Evgeniy Polyakov <johnpol@2ka.mipt.ru> wrote:
> > > > --- /tmp/empty/crypto_dev.c 1970-01-01 03:00:00.000000000 +0300
> > > > +++ ./acrypto/crypto_dev.c 2005-03-07 20:35:36.000000000 +0300
> > > > @@ -0,0 +1,421 @@
> > > > +/*
> > > > + * crypto_dev.c
> > >
> > > <snip>
> > >
> > > > + while (atomic_read(&__dev->refcnt)) {
>
> <snip>
>
> > > > + set_current_state(TASK_UNINTERRUPTIBLE);
> > > > + schedule_timeout(HZ);
> > >
> > > I don't see any wait-queues in the immediate area of this code. Can
> > > this be an ssleep(1)?
> >
> > Yes, you are right, this loop just spins until all pending sessions
> > are removed from given crypto device, so it can just ssleep(1) here.
>
> Would you like me to send an incremental patch or will you be changing
> it yourself?

That would be nice to see your changes in the acrypto.
If it will be commited...

> Thanks,
> Nish


Evgeniy Polyakov

Only failure makes us experts. -- Theo de Raadt
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:10    [W:2.100 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site