Re: [PATCH] drivers/base: export gpl (un)register_memory_notifier

2008-02-18 Thread Jan-Bernd Themann
switching to proper mail client... Dave Hansen [EMAIL PROTECTED] wrote on 15.02.2008 17:55:38: I've been thinking about that, and I don't think you really *need* to keep a comprehensive map like that. When the memory is in a particular configuration (range of memory present along with

Re: [PATCH] drivers/base: export gpl (un)register_memory_notifier

2008-02-13 Thread Jan-Bernd Themann
Hi Dave, On Monday 11 February 2008 17:47, Dave Hansen wrote: Also, just ripping down and completely re-doing the entire mass of cards every time a 16MB area of memory is added or removed seems like an awfully big sledgehammer to me. I would *HATE* to see anybody else using this driver as an

[PATCH] drivers/base: export gpl (un)register_memory_notifier

2008-02-11 Thread Jan-Bernd Themann
Drivers like eHEA need memory notifiers in order to update their internal DMA memory map when memory is added to or removed from the system. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- Hi, this is the modified version with EXPORT_SYMBOL_GPL Regards, Jan-Bernd drivers/base

[PATCH] drivers/base: export gpl (un)register_memory_notifier

2008-02-11 Thread Jan-Bernd Themann
Drivers like eHEA need memory notifiers in order to update their internal DMA memory map when memory is added to or removed from the system. Patch for eHEA memory hotplug support that uses these functions: http://www.spinics.net/lists/netdev/msg54484.html Signed-off-by: Jan-Bernd Themann [EMAIL

Re: [PATCH][RESEND] drivers/base: export (un)register_memory_notifier

2008-02-11 Thread Jan-Bernd Themann
On Monday 11 February 2008 11:12, Dave Hansen wrote: On Mon, 2008-02-11 at 10:49 +0100, Jan-Bernd Themann wrote: are you the right person to address this patch to? You might want to check the top of the file. ;) --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -52,11 +52,13

[PATCH][RESEND] drivers/base: export (un)register_memory_notifier

2008-02-11 Thread Jan-Bernd Themann
Drivers like eHEA need memory notifiers in order to update their internal DMA memory map when memory is added to or removed from the system. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- Hi Greg, are you the right person to address this patch to? Regards, Jan-Bernd drivers/base

Re: LRO ip_summed

2008-02-05 Thread Jan-Bernd Themann
On Tuesday 05 February 2008 09:56, Kostya B wrote: 1. Let's assume a driver which has a HW csum capability, however it has a privilege to assign any value for ip_summed. For example the driver assigns unnecessary for most of the packets, however it fails to verify the packets less than 96

Re: LRO ip_summed

2008-02-04 Thread Jan-Bernd Themann
On Sunday 03 February 2008 10:48, Kostya B wrote: Hi, The mail is related to the way LRO manipulates the ip_summed value. Could anybody (author) explain why to overwrite the original value of skb-ip_summed, when it's processing by __lro_proc_skb ? E.g. in out: label Why not to

[PATCH 2/2] ehea: add memory remove hotplug support

2008-02-04 Thread Jan-Bernd Themann
Add memory remove hotplug support Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- Comment: This patch depends on the following patch that exports the symbols register_memory_notifier() unregister_memory_notifier() http://lkml.org/lkml/2008/2/1/293 drivers/net/ehea/ehea_main.c

Re: [PATCH 2/2] ehea: add memory remove hotplug support

