lofi.c, line 1199 (new file)
uncompressed_seg_index is uninitialized, the initialization for it[*]
was lost during merge of the suggested changes; due to this bug the
decompression cache didn't work at all with this webrev.
[*]
diff --git a/usr/src/uts/common/io/lofi.c b/usr/src/uts/common/io/lofi.c
--- a/usr/src/uts/common/io/lofi.c
+++ b/usr/src/uts/common/io/lofi.c
...
@@ -991,6 +1175,7 @@
goto done;
}
}
+ uncompressed_seg_index = i;
/*
* Determine how much uncompressed data we
...
2009/3/10 sanjay nadkarni (Laptop) <Sanjay.Nadkarni at sun.com>:
> Hopefully third time is a charm..the review is located at:
>
> http://cr.opensolaris.org/~nadkarni/bug6679115+6805505/
>
> Dina wrote:
>>
>> Was there a link to ...? ?I looked at the CR for 6679115 in the
>> suggested fix and had no comments.
>>
>> Thanks,
>> D.
>>
>> sanjay nadkarni (Laptop) wrote:
>>>
>>> I am sponsoring the following changes for Juergen Keil. ?Please send your
>>> comments to caiman-discuss and cc jrgn.keil at sun.com. ?I would like these
>>> to
>>> be included in build 111 since that will help us reduce the number of ISOs
>>> for OpenSolaris.
>>>
>>> The bugs are:
>>> 6679115 lofi(7) shouldn't accept non-powers of 2 for a segment size
>>> ? ? ? 6805505 lofi needs a performance boost
>>>