Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-07-09 Thread Andreas Färber
Hi Marc-André, Am 09.07.2015 um 16:37 schrieb Marc-André Lureau: > On Fri, Jun 26, 2015 at 6:02 PM, Marc-André Lureau > wrote: >>> Also, see the discussion on my RFC: CONFIG_LINUX is wrong here. It would >>> need to depend on KVM/eventfd, and for that there is no easy config >>> option available

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-07-09 Thread Marc-André Lureau
Hi Andreas On Fri, Jun 26, 2015 at 6:02 PM, Marc-André Lureau wrote: >> Also, see the discussion on my RFC: CONFIG_LINUX is wrong here. It would >> need to depend on KVM/eventfd, and for that there is no easy config >> option available here, which stopped my patch at the time. > > > Right, I don'

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Andreas Färber
Am 26.06.2015 um 18:02 schrieb Marc-André Lureau: > On Fri, Jun 26, 2015 at 5:53 PM, Andreas Färber > wrote: > > Please don't use HTML mails, that breaks quoting as you can see below. > > > I mostly use gmail, afaik it's multipart, never had issues with that. The r

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Andreas Färber
Am 26.06.2015 um 17:44 schrieb Marc-André Lureau: > On Fri, Jun 26, 2015 at 5:27 PM, Marc-André Lureau > mailto:marcandre.lur...@gmail.com>> wrote: > > Sorry which patch are you refering to? > > > You probably mean about the missing include: I wasn't aware such patch > was already submitted,

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Marc-André Lureau
Hi On Fri, Jun 26, 2015 at 5:53 PM, Andreas Färber wrote: > Please don't use HTML mails, that breaks quoting as you can see below. > > I mostly use gmail, afaik it's multipart, never had issues with that. Am 26.06.2015 um 17:27 schrieb Marc-André Lureau: > > On Fri, Jun 26, 2015 at 5:08 PM, And

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Andreas Färber
Hi, Please don't use HTML mails, that breaks quoting as you can see below. Am 26.06.2015 um 17:27 schrieb Marc-André Lureau: > On Fri, Jun 26, 2015 at 5:08 PM, Andreas Färber > wrote: > > > For the second time within days you're posting a patch for something >

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Marc-André Lureau
On Fri, Jun 26, 2015 at 5:27 PM, Marc-André Lureau < marcandre.lur...@gmail.com> wrote: > Sorry which patch are you refering to? You probably mean about the missing include: I wasn't aware such patch was already submitted, and I don't think it's a problem if two people identify the same issue an

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Marc-André Lureau
Hi On Fri, Jun 26, 2015 at 5:08 PM, Andreas Färber wrote: > > For the second time within days you're posting a patch for something > that's been on the list already... The commit message above is entirely > Sorry which patch are you refering to? Is it wrong to repost a old patch and adding, wh

[Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Marc-André Lureau
Cc: Cam Macdonell CC: Andreas Färber Signed-off-by: Marc-André Lureau --- tests/Makefile | 3 + tests/ivshmem-test.c | 475 +++ 2 files changed, 478 insertions(+) create mode 100644 tests/ivshmem-test.c diff --git a/tests/Makefile b/test

Re: [Qemu-devel] [PATCH 39/39] tests: Add ivshmem qtest

2015-06-26 Thread Andreas Färber
Marc-André, Am 26.06.2015 um 16:49 schrieb Marc-André Lureau: > Cc: Cam Macdonell > CC: Andreas Färber > Signed-off-by: Marc-André Lureau > --- > tests/Makefile | 3 + > tests/ivshmem-test.c | 475 > +++ > 2 files changed, 478 insertions