2002-10-08 18:33:01

by Andrew Morton

[permalink] [raw]
Subject: Re: [Ext2-devel] [RFC] [PATCH 1/4] Add extended attributes to ext2/3

Christoph Hellwig wrote:
>
> On Tue, Oct 08, 2002 at 02:08:11PM -0400, [email protected] wrote:
> >
> > This is the first of four patches which add extended attribute support
> > to the ext2 and ext3 filesystems. It is a port of Andreas Gruenbacher's
> > patches, which have been well tested and in a number of distributions
> > (including RH 8, if I'm not mistaken) already.
>
> RH backed it out after the second or third beta due to bugginess..
>
> > This first patch creates a generic interface for registering caches with
> > the VM subsystem so that they can react appropriately to memory
> > pressure.
>
> I'd suggest Ed Tomlinson's much saner interface that adds a third callbackj
> to kmem_cache_t (similar to the Solaris implementation) instead.
>
> Doing this outside slab is not a good idea (and XFS currently does
> it too - in it's own code which should be replaced with Ed's one)

Yup, although that's a fairly minor point in this context..

The shrinker callback code is not actually in Linus's kernel
at present. I'm kind of sitting on it until I've had time
to ponder the dirty great lock which Ed added ;)


2002-10-08 18:43:20

by Andreas Gruenbacher

[permalink] [raw]
Subject: Re: [Ext2-devel] [RFC] [PATCH 1/4] Add extended attributes to ext2/3

On Tuesday 08 October 2002 20:38, Andrew Morton wrote:
> [...]
> > > This first patch creates a generic interface for registering caches
> > > with the VM subsystem so that they can react appropriately to memory
> > > pressure.
> >
> > I'd suggest Ed Tomlinson's much saner interface that adds a third
> > callbackj to kmem_cache_t (similar to the Solaris implementation)
> > instead.
> >
> > Doing this outside slab is not a good idea (and XFS currently does
> > it too - in it's own code which should be replaced with Ed's one)
>
> Yup, although that's a fairly minor point in this context..
>
> The shrinker callback code is not actually in Linus's kernel
> at present. I'm kind of sitting on it until I've had time
> to ponder the dirty great lock which Ed added ;)

Switching to Ed's code once it's in the kernel may be worthwhile; until then
the dumb shrinking approaah doesn't to do much harm IMHO.

--Andreas.

2002-10-08 19:12:15

by Rik van Riel

[permalink] [raw]
Subject: Re: [Ext2-devel] [RFC] [PATCH 1/4] Add extended attributes to ext2/3

On Tue, 8 Oct 2002, Andreas Gruenbacher wrote:

> Switching to Ed's code once it's in the kernel may be worthwhile; until
> then the dumb shrinking approaah doesn't to do much harm IMHO.

Agreed.

Rik
--
A: No.
Q: Should I include quotations after my reply?

http://www.surriel.com/ http://distro.conectiva.com/