[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] A few race conditions in E1000 device fixed
From: |
Dmitry Fleytman |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] A few race conditions in E1000 device fixed |
Date: |
Mon, 15 Oct 2012 22:30:14 +0200 |
Hello,
Please, ignore 1st patch for now.
Although it fixes the problem observed it looks like there is a better
and easier solution (many thanks to Intel Guys that explained e1000
operation in details:
http://sourceforge.net/mailarchive/forum.php?thread_name=CAGHCxhcad%3Dzx7ihX5zoDB%3DZOLGGuZty%3DBck6zSoMQ-9S3ZJo7w%40mail.gmail.com&forum_name=e1000-devel).
We'll submit the final patch soon.
Dmitry.
On Mon, Oct 15, 2012 at 6:48 PM, Dmitry Fleytman <address@hidden> wrote:
> Following patches fix a few race conditions in E1000 code:
>
> 1st patch fixes race condition between driver shutdown and device shutdown
> (see patch comment)
> It also work-arounds race condition in e1000 Linux driver between RX enable
> and RX rings init
> (Separate patch for the second problem sent to e1000-devel/linux/kernel and
> accepted by maintainers, see
>
> http://sourceforge.net/mailarchive/forum.php?thread_name=1350280341.2152.12.camel%40jtkirshe-mobl&forum_name=e1000-devel)
>
> 2nd patch is pretty trivial and adds forgotten field into live migration
> list thus fixing another race condition.
>
> Dmitry Fleytman (2):
> Fix a race condition in E1000 device implementation:
> Fix a race condition in E1000 device live migration. One of
> data-transfer related flags not in migrated fields list.
>
> hw/e1000.c | 25 +++++++++++++++++++++++--
> 1 file changed, 23 insertions(+), 2 deletions(-)
>
> --
> 1.7.11.4
>
--
Dmitry Fleytman
Technology Expert and Consultant,
Daynix Computing Ltd.
Cell: +972-54-2819481
Skype: dmitry.fleytman