RE: [PATCH 1/2] NET: Multiple queue network device support

2007-03-12 Thread Waskiewicz Jr, Peter P
randeburg, Jesse; Kok, Auke; Ronciak, John > Subject: Re: [PATCH 1/2] NET: Multiple queue network device support > > On 09-03-2007 14:40, Thomas Graf wrote: > > * Kok, Auke <[EMAIL PROTECTED]> 2007-02-08 16:09 > >> diff --git a/net/core/dev.c b/net/core/dev.c index

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-03-12 Thread Jarek Poplawski
On 09-03-2007 14:40, Thomas Graf wrote: > * Kok, Auke <[EMAIL PROTECTED]> 2007-02-08 16:09 >> diff --git a/net/core/dev.c b/net/core/dev.c >> index 455d589..42b635c 100644 >> --- a/net/core/dev.c >> +++ b/net/core/dev.c >> @@ -1477,6 +1477,49 @@ gso: >> skb->tc_verd = SET_TC_AT(skb->tc_verd,AT

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-03-10 Thread Waskiewicz Jr, Peter P
, Auke; Ronciak, John > Subject: Re: [PATCH 1/2] NET: Multiple queue network device support > > * Waskiewicz Jr, Peter P <[EMAIL PROTECTED]> > 2007-03-09 15:27 > > That's the entire point of this extra locking. enqueue() > is going to > > put an skb int

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-03-09 Thread Thomas Graf
* Waskiewicz Jr, Peter P <[EMAIL PROTECTED]> 2007-03-09 15:27 > That's the entire point of this extra locking. enqueue() is going to > put an skb into a band somewhere that maps to some queue, and there is > no way to guarantee the skb I retrieve from dequeue() is headed for the > same queue. The

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-03-09 Thread Waskiewicz Jr, Peter P
> * Waskiewicz Jr, Peter P <[EMAIL PROTECTED]> > 2007-03-09 11:25 > > > > + } > > > > + } else { > > > > + /* We're not a multi-queue device. */ > > > > + spin_lock(&dev->queue_lock); > > > > + q

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-03-09 Thread Thomas Graf
* Waskiewicz Jr, Peter P <[EMAIL PROTECTED]> 2007-03-09 11:25 > > > + } > > > + } else { > > > + /* We're not a multi-queue device. */ > > > + spin_lock(&dev->queue_lock); > > > + q = dev->qdisc; > > > + if (q->

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-03-09 Thread Waskiewicz Jr, Peter P
, Auke; Ronciak, John > Subject: Re: [PATCH 1/2] NET: Multiple queue network device support > > * Kok, Auke <[EMAIL PROTECTED]> 2007-02-08 16:09 > > diff --git a/net/core/dev.c b/net/core/dev.c index 455d589..42b635c > > 100644 > > --- a/net/core/dev.c &

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-03-09 Thread Thomas Graf
* Kok, Auke <[EMAIL PROTECTED]> 2007-02-08 16:09 > diff --git a/net/core/dev.c b/net/core/dev.c > index 455d589..42b635c 100644 > --- a/net/core/dev.c > +++ b/net/core/dev.c > @@ -1477,6 +1477,49 @@ gso: > skb->tc_verd = SET_TC_AT(skb->tc_verd,AT_EGRESS); > #endif > if (q->enqueue) { >

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-03-08 Thread Jarek Poplawski
On 07-03-2007 23:42, David Miller wrote: > I didn't say to use skb->priority, I said to shrink skb->priority down > to a u16 and then make another u16 which will store your queue mapping > value. Peter is right: this is fully used by schedulers (prio, CBQ, HTB, HFSC...) and would break users' scri

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-03-07 Thread David Miller
I didn't say to use skb->priority, I said to shrink skb->priority down to a u16 and then make another u16 which will store your queue mapping value. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-03-07 Thread Waskiewicz Jr, Peter P
IL PROTECTED] > Sent: Monday, February 26, 2007 5:03 PM > To: Kok, Auke-jan H > Cc: [EMAIL PROTECTED]; netdev@vger.kernel.org; > linux-kernel@vger.kernel.org; Waskiewicz Jr, Peter P; > Brandeburg, Jesse; [EMAIL PROTECTED]; Ronciak, John > Subject: Re: [PATCH 1/2] NET: Multiple

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-02-27 Thread Waskiewicz Jr, Peter P
Dave, Thanks for the feedback. Please see below for my comments / questions. PJ Waskiewicz Intel Corporation > > From: Peter Waskiewicz Jr <[EMAIL PROTECTED]> > > > > Added an API and associated supporting routines for > multiqueue network > > devices. This allows network devices sup

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-02-26 Thread David Miller
From: "Kok, Auke" <[EMAIL PROTECTED]> Date: Thu, 08 Feb 2007 16:09:50 -0800 > From: Peter Waskiewicz Jr <[EMAIL PROTECTED]> > > Added an API and associated supporting routines for multiqueue network > devices. This allows network devices supporting multiple TX queues to > configure each queue wi

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-02-23 Thread Kok, Auke
Stephen Hemminger wrote: On Fri, 23 Feb 2007 04:00:55 -0500 "Sreenivasa Honnur" <[EMAIL PROTECTED]> wrote: Fucntion "map_queue" returns queue index as '0'. There is no support to return different queue indexes. -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Beh

Re: [PATCH 1/2] NET: Multiple queue network device support

2007-02-23 Thread Stephen Hemminger
On Fri, 23 Feb 2007 04:00:55 -0500 "Sreenivasa Honnur" <[EMAIL PROTECTED]> wrote: > Fucntion "map_queue" returns queue index as '0'. There is no support to > return different queue indexes. > > -Original Message- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > On Behalf Of Kok, Auke

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-02-23 Thread Waskiewicz Jr, Peter P
g > Cc: Waskiewicz Jr, Peter P; Brandeburg, Jesse; Kok, Auke; > Ronciak, John > Subject: RE: [PATCH 1/2] NET: Multiple queue network device support > > Fucntion "map_queue" returns queue index as '0'. There is no > support to return different queue indexes.

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-02-23 Thread Sreenivasa Honnur
Fucntion "map_queue" returns queue index as '0'. There is no support to return different queue indexes. -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Kok, Auke Sent: Friday, February 09, 2007 5:40 AM To: David Miller; Garzik, Jeff; netdev@vger.kernel.org

RE: [PATCH 1/2] NET: Multiple queue network device support

2007-02-23 Thread Sreenivasa Honnur
Function "map_queue" returns queue_index as 0 always. There is no support to return different queue numbers. -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Kok, Auke Sent: Friday, February 09, 2007 5:40 AM To: David Miller; Garzik, Jeff; netdev@vger.kerne