Ilya Yanok said the following:
> Hi Wolfgang,
>
> Wolfgang Denk wrote:
>>> here is a set of changes we made to improve U-Boot JFFS2 code
>>> performance. We still can't reach Linux's performance but improvements
>>> are significant.
>>>
>>> Any comments are welcome.
>>>
>>
>> Are these patches independent of each other, or are all of them
>> needed, and if so, is there any specific order in which they have to
>> be applied?
>>
> Argh... I forgot to create patches with xxx/nnn notation...
> Here is the order of patches:
> 0001-jffs2-add-sector_size-field-to-part_info-structure.patch
> 0002-jffs2-rewrite-jffs2-scanning-code-based-on-Linux-on.patch
> 0003-jffs2-summary-support.patch
> 0004-jffs2-fix-searching-for-latest-version-in-jffs2_1pa.patch
> 0005-jffs2-add-buffer-to-cache-flash-accesses.patch
> 0006-jffs2-cache-data_crc-results.patch
>
> It's the order of changes in my tree and thus it's the simplest way to
> apply this patches. But actual dependencies are as follows:
> 0001 and 0004 don't depend on anything
> 0002 depends on 0001
> 0003, 0005 and 0006 depend on 0002
>
> Regards, Ilya.
I have put these patches to our board (MPC8548, NOR Flash, MTD partition
size 48MB).
a) performance has improved (execution time of 'ls' reduced from ~16s to
~1.5s)
b) there is a bug in the patch
0005-jffs2-add-buffer-to-cache-flash-accesses:
The passage
@@ -724,12 +731,14 @@ jffs2_1pass_read_inode(struct b_lists *pL, u32
inode, char *dest)
latestVersion = jNode->version;
}
}
- put_fl_mem(jNode);
+ if (pL->readbuf == NULL)
+ put_fl_mem(jNode);
}
#endif
should be
@@ -724,12 +731,14 @@ jffs2_1pass_read_inode(struct b_lists *pL, u32
inode, char *dest)
latestVersion = jNode->version;
}
}
- put_fl_mem(jNode);
+ if (pL->readbuf == NULL)
+ put_fl_mem(jNode, NULL);
}
#endif
c) I found no improvement with CFG_JFFS2_SORT_FRAGMENTS set (the way I
detected b) ;-)
d) output of 'ls' is corrupted: no date and access right output,
directories in / are shown multiple times.
--
Michael Lawnick
Software Design Engineer
Lise-Meitner-Str. 7/1
89081 Ulm
Tel: +49 731 9533 2066
[EMAIL PROTECTED]
http://www.nokiasiemensnetworks.com/global/
Think before you print
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
