Re: [PATCH net-next 02/12] ibmvnic: Introduce indirect subordinate Command Response Queue buffer

2020-11-13 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center

Re: [PATCH net-next 01/12] ibmvnic: Ensure that subCRQ entry reads are ordered

2020-11-13 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center

Re: [EXT] [PATCH] bnx2x: Prevent load reordering in tx completion processing

2019-07-18 Thread Brian King
On 7/18/19 5:12 AM, Manish Chopra wrote: >> -Original Message- >> From: Brian King >> Sent: Tuesday, July 16, 2019 3:12 AM >> To: GR-everest-linux-l2 >> Cc: Sudarsana Reddy Kalluru ; Ariel Elior >> ; netdev@vger.kernel.org; Brian King >> >

[PATCH] bnx2x: Prevent load reordering in tx completion processing

2019-07-15 Thread Brian King
: Brian King --- drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c index 656ed80..e2be5a6 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c

[PATCH] tg3: Fix rx hang on MTU change with 5717/5719

2017-12-15 Thread Brian King
8 and Power 9 systems as well as by Broadcom on x86 hardware and has been confirmed to resolve the hang issue. Cc: stable Signed-off-by: Brian King --- drivers/net/ethernet/broadcom/tg3.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/broadcom/tg3

Re: [PATCH net-next] ibmveth: v1 calculate correct gso_size and set gso_type

2016-11-11 Thread Brian King
s of real MSS used by the TCP > sender. > > Don't you have the MSS provided in RX descriptor, instead of guessing > the value ? Eric, We are currently pursuing making changes to the Power Virtual I/O Server to provide the MSS to the ibmveth driver. However, this will take time to go through test and ultimately get released. Although imperfect, this patch does help a real customer hitting this issue right now. Would you object to this patch getting merged as is, with the understanding that when we get the change in the Virtual I/O Server released, we will revert this interim change and apply the new method? Thanks, Brian -- Brian King Power Linux I/O IBM Linux Technology Center

Re: [PATCH net-next] ibmveth: v1 calculate correct gso_size and set gso_type

2016-11-09 Thread Brian King
On 11/06/2016 03:22 PM, Jonathan Maxwell wrote: > On Thu, Nov 3, 2016 at 8:40 AM, Brian King wrote: >> On 10/27/2016 10:26 AM, Eric Dumazet wrote: >>> On Wed, 2016-10-26 at 11:09 +1100, Jon Maxwell wrote: >>>> We recently encountered a bug where a few customers using

Re: [PATCH net-next] ibmveth: v1 calculate correct gso_size and set gso_type

2016-11-02 Thread Brian King
) is actually being done by software in the VIOS. What they may be able to do is when performing this aggregation, they could look at the packet lengths of all the packets being aggregated and take the largest packet size within the aggregation unit, minus the header length and return that to the

[PATCH 6/6] ibmveth: Remove use of bitfields

2007-08-17 Thread Brian King
Removes the use of bitfields from the ibmveth driver. This results in slightly smaller object code. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 90 linux-2.6-bjking1/drivers/net/ibmveth.h

[PATCH 4/6] ibmveth: Add ethtool driver stats hooks

2007-08-17 Thread Brian King
Add ethtool hooks to ibmveth to retrieve driver statistics. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 51 1 file changed, 51 insertions(+) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_driver_stats d

[PATCH 5/6] ibmveth: Remove dead frag processing code

