[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v2 0/3] Cache modelling TCG plugin
From: |
Alex Bennée |
Subject: |
Re: [RFC PATCH v2 0/3] Cache modelling TCG plugin |
Date: |
Tue, 01 Jun 2021 14:30:31 +0100 |
User-agent: |
mu4e 1.5.13; emacs 28.0.50 |
Mahmoud Mandour <ma.mandourr@gmail.com> writes:
> In this RFC patch series, I propose an initial cache modelling TCG
> plugin. As of now, it models separate L1 data cache and L1 instruction
> cache. It supports three eviction policies: LRU, random, and FIFO. Once
> a policy is chosen, it's used for both instruction and data caches.
>
> v1 -> v2: Unlocked dmtx on early return in vcpu_mem_access & removed a
> (probably?) bad InsnData free.
> This is probably still problematic since it does not free the
> ``idata`` allocated for the vcpu_mem_access callback even
> once, but if it's placed, it would double-free it.
> How do I mitigate this? I need to free the InsnData passed to
> vcpu_mem_access only once if we find out that it's an IO
> access since we do not need it anymore and it will early
> return every time.
OK I've done my review, sorry I reviewed 1/3 from the previous series.
I've made some comments inline in those patches but I think this is an
excellent start to the project. I think we can get the core plugin
up-streamed fairly quickly and then spend some time examining better
integration and possibly enhancing the plugin API.
--
Alex Bennée
- Re: [RFC PATCH v2 0/3] Cache modelling TCG plugin,
Alex Bennée <=