[dts] [PATCH V1]framework/dut: fix parameter error on some OS

Liu, Yong yong.liu at intel.com
Wed Jan 18 09:03:17 CET 2017


Hi Lijuan,
Function destroy_all_sriov_vfs should be workable on different OSs. 
If FreeBSD not support virtualization cases, we can just add one blank function for FreeBSD OS to work around this issue.

Thanks,
Marvin
> -----Original Message-----
> From: Tu, LijuanX A
> Sent: Wednesday, January 18, 2017 10:49 AM
> To: Liu, Yong <yong.liu at intel.com>; dts at dpdk.org
> Subject: RE: [dts] [PATCH V1]framework/dut: fix parameter error on some OS
> 
> Hi yong,
> We have function "get_sriov_vfs_pci_linux"
> but not have function "get_sriov_vfs_pci_freebsd"
> in framework.
> So I think we don't support on FreeBSD now.
> 
>  > -----Original Message-----
>  > From: Liu, Yong
>  > Sent: Wednesday, January 18, 2017 10:41 AM  > To: Tu, LijuanX A;
> dts at dpdk.org  > Cc: Tu, LijuanX A  > Subject: RE: [dts] [PATCH
> V1]framework/dut: fix parameter error on some OS  >  > Hi Lijuan,  >
> Virtualization case may not support on FreeBSD, but I think  >
> destroy_all_sriov_vfs this function should has FreeBSD and Linux version.
>  >
>  > > -----Original Message-----
>  > > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of Lijuan Tu  > > Sent:
> Tuesday, January 17, 2017 3:09 PM  > > To: dts at dpdk.org  > > Cc: Tu, LijuanX A
> <lijuanx.a.tu at intel.com>  > > Subject: [dts] [PATCH V1]framework/dut: fix
> parameter error on some OS  > >  > > Virtual test is only support linux now.
>  > > Framework not support freebsd
>  > >
>  > > Signed-off-by: Lijuan Tu <lijuanx.a.tu at intel.com>  > > ---  > >
> framework/dut.py | 3 +++  > >  1 file changed, 3 insertions(+)  > >  > > diff --git
> a/framework/dut.py b/framework/dut.py index 9cbdaf7..0cc45dd  > >
> 100644  > > --- a/framework/dut.py  > > +++ b/framework/dut.py  > > @@ -
> 820,6 +820,9 @@ class Dut(Crb):
>  > >
>  > >      def destroy_all_sriov_vfs(self):
>  > >
>  > > +        # virtual test is only support linux now.
>  > > +        if self.get_os_type() != "linux":
>  > > +
>  > >          if self.ports_info == None:
>  > >              return
>  > >          for port_id in range(len(self.ports_info)):
>  > > --
>  > > 1.9.3



More information about the dts mailing list