2007-08-17 Thread Brian King
Removes dead frag processing code from ibmveth. Since NETIF_F_SG was not set, this code was never executed. Also, since the ibmveth interface can only handle 6 fragments, core networking code would need to be modified in order to efficiently enable this support. Signed-off-by: Brian King <[EM

[PATCH 3/6] ibmveth: Add ethtool TSO handlers

2007-08-17 Thread Brian King
Add handlers for get_tso and get_ufo to prevent errors being printed by ethtool. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c |2 ++ 1 file changed, 2 insertions(+) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_get_tso drivers/net/ibm

[PATCH 2/6] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-08-17 Thread Brian King
This patch adds the appropriate ethtool hooks to allow for enabling/disabling of hypervisor assisted checksum offload for TCP. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 125 +++- linux-2.6-bjking1/drive

[PATCH 1/6] ibmveth: Enable TCP checksum offload

2007-08-17 Thread Brian King
yielded a ~30% increase in throughput. This feature is enabled by default on systems that support it, but can be disabled with a module option. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 linux-2.6-bjking1/d

[PATCH 6/6] ibmveth: Remove use of bitfields

2007-08-10 Thread Brian King
Removes the use of bitfields from the ibmveth driver. This results in slightly smaller object code. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 90 linux-2.6-bjking1/drivers/net/ibmveth.h

[PATCH 5/6] ibmveth: Remove dead frag processing code

2007-08-10 Thread Brian King
Removes dead frag processing code from ibmveth. Since NETIF_F_SG was not set, this code was never executed. Also, since the ibmveth interface can only handle 6 fragments, core networking code would need to be modified in order to efficiently enable this support. Signed-off-by: Brian King <[EM

[PATCH 4/6] ibmveth: Add ethtool driver stats hooks

2007-08-10 Thread Brian King
Add ethtool hooks to ibmveth to retrieve driver statistics. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 51 1 file changed, 51 insertions(+) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_driver_stats d

[PATCH 3/6] ibmveth: Add ethtool TSO handlers

2007-08-10 Thread Brian King
Add handlers for get_tso and get_ufo to prevent errors being printed by ethtool. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c |2 ++ 1 file changed, 2 insertions(+) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_get_tso drivers/net/ibm

[PATCH 2/6] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-08-10 Thread Brian King
This patch adds the appropriate ethtool hooks to allow for enabling/disabling of hypervisor assisted checksum offload for TCP. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 125 +++- linux-2.6-bjking1/drive

[PATCH 1/6] ibmveth: Enable TCP checksum offload

2007-08-10 Thread Brian King
yielded a ~30% increase in throughput. This feature is enabled by default on systems that support it, but can be disabled with a module option. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 linux-2.6-bjking1/d

Re: [PATCH 2/6] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-08-07 Thread Brian King
Jeff Garzik wrote: > Brian King wrote: >> This patch adds the appropriate ethtool hooks to allow for enabling/disabling >> of hypervisor assisted checksum offload for TCP. >> >> Signed-off-by: Brian King <[EMAIL PROTECTED]> >> --- >> >&

Re: [PATCH] drivers/net/ibmveth.c: memset fix

2007-08-07 Thread Brian King
netdev->priv; > - memset(adapter, 0, sizeof(adapter)); > dev->dev.driver_data = netdev; > > adapter->vdev = dev; > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [EMAIL PROTECTED] > More ma

Re: [PATCH] drivers/net/ibmveth.c: memset fix

2007-08-06 Thread Brian King
r_data = netdev; > > adapter->vdev = dev; > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Brian King Linux on Pow

[PATCH 6/6] ibmveth: Remove use of bitfields

2007-08-06 Thread Brian King
Removes the use of bitfields from the ibmveth driver. This results in slightly smaller object code. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 90 linux-2.6-bjking1/drivers/net/ibmveth.h

[PATCH 4/6] ibmveth: Add ethtool driver stats hooks

2007-08-06 Thread Brian King
Add ethtool hooks to ibmveth to retrieve driver statistics. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- drivers/net/ibmveth.c | 53 +- 1 file changed, 52 insertions(+), 1 deletion(-) diff -puN drivers/net/ibm

[PATCH 5/6] ibmveth: Remove dead frag processing code

2007-08-06 Thread Brian King
Removes dead frag processing code from ibmveth. Since NETIF_F_SG was not set, this code was never executed. Also, since the ibmveth interface can only handle 6 fragments, core networking code would need to be modified in order to efficiently enable this support. Signed-off-by: Brian King <[EM

[PATCH 2/6] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-08-06 Thread Brian King
This patch adds the appropriate ethtool hooks to allow for enabling/disabling of hypervisor assisted checksum offload for TCP. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 118 +++- linux-2.6-bjking1/drive

[PATCH 3/6] ibmveth: Add ethtool TSO handlers

2007-08-06 Thread Brian King
Add handlers for get_tso and get_ufo to prevent errors being printed by ethtool. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- drivers/net/ibmveth.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_get_tso drivers/net/ibm

[PATCH 1/6] ibmveth: Enable TCP checksum offload

2007-08-06 Thread Brian King
yielded a ~30% increase in throughput. This feature is enabled by default on systems that support it, but can be disabled with a module option. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 linux-2.6-bjking1/d

[PATCH 1/1] ibmveth: Fix rx pool deactivate oops

2007-08-01 Thread Brian King
1 789c0020 1d3d00a8 7b8a1f24 3800 7c7f1b78 7d291a14 e9690128 <7c0a592a> e803 e9690120 80a90100 Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 24 ++-- linux-2.6-bjking1/drivers/net/ibmveth.h |3 --- 2 files

Re: [PATCH 2/4] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-07-19 Thread Brian King
Ragner Magalhaes wrote: > ext Brian King wrote: >> Ragner Magalhaes wrote: >>> here also, as above ... >>>> + if (data && (dev->features & NETIF_F_IP_CSUM)) >>>> + return 0; >>>> + if (!data && !(dev->feat

Re: [PATCH 2/4] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-07-19 Thread Brian King
Ragner Magalhaes wrote: > ext Brian King wrote: > >> + >> +static int ibmveth_set_rx_csum(struct net_device *dev, u32 data) >> +{ >> +struct ibmveth_adapter *adapter = dev->priv; >> + > > Why do not to do > > if ((data

[PATCH 3/4] ibmveth: Add ethtool TSO handlers

2007-07-19 Thread Brian King
Add handlers for get_tso and get_ufo to prevent errors being printed by ethtool. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_g

[PATCH 4/4] ibmveth: Add ethtool driver stats hooks

2007-07-19 Thread Brian King
Add ethtool hooks to ibmveth to retrieve driver statistics. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 +++- 1 file changed, 52 insertions(+), 1 deletion(-) diff -puN drivers/net/ibm

[PATCH 1/4] ibmveth: Enable TCP checksum offload

2007-07-19 Thread Brian King
yielded a ~30% increase in throughput. This feature is enabled by default on systems that support it, but can be disabled with a module option. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 linux-2.6-bjking1/d

[PATCH 2/4] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-07-19 Thread Brian King
This patch adds the appropriate ethtool hooks to allow for enabling/disabling of hypervisor assisted checksum offload for TCP. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 120 +++- linux-2.6-bjking1/drive

Re: [PATCH 1/4] ibmveth: Enable TCP checksum offload

2007-07-19 Thread Brian King
Jeff Garzik wrote: > Brian King wrote: >> This patchset enables TCP checksum offload support for IPV4 >> on ibmveth. This completely eliminates the generation and checking of >> the checksum for packets that are completely virtual and never >> touch a physical network.

[PATCH 4/4] ibmveth: Add ethtool driver stats hooks

2007-07-17 Thread Brian King
Add ethtool hooks to ibmveth to retrieve driver statistics. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 +++- 1 file changed, 52 insertions(+), 1 deletion(-) diff -puN drivers/net/ibm

[PATCH 1/4] ibmveth: Enable TCP checksum offload

2007-07-17 Thread Brian King
yielded a ~30% increase in throughput. This feature is enabled by default on systems that support it, but can be disabled with a module option. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 58 linux-2.6-bjking1/d

[PATCH 2/4] ibmveth: Implement ethtool hooks to enable/disable checksum offload

2007-07-17 Thread Brian King
This patch adds the appropriate ethtool hooks to allow for enabling/disabling of hypervisor assisted checksum offload for TCP. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 120 +++- linux-2.6-bjking1/drive

[PATCH 3/4] ibmveth: Add ethtool TSO handlers

2007-07-17 Thread Brian King
Add handlers for get_tso and get_ufo to prevent errors being printed by ethtool. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff -puN drivers/net/ibmveth.c~ibmveth_ethtool_g

[PATCH 2/2] ibmveth: Automatically enable larger rx buffer pools for larger mtu

2007-06-08 Thread Brian King
patch makes ibmveth automatically allocate these larger buffer pools when the mtu is changed. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 27 +++ 1 file changed, 23 insertions(+), 4 deletions(-) diff -puN drive

[PATCH 1/2] ibmveth: Fix h_free_logical_lan error on pool resize

2007-06-08 Thread Brian King
48001531 e8410028 e93d00e0 7fa3eb78 f81d0430 4bfffdc9 38210090 Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 ++-- 1 file changed, 31 insertions(+), 22 deletions(-) diff -puN drivers/net/ibmveth.c~ibmveth_large_fr

[PATCH 2/2] ibmveth: Automatically enable larger rx buffer pools for larger mtu

2007-05-16 Thread Brian King
patch makes ibmveth automatically allocate these larger buffer pools when the mtu is changed. Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 27 +++ 1 file changed, 23 insertions(+), 4 deletions(-) diff -puN drive

[PATCH 1/2] ibmveth: Fix h_free_logical_lan error on pool resize

2007-05-16 Thread Brian King
48001531 e8410028 e93d00e0 7fa3eb78 f81d0430 4bfffdc9 38210090 Signed-off-by: Brian King <[EMAIL PROTECTED]> --- linux-2.6-bjking1/drivers/net/ibmveth.c | 53 ++-- 1 file changed, 31 insertions(+), 22 deletions(-) diff -puN drivers/net/ibmveth.c~ibmveth_large_fr