Re: [E1000-devel] [PATCH v4 net-next 2/4] net: convert low latency sockets to sched_clock()

2013-06-13 Thread Eliezer Tamir
On 14/06/2013 06:12, Eric Dumazet wrote: > On Fri, 2013-06-14 at 04:57 +0300, Eliezer Tamir wrote: >> Use sched_clock() instead of get_cycles(). >> We can use sched_clock() because we don't care much about accuracy. >> Remove the dependency on X86_TSC >> >> Signed-off-by: Eliezer Tamir >> --- > >>

Re: [E1000-devel] [net-next 1/8] i40e: main driver core

2013-06-13 Thread Joe Perches
On Thu, 2013-06-13 at 20:55 -0700, Jeff Kirsher wrote: > From: Jesse Brandeburg > > This is the driver for the Intel(R) Ethernet Controller XL710 Family. This code looks very generic and not tailored to linux. Are you intending to fix it to be more linux-kernel like? > diff --git a/drivers/net/

[E1000-devel] [Fwd: [net-next 0/8][pull request] Intel Wired LAN Driver Updates]

2013-06-13 Thread Jeff Kirsher
Forwarded Message From: Jeff Kirsher To: da...@davemloft.net Cc: Jeff Kirsher , net...@vger.kernel.org, go...@redhat.com, sassm...@redhat.com Subject: [net-next 0/8][pull request] Intel Wired LAN Driver Updates Date: Thu, 13 Jun 2013 20:55:14 -0700 This series implements the new

[E1000-devel] [net-next 4/8] i40e: driver core headers

2013-06-13 Thread Jeff Kirsher
From: Jesse Brandeburg This patch contains the main driver header files, containing structures and data types specific to the Linux driver. i40e_osdep.h contains some code that helps us adapt our OS agnostic code to Linux. Signed-off-by: Jesse Brandeburg Signed-off-by: Shannon Nelson CC: PJ W

[E1000-devel] [net-next 8/8] i40e: include i40e in kernel proper

2013-06-13 Thread Jeff Kirsher
From: Jesse Brandeburg This patch adds the Kconfig, readme, MAINTAINERS and Kbuild changes to build i40e with the kernel. New driver build option is CONFIG_I40E Signed-off-by: Jesse Brandeburg Signed-off-by: Shannon Nelson CC: PJ Waskiewicz CC: e1000-devel@lists.sourceforge.net Signed-off-by

[E1000-devel] [net-next 2/8] i40e: transmit, receive, and napi

2013-06-13 Thread Jeff Kirsher
From: Jesse Brandeburg This patch contains the transmit, receive, and napi routines, as well as ancillary routines. This file is code that is (will be) shared between the VF and PF drivers. Signed-off-by: Jesse Brandeburg Signed-off-by: Shannon Nelson CC: PJ Waskiewicz CC: e1000-devel@lists.

[E1000-devel] [net-next 3/8] i40e: driver ethtool core

2013-06-13 Thread Jeff Kirsher
From: Jesse Brandeburg This patch contains the ethtool interface and implementation. The goal in this patch series is minimal functionality while not including much in the way of "set support." Signed-off-by: Jesse Brandeburg Signed-off-by: Shannon Nelson CC: PJ Waskiewicz CC: e1000-devel@li

Re: [E1000-devel] [PATCH v4 net-next 2/4] net: convert low latency sockets to sched_clock()

2013-06-13 Thread Eric Dumazet
On Fri, 2013-06-14 at 04:57 +0300, Eliezer Tamir wrote: > Use sched_clock() instead of get_cycles(). > We can use sched_clock() because we don't care much about accuracy. > Remove the dependency on X86_TSC > > Signed-off-by: Eliezer Tamir > --- > > -static inline bool can_poll_ll(cycles_t end_

Re: [E1000-devel] [PATCH v4 net-next 1/4] net: change sysctl_net_ll_poll into an unsigned int

2013-06-13 Thread Eric Dumazet
On Fri, 2013-06-14 at 04:56 +0300, Eliezer Tamir wrote: > There is no reason for sysctl_net_ll_poll to be an unsigned long. > Change it into an unsigned int. > Fix the proc handler. > > Signed-off-by: Eliezer Tamir > --- Acked-by: Eric Dumazet

Re: [E1000-devel] Regarding 82599 SR-IOV support in MSI

2013-06-13 Thread mhban
Dear John and Don, Thanks for kind reply. This ARM SoC doesn’t have additional features for virtualization like VT-d. And I'm not going to use SR-IOV for virtualization. I just want to do some experimental VF test only with limited 2 or 4. BTW, I have one more question about this. Do you mean roo

[E1000-devel] [PATCH v2 sockperf] sockperf: add SO_LL socketop support

2013-06-13 Thread Eliezer Tamir
Add lls socket option support. use --lls (value in usecs) to override global setting. Right now we always get and print the value of the option before we set, to show that the option works properly. This should be removed in an official release. --- src/Defs.h |3 +++ src/SockPerf.cpp

