[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [External] Re: [PATCH v8 4/5] migration: Introduce 'qatzip' compress
From: |
Prasad Pandit |
Subject: |
Re: [External] Re: [PATCH v8 4/5] migration: Introduce 'qatzip' compression method |
Date: |
Mon, 26 Aug 2024 12:44:03 +0530 |
On Sat, 24 Aug 2024 at 04:22, Yichen Wang <yichen.wang@bytedance.com> wrote:
> The patch was originally written exactly like what you suggest,
> cleanup in the error path of the same function. However, later I
> realized in gdb that I was wrong. The qatzip_send_cleanup() function
> will be called later in another thread in both normal and error paths.
> So I revised the patch to this behavior, otherwise I will run into
> double free in the error path.
>
* I see, okay, in that case:
Reviewed-by: Prasad Pandit <pjp@fedoraproject.org>
Thank you.
---
- Prasad
[PATCH v8 5/5] tests/migration: Add integration test for 'qatzip' compression method, Yichen Wang, 2024/08/20