[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/48] ivshmem series
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PULL 00/48] ivshmem series |
Date: |
Wed, 7 Oct 2015 08:44:13 -0400 (EDT) |
Hi
----- Original Message -----
> Am 07.10.2015 um 14:16 schrieb Marc-André Lureau:
> > ----- Original Message -----
> >> Am 06.10.2015 um 21:18 schrieb address@hidden:
> >>> Marc-André Lureau (45):
> >> [...]
> >>> tests: add ivshmem qtest
> >>
> >> I had NAK'ed this patch in v1 and it has not been fixed. If this pull
> >> gets merged I will immediately revert it. Not funny.
> >>
> >
> >
> > Could stick to technical review, please. The test runs fine without kvm.
> > Regarding your copyright claim, I already explain that your older version
> > of boilerplate test is really nothing compare to this one. But if you feel
> > so strongly about it, I don't care you add a copyright line.
>
> It is non-technical and called plagiarism.
All tests share common boilerplate. Your test is not even in my tests. You
could add it back!
> This is not about adding a copyright line to the file, it's about having
> a Signed-off-by on your patch. I had the same discussion with Paolo
> before, when he supposedly saw-but-not-read my patch. The common
> denominator is that every time this happens to me it's address@hidden
they are everywhere :)
> You were arguing that because your patch does more than mine you don't
> need to carry my copyright and Sob - that's an invalid argument given
> that even trivial refactoring changes by copyright holder IBM have been
> blocking our relicensing efforts. We chose not to define a threshold.
I propose to add your patch first, that way you get your test and your
copyright. Would that work for you?
- [Qemu-devel] [PULL 23/48] ivshmem: migrate with VMStateDescription, (continued)
- [Qemu-devel] [PULL 23/48] ivshmem: migrate with VMStateDescription, marcandre . lureau, 2015/10/08
- [Qemu-devel] [PULL 31/48] contrib: add ivshmem client and server, marcandre . lureau, 2015/10/08
- [Qemu-devel] [PULL 20/48] ivshmem: simplify a bit the code, marcandre . lureau, 2015/10/08
- [Qemu-devel] [PULL 22/48] ivshmem: use common is_power_of_2(), marcandre . lureau, 2015/10/08
- [Qemu-devel] [PULL 42/48] ivshmem: do not keep shm_fd open, marcandre . lureau, 2015/10/08
- [Qemu-devel] [PULL 16/48] ivshmem: remove max_peer field, marcandre . lureau, 2015/10/08
- [Qemu-devel] [PULL 36/48] ivshmem: add check on protocol version in QEMU, marcandre . lureau, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Andreas Färber, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Marc-André Lureau, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Andreas Färber, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series,
Marc-André Lureau <=
- [Qemu-devel] [PATCH 1/2] tests: Add ivshmem qtest, Andreas Färber, 2015/10/11
- Re: [Qemu-devel] [PATCH 1/2] tests: Add ivshmem qtest, Marc-André Lureau, 2015/10/10
- [Qemu-devel] [PATCH 2/2] ivshmem-test: Implement tests, Andreas Färber, 2015/10/11
- Re: [Qemu-devel] [PATCH 2/2] ivshmem-test: Implement tests, Andreas Färber, 2015/10/11
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Paolo Bonzini, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Andrew Jones, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Andreas Färber, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Peter Maydell, 2015/10/08
- Re: [Qemu-devel] [PULL 00/48] ivshmem series, Marc-André Lureau, 2015/10/08
[Qemu-devel] [PULL 01/48] char: add qemu_chr_free(), marcandre . lureau, 2015/10/08