qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [RFC PATCH v3] tests: rtl8139: test timers and interrup


From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [RFC PATCH v3] tests: rtl8139: test timers and interrupt
Date: Mon, 9 Feb 2015 10:21:15 +0000
User-agent: Mutt/1.5.23 (2014-03-12)

On Fri, Feb 06, 2015 at 05:07:17PM +0000, Frediano Ziglio wrote:
> 2015-02-06 16:54 GMT+00:00 Stefan Hajnoczi <address@hidden>:
> > On Thu, Jan 08, 2015 at 06:38:23PM +0000, Frediano Ziglio wrote:
> >> Test behaviour of timers and interrupts related to timeouts.
> >>
> >> Signed-off-by: Frediano Ziglio <address@hidden>
> >> ---
> >>  tests/Makefile       |   2 +-
> >>  tests/rtl8139-test.c | 181 
> >> +++++++++++++++++++++++++++++++++++++++++++++++++++
> >>  2 files changed, 182 insertions(+), 1 deletion(-)
> >>
> >> This patch was derived from a test I did while implementing timer in
> >> rtl8139 code. Now that there is support for integrated testing I converted
> >> it. The test was tested on a real NIC.
> >>
> >> As if it's the first test I wrote I don't know if syntax and details are
> >> fine. For instance should I remove nop test? Should I split my test?
> >>
> >> Changed from v2:
> >> - style (variable declaration, Perl script not able to spot it)
> >>
> >> Changed from v1:
> >> - style
> >
> > Thanks, applied to my net tree:
> > https://github.com/stefanha/qemu/commits/net
> 
> Hi,
>   all the comments seems to refer to my patch for testing the card but
> from https://github.com/stefanha/qemu/commits/net looks like you
> applied Paolo patch.
> 
> Is it expected?

Sorry, it didn't make it into last week's net pull request due to a
mistake.

It will be included in the next net pull request:
https://github.com/stefanha/qemu/commits/net

Stefan

Attachment: pgpLJ9rxXR5mR.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]