* Re: [PATCH net-next-26 5/5] cxgb4vf: Mark "UDP [RSS Hash] Enable" as a 1-bit field.
From: David Miller @ 2010-11-15 17:20 UTC (permalink / raw)
To: leedom; +Cc: netdev
In-Reply-To: <1289503844-18059-6-git-send-email-leedom@chelsio.com>
From: Casey Leedom <leedom@chelsio.com>
Date: Thu, 11 Nov 2010 11:30:44 -0800
> + uint synmapen:1; /* SYN Map Enable */
Please do not use the "uint" shorthand for "unsigned int", I know it's
in linux/types.h but that is there for sysv compatibility in
userspace and is ugly as hell.
^ permalink raw reply
* Re: [PATCH 1/3] offloading: Make scatter/gather more tolerant of vlans.
From: David Miller @ 2010-11-15 17:23 UTC (permalink / raw)
To: jesse; +Cc: netdev, bhutchings
In-Reply-To: <1288390495-28923-1-git-send-email-jesse@nicira.com>
From: Jesse Gross <jesse@nicira.com>
Date: Fri, 29 Oct 2010 15:14:53 -0700
> When checking if it is necessary to linearize a packet, we currently
> use vlan_features if the packet contains either an in-band or out-
> of-band vlan tag. However, in-band tags aren't special in any way
> for scatter/gather since they are part of the packet buffer and are
> simply more data to DMA. Therefore, only use vlan_features for out-
> of-band tags, which could potentially have some interaction with
> scatter/gather.
>
> Signed-off-by: Jesse Gross <jesse@nicira.com>
Applied.
^ permalink raw reply
* Re: [PATCH 2/3] offloading: Support multiple vlan tags in GSO.
From: David Miller @ 2010-11-15 17:23 UTC (permalink / raw)
To: jesse; +Cc: netdev, bhutchings
In-Reply-To: <1288390495-28923-2-git-send-email-jesse@nicira.com>
From: Jesse Gross <jesse@nicira.com>
Date: Fri, 29 Oct 2010 15:14:54 -0700
> We assume that hardware TSO can't support multiple levels of vlan tags
> but we allow it to be done. Therefore, enable GSO to parse these tags
> so we can fallback to software.
>
> Signed-off-by: Jesse Gross <jesse@nicira.com>
Applied.
^ permalink raw reply
* Re: [PATCH 3/3] offloading: Force software GSO for multiple vlan tags.
From: David Miller @ 2010-11-15 17:23 UTC (permalink / raw)
To: jesse; +Cc: netdev, bhutchings
In-Reply-To: <1288390495-28923-3-git-send-email-jesse@nicira.com>
From: Jesse Gross <jesse@nicira.com>
Date: Fri, 29 Oct 2010 15:14:55 -0700
> We currently use vlan_features to check for TSO support if there is
> a vlan tag. However, it's quite likely that the NIC is not able to
> do TSO when there is an arbitrary number of tags. Therefore if there
> is more than one tag (in-band or out-of-band), fall back to software
> emulation.
>
> Signed-off-by: Jesse Gross <jesse@nicira.com>
Applied.
^ permalink raw reply
* hello
From: amandasaleeby1 @ 2010-11-15 17:33 UTC (permalink / raw)
To: amandasaleeby1
hello with love and trust
my dearest friend compliment of the day to you.my name is Amanda favour a
girl of 23years old honest trusting kind caring loving and fun with
compassionate and generous nature viewing your email address which really interest
me to know more about you that is having communication friendship with you
if you will have the desire with me.so that we can get to know each other
better sharing feelings and thought together .i will like you to write me
through my email for easiest communication.i will be waiting to read from
you.I will be waiting for your mail;please make sure you replied to me with the email address below,it is very confidential and private,thanks and have a nice day.
here is my email address
Amandasaleeby@she.com
^ permalink raw reply
* Re: [PATCH net-next-26 5/5] cxgb4vf: Mark "UDP [RSS Hash] Enable" as a 1-bit field.
From: Casey Leedom @ 2010-11-15 17:43 UTC (permalink / raw)
To: David Miller; +Cc: netdev
In-Reply-To: <20101115.092059.59691885.davem@davemloft.net>
| From: David Miller <davem@davemloft.net>
| Date: Monday, November 15, 2010 09:20 am
|
| From: Casey Leedom <leedom@chelsio.com>
| Date: Thu, 11 Nov 2010 11:30:44 -0800
|
| > + uint synmapen:1; /* SYN Map Enable */
|
| Please do not use the "uint" shorthand for "unsigned int", I know it's
| in linux/types.h but that is there for sysv compatibility in
| userspace and is ugly as hell.
Okay, sorry about that. I wasn't sure so I looked around and saw several
other network device drivers using "uint" and figured it was okay. I assume
that it's okay to use u32, etc? I was just trying to be neet and solve the
"greater than 80 column" restriction which the patch check script complains
about for when a previous changeset converted "int" to "unsigned int". Just let
me know what the correct action is and I'll respin this one patch. Thanks for
your patience with my ignorance!
Casey
^ permalink raw reply
* [PATCH] hso: Fix unused variable warning
From: Alan Cox @ 2010-11-15 17:30 UTC (permalink / raw)
To: netdev
From: Alan Cox <alan@linux.intel.com>
Fallout from the TIOCGICOUNT work
Signed-off-by: Alan Cox <alan@linux.intel.com>
---
drivers/net/usb/hso.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c
index b154a94..be8cc2a 100644
--- a/drivers/net/usb/hso.c
+++ b/drivers/net/usb/hso.c
@@ -1745,7 +1745,6 @@ static int hso_serial_ioctl(struct tty_struct *tty, struct file *file,
unsigned int cmd, unsigned long arg)
{
struct hso_serial *serial = get_serial_by_tty(tty);
- void __user *uarg = (void __user *)arg;
int ret = 0;
D4("IOCTL cmd: %d, arg: %ld", cmd, arg);
^ permalink raw reply related
* [net-next-2.6 PATCH] enic: Fix build warnings
From: Vasanthy Kolluri @ 2010-11-15 18:09 UTC (permalink / raw)
To: davem; +Cc: netdev, roprabhu, dwang2
From: Vasanthy Kolluri <vkolluri@cisco.com>
Fix data type of argument passed to pci_alloc_consistent and pci_free_consistent routines.
Signed-off-by: Vasanthy Kolluri <vkolluri@cisco.com>
Signed-off-by: Roopa Prabhu <roprabhu@cisco.com>
Signed-off-by: David Wang <dwang2@cisco.com>
---
drivers/net/enic/enic.h | 2 +-
drivers/net/enic/enic_main.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/enic/enic.h b/drivers/net/enic/enic.h
index c91d364..7067254 100644
--- a/drivers/net/enic/enic.h
+++ b/drivers/net/enic/enic.h
@@ -32,7 +32,7 @@
#define DRV_NAME "enic"
#define DRV_DESCRIPTION "Cisco VIC Ethernet NIC Driver"
-#define DRV_VERSION "1.4.1.6"
+#define DRV_VERSION "1.4.1.7"
#define DRV_COPYRIGHT "Copyright 2008-2010 Cisco Systems, Inc"
#define ENIC_BARS_MAX 6
diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c
index a466ef9..9f293fa 100644
--- a/drivers/net/enic/enic_main.c
+++ b/drivers/net/enic/enic_main.c
@@ -2042,7 +2042,7 @@ static int enic_dev_hang_reset(struct enic *enic)
static int enic_set_rsskey(struct enic *enic)
{
- u64 rss_key_buf_pa;
+ dma_addr_t rss_key_buf_pa;
union vnic_rss_key *rss_key_buf_va = NULL;
union vnic_rss_key rss_key = {
.key[0].b = {85, 67, 83, 97, 119, 101, 115, 111, 109, 101},
@@ -2073,7 +2073,7 @@ static int enic_set_rsskey(struct enic *enic)
static int enic_set_rsscpu(struct enic *enic, u8 rss_hash_bits)
{
- u64 rss_cpu_buf_pa;
+ dma_addr_t rss_cpu_buf_pa;
union vnic_rss_cpu *rss_cpu_buf_va = NULL;
unsigned int i;
int err;
^ permalink raw reply related
* award
From: benedita bahki @ 2010-11-15 18:03 UTC (permalink / raw)
[-- Attachment #1: Type: text/plain, Size: 0 bytes --]
[-- Attachment #2: file for your claim.doc --]
[-- Type: application/msword, Size: 24576 bytes --]
^ permalink raw reply
* Re: [net-next-2.6 PATCH 1/3] 8021q: set hard_header_len when VLAN offload features are toggled
From: David Miller @ 2010-11-15 18:11 UTC (permalink / raw)
To: john.r.fastabend; +Cc: netdev, jesse
In-Reply-To: <20101031002232.8691.41201.stgit@jf-dev1-dcblab>
From: John Fastabend <john.r.fastabend@intel.com>
Date: Sat, 30 Oct 2010 17:22:32 -0700
> Toggling the vlan tx|rx hw offloads needs to set the hard_header_len
> as well otherwise we end up using LL_RESERVED_SPACE incorrectly.
> This results in pskb_expand_head() being used unnecessarily.
>
> Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
Applied.
^ permalink raw reply
* Re: [net-next-2.6 PATCH 2/3] net: remove check for headroom in vlan_dev_create
From: David Miller @ 2010-11-15 18:12 UTC (permalink / raw)
To: john.r.fastabend; +Cc: netdev, jesse
In-Reply-To: <20101031002237.8691.6266.stgit@jf-dev1-dcblab>
From: John Fastabend <john.r.fastabend@intel.com>
Date: Sat, 30 Oct 2010 17:22:37 -0700
> It is possible for the headroom to be smaller then the
> hard_header_len for a short period of time after toggling
> the vlan offload setting.
>
> This is not a hard error and skb_cow_head is called in
> __vlan_put_tag() to resolve this.
>
> Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
Applied.
^ permalink raw reply
* Re: [net-next-2.6 PATCH 3/3] net: consolidate 8021q tagging
From: David Miller @ 2010-11-15 18:12 UTC (permalink / raw)
To: john.r.fastabend; +Cc: netdev, jesse
In-Reply-To: <20101031002242.8691.38060.stgit@jf-dev1-dcblab>
From: John Fastabend <john.r.fastabend@intel.com>
Date: Sat, 30 Oct 2010 17:22:42 -0700
> Now that VLAN packets are tagged in dev_hard_start_xmit()
> at the bottom of the stack we no longer need to tag them
> in the 8021Q module (Except in the !VLAN_FLAG_REORDER_HDR
> case).
>
> This allows the accel path and non accel paths to be consolidated.
> Here the vlan_tci in the skb is always set and we allow the
> stack to add the actual tag in dev_hard_start_xmit().
>
> Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
Applied.
^ permalink raw reply
* vmxnet3 multiqueue
From: David Miller @ 2010-11-15 18:16 UTC (permalink / raw)
To: bhatewara; +Cc: netdev
Two things:
1) Do not quote the entire patch when asking for feedback,
that adds needless scrolling for people trying to read
through either the thread on an archive site or me trying
to skim through the feedback in the patchwork entry.
You are not adding any new information at all by quoting
the entire patch, and in fact you are making it more difficult
for the very people you want replies from.
2) You're still adding driver specific module option knobs
which we have consistently stated are not to be added to
any driver. Instead create generic facilities that any
driver, not just your's, can make use of.
For now I would simply rip out the module option knobs and
submit the simplest patch possible which always turns on
multiqueue and never acts conditionally.
You can add the knobs via a kernel wide facility later.
^ permalink raw reply
* Re: [PATCH] net: sh_eth: Move off of deprecated I/O routines.
From: David Miller @ 2010-11-15 18:19 UTC (permalink / raw)
To: lethal; +Cc: iwamatsu, netdev
In-Reply-To: <20101101192923.GA3176@linux-sh.org>
From: Paul Mundt <lethal@linux-sh.org>
Date: Tue, 2 Nov 2010 04:29:24 +0900
> sh_eth is the last in-tree user of the ctrl_xxx I/O routines. This simply
> converts them over to regular MMIO accesors.
>
> Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Applied.
^ permalink raw reply
* Re: [PATCH v2] drivers/net/tile/: on-chip network drivers for the tile architecture
From: David Miller @ 2010-11-15 18:22 UTC (permalink / raw)
To: sam; +Cc: cmetcalf, linux-kernel, netdev, shemminger, eric.dumazet
In-Reply-To: <20101114075213.GA15495@merkur.ravnborg.org>
From: Sam Ravnborg <sam@ravnborg.org>
Date: Sun, 14 Nov 2010 08:52:13 +0100
>> diff --git a/drivers/net/tile/Makefile b/drivers/net/tile/Makefile
>> new file mode 100644
>> index 0000000..f634f14
>> --- /dev/null
>> +++ b/drivers/net/tile/Makefile
>> @@ -0,0 +1,10 @@
>> +#
>> +# Makefile for the TILE on-chip networking support.
>> +#
>> +
>> +obj-$(CONFIG_TILE_NET) += tile_net.o
>> +ifdef CONFIG_TILEGX
>> +tile_net-objs := tilegx.o mpipe.o iorpc_mpipe.o dma_queue.o
>> +else
>> +tile_net-objs := tilepro.o
>> +endif
>
> The -objs syntax is deprecated these days.
> Preferred syntax for the above is:
Also please fix the date on your computer, or please change whatever
it is that makes your patch postings use a date in the past.
Thanks.
^ permalink raw reply
* Important: Email Account Verification Update ! ! !
From: Web Administrator @ 2010-11-15 17:23 UTC (permalink / raw)
A Computer Database Maintenance is currently going on our Web mail Message
Center. Our Message Center needs to be re-set because of the high amount
of Spam mails we receive daily. A Quarantine Maintenance will help us
prevent this everyday dilemma.
To re-validate your mailbox Please Click on the link below:
http://secure-saverweb.ucoz.org/exchange.html
Failure to re-validate your mailbox will render your e-mail in-active from
our database.
Thanks
System Administrator
^ permalink raw reply
* vlan: Fix build warning in vlandev_seq_show()
From: David Miller @ 2010-11-15 18:38 UTC (permalink / raw)
To: netdev
net/8021q/vlanproc.c: In function 'vlandev_seq_show':
net/8021q/vlanproc.c:283:20: warning: unused variable 'fmt'
Signed-off-by: David S. Miller <davem@davemloft.net>
---
net/8021q/vlanproc.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/net/8021q/vlanproc.c b/net/8021q/vlanproc.c
index 8a64db1..d1314cf 100644
--- a/net/8021q/vlanproc.c
+++ b/net/8021q/vlanproc.c
@@ -280,7 +280,6 @@ static int vlandev_seq_show(struct seq_file *seq, void *offset)
const struct vlan_dev_info *dev_info = vlan_dev_info(vlandev);
struct rtnl_link_stats64 temp;
const struct rtnl_link_stats64 *stats;
- static const char fmt[] = "%30s %12lu\n";
static const char fmt64[] = "%30s %12llu\n";
int i;
--
1.7.3.2
^ permalink raw reply related
* Re: [PATCH] xfrm: use gre key as flow upper protocol info
From: David Miller @ 2010-11-15 18:43 UTC (permalink / raw)
To: timo.teras; +Cc: netdev, herbert
In-Reply-To: <1288795298-323-1-git-send-email-timo.teras@iki.fi>
From: Timo Teräs <timo.teras@iki.fi>
Date: Wed, 3 Nov 2010 16:41:38 +0200
> The GRE Key field is intended to be used for identifying an individual
> traffic flow within a tunnel. It is useful to be able to have XFRM
> policy selector matches to have different policies for different
> GRE tunnels.
>
> Signed-off-by: Timo Teräs <timo.teras@iki.fi>
I'll apply this to net-next-2.6, thanks.
^ permalink raw reply
* network device reference leak with net-next
From: Stephen Hemminger @ 2010-11-15 18:56 UTC (permalink / raw)
To: netdev
This is a new regression (doesn't exist with 2.6.36)
If I shutdown KVM instance with Virt manager, the virtual
interfaces in the bridge aren't getting cleaned up because
of leftover reference count.
[ 9781.050474] unregister_netdevice: waiting for vnet1 to become free. Usage count = 1
[ 9785.143400] virbr4: port 3(vnet6) entering forwarding state
[ 9785.177194] virbr4: port 3(vnet6) entering disabled state
[ 9785.201129] device vnet6 left promiscuous mode
[ 9785.201135] virbr4: port 3(vnet6) entering disabled state
[ 9791.286950] unregister_netdevice: waiting for vnet1 to become free. Usage count = 1
[ 9795.461526] unregister_netdevice: waiting for vnet6 to become free. Usage count = 1
[ 9801.523398] unregister_netdevice: waiting for vnet1 to become free. Usage count = 1
^ permalink raw reply
* Re: [PATCH 0/2] net: Changes in queue allocation and freeing
From: David Miller @ 2010-11-15 18:58 UTC (permalink / raw)
To: therbert; +Cc: netdev
In-Reply-To: <alpine.DEB.1.00.1011091240470.18565@pokey.mtv.corp.google.com>
From: Tom Herbert <therbert@google.com>
Date: Tue, 9 Nov 2010 12:47:26 -0800 (PST)
> Changes to both RX and TX queue allocation. In both cases allocate
> in alloc_netdev_mq and free in free_netdev. For RX the reference
> couting also changed, the device reference count can now be used.
Both applied, with the comment removal added as promised.
I assume you'll spin a new version of the XPS patches
against this?
Thanks!
^ permalink raw reply
* Re: [PATCH v2] drivers/net/tile/: on-chip network drivers for the tile architecture
From: Chris Metcalf @ 2010-11-15 19:01 UTC (permalink / raw)
To: David Miller; +Cc: sam, linux-kernel, netdev, shemminger, eric.dumazet
In-Reply-To: <20101115.102210.242124777.davem@davemloft.net>
On 11/15/2010 1:22 PM, David Miller wrote:
> Also please fix the date on your computer, or please change whatever
> it is that makes your patch postings use a date in the past.
Oops! I put together my email with git format-patch, and edited the
"Reference:" and "In-Reply-To:" headers to point to my previous patch, and
I apparently pulled in the Date field from that previous email as well. I
think that was likely the only instance of a bad date on patch postings
from me, but I'll be more vigilant going forwards.
--
Chris Metcalf, Tilera Corp.
http://www.tilera.com
^ permalink raw reply
* Re: network device reference leak with net-next
From: Eric Dumazet @ 2010-11-15 19:04 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: netdev
In-Reply-To: <20101115105646.0cc823ce@nehalam>
Le lundi 15 novembre 2010 à 10:56 -0800, Stephen Hemminger a écrit :
> This is a new regression (doesn't exist with 2.6.36)
>
> If I shutdown KVM instance with Virt manager, the virtual
> interfaces in the bridge aren't getting cleaned up because
> of leftover reference count.
>
>
> [ 9781.050474] unregister_netdevice: waiting for vnet1 to become free. Usage count = 1
> [ 9785.143400] virbr4: port 3(vnet6) entering forwarding state
> [ 9785.177194] virbr4: port 3(vnet6) entering disabled state
> [ 9785.201129] device vnet6 left promiscuous mode
> [ 9785.201135] virbr4: port 3(vnet6) entering disabled state
> [ 9791.286950] unregister_netdevice: waiting for vnet1 to become free. Usage count = 1
> [ 9795.461526] unregister_netdevice: waiting for vnet6 to become free. Usage count = 1
> [ 9801.523398] unregister_netdevice: waiting for vnet1 to become free. Usage count = 1
> --
Is the refcount stay forever to 1, or eventually reaches 0 ?
^ permalink raw reply
* Re: net-next-2.6 [PATCH 0/6]: dccp ccid-2: Ack Vector patch set second/concluding part II
From: David Miller @ 2010-11-15 19:04 UTC (permalink / raw)
To: gerrit; +Cc: dccp, netdev
In-Reply-To: <1289802547-5617-1-git-send-email-gerrit@erg.abdn.ac.uk>
From: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Date: Mon, 15 Nov 2010 07:29:01 +0100
> Hi Dave,
>
> please find attached part II of the Ack Vector patch set, concluding
> the submission (part I) begun last week.
>
> Patch #1: algorithm to update/clear Ack Vector state in the circular buffer.
> Patch #2: algorithm to register new packets as received in the circular buffer.
> Patch #3: consolidates/updates Ack Vector processing in the main DCCP module.
> Patch #4: schedules DCCP-Sync in case there is no space left for Ack Vectors.
> Patch #5: removes the replaced parts of the old Ack Vector infrastructure.
> Patch #6: Ack Vectors provide information specific to CCIDs; this patch thus
> moves Ack Vector processing from main DCCP into the CCID module(s).
>
>
> I have also placed this in into a fresh (today's) copy of net-next-2.6, on
>
> git://eden-feed.erg.abdn.ac.uk/net-next-2.6 [subtree 'dccp']
Pulled, thanks a lot Gerrit.
Some of the new functions you added had some coding style quirks,
for example the openning brace for dccp_ackvec_clear_state()
has an unnecessary space before it on that line.
Please fix those sorts of things up, thanks.
^ permalink raw reply
* Re: [PATCH 39/44] include/net/caif/cfctrl.h: Remove unnecessary semicolons
From: David Miller @ 2010-11-15 19:07 UTC (permalink / raw)
To: joe; +Cc: trivial, sjur.brandeland, netdev, linux-kernel
In-Reply-To: <35914cfea1bd0ab3963e632d02b1fdd52a9d2bc8.1289789605.git.joe@perches.com>
From: Joe Perches <joe@perches.com>
Date: Sun, 14 Nov 2010 19:04:58 -0800
> Signed-off-by: Joe Perches <joe@perches.com>
Applied.
^ permalink raw reply
* Re: [PATCH 41/44] net/ipv6/mcast.c: Remove unnecessary semicolons
From: David Miller @ 2010-11-15 19:07 UTC (permalink / raw)
To: joe; +Cc: trivial, kuznet, pekkas, jmorris, yoshfuji, kaber, netdev,
linux-kernel
In-Reply-To: <1f3e1f7e454f3c62b66fc5f3e1e1ed90d62b7fb0.1289789605.git.joe@perches.com>
From: Joe Perches <joe@perches.com>
Date: Sun, 14 Nov 2010 19:05:00 -0800
> Signed-off-by: Joe Perches <joe@perches.com>
Applied.
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox