Re: [Qemu-devel] Re: [PATCH v4 3/5] qed: Table, L2 cache, and cluster functions

[Stefan Hajnoczi]

On Fri, Nov 12, 2010 at 5:26 PM, Kevin Wolf <address@hidden> wrote:
> Am 28.10.2010 13:01, schrieb Stefan Hajnoczi:
>> This patch adds code to look up data cluster offsets in the image via
>> the L1/L2 tables.  The L2 tables are writethrough cached in memory for
>> performance (each read/write requires a lookup so it is essential to
>> cache the tables).
>>
>> With cluster lookup code in place it is possible to implement
>> bdrv_is_allocated() to query the number of contiguous
>> allocated/unallocated clusters.
>>
>> Signed-off-by: Stefan Hajnoczi <address@hidden>
>
> Didn't find any obvious problems with this one, but maybe that just
> means that I'm tired and should stop working for today. ;-)

Thanks for your review effort, Kevin.

Stefan