On Wed, Jan 23 2008, Andrew Morton wrote:
> > On Tue, 22 Jan 2008 10:49:17 +0100 Jens Axboe <[EMAIL PROTECTED]> wrote:
> > -void put_io_context(struct io_context *ioc)
> > +int put_io_context(struct io_context *ioc)
> > {
> > if (ioc == NULL)
> > - return;
> > + return 1;
> >
> > BUG_ON(atomic_read(&ioc->refcount) == 0);
> >
> > @@ -3856,7 +3856,9 @@ void put_io_context(struct io_context *ioc)
> > rcu_read_unlock();
> >
> > kmem_cache_free(iocontext_cachep, ioc);
> > + return 1;
> > }
> > + return 0;
> > }
>
> Document the return value? (and the function)
Will do.
> I assume this return value gets used in some other patch.
Yeah, it is.
--
Jens Axboe
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