2008-02-04 Thread Jan-Bernd Themann
On Monday 04 February 2008 15:46, Michael Ellerman wrote: On Mon, 2008-02-04 at 14:04 +0100, Jan-Bernd Themann wrote: Add memory remove hotplug support @@ -3559,6 +3578,10 @@ int __init ehea_module_init(void) if (ret) ehea_info(failed registering reboot notifier

[PATCH 1/2] ehea: kdump support

2008-02-04 Thread Jan-Bernd Themann
there. The arrays are kept up-to-date during normal runtime. The crash handler fn is triggered by the recently introduced PPC crash shutdown reg/unreg functions. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h | 34 +- drivers/net/ehea/ehea_main.c | 281

[PATCH 0/2] ehea: kdump memory remove support

2008-02-04 Thread Jan-Bernd Themann
This patch set adds support for kdump and hotplug memory remove to the eHEA driver. The memory remove patch depends on the following patch that has been posted a few days ago. That patch exports the symbols  - register_memory_notifier()  - unregister_memory_notifier()

[PATCH] ehea: fix sysfs link compile problem

2008-02-01 Thread Jan-Bernd Themann
Due to changes in the struct device_driver there is no direct access to its kobj any longer. The kobj was used to create sysfs links between eHEA ethernet devices and the driver. This patch removes the affected sysfs links to resolve the build problems. Signed-off-by: Jan-Bernd Themann [EMAIL

[PATCH] drivers/base: export (un)register_memory_notifier

2008-02-01 Thread Jan-Bernd Themann
Drivers like eHEA need memory notifiers in order to update their internal DMA memory map when memory is added to or removed from the system. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- Comment: eHEA patches that exploit these functions will follow drivers/base/memory.c |2

Re: cqe, cqe_skb: return when both or when either NULL?

2007-11-26 Thread Jan-Bernd Themann
Hi On Friday 23 November 2007 22:54, Roel Kluin wrote: In function ehea_poll() drivers/net/ehea/ehea_main.c:667, in a loop cqe and cqe_skb - both struct ehea_cqe pointers - are assigned: -- cqe = ehea_poll_rq1(pr-qp, wqe_index); cqe_skb = ehea_poll_cq(pr-send_cq); if (!cqe !cqe_skb)

[PATCH] ehea: add kexec support

2007-10-26 Thread Jan-Bernd Themann
eHEA resources that are allocated via H_CALLs have a unique identifier each. These identifiers are necessary to free the resources. A reboot notifier is used to free all eHEA resources before the indentifiers get lost, i.e before kexec starts a new kernel. Signed-off-by: Jan-Bernd Themann [EMAIL

[PATCH] ehea: use kernel event queue

2007-10-08 Thread Jan-Bernd Themann
eHEA recovery and DLPAR functions are called seldomly. The eHEA workqueues are replaced by the kernel event queue. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- The patch has been built against upstream git drivers/net/ehea/ehea.h |3 +-- drivers/net/ehea/ehea_main.c | 28

[PATCH] ehea: DLPAR memory add fix

2007-10-01 Thread Jan-Bernd Themann
Due to stability issues in high load situations the HW queue handling has to be changed. The HW queues are now stopped and restarted again instead of destroying and allocating new HW queues. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |4 +- drivers

Re: ehea work queues

2007-10-01 Thread Jan-Bernd Themann
Hi On Sunday 30 September 2007 18:20, Anton Blanchard wrote: Hi, I booted 2.6.23-rc8 and noticed that ehea loves its workqueues: (notice also that the ehea_driver_wq/XXX exceeds TASK_COMM_LEN). Since they are both infrequent events and not performance critical (memory hotplug and

Re: [PATCH] ehea: DLPAR memory add fix

2007-10-01 Thread Jan-Bernd Themann
Hi, On Monday 01 October 2007 16:44, Jeff Garzik wrote: Jan-Bernd Themann wrote: Due to stability issues in high load situations the HW queue handling has to be changed. The HW queues are now stopped and restarted again instead of destroying and allocating new HW queues. Signed-off

Re: [Bug, PATCH and another Bug] Was: Fix refcounting problem with netif_rx_reschedule()

2007-09-19 Thread Jan-Bernd Themann
Hi, On Wednesday 19 September 2007 13:54, Krishna Kumar wrote: CPU#1: ipoib_poll(budget=100) { A. process 100 skbs B. netif_rx_complete() Process unrelated interrupts; executes slower than steps C, D, E on CPU#2 F. ib_req_notify_cq() (no missed completions,

[PATCH net-2.6.24] eHEA: poll function update for new NAPI scheme

2007-09-19 Thread Jan-Bernd Themann
Update of ehea_poll function to work with new NAPI scheme. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- Hi David, this patch is built upon the patches provided by Mel Gorman (2.6.23-rc6-mm1: Build failure on ppc64 drivers/net/ehea/ehea_main.c) and Roland Dreier ([PATCH net-2.6.24

Re: new NAPI interface broken

2007-09-19 Thread Jan-Bernd Themann
On Wednesday 19 September 2007 17:33, Roland Dreier wrote: One other thing I observed is that I can not unload the module as the ref counter of the eth device is too low. I haven't tracked down the source of this problem yet. I suspect that this is because netif_rx_reschedule() was

Re: new NAPI interface broken

2007-09-18 Thread Jan-Bernd Themann
Hi, On Saturday 15 September 2007 00:12, David Miller wrote: Ok, for now I'm going to try and deal with this by reverting the list handling to something approximating the old NAPI code, as per the patch below. I've only quickly test booted into this kernel on my workstation. So take care

new NAPI interface broken

2007-09-07 Thread Jan-Bernd Themann
Hi Stephen, I saw that you developed most of the new NAPI interface. I already addressed this issue a while ago. Please correct me if I got it wrong. I think there is still a serious problem with the NAPI changes to make NAPI polling independent of struct net_device objects. Its about the

[PATCH 2/2][RESEND] ehea: fix last_rx update

2007-09-07 Thread Jan-Bernd Themann
Update last_rx in registered device struct instead of in the dummy device. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea_main.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c

Re: RFC: issues concerning the next NAPI interface

2007-08-29 Thread Jan-Bernd Themann
Hi David David Miller schrieb: Interrupt mitigation only works if it helps you avoid interrupts. This scheme potentially makes more of them happen. The hrtimer is just another interrupt, a cpu locally triggered one, but it has much of the same costs nonetheless. So if you set this timer, it

Re: RFC: issues concerning the next NAPI interface

2007-08-29 Thread Jan-Bernd Themann
On Wednesday 29 August 2007 10:29, David Miller wrote: From: Jan-Bernd Themann [EMAIL PROTECTED] Date: Wed, 29 Aug 2007 09:10:15 +0200 In the end I want to reduce the CPU utilization. And one way to do that is LRO which also works only well if there are more then just a very few packets

Re: RFC: issues concerning the next NAPI interface

2007-08-29 Thread Jan-Bernd Themann
On Wednesday 29 August 2007 10:15, James Chapman wrote: Jan-Bernd Themann wrote: What I'm trying to improve with this approach is interrupt mitigation for NICs where the hardware support for interrupt mitigation is limited. I'm not trying to improve this for NICs that work well

Re: RFC: issues concerning the next NAPI interface

2007-08-28 Thread Jan-Bernd Themann
On Monday 27 August 2007 22:37, David Miller wrote: From: Jan-Bernd Themann [EMAIL PROTECTED] Date: Mon, 27 Aug 2007 11:47:01 +0200 So the question is simply: Do we want drivers that need (benefit from) a timer based polling support to implement their own timers each, or should

Re: RFC: issues concerning the next NAPI interface

2007-08-28 Thread Jan-Bernd Themann
Hi On Monday 27 August 2007 23:02, David Miller wrote: But there are huger fish to fry for you I think. Talk to your platform maintainers and ask for an interface for obtaining a flat static distribution of interrupts to cpus in order to support multiqueue NAPI better. In your previous

Re: RFC: issues concerning the next NAPI interface

2007-08-28 Thread Jan-Bernd Themann
On Tuesday 28 August 2007 11:22, James Chapman wrote: So in this scheme what runs -poll() to process incoming packets? The hrtimer? No, the regular NAPI networking core calls -poll() as usual; no timers are involved. This scheme simply delays the napi_complete() from the driver so the

Re: RFC: issues concerning the next NAPI interface

2007-08-27 Thread Jan-Bernd Themann
On Monday 27 August 2007 03:58, David Miller wrote: From: James Chapman [EMAIL PROTECTED] Date: Sun, 26 Aug 2007 20:36:20 +0100 David Miller wrote: From: James Chapman [EMAIL PROTECTED] Date: Fri, 24 Aug 2007 18:16:45 +0100 Does hardware interrupt mitigation really interact well

[PATCH 1/2] ehea: propagate physical port state

2007-08-27 Thread Jan-Bernd Themann
the physical port state is. Thus eHEA can be considered as a switch there. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |5 - drivers/net/ehea/ehea_main.c | 14 +- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 2/2] ehea: fix last_rx update

2007-08-27 Thread Jan-Bernd Themann
Update last_rx in registered device struct instead of in the dummy device. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea_main.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c

Re: RFC: issues concerning the next NAPI interface

2007-08-27 Thread Jan-Bernd Themann
On Monday 27 August 2007 17:51, James Chapman wrote: In the second half of my previous reply (which seems to have been deleted), I suggest a way to avoid this problem without using hardware interrupt mitigation / coalescing. Original text is quoted below. I've seen the same and I'm

RFC: issues concerning the next NAPI interface

2007-08-24 Thread Jan-Bernd Themann
Hi, when I tried to get the eHEA driver working with the new interface, the following issues came up. 1) The current implementation of netif_rx_schedule, netif_rx_complete    and the net_rx_action have the following problem: netif_rx_schedule    sets the NAPI_STATE_SCHED flag and adds the NAPI

Re: RFC: issues concerning the next NAPI interface

2007-08-24 Thread Jan-Bernd Themann
Hi, On Friday 24 August 2007 17:37, [EMAIL PROTECTED] wrote: On Fri, Aug 24, 2007 at 03:59:16PM +0200, Jan-Bernd Themann wrote: ... 3) On modern systems the incoming packets are processed very fast. Especially    on SMP systems when we use multiple queues we process only a few

Re: RFC: issues concerning the next NAPI interface

2007-08-24 Thread Jan-Bernd Themann
James Chapman schrieb: Stephen Hemminger wrote: On Fri, 24 Aug 2007 17:47:15 +0200 Jan-Bernd Themann [EMAIL PROTECTED] wrote: Hi, On Friday 24 August 2007 17:37, [EMAIL PROTECTED] wrote: On Fri, Aug 24, 2007 at 03:59:16PM +0200, Jan-Bernd Themann wrote: ... 3) On modern systems

Re: RFC: issues concerning the next NAPI interface

2007-08-24 Thread Jan-Bernd Themann
Linas Vepstas schrieb: On Fri, Aug 24, 2007 at 09:04:56PM +0200, Bodo Eggert wrote: Linas Vepstas [EMAIL PROTECTED] wrote: On Fri, Aug 24, 2007 at 03:59:16PM +0200, Jan-Bernd Themann wrote: 3) On modern systems the incoming packets are processed very fast. Especially on SMP

Re: eHEA driver issues from net-2.6.24

2007-08-23 Thread Jan-Bernd Themann
On Thursday 23 August 2007 00:20, Andrew Theurer wrote: David Miller wrote: From: Andrew Theurer [EMAIL PROTECTED] Date: Wed, 22 Aug 2007 16:55:03 -0500 Thanks for finally getting to test this, I thought nobody would test this until it got merged into 2.6.24 :-/ Yes, sorry for the

Re: eHEA driver issues from net-2.6.24

2007-08-23 Thread Jan-Bernd Themann
Hi David, On Thursday 23 August 2007 10:17, David Miller wrote: From: Jan-Bernd Themann [EMAIL PROTECTED] Date: Thu, 23 Aug 2007 08:55:29 +0200 We'd like to keep the possibility to switch back to a single queue for now. Please do not do this, we already have way too much

New NAPI interface: netif_rx_reschedule not working

2007-08-23 Thread Jan-Bernd Themann
Hi David, when trying to get our driver working with the new interface, I found the following issue where I'm not sure how to solve it best: netif_rx_reschedule() does not work when called after netif_rx_complete(). The problem is that netif_rx_reschedule currently adds the napi struct once

[PATCH 1/4] ehea: fix interface to DLPAR tools

2007-08-22 Thread Jan-Bernd Themann
Userspace DLPAR tool expects decimal numbers to be written to and read from sysfs entries. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea_main.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/ehea/ehea_main.c b/drivers

[PATCH 2/4] ehea: fix module parameter description

2007-08-22 Thread Jan-Bernd Themann
Update the module parameter description of use_mcs to show correct default value Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea_main.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea

[PATCH 3/4] ehea: fix queue destructor

2007-08-22 Thread Jan-Bernd Themann
Includes hcp_epas_dtor in eq/cq/qp destructors to unmap HW register. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea_qmr.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/net/ehea/ehea_qmr.c b/drivers/net/ehea/ehea_qmr.c index

[PATCH 4/4] ehea: show physical port state

2007-08-22 Thread Jan-Bernd Themann
the physical port state is. Thus eHEA can be considered as a switch there. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |5 - drivers/net/ehea/ehea_main.c | 14 +- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/drivers

Re: [PATCH] [NET] ethtool: Add LRO support

2007-08-10 Thread Jan-Bernd Themann
Hi On Friday 10 August 2007 00:00, David Miller wrote: From: Auke Kok [EMAIL PROTECTED] Date: Thu, 09 Aug 2007 09:41:17 -0700 Signed-off-by: Auke Kok [EMAIL PROTECTED] I think this is definitely how we should handle LRO configuration instead of the ad-hoc module parameters current LRO

Re: [PATCH] lro: eHEA example how to use LRO

2007-08-06 Thread Jan-Bernd Themann
Hi Auke, On Friday 03 August 2007 22:29, Kok, Auke wrote: Jan-Bernd Themann wrote: This patch shows how the generic LRO interface is used for SKB mode Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/Kconfig |1 + drivers/net/ehea/ehea.h

Re: [PATCH 1/1] lro: Generic Large Receive Offload for TCP traffic

2007-08-06 Thread Jan-Bernd Themann
Hi Jörn On Friday 03 August 2007 15:41, Jörn Engel wrote: On Fri, 3 August 2007 14:41:19 +0200, Jan-Bernd Themann wrote: This patch provides generic Large Receive Offload (LRO) functionality for IPv4/TCP traffic. LRO combines received tcp packets to a single larger tcp packet

[PATCH] lro: eHEA example how to use LRO

2007-08-03 Thread Jan-Bernd Themann
This patch shows how the generic LRO interface is used for SKB mode Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/Kconfig |1 + drivers/net/ehea/ehea.h |9 - drivers/net/ehea/ehea_ethtool.c | 15 +++ drivers/net/ehea/ehea_main.c

[PATCH 1/1] lro: Generic Large Receive Offload for TCP traffic

2007-08-03 Thread Jan-Bernd Themann
either pass SKBs or fragment lists to the LRO engine. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- include/linux/inet_lro.h | 177 ++ net/ipv4/Kconfig |8 + net/ipv4/Makefile|1 + net/ipv4/inet_lro.c | 600

[PATCH 0/1] lro: Generic Large Receive Offload for TCP traffic

2007-08-03 Thread Jan-Bernd Themann
Hi, I think this patch could be the final version for now. It has been tested on two platforms (power and x86_64) and works very well. Apart from David Miller and Evgeniy Polaykov, we'd like to thank especially Andrew Gallatin for his great reviews and help to make that happen. After some

Re: [PATCH 0/4][RFC] lro: Generic Large Receive Offload for TCP traffic

2007-07-31 Thread Jan-Bernd Themann
Hi, Thanks for finding these bugs! I'll post an updated version soon (2 patches with no separate Kconfig patches, one LRO and one eHEA patch). See comments below. Thanks, Jan-Bernd On Monday 30 July 2007 22:32, Andrew Gallatin wrote: I was working on testing the myri10ge patch, and I ran into

[PATCH 1/4][RFC] lro: Generic Large Receive Offload for TCP traffic

2007-07-30 Thread Jan-Bernd Themann
Generic Large Receive Offload for TCP traffic Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- include/linux/inet_lro.h | 173 ++ net/ipv4/inet_lro.c | 590 ++ 2 files changed, 763 insertions(+), 0 deletions(-) create mode

[PATCH 2/4][RFC] lro: Kconfig and Makefile

2007-07-30 Thread Jan-Bernd Themann
Kconfig and Makefile for LRO Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- net/ipv4/Kconfig |8 net/ipv4/Makefile |1 + 2 files changed, 9 insertions(+), 0 deletions(-) diff --git a/net/ipv4/Kconfig b/net/ipv4/Kconfig index fb79097..d894f61 100644 --- a/net/ipv4

[PATCH 0/4][RFC] lro: Generic Large Receive Offload for TCP traffic

2007-07-30 Thread Jan-Bernd Themann
Hi, this patch set contains the latest generic LRO code, a Kconfig / Makefile and an eHEA patch demonstrating how the aggregate SKB interface has to to be used. Drew, could you provide a patch for the myri10ge driver to show how the receive in page interface works? Please check the Kconfig /

[PATCH 3/4][RFC] ehea: LRO support

2007-07-30 Thread Jan-Bernd Themann
Added LRO support using the SKB aggregate interface Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |9 - drivers/net/ehea/ehea_ethtool.c | 15 +++ drivers/net/ehea/ehea_main.c| 82 +++--- 3 files changed

[PATCH 4/4][RFC] ehea: Kconfig

2007-07-30 Thread Jan-Bernd Themann
Kconfig changes for LRO Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/Kconfig |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig index f8a602c..fec4004 100644 --- a/drivers/net/Kconfig +++ b/drivers/net/Kconfig

Re: [RFC 0/1] lro: Generic Large Receive Offload for TCP traffic

2007-07-27 Thread Jan-Bernd Themann
Hi Drew, thanks a lot for your good feedback. See comments below. I'll try to provide an updated version next week. It would be nice if you could post a patch for your driver once we have addressed the issues you mentioned. Then we would have the eHEA driver for the SKB interface, and your driver

[PATCH] eHEA: net_poll support

2007-07-23 Thread Jan-Bernd Themann
net_poll support for eHEA added Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |2 +- drivers/net/ehea/ehea_main.c | 22 +- 2 files changed, 22 insertions(+), 2 deletions(-) diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea

[RFC 0/1] lro: Generic Large Receive Offload for TCP traffic

2007-07-20 Thread Jan-Bernd Themann
Hi, Thanks a lot for your comments so far. This generic LRO patch differs from the last one in several points. A new interface for a receive in pages mode has been added and tested with an eHEA prototype. Seems to work well. Does this extended interface seem to be sufficient? Below some more

[RFC 1/1] lro: Generic Large Receive Offload for TCP traffic

2007-07-20 Thread Jan-Bernd Themann
Generic LRO patch Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- include/linux/inet_lro.h | 154 + net/ipv4/inet_lro.c | 549 ++ 2 files changed, 703 insertions(+), 0 deletions(-) create mode 100644 include/linux/inet_lro.h

Re: [RFC 0/3] lro: Generic Large Receive Offload for TCP traffic (IPv6)

2007-07-18 Thread Jan-Bernd Themann
Hi, I suggest we keep the interface open for IPv6 support by adding an additional parameter but first just get IPv4 support only into the kernel. IPv6 support can then incrementially be added. Would that be ok? On Sunday 15 July 2007 11:40, David Miller wrote: From: Christoph Hellwig [EMAIL

Re: [RFC 1/3] lro: Generic LRO for TCP traffic

2007-07-12 Thread Jan-Bernd Themann
Hi Evgeniy On Thursday 12 July 2007 10:01, Evgeniy Polyakov wrote: + + if (tcph-cwr || tcph-ece || tcph-urg || !tcph-ack || tcph-psh + || tcph-rst || tcph-syn || tcph-fin) + return -1; I think you do not want to break lro frame because of push flag - it is pretty

[RFC 0/3] lro: Generic Large Receive Offload for TCP traffic

2007-07-11 Thread Jan-Bernd Themann
Generic Large Receive Offload proposal After some discussions on the mailing list concerning our LRO approach, we agreed to provide a generic LRO patch. The algorithm is based on the version we developed for eHEA. The performance improvements we observed were significant. The LRO functionality

[RFC 3/3] lro: LRO support for eHEA

2007-07-11 Thread Jan-Bernd Themann
LRO support for eHEA Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |9 +++- drivers/net/ehea/ehea_main.c | 102 +++--- 2 files changed, 104 insertions(+), 7 deletions(-) diff --git a/drivers/net/ehea/ehea.h b/drivers

[RFC 2/3] lro: Kconfig and Makefile

2007-07-11 Thread Jan-Bernd Themann
Kconfig and Makefile changes for LRO Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- net/ipv4/Kconfig |9 + net/ipv4/Makefile |1 + 2 files changed, 10 insertions(+), 0 deletions(-) diff --git a/net/ipv4/Kconfig b/net/ipv4/Kconfig index 010fbb2..25279f4 100644 --- a/net

[PATCH 1/2] eHEA: Capability flag for DLPAR support

2007-07-05 Thread Jan-Bernd Themann
This patch introduces a capability flag that is used by the DLPAR userspace tool to check which DLPAR features are supported by the eHEA driver. Missing goto has been included. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |8 +++- drivers/net/ehea

[PATCH 2/2] eHEA: Receive SKB Aggregation

2007-07-05 Thread Jan-Bernd Themann
and improved throughput for small numbers of parallel TCP connections. As this feature is considered as experimental it is switched off by default and can be activated via a module parameter. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h | 30 drivers/net

[PATCH 1/2] eHEA: Capability flag for DLPAR support

2007-07-04 Thread Jan-Bernd Themann
This patch introduces a capability flag that is used by the DLPAR userspace tool to check which DLPAR features are supported by the eHEA driver. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |8 +++- drivers/net/ehea/ehea_main.c | 17

[PATCH 2/2] eHEA: Receive SKB Aggregation

2007-07-04 Thread Jan-Bernd Themann
and improved throughput for small numbers of parallel TCP connections. As this feature is considered as experimental it is switched off by default and can be activated via a module parameter. Some additional security checks have been added since the last posting. Signed-off-by: Jan-Bernd Themann [EMAIL

Re: [PATCH 1/2] eHEA: Capability flag for DLPAR support

2007-07-04 Thread Jan-Bernd Themann
Hi, good catch. Thanks On Wednesday 04 July 2007 14:53, Akinobu Mita wrote: 2007/7/4, Jan-Bernd Themann [EMAIL PROTECTED]: diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c index bdb5241..f8c0908 100644 --- a/drivers/net/ehea/ehea_main.c +++ b/drivers/net/ehea

[PATCH/RESENT] ehea: Whitespace cleanup

2007-07-02 Thread Jan-Bernd Themann
This patch fixes several whitespace issues. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index c0f81b5..abaf3ac 100644 --- a/drivers/net/ehea/ehea.h +++ b/drivers/net/ehea/ehea.h @@ -39,7 +39,7 @@ #include asm/io.h

RE: [PATCH] NET: Multiple queue hardware support

2007-06-20 Thread Jan-Bernd Themann
Hi, to me it seems that this patch set only include multiple transmit queue support (for qdisc). Am I right with this observation? If so, are there also plans to support multiple receive queues to allow the queues to be processed in parallel on different CPUs via a standard interface? Currently,

[PATCH] ehea: Whitespace cleanup

2007-06-14 Thread Jan-Bernd Themann
This patch fixes several whitespace issues. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index c0f81b5..abaf3ac 100644 --- a/drivers/net/ehea/ehea.h +++ b/drivers/net/ehea/ehea.h @@ -39,7 +39,7 @@ #include asm/io.h

[PATCH/RESEND] ehea: fix for dlpar and sysfs entries

2007-04-23 Thread Jan-Bernd Themann
-by: Jan-Bernd Themann [EMAIL PROTECTED] --- This patch applies on top of the netdev upstream branch for 2.6.22 diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index 1405d0b..173994d 100644 --- a/drivers/net/ehea/ehea.h +++ b/drivers/net/ehea/ehea.h @@ -39,7 +39,7 @@ #include asm

[PATCH] ehea: fix for dlpar and sysfs entries

2007-04-05 Thread Jan-Bernd Themann
-by: Jan-Bernd Themann [EMAIL PROTECTED] --- This patch applies on top of the netdev upstream branch for 2.6.22 diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index 1405d0b..173994d 100644 --- a/drivers/net/ehea/ehea.h +++ b/drivers/net/ehea/ehea.h @@ -39,7 +39,7 @@ #include asm

[PATCH] ehea: removing unused functionality

2007-03-23 Thread Jan-Bernd Themann
This patch includes: - removal of unused fields in structs - ethtool statistics cleanup - removes unsed functionality from send path Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- This patch applies on top of the netdev upstream branch for 2.6.22 drivers/net/ehea/ehea.h

[PATCH 1/2] ehea: fix for dynamic lpar support

2007-03-22 Thread Jan-Bernd Themann
The patch fixes bugs related to the probe / remove adapter functionality (handling of OFDT nodes) Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- This patch applies on top of the netdev upstream branch for 2.6.22 diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index

[PATCH 2/2] ehea: code cleanup

2007-03-22 Thread Jan-Bernd Themann
This patch includes: - code cleanup related to resource management - extended error data gathering for resource management - removing trailing whitespaces Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- This patch applies on top of the netdev upstream branch for 2.6.22 drivers/net

Re: [PATCH 2/2] ehea: NAPI multi queue TX/RX path for SMP

2007-02-28 Thread Jan-Bernd Themann
Hi, +static inline int ehea_hash_skb(struct sk_buff *skb, int num_qps) +{ + u32 tmp; + if ((skb-nh.iph-protocol == IPPROTO_TCP) +skb-protocol == ETH_P_IP) { skb-protocol has network byte order. The ETH_P_IP test should also logically come before checking the IP

[PATCH 0/2] ehea: dynamic port SMP support

2007-02-28 Thread Jan-Bernd Themann
.html) please apply. Jan-Bernd Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] - 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

[PATCH 1/2] ehea: dynamic add / remove port

2007-02-28 Thread Jan-Bernd Themann
This patch introduces functionality to dynamically add / remove ehea ports via an userspace DLPAR tool. It creates a subnode for each logical port in the sysfs. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index 42295d6

[PATCH 2/2] ehea: NAPI multi queue TX/RX path for SMP

2007-02-28 Thread Jan-Bernd Themann
This patch provides a functionality that allows parallel RX processing on multiple RX queues by using dummy netdevices. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff -Nurp -X dontdiff linux-2.6.21-rc1/drivers/net/ehea/ehea.h patched_kernel/drivers/net/ehea/ehea.h --- linux

[PATCH 0/2] ehea: dynamic port SMP support

2007-02-27 Thread Jan-Bernd Themann
) As soon as discussions about splitting NAPI from netdevice have settled and this functionality is in kernel, we'll provide a patch for the new interface. (see: http://www.spinics.net/lists/netdev/msg25647.html) please apply. Jan-Bernd Signed-off-by: Jan-Bernd Themann [EMAIL

[PATCH 1/2] ehea: dynamic add / remove port

2007-02-27 Thread Jan-Bernd Themann
This patch introduces functionality to dynamically add / remove ehea ports via an userspace DLPAR tool. It creates a subnode for each logical port in the sysfs. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index 42295d6

[PATCH 2/2] ehea: NAPI multi queue TX/RX path for SMP

2007-02-27 Thread Jan-Bernd Themann
This patch provides a functionality that allows parallel RX processing on multiple RX queues by using dummy netdevices. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff -Nurp -X dontdiff linux-2.6.21-rc1/drivers/net/ehea/ehea.h patched_kernel/drivers/net/ehea/ehea.h --- linux

Re: [PATCH] ehea: Optional TX/RX path optimized for SMP

2007-02-26 Thread Jan-Bernd Themann
Hi Also, as far as the approach of using tasklets, I think it would be better to use the fake netdev approach to continue to use NAPI. Basically you create a pseudo-netdev for each receive queue and have NAPI handle the polling for you -- you could look for drivers/net/cxgb3 for an example

[PATCH] ehea: NAPI multi queue TX/RX path for SMP

2007-02-26 Thread Jan-Bernd Themann
. Regards, Jan-Bernd Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff -Nurp -X dontdiff linux-2.6.21-rc1/drivers/net/ehea/ehea.h patched_kernel/drivers/net/ehea/ehea.h --- linux-2.6.21-rc1/drivers/net/ehea/ehea.h2007-02-26 16:17:14.0 +0100 +++ patched_kernel/drivers/net/ehea

[PATCH] ehea: Optional TX/RX path optimized for SMP

2007-02-23 Thread Jan-Bernd Themann
performance improvements with ehea. Are there any concerns about this approach? Regards, Jan-Bernd Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff -Nurp -X dontdiff linux-2.6.21-rc1/drivers/net/ehea/ehea.h patched_kernel/drivers/net/ehea/ehea.h --- linux-2.6.21-rc1/drivers/net/ehea

[PATCH] ehea: dynamic add / remove port

2007-02-21 Thread Jan-Bernd Themann
(devspec) Logical ports can be added / removed by writing the logical port id into the probe_port / remove_port file located in the lhea adapter directory. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h index 42295d6..e595d6b

[PATCH] ehea: dynamic add / remove port

2007-02-20 Thread Jan-Bernd Themann
be added / removed by writing the logical port id into the probe_port / remove_port file. Would that interface be sufficient for the userspace application? Regards, Jan-Bernd Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h

Re: [PATCH] ehea: dynamic add / remove port

2007-02-20 Thread Jan-Bernd Themann
Hi, This looks great! Two questions. I would assume that the of_device results in a devspec file in the sysfs directory for the port? Also, yes, that is true. There is a devspec file containing the ofdt path in the port directory. is there a way to correlate a logical port to an ethernet

Re: [PATCH 2.6.21-rc1] ehea: dynamic add / remove port

2007-02-16 Thread Jan-Bernd Themann
Hi, I agree with most points. Here the new design proposal: On Wednesday 14 February 2007 23:25, John Rose wrote: Hi- A few high level comments, then some really insignificant ones. First, is there a reason why we shouldn't have a sysfs entry/kobject for each logical port? How is it

[PATCH 2.6.21-rc1] ehea: dynamic add / remove port

2007-02-14 Thread Jan-Bernd Themann
This patch enables dynamic adding / removing of ehea ports by DLPAR tool. Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff -Nurp -X dontdiff linux-2.6.20/drivers/net/ehea/ehea.h patched_kernel/drivers/net/ehea/ehea.h --- linux-2.6.20/drivers/net/ehea/ehea.h2007-02-12 14:44

[PATCH 2.6.21-rc1] ehea: Fixed error recovery

2007-02-09 Thread Jan-Bernd Themann
Error recovery for QP errors: Reset QPs and dump error information Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |2 +- drivers/net/ehea/ehea_main.c |8 +++- drivers/net/ehea/ehea_phyp.c | 10 ++ drivers/net/ehea/ehea_phyp.h |3

[PATCH 2.6.21-rc1] ehea: Fixed error recovery

2007-02-05 Thread Jan-Bernd Themann
Error recovery for QP errors: Reset QPs and dump error information Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- drivers/net/ehea/ehea.h |2 +- drivers/net/ehea/ehea_main.c |8 +++- drivers/net/ehea/ehea_phyp.c | 10 ++ drivers/net/ehea/ehea_phyp.h |3

Re: [PATCH 2.6.19-rc3 2/2] ehea: 64K page support fix

2006-10-26 Thread Jan-Bernd Themann
Hi, that is right, I'll send a new patch Thanks, Jan-Bernd On Wednesday 25 October 2006 18:21, Anton Blanchard wrote: Hi, +#ifdef CONFIG_PPC_64K_PAGES + /* To support 64k pages we must round to 64k page boundary */ + epas-kernel.addr = + ioremap((paddr_kernel

[PATCH 2.6.19-rc3 1/2] ehea: kzalloc GFP_ATOMIC fix

2006-10-25 Thread Jan-Bernd Themann
This patch fixes kzalloc parameters (GFP_ATOMIC instead of GFP_KERNEL) Signed-off-by: Jan-Bernd Themann [EMAIL PROTECTED] --- diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c index eb7d44d..4538c99 100644 --- a/drivers/net/ehea/ehea_main.c +++ b/drivers/net/ehea

  1   2   >