[PATCH] LogFS take three

Wed May 16 08:36:44 EDT 2007

On 5/16/07, Jörn Engel <joern at lazybastard.org> wrote:
> > > +/* FIXME: all this mess should get replaced by using the page cache */
> > > +static void fixup_from_wbuf(struct super_block *sb, struct logfs_area
> > *area,
> > > + void *read, u64 ofs, size_t readlen)
> > > +{
> >
> > Indeed. And I think you're getting some more trouble because of this...
>> More trouble?

Forgot to add (see below). Seems logfs_segment_read would be simpler
too if you fixed this.
> [ Objects are the units that get compressed. Segments can contain both
> compressed and uncompressed objects. ]
>> It is a trade-off. Each object has a 24 Byte header plus X Bytes of
> data. Whether the data is compressed or not is indicated in the header.

Was my point really. Why do segments contain both compressed and
uncompressed objects?