qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [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).