* [PATCH] Test netmap on FreeBSD
@ 2019-10-08 8:49 Thomas Huth
2019-10-08 9:48 ` Stefano Garzarella
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Huth @ 2019-10-08 8:49 UTC (permalink / raw)
To: Ed Maste, Li-Wen Hsu, Alex Bennée, qemu-devel
Cc: qemu-trivial, Markus Armbruster, Giuseppe Lettieri,
Vincenzo Maffione, Philippe Mathieu-Daudé,
Stefano Garzarella
FreeBSD offers a package for "netmap", thus let's install it in our
FreeBSD tests to get some compile test coverage for net/netmap.c, too.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
.cirrus.yml | 2 +-
tests/vm/freebsd | 3 +++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/.cirrus.yml b/.cirrus.yml
index 8326a3a4b1..44fea2c29e 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -8,7 +8,7 @@ freebsd_12_task:
memory: 8G
install_script: pkg install -y
bash bison curl cyrus-sasl git glib gmake gnutls gsed
- nettle perl5 pixman pkgconf png usbredir
+ netmap nettle perl5 pixman pkgconf png usbredir
script:
- mkdir build
- cd build
diff --git a/tests/vm/freebsd b/tests/vm/freebsd
index 2a19461a90..45821702bd 100755
--- a/tests/vm/freebsd
+++ b/tests/vm/freebsd
@@ -54,6 +54,9 @@ class FreeBSDVM(basevm.BaseVM):
# libs: opengl
"libepoxy",
"mesa-libs",
+
+ # libs: network
+ "netmap",
]
BUILD_SCRIPT = """
--
2.18.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] Test netmap on FreeBSD
2019-10-08 8:49 [PATCH] Test netmap on FreeBSD Thomas Huth
@ 2019-10-08 9:48 ` Stefano Garzarella
2019-10-08 9:52 ` Li-Wen Hsu
2019-10-08 9:57 ` Thomas Huth
0 siblings, 2 replies; 6+ messages in thread
From: Stefano Garzarella @ 2019-10-08 9:48 UTC (permalink / raw)
To: Thomas Huth
Cc: Ed Maste, qemu-trivial, Philippe Mathieu-Daudé,
qemu devel list, Markus Armbruster, Giuseppe Lettieri,
Alex Bennée, Li-Wen Hsu, Vincenzo Maffione
Hi Thomas,
On Tue, Oct 8, 2019 at 10:49 AM Thomas Huth <thuth@redhat.com> wrote:
>
> FreeBSD offers a package for "netmap", thus let's install it in our
> FreeBSD tests to get some compile test coverage for net/netmap.c, too.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .cirrus.yml | 2 +-
> tests/vm/freebsd | 3 +++
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/.cirrus.yml b/.cirrus.yml
> index 8326a3a4b1..44fea2c29e 100644
> --- a/.cirrus.yml
> +++ b/.cirrus.yml
> @@ -8,7 +8,7 @@ freebsd_12_task:
> memory: 8G
> install_script: pkg install -y
> bash bison curl cyrus-sasl git glib gmake gnutls gsed
> - nettle perl5 pixman pkgconf png usbredir
> + netmap nettle perl5 pixman pkgconf png usbredir
> script:
> - mkdir build
> - cd build
> diff --git a/tests/vm/freebsd b/tests/vm/freebsd
> index 2a19461a90..45821702bd 100755
> --- a/tests/vm/freebsd
> +++ b/tests/vm/freebsd
> @@ -54,6 +54,9 @@ class FreeBSDVM(basevm.BaseVM):
> # libs: opengl
> "libepoxy",
> "mesa-libs",
> +
> + # libs: network
> + "netmap",
IIRC netmap is shipped with the FreeBSD kernel, so we shouldn't need to
install it.
I tried to understand what is the 'netmap' pkg and I found the following
in my FreeBSD 12 VM:
# pkg search netmap
netmap-0.1.3_1 Make a graphical representation of the surrounding network
I don't think we are installing the right 'netmap' (I know, it is a bit
confusing :) )
I'll tried "make vm-build-freebsd" without this patch and I noticed
"netmap support yes" in the configure output. So I think we're
already compiling the netmap backend in the 'vm-buil-freebsd' target and
maybe also in cirrus.
Maybe Ed and Li-Wen can confirm (or not).
Thanks,
Stefano
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Test netmap on FreeBSD
2019-10-08 9:48 ` Stefano Garzarella
@ 2019-10-08 9:52 ` Li-Wen Hsu
2019-10-08 17:07 ` Vincenzo Maffione
2019-10-08 17:12 ` Vincenzo Maffione
2019-10-08 9:57 ` Thomas Huth
1 sibling, 2 replies; 6+ messages in thread
From: Li-Wen Hsu @ 2019-10-08 9:52 UTC (permalink / raw)
To: Stefano Garzarella
Cc: Thomas Huth, Ed Maste, qemu-trivial, Philippe Mathieu-Daudé,
qemu devel list, Markus Armbruster, Giuseppe Lettieri,
Alex Bennée, Vincenzo Maffione
On Tue, Oct 8, 2019 at 5:49 PM Stefano Garzarella <sgarzare@redhat.com> wrote:
>
> Hi Thomas,
>
> On Tue, Oct 8, 2019 at 10:49 AM Thomas Huth <thuth@redhat.com> wrote:
> >
> > FreeBSD offers a package for "netmap", thus let's install it in our
> > FreeBSD tests to get some compile test coverage for net/netmap.c, too.
> >
> > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > ---
> > .cirrus.yml | 2 +-
> > tests/vm/freebsd | 3 +++
> > 2 files changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/.cirrus.yml b/.cirrus.yml
> > index 8326a3a4b1..44fea2c29e 100644
> > --- a/.cirrus.yml
> > +++ b/.cirrus.yml
> > @@ -8,7 +8,7 @@ freebsd_12_task:
> > memory: 8G
> > install_script: pkg install -y
> > bash bison curl cyrus-sasl git glib gmake gnutls gsed
> > - nettle perl5 pixman pkgconf png usbredir
> > + netmap nettle perl5 pixman pkgconf png usbredir
>
> > script:
> > - mkdir build
> > - cd build
> > diff --git a/tests/vm/freebsd b/tests/vm/freebsd
> > index 2a19461a90..45821702bd 100755
> > --- a/tests/vm/freebsd
> > +++ b/tests/vm/freebsd
> > @@ -54,6 +54,9 @@ class FreeBSDVM(basevm.BaseVM):
> > # libs: opengl
> > "libepoxy",
> > "mesa-libs",
> > +
> > + # libs: network
> > + "netmap",
>
> IIRC netmap is shipped with the FreeBSD kernel, so we shouldn't need to
> install it.
>
> I tried to understand what is the 'netmap' pkg and I found the following
> in my FreeBSD 12 VM:
>
> # pkg search netmap
> netmap-0.1.3_1 Make a graphical representation of the surrounding network
>
> I don't think we are installing the right 'netmap' (I know, it is a bit
> confusing :) )
>
> I'll tried "make vm-build-freebsd" without this patch and I noticed
> "netmap support yes" in the configure output. So I think we're
> already compiling the netmap backend in the 'vm-buil-freebsd' target and
> maybe also in cirrus.
>
> Maybe Ed and Li-Wen can confirm (or not).
Yes netmap(4) is a built-in and default feature on FreeBSD. netmap
package from ports is not related to it.
BTW, I haven't tried qemu's netmap feature on FreeBSD. Is there
anything we can test this more?
Best,
Li-Wen
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Test netmap on FreeBSD
2019-10-08 9:48 ` Stefano Garzarella
2019-10-08 9:52 ` Li-Wen Hsu
@ 2019-10-08 9:57 ` Thomas Huth
1 sibling, 0 replies; 6+ messages in thread
From: Thomas Huth @ 2019-10-08 9:57 UTC (permalink / raw)
To: Stefano Garzarella
Cc: Ed Maste, qemu-trivial, Philippe Mathieu-Daudé,
qemu devel list, Markus Armbruster, Giuseppe Lettieri,
Alex Bennée, Li-Wen Hsu, Vincenzo Maffione
On 08/10/2019 11.48, Stefano Garzarella wrote:
> Hi Thomas,
>
> On Tue, Oct 8, 2019 at 10:49 AM Thomas Huth <thuth@redhat.com> wrote:
>>
>> FreeBSD offers a package for "netmap", thus let's install it in our
>> FreeBSD tests to get some compile test coverage for net/netmap.c, too.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> .cirrus.yml | 2 +-
>> tests/vm/freebsd | 3 +++
>> 2 files changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/.cirrus.yml b/.cirrus.yml
>> index 8326a3a4b1..44fea2c29e 100644
>> --- a/.cirrus.yml
>> +++ b/.cirrus.yml
>> @@ -8,7 +8,7 @@ freebsd_12_task:
>> memory: 8G
>> install_script: pkg install -y
>> bash bison curl cyrus-sasl git glib gmake gnutls gsed
>> - nettle perl5 pixman pkgconf png usbredir
>> + netmap nettle perl5 pixman pkgconf png usbredir
>
>> script:
>> - mkdir build
>> - cd build
>> diff --git a/tests/vm/freebsd b/tests/vm/freebsd
>> index 2a19461a90..45821702bd 100755
>> --- a/tests/vm/freebsd
>> +++ b/tests/vm/freebsd
>> @@ -54,6 +54,9 @@ class FreeBSDVM(basevm.BaseVM):
>> # libs: opengl
>> "libepoxy",
>> "mesa-libs",
>> +
>> + # libs: network
>> + "netmap",
>
> IIRC netmap is shipped with the FreeBSD kernel, so we shouldn't need to
> install it.
>
> I tried to understand what is the 'netmap' pkg and I found the following
> in my FreeBSD 12 VM:
>
> # pkg search netmap
> netmap-0.1.3_1 Make a graphical representation of the surrounding network
>
> I don't think we are installing the right 'netmap' (I know, it is a bit
> confusing :) )
>
> I'll tried "make vm-build-freebsd" without this patch and I noticed
> "netmap support yes" in the configure output. So I think we're
> already compiling the netmap backend in the 'vm-buil-freebsd' target and
> maybe also in cirrus.
Oops, I think you're right. I just checked an older build on cirrus-ci
(without this patch), and indeed, it also says "netmap support yes"
in the output of "configure". So we already have a compile test for
netmap :-)
Thus, please disregard this patch, it's not necessary indeed.
Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Test netmap on FreeBSD
2019-10-08 9:52 ` Li-Wen Hsu
@ 2019-10-08 17:07 ` Vincenzo Maffione
2019-10-08 17:12 ` Vincenzo Maffione
1 sibling, 0 replies; 6+ messages in thread
From: Vincenzo Maffione @ 2019-10-08 17:07 UTC (permalink / raw)
To: Li-Wen Hsu
Cc: Thomas Huth, Ed Maste, qemu-trivial, Philippe Mathieu-Daudé,
qemu devel list, Markus Armbruster, Giuseppe Lettieri,
Alex Bennée, Stefano Garzarella
[-- Attachment #1: Type: text/plain, Size: 2837 bytes --]
Il giorno mar 8 ott 2019 alle ore 11:53 Li-Wen Hsu <lwhsu@freebsd.org> ha
scritto:
> On Tue, Oct 8, 2019 at 5:49 PM Stefano Garzarella <sgarzare@redhat.com>
> wrote:
> >
> > Hi Thomas,
> >
> > On Tue, Oct 8, 2019 at 10:49 AM Thomas Huth <thuth@redhat.com> wrote:
> > >
> > > FreeBSD offers a package for "netmap", thus let's install it in our
> > > FreeBSD tests to get some compile test coverage for net/netmap.c, too.
> > >
> > > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > > ---
> > > .cirrus.yml | 2 +-
> > > tests/vm/freebsd | 3 +++
> > > 2 files changed, 4 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/.cirrus.yml b/.cirrus.yml
> > > index 8326a3a4b1..44fea2c29e 100644
> > > --- a/.cirrus.yml
> > > +++ b/.cirrus.yml
> > > @@ -8,7 +8,7 @@ freebsd_12_task:
> > > memory: 8G
> > > install_script: pkg install -y
> > > bash bison curl cyrus-sasl git glib gmake gnutls gsed
> > > - nettle perl5 pixman pkgconf png usbredir
> > > + netmap nettle perl5 pixman pkgconf png usbredir
> >
> > > script:
> > > - mkdir build
> > > - cd build
> > > diff --git a/tests/vm/freebsd b/tests/vm/freebsd
> > > index 2a19461a90..45821702bd 100755
> > > --- a/tests/vm/freebsd
> > > +++ b/tests/vm/freebsd
> > > @@ -54,6 +54,9 @@ class FreeBSDVM(basevm.BaseVM):
> > > # libs: opengl
> > > "libepoxy",
> > > "mesa-libs",
> > > +
> > > + # libs: network
> > > + "netmap",
> >
> > IIRC netmap is shipped with the FreeBSD kernel, so we shouldn't need to
> > install it.
> >
> > I tried to understand what is the 'netmap' pkg and I found the following
> > in my FreeBSD 12 VM:
> >
> > # pkg search netmap
> > netmap-0.1.3_1 Make a graphical representation of the
> surrounding network
> >
> > I don't think we are installing the right 'netmap' (I know, it is a bit
> > confusing :) )
> >
> > I'll tried "make vm-build-freebsd" without this patch and I noticed
> > "netmap support yes" in the configure output. So I think we're
> > already compiling the netmap backend in the 'vm-buil-freebsd' target and
> > maybe also in cirrus.
> >
> > Maybe Ed and Li-Wen can confirm (or not).
>
> Yes netmap(4) is a built-in and default feature on FreeBSD. netmap
> package from ports is not related to it.
>
> BTW, I haven't tried qemu's netmap feature on FreeBSD. Is there
> anything we can test this more?
>
This is an example command line for a qemu VM with a vtnet interface
connected to a port of a VALE switch called 'vale1':
qemu-system-x86_64 fbsd-head.qcow2 -smp 2 -m 2G -vga std -device
e1000,netdev=mgmt,mac=00:AA:BB:CC:0a:99 -netdev
user,id=mgmt,hostfwd=tcp::20010-:22 -device
virtio-net-pci,netdev=data10,mac=00:AA:BB:CC:0a:0a,ioeventfd=on,mrg_rxbuf=on
-netdev netmap,ifname=vale1:10,id=data10
Cheers,
Vincenzo
>
> Best,
> Li-Wen
>
[-- Attachment #2: Type: text/html, Size: 4448 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Test netmap on FreeBSD
2019-10-08 9:52 ` Li-Wen Hsu
2019-10-08 17:07 ` Vincenzo Maffione
@ 2019-10-08 17:12 ` Vincenzo Maffione
1 sibling, 0 replies; 6+ messages in thread
From: Vincenzo Maffione @ 2019-10-08 17:12 UTC (permalink / raw)
To: Li-Wen Hsu
Cc: Thomas Huth, Ed Maste, qemu-trivial, Philippe Mathieu-Daudé,
qemu devel list, Markus Armbruster, Giuseppe Lettieri,
Alex Bennée, Stefano Garzarella
[-- Attachment #1: Type: text/plain, Size: 418 bytes --]
This is an example command line for a qemu VM with a vtnet interface
connected to a port of a VALE switch called 'vale1':
qemu-system-x86_64 fbsd-head.qcow2 -smp 2 -m 2G -vga std -device
e1000,netdev=mgmt,mac=00:AA:BB:CC:0a:99 -netdev
user,id=mgmt,hostfwd=tcp::20010-:22 -device
virtio-net-pci,netdev=data10,mac=00:AA:BB:CC:0a:0a,ioeventfd=on,mrg_rxbuf=on
-netdev netmap,ifname=vale1:10,id=data10
Cheers,
Vincenzo
[-- Attachment #2: Type: text/html, Size: 464 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-10-08 17:13 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-08 8:49 [PATCH] Test netmap on FreeBSD Thomas Huth
2019-10-08 9:48 ` Stefano Garzarella
2019-10-08 9:52 ` Li-Wen Hsu
2019-10-08 17:07 ` Vincenzo Maffione
2019-10-08 17:12 ` Vincenzo Maffione
2019-10-08 9:57 ` Thomas Huth
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).