Jan-Bernd Themann wrote:
This patch enables the receive side processing to aggregate TCP packets within
the HEA device driver. It analyses the packets already received after an
interrupt arrived and forwards these as chains of SKBs for the same TCP
connection with modified header field. We have s
On Thu, Jul 05, 2007 at 04:24:46PM +0200, Jan-Bernd Themann ([EMAIL PROTECTED])
wrote:
> > I've couple of comments on the driver, but mainly the fact of decreased
> > CPU usage itself - what was the magnitude of the win with this driver,
> > it looks like because of per-packet receive code path in
Hi,
thanks for your comment.
Jan-Bernd
On Thursday 05 July 2007 10:20, you wrote:
> Hi Jan-Bernd.
>
> [ Dropped spambot/i.e. unrelated mail lists ]
>
> On Thu, Jul 05, 2007 at 09:26:30AM +0200, Jan-Bernd Themann ([EMAIL
> PROTECTED]) wrote:
> > This patch enables the receive side processing
Christoph Hellwig wrote on 31.05.2007 15:41:18:
> I'm still very unhappy with having all this in various drivers. There's
> a lot of code that can be turned into generic library functions, and even
> more code that could be made generic with some amount of refactoring.
Yes, we'd also prefer to
Stephen Hemminger wrote on 31.05.2007 18:37:03:
>
> >
> >
> > +static int try_get_ip_tcp_hdr(struct ehea_cqe *cqe, struct sk_buff
*skb,
> > + struct iphdr **iph, struct tcphdr **tcph)
> > +{
> > + int ip_len;
> > +
> > + /* non tcp/udp packets */
> > + if (!cqe->header_lengt
>
>
> +static int try_get_ip_tcp_hdr(struct ehea_cqe *cqe, struct sk_buff *skb,
> + struct iphdr **iph, struct tcphdr **tcph)
> +{
> + int ip_len;
> +
> + /* non tcp/udp packets */
> + if (!cqe->header_length)
> + return -1;
> +
> + /* non t
I'm still very unhappy with having all this in various drivers. There's
a lot of code that can be turned into generic library functions, and even
more code that could be made generic with some amount of refactoring.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the b
7 matches
Mail list logo