[E1000-devel] [PATCH v4 net-next 2/4] net: convert low latency sockets to sched_clock()

2013-06-13 Thread Eliezer Tamir
Use sched_clock() instead of get_cycles(). We can use sched_clock() because we don't care much about accuracy. Remove the dependency on X86_TSC Signed-off-by: Eliezer Tamir --- include/net/ll_poll.h | 32 +--- net/Kconfig |1 - 2 files changed, 17 ins

[E1000-devel] [PATCH v4 net-next 3/4] net: remove NET_LL_RX_POLL config menue

2013-06-13 Thread Eliezer Tamir
Remove NET_LL_RX_POLL from the config menu. Change default to y. Busy polling still needs to be enabled at run time. Signed-off-by: Eliezer Tamir --- net/Kconfig | 11 ++- 1 files changed, 2 insertions(+), 9 deletions(-) diff --git a/net/Kconfig b/net/Kconfig index e591668..51da839 1

[E1000-devel] [PATCH v4 net-next 4/4] net: add socket option for low latency polling

2013-06-13 Thread Eliezer Tamir
adds a socket option for low latency polling. This allows overriding the global sysctl value with a per-socket one. Unexport sysctl_net_ll_poll since for now it's not needed in modules. Signed-off-by: Eliezer Tamir --- arch/alpha/include/uapi/asm/socket.h |2 ++ arch/avr32/include/uapi/as

[E1000-devel] [PATCH v4 net-next 0/4] net: low latency sockets follow ups

2013-06-13 Thread Eliezer Tamir
David, Here are three clean up patches, plus one that adds a socket option for LLS. Patch 1 converts sysctl_net_ll_poll into an unsigned int Patch 2 converts low latency sockets to sched_clock. Patch 3 removes the config menu for NET_LL_RX_POLL and defaults to y. Patch 4 adds an SO_LL socket opti

[E1000-devel] [PATCH v4 net-next 1/4] net: change sysctl_net_ll_poll into an unsigned int

2013-06-13 Thread Eliezer Tamir
There is no reason for sysctl_net_ll_poll to be an unsigned long. Change it into an unsigned int. Fix the proc handler. Signed-off-by: Eliezer Tamir --- include/net/ll_poll.h |5 +++-- net/core/sysctl_net_core.c |4 ++-- net/socket.c |2 +- 3 files changed, 6 inse

Re: [E1000-devel] [PATCH v3 net-next 0/4] net: low latency sockets follow ups

2013-06-13 Thread David Miller
From: Eliezer Tamir Date: Thu, 13 Jun 2013 17:46:13 +0300 > Here are three clean up patches, plus one that adds a socket option for LLS. > > Patch 1 converts low latency sockets to sched_clock. > Patch 2 removes the config menu for NET_LL_RX_POLL and defaults to y. > Patch 3 converts sysctl_net_

Re: [E1000-devel] [PATCH v3 net-next 3/4] net: change sysctl_net_ll_poll into an unsigned int

2013-06-13 Thread Eliezer Tamir
On 13/06/2013 18:31, Eric Dumazet wrote: > On Thu, 2013-06-13 at 17:46 +0300, Eliezer Tamir wrote: >> +extern unsigned int sysctl_net_ll_poll __read_mostly; >> > > Oh well, you re-introduce the overflow > > hint : you defined > > #define USECS_TO_NS(us) (us << 10) ouch! (thanks) -

Re: [E1000-devel] [PATCH v3 net-next 3/4] net: change sysctl_net_ll_poll into an unsigned int

2013-06-13 Thread Eric Dumazet
On Thu, 2013-06-13 at 17:46 +0300, Eliezer Tamir wrote: > There is no reason for sysctl_net_ll_poll to be an unsigned long. > Change it into an unsigned int. > Fix the proc handler. > > Signed-off-by: Eliezer Tamir > --- > > include/net/ll_poll.h |2 +- > net/core/sysctl_net_core.c |

Re: [E1000-devel] Regarding 82599 SR-IOV support in MSI

2013-06-13 Thread Ronciak, John
Hello, No it's not possible. MSI-X is needed so that interrupts can be routed to each guest. With MSI there is only a single interrupt so there is no way to route it to multiple VF's. MSI-X is a requirement for SR-IOV support. So don's comments below are correct. Sorry. Does this system

[E1000-devel] [PATCH corrected v2 sockperf] sockperf: add SO_LL socketop support

2013-06-13 Thread Eliezer Tamir
Add lls socket option support. use --lls (value in usecs) to override global setting. Right now we always get and print the value of the option before we set, to show that the option works properly. This should be removed in an official release. --- src/Defs.h |3 +++ src/SockPerf.cpp

Re: [E1000-devel] [PATCH] sockperf: add SO_LL socketop support

