Hi Michael and Stefan,
I thought about this and discussed it with my colleague Cong Wang.
One idea is to make current asynchronous send_pkt flow to be synchronous,
then if the virtqueue is full, the function can return ENOMEM all the way back
to the caller and the caller can check the return
On 26.03.21 10:53, Liu Xiang wrote:
Since pages have been deflated to a local list,
there is no race between fill and leak.
Signed-off-by: Liu Xiang
---
drivers/virtio/virtio_balloon.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/virtio/virtio_balloon.c
在 2021/3/26 下午2:56, Yongji Xie 写道:
On Fri, Mar 26, 2021 at 2:16 PM Jason Wang wrote:
在 2021/3/26 下午1:14, Yongji Xie 写道:
+ }
+ map->bounce_page = page;
+
+ /* paired with vduse_domain_map_page() */
+ smp_mb();
So this is suspicious. It's better to explain like, we need make
在 2021/3/26 下午1:14, Yongji Xie 写道:
+ }
+ map->bounce_page = page;
+
+ /* paired with vduse_domain_map_page() */
+ smp_mb();
So this is suspicious. It's better to explain like, we need make sure A
must be done after B.
OK. I see. It's used to protect this pattern:
