[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/6] Move paio_cancel() to new infrastructure.
From: |
Venkateswararao Jujjuri (JV) |
Subject: |
Re: [Qemu-devel] [PATCH 2/6] Move paio_cancel() to new infrastructure. |
Date: |
Tue, 30 Nov 2010 15:20:33 -0800 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.12) Gecko/20101027 Thunderbird/3.1.6 |
On 11/19/2010 2:07 AM, Stefan Hajnoczi wrote:
> On Thu, Nov 18, 2010 at 6:06 PM, Arun R Bharadwaj
> <address@hidden> wrote:
>> Move paio_cancel() to new infrastructure and introduce
>> the necessary APIs for this.
>>
>> Signed-off-by: Arun R Bharadwaj <address@hidden>
>> ---
>> posix-aio-compat.c | 92
>> ++++++++++++++++++++++++++++++++++++++++++----------
>> 1 files changed, 74 insertions(+), 18 deletions(-)
>
> This commit is not a safe step to make. paio_cancel() is going to
> cause uninitialized threadlet structures to be accessed.
>
> You split the patches up into smaller commits for easier review.
> Unfortunately this particular commit will result in a QEMU which
> builds successfully but has undefined behavior at runtime - it may
> crash or do random things.
>
> That's a problem for git-bisect(1) and in general for anyone who
> assumes they can build QEMU at an arbitrary point in the commit
> history. It's really important to preserve bisectability, because
> once commits get introduced that build bad QEMUs we can no longer have
> confidence in bisect!
>
> Please split up commits so they introduce a new feature incrementally
> but work properly at each step in the series.
Arun, you may move this to later part of the series to take care of this bisect
issue.
Thanks,
JV
>
> Stefan
>
- [Qemu-devel] [PATCH 0/6] v12: Threadlets: A generic task offloading framework., Arun R Bharadwaj, 2010/11/18
- [Qemu-devel] [PATCH 1/6] Make the necessary changes in Makefile and configure file., Arun R Bharadwaj, 2010/11/18
- [Qemu-devel] [PATCH 2/6] Move paio_cancel() to new infrastructure., Arun R Bharadwaj, 2010/11/18
- [Qemu-devel] [PATCH 3/6] Move qemu_paio_submit() to the new infrastructure., Arun R Bharadwaj, 2010/11/18
- [Qemu-devel] [PATCH 4/6] Cleanup posix-aio.compat.c off all the old code., Arun R Bharadwaj, 2010/11/18
- [Qemu-devel] [PATCH 5/6] Move threadlets infrastructure to qemu-threadlets.c, Arun R Bharadwaj, 2010/11/18
- [Qemu-devel] [PATCH 6/6] Add helper functions to enable virtio-9p make use of the threadlets, Arun R Bharadwaj, 2010/11/18