RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-07 Thread Eric Dumazet
pc-...@ozlabs.org; > grant.lik...@secretlab.ca; > > jwbo...@linux.vnet.ibm.com; john.willi...@petalogix.com; > michal.si...@petalogix.com; jty...@cs.ucr.edu > > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > > > From: Eric Dumazet > > Da

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-07 Thread John Linn
i...@petalogix.com; michal.si...@petalogix.com; jty...@cs.ucr.edu > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > From: Eric Dumazet > Date: Mon, 05 Apr 2010 23:29:53 +0200 > > > So, I ask, cant you use netdev_alloc_skb_ip_align() in this drive

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread David Miller
From: Eric Dumazet Date: Mon, 05 Apr 2010 23:29:53 +0200 > So, I ask, cant you use netdev_alloc_skb_ip_align() in this driver ? Thanks to everyone for getting this patch into shape. I applied version 4 of the patch to net-next-2.6, thanks! ___ Linuxpp

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread John Linn
alogix.com; michal.si...@petalogix.com; John Tyner > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > Le mardi 06 avril 2010 à 12:53 -0600, Grant Likely a écrit : > > > Hold on BUFFER_ALIGN is being used to align the DMA buffer on a > > cache line boun

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Eric Dumazet
Le mardi 06 avril 2010 à 12:53 -0600, Grant Likely a écrit : > Hold on BUFFER_ALIGN is being used to align the DMA buffer on a > cache line boundary. I don't think netdev_alloc_skb() makes any > guarantees about how the start of the IP header lines up against cache > line boundaries. The amo

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread John Linn
i...@petalogix.com; michal.si...@petalogix.com; John Tyner > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > On Tue, 2010-04-06 at 12:53 -0600, Grant Likely wrote: > > > Hold on BUFFER_ALIGN is being used to align the DMA buffer on a >

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread John Linn
ibm.com; > john.willi...@petalogix.com; michal.si...@petalogix.com; John Tyner > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > On Tue, Apr 6, 2010 at 11:11 AM, John Linn wrote: > >> -Original Message- > >> From: Eric Dumazet [mailto:eric.duma.

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Steven J. Magnani
On Tue, 2010-04-06 at 12:53 -0600, Grant Likely wrote: > Hold on BUFFER_ALIGN is being used to align the DMA buffer on a > cache line boundary. I don't think netdev_alloc_skb() makes any > guarantees about how the start of the IP header lines up against cache > line boundaries. The amount of

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Grant Likely
; grant.lik...@secretlab.ca; >> jwbo...@linux.vnet.ibm.com; john.willi...@petalogix.com; >> michal.si...@petalogix.com; John Tyner >> Subject: RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver >> >> Le mardi 06 avril 2010 à 10:12 -0600, John Linn a écrit : >> &g

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Grant Likely
On Mon, Apr 5, 2010 at 3:33 PM, John Linn wrote: > From: Eric Dumazet [mailto:eric.duma...@gmail.com] >> > +/* Align the IP data in the packet on word boundaries as MicroBlaze >> > + * needs it. >> > + */ >> > + >> >  #define XTE_ALIGN       32 >> > -#define BUFFER_ALIGN(adr) ((XTE_ALIGN - ((u32)

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Eric Dumazet
> > Yes I see how it's used, but it only allows you to reserve 2 bytes in the skb > with no options. Really ? This would be a bug ! __alloc_skb() uses kmalloc(), this gives you the guarantee you want, or maybe comment you wrote is not what is _really_ necessary ? /* Align the IP data in

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread John Linn
gix.com; > michal.si...@petalogix.com; John Tyner > Subject: RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > Le mardi 06 avril 2010 à 10:12 -0600, John Linn a écrit : > > > -Original Message- > > > From: Eric Dumazet [mailto:eric.duma...@gm

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Eric Dumazet
> > grant.lik...@secretlab.ca; > > jwbo...@linux.vnet.ibm.com; john.willi...@petalogix.com; > > michal.si...@petalogix.com; John Tyner > > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > > > Le lundi 05 avril 2010 à 15:11 -0600, John L

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread John Linn
gix.com; > michal.si...@petalogix.com; John Tyner > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > Le lundi 05 avril 2010 à 15:11 -0600, John Linn a écrit : > > This patch adds support for using the LL TEMAC Ethernet driver on > > non-Virtex 5 platf

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-06 Thread Steven J. Magnani
On Mon, 2010-04-05 at 15:33 -0600, John Linn wrote: > > > +/* Align the IP data in the packet on word boundaries as MicroBlaze > > > + * needs it. > > > + */ > > > + > > > #define XTE_ALIGN 32 > > > -#define BUFFER_ALIGN(adr) ((XTE_ALIGN - ((u32) adr)) % XTE_ALIGN) > > > +#define BUFFER_ALIG

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-05 Thread Grant Likely
On Mon, Apr 5, 2010 at 3:11 PM, John Linn wrote: > This patch adds support for using the LL TEMAC Ethernet driver on > non-Virtex 5 platforms by adding support for accessing the Soft DMA > registers as if they were memory mapped instead of solely through the > DCR's (available on the Virtex 5). >

RE: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-05 Thread John Linn
gix.com; > michal.si...@petalogix.com; John Tyner > Subject: Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver > > Le lundi 05 avril 2010 à 15:11 -0600, John Linn a écrit : > > This patch adds support for using the LL TEMAC Ethernet driver on > > non-Virtex 5 platf

Re: [PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-05 Thread Eric Dumazet
Le lundi 05 avril 2010 à 15:11 -0600, John Linn a écrit : > This patch adds support for using the LL TEMAC Ethernet driver on > non-Virtex 5 platforms by adding support for accessing the Soft DMA > registers as if they were memory mapped instead of solely through the > DCR's (available on the Virte

[PATCH] [V3] Add non-Virtex5 support for LL TEMAC driver

2010-04-05 Thread John Linn
This patch adds support for using the LL TEMAC Ethernet driver on non-Virtex 5 platforms by adding support for accessing the Soft DMA registers as if they were memory mapped instead of solely through the DCR's (available on the Virtex 5). The patch also updates the driver so that it runs on the Mi