Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-27 Thread Alexander Duyck
On 01/27/2015 04:53 AM, Hiroshi Shimamoto wrote: >>> On 01/22/2015 04:32 PM, Hiroshi Shimamoto wrote: >>>>> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast >>>>> promiscuous mode control >>>>>> "Skidmore, Donald C"

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-27 Thread Hiroshi Shimamoto
> > On 01/22/2015 04:32 PM, Hiroshi Shimamoto wrote: > > >> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast > > >> promiscuous mode control > > >>> "Skidmore, Donald C" writes: > > >>> > > >>>&

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-23 Thread Skidmore, Donald C
Jong; linux-kernel@vger.kernel.org; David Laight; Hayato Momma > Subject: Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > On 01/22/2015 04:32 PM, Hiroshi Shimamoto wrote: > >> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast > &

Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Alexander Duyck
On 01/22/2015 04:32 PM, Hiroshi Shimamoto wrote: >> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous >> mode control >>> "Skidmore, Donald C" writes: >>> >>>> My hang up is more related to: without the nob to e

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Skidmore, Donald C
el@vger.kernel.org; > Hayato Momma > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast > > promiscuous mode control > > > > > > "Skidmor

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Hiroshi Shimamoto
> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > > "Skidmore, Donald C" writes: > > > > > My hang up is more related to: without the nob to enable it (off by > > > default) we are letting one V

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Skidmore, Donald C
rnel.org; > Hayato Momma > Subject: Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > "Skidmore, Donald C" writes: > > > My hang up is more related to: without the nob to enable it (off by > > default) we are letting one V

Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Bjørn Mork
"Skidmore, Donald C" writes: > My hang up is more related to: without the nob to enable it (off by > default) we are letting one VF dictate policy for all the other VFs > and the PF. If one VF needs to be in promiscuous multicast so is > everyone else. Their stacks now needs to deal with all th

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Skidmore, Donald C
el@vger.kernel.org; Hayato Momma > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > From: Skidmore, Donald C > > > > From: Hiroshi Shimamoto > > > > > My concern is what is the real issue that VF multicast > > >

Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread Jeff Kirsher
On Thu, 2015-01-22 at 09:50 +, David Laight wrote: > From: Skidmore, Donald C > > > > From: Hiroshi Shimamoto > > > > > My concern is what is the real issue that VF multicast > promiscuous mode > > > can cause. > > > > > I think there is the 4k entries to filter multicast address, > and the >

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-22 Thread David Laight
From: Skidmore, Donald C > > > From: Hiroshi Shimamoto > > > > My concern is what is the real issue that VF multicast promiscuous mode > > can cause. > > > > I think there is the 4k entries to filter multicast address, and the > > > > current ixgbe/ixgbevf can turn all bits on from VM. That is alm

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-21 Thread Skidmore, Donald C
el@vger.kernel.org; Hayato Momma > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast > > promiscuous mode control > > > > From: Hiroshi Shimamoto

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-21 Thread Hiroshi Shimamoto
> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > From: Hiroshi Shimamoto > > My concern is what is the real issue that VF multicast promiscuous mode can > > cause. > > I think there is the 4k entries to fil

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-21 Thread David Laight
From: Hiroshi Shimamoto > My concern is what is the real issue that VF multicast promiscuous mode can > cause. > I think there is the 4k entries to filter multicast address, and the current > ixgbe/ixgbevf > can turn all bits on from VM. That is almost same as enabling multicast > promiscuous mo

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-21 Thread Hiroshi Shimamoto
> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > > > -Original Message- > > From: Hiroshi Shimamoto [mailto:h-shimam...@ct.jp.nec.com] > > Sent: Tuesday, January 20, 2015 5:07 PM > > To: Skidmore

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-20 Thread Skidmore, Donald C
.org; Hayato Momma > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast > > promiscuous mode control > > > > > > > > > -Or

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-20 Thread Hiroshi Shimamoto
> Subject: RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > > > -Original Message- > > From: Hiroshi Shimamoto [mailto:h-shimam...@ct.jp.nec.com] > > Sent: Tuesday, January 20, 2015 3:40 PM &g

RE: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous mode control

2015-01-20 Thread Skidmore, Donald C
; Subject: Re: [E1000-devel] [PATCH 1/2] if_link: Add VF multicast promiscuous > mode control > > > Subject: Re: [PATCH 1/2] if_link: Add VF multicast promiscuous mode > > control > > > > Hiroshi Shimamoto writes: > > > > > From: Hiroshi Shimamo