2013-06-13 Thread Eliezer Tamir
On 13/06/2013 17:49, Eliezer Tamir wrote: > Add lls socket option support to sockperf. > Right now we always get the option before set to show the option is > working properly. We should probably remove that in an official release. > use --lls (value in usecs) to override global setting. Sorry, th

[E1000-devel] [PATCH] sockperf: add SO_LL socketop support

2013-06-13 Thread Eliezer Tamir
Add lls socket option support to sockperf. Right now we always get the option before set to show the option is working properly. We should probably remove that in an official release. use --lls (value in usecs) to override global setting. --- src/Defs.h |3 +++ src/SockPerf.cpp | 5

[E1000-devel] [PATCH v3 net-next 1/4] net: convert low latency sockets to sched_clock()

2013-06-13 Thread Eliezer Tamir
Use sched_clock() instead of get_cycles(). We can use sched_clock() because we don't care much about accuracy. Remove the dependency on X86_TSC Signed-off-by: Eliezer Tamir --- include/net/ll_poll.h | 23 +++ net/Kconfig |1 - 2 files changed, 11 insertions(+

[E1000-devel] [PATCH v3 net-next 3/4] net: change sysctl_net_ll_poll into an unsigned int

2013-06-13 Thread Eliezer Tamir
There is no reason for sysctl_net_ll_poll to be an unsigned long. Change it into an unsigned int. Fix the proc handler. Signed-off-by: Eliezer Tamir --- include/net/ll_poll.h |2 +- net/core/sysctl_net_core.c |4 ++-- net/socket.c |2 +- 3 files changed, 4 inserti

[E1000-devel] [PATCH v3 net-next 4/4] net: add socket option for low latency polling

2013-06-13 Thread Eliezer Tamir
adds a socket option for low latency polling. This allows overriding the global sysctl value with a per-socket one. Unexport sysctl_net_ll_poll since for now it's not needed in modules. Signed-off-by: Eliezer Tamir --- arch/alpha/include/uapi/asm/socket.h |2 ++ arch/avr32/include/uapi/as

[E1000-devel] [PATCH v3 net-next 0/4] net: low latency sockets follow ups

2013-06-13 Thread Eliezer Tamir
David, Here are three clean up patches, plus one that adds a socket option for LLS. Patch 1 converts low latency sockets to sched_clock. Patch 2 removes the config menu for NET_LL_RX_POLL and defaults to y. Patch 3 converts sysctl_net_ll_poll into an unsigned int Patch 4 adds an SO_LL socket opti

[E1000-devel] [PATCH v3 net-next 2/4] net: remove NET_LL_RX_POLL config menue

2013-06-13 Thread Eliezer Tamir
Remove NET_LL_RX_POLL from the config menu. Change default to y. Busy polling still needs to be enabled at run time. Signed-off-by: Eliezer Tamir --- net/Kconfig | 11 ++- 1 files changed, 2 insertions(+), 9 deletions(-) diff --git a/net/Kconfig b/net/Kconfig index e591668..51da839 1

Re: [E1000-devel] Regarding 82599 SR-IOV support in MSI

2013-06-13 Thread Skidmore, Donald C
Hey Minho, I believe that it would be problematic at best since as MSI uses identical address for all vectors in a Function and differentiates vectors in a Function only by LSBs in data. Thanks, -Don Skidmore > -Original Message- > From: mhban [mailto:mh...@samsung.com] > Sent: Thur

Re: [E1000-devel] [PATCH net-next 1/2] net: remove NET_LL_RX_POLL config menue

2013-06-13 Thread Eliezer Tamir
On 13/06/2013 11:00, Daniel Borkmann wrote: > On 06/13/2013 04:13 AM, Eliezer Tamir wrote: >> On 13/06/2013 05:01, Stephen Hemminger wrote: >>> On Wed, 12 Jun 2013 15:12:05 -0700 (PDT) >>> David Miller wrote: >>> From: Eliezer Tamir Date: Tue, 11 Jun 2013 17:24:28 +0300 >

Re: [E1000-devel] [PATCH net-next 1/2] net: remove NET_LL_RX_POLL config menue

2013-06-13 Thread Daniel Borkmann
On 06/13/2013 04:13 AM, Eliezer Tamir wrote: > On 13/06/2013 05:01, Stephen Hemminger wrote: >> On Wed, 12 Jun 2013 15:12:05 -0700 (PDT) >> David Miller wrote: >> >>> From: Eliezer Tamir >>> Date: Tue, 11 Jun 2013 17:24:28 +0300 >>> depends on X86_TSC >>> >>> Wait a second, I didn't no

[E1000-devel] Regarding 82599 SR-IOV support in MSI

2013-06-13 Thread mhban
Dear maintainer and all SR-IOV experts, I’m developing a board which is based on ARM SoC which support PCI Express root-port. I’m trying to install and activate SR-IOV with Intel X520-SR2 on this board(/w Linux). Up to now, ixgbe and ixgbevf driver is loaded successfully but driver prints error wh