* Re: [patch] korina: use resource_size()
From: David Miller @ 2010-03-23 1:33 UTC (permalink / raw)
To: error27; +Cc: netdev, kernel-janitors, n0-1, jpirko, shemminger, joe
In-Reply-To: <20100322121145.GK21571@bicker>
From: Dan Carpenter <error27@gmail.com>
Date: Mon, 22 Mar 2010 15:11:45 +0300
> This size calculation is wrong. It should be end - start + 1.
> Use resource_size() to calculate it correctly.
>
> Signed-off-by: Dan Carpenter <error27@gmail.com>
Applied.
^ permalink raw reply
* Re: [patch] stmmac: use resource_size()
From: David Miller @ 2010-03-23 1:33 UTC (permalink / raw)
To: peppe.cavallaro; +Cc: error27, netdev, joe, linux-kernel, kernel-janitors
In-Reply-To: <4BA78977.6020801@st.com>
From: Giuseppe CAVALLARO <peppe.cavallaro@st.com>
Date: Mon, 22 Mar 2010 16:15:03 +0100
> Hi Dan
> Thanks, the patch can also applied against the net-next driver and it's ok.
Applied, thanks everyone.
^ permalink raw reply
* Re: [patch] ewrk3: range checking problem
From: David Miller @ 2010-03-23 1:33 UTC (permalink / raw)
To: error27
Cc: netdev, shemminger, kaber, jpirko, adobriyan, linux-kernel,
kernel-janitors
In-Reply-To: <20100322120728.GD21571@bicker>
From: Dan Carpenter <error27@gmail.com>
Date: Mon, 22 Mar 2010 15:07:28 +0300
> The range checking here is wrong. It should be HASH_TABLE_LEN which
> is 512.
>
> Signed-off-by: Dan Carpenter <error27@gmail.com>
Applied.
^ permalink raw reply
* Re: [Patch] netconsole: do not depend on experimental
From: David Miller @ 2010-03-23 1:33 UTC (permalink / raw)
To: amwang; +Cc: linux-kernel, netdev
In-Reply-To: <20100322100305.5518.29500.sendpatchset@localhost.localdomain>
From: Amerigo Wang <amwang@redhat.com>
Date: Mon, 22 Mar 2010 05:59:23 -0400
>
> Nowadays, most distributions enable netconsole by default,
> including RHEL, Fedora, Debian, Arch, Opensuse. And
> we don't have any bug reports about it. So I think there
> is no need to mark it as experimental any more.
>
> Cc: David S. Miller <davem@davemloft.net>
> Signed-off-by: WANG Cong <amwang@redhat.com>
Applied.
^ permalink raw reply
* Re: [net-next-2.6 PATCH] ipoib: remove addrlen check for mc addresses
From: David Miller @ 2010-03-23 1:34 UTC (permalink / raw)
To: jpirko-H+wXaHxf7aLQT0dZR+AlfA
Cc: jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/,
netdev-u79uwXL29TY76Z2rM5mHXA, ogerlitz-smomgflXvOZWk0Htik3J/w,
linux-rdma-u79uwXL29TY76Z2rM5mHXA, monis-smomgflXvOZWk0Htik3J/w
In-Reply-To: <20100322172613.GA2884-YzwxZg+R7et1/kRsl7OVgNvLeJWuRmrY@public.gmane.org>
From: Jiri Pirko <jpirko-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Date: Mon, 22 Mar 2010 18:26:14 +0100
> Mon, Mar 22, 2010 at 05:59:16PM CET, jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org wrote:
>>On Mon, Mar 22, 2010 at 02:21:39PM +0100, Jiri Pirko wrote:
>>> Finally this bit can be removed. Currently, after the bonding driver is
>>> changed/fixed (32a806c194ea112cfab00f558482dd97bee5e44e net-next-2.6),
>>> that's not possible for an addr with different length than dev->addr_len
>>> to be present in list. Removing this check as in new mc_list there will be
>>> no addrlen in the record.
>>
>>Maybe just make this check a WARN_ON?
>
> As I said, addrlen will no longer be in record (because it would have no
> meaning since length ot the addr is always dev->addr_len)
>
>>Can userspace create a mc_list
>>entry with the wrong size via netlink?
>
> Nope - this is not possible. dev->addr_len is used.
I've applied your patch, thanks Jiri.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply
* [BUG] XFRM is not updating RTAX_ADVMSS metric
From: Eduardo Panisset @ 2010-03-23 1:35 UTC (permalink / raw)
To: netdev
Hi All,
I have been testing DSMIPv6 code which uses all kind of advanced
features of XFRM framework and I believe I have found a bug related to
update RTAX_ADVMSS route metric.
The XFRM code on net/xfrm/xfrm_policy.c by its functions
xfrm_init_pmtu and xfrm_bundle_ok updates RTAX_MTU route caching
metric however I believe it must update RTAX_ADVMSS as this later is
used by tcp connect function for adverting the MSS value on SYN
messages.
As MSS is not being updated by XFRM the TCP SYN messages (e.g.
originated from a internet browser) is erroneously informing its MSS
(without taking into account the overhead added to IP packet size by
XFRM transformations). One result of that is the browser gets
"frozen" after starts a TCP connection because TCP messages sent by
TCP server will never get to it (TCP server is sending too large
segments to browser).
Below I describe the changes I have done (on xfrm_init_pmtu and
xfrm_bundle_ok) and that seem to fix this problem:
xfrm_init_pmtu:
.
.
.
dst->metrics[RTAX_MTU-1] = pmtu; // original code, below my changes
if (dst->xfrm->props.mode == XFRM_MODE_TUNNEL)
switch (dst->xfrm->props.family)
{
case AF_INET:
dst->metrics[RTAX_ADVMSS-1] = max_t(unsigned int,
pmtu - sizeof(struct iphdr) - sizeof(struct tcphdr), 256);
break;
case AF_INET6:
dst->metrics[RTAX_ADVMSS-1] = max_t(unsigned int,
pmtu - sizeof(struct ipv6hdr) - sizeof(struct tcphdr),
dev_net(dst->dev)->ipv6.
sysctl.ip6_rt_min_advmss);
break;
}
xfrm_bundle_ok:
.
.
.
dst->metrics[RTAX_MTU-1] = mtu; // original code, below my changes
if (dst->xfrm->props.mode == XFRM_MODE_TUNNEL)
switch (dst->xfrm->props.family)
{
case AF_INET:
dst->metrics[RTAX_ADVMSS-1] = max_t(unsigned
int, mtu - sizeof(struct iphdr) - sizeof(struct tcphdr), 256);
break;
case AF_INET6:
dst->metrics[RTAX_ADVMSS-1] = max_t(unsigned
int, mtu - sizeof(struct ipv6hdr) - sizeof(struct tcphdr),
dev_net(dst->dev)->ipv6.sysctl.ip6_rt_min_advmss);
break;
}
Regards,
Eduardo Panisset.
^ permalink raw reply
* Re: [Uclinux-dist-devel] [PATCH] can: bfin_can: switch to common Blackfin can header
From: Mike Frysinger @ 2010-03-23 1:42 UTC (permalink / raw)
To: David Miller
Cc: socketcan-core, netdev, uclinux-dist-devel, oliver.hartkopp,
urs.thuermann
In-Reply-To: <20100322.163637.73665927.davem@davemloft.net>
On Mon, Mar 22, 2010 at 19:36, David Miller wrote:
> From: Mike Frysinger <vapier.adi@gmail.com>
>> On Mon, Mar 22, 2010 at 13:17, David Miller wrote:
>>> From: Mike Frysinger <vapier.adi@gmail.com>
>>> Do you still feel this way after people other than me also asked you
>>> to do the same exact thing for me? Do you think they are making
>>> arbitrary requests as well?
>>
>> the only other person to say anything did not say "you now need to
>> resend your patch". it was "please wait until Linus has pulled things
>> and then Dave has pulled things". which was not relevant at the point
>> it was sent.
>
> Wolfgang told you to "just resubmit the patch", just like I asked you
> to, when you said "the code is there just apply the old one."
i see no e-mail from Wolfgang. did he miss some CC's ? or do you mean Oliver ?
how about we skip to the end where we agree i'm wrong, you're right,
and you just merge the patch that i already resent ...
-mike
^ permalink raw reply
* Re: [Patch v2] netpoll: warn when there are spaces in parameters
From: Cong Wang @ 2010-03-23 1:44 UTC (permalink / raw)
To: Matt Mackall; +Cc: linux-kernel, netdev, elendil, David Miller
In-Reply-To: <1269296535.3552.10.camel@calx>
Matt Mackall wrote:
> On Mon, 2010-03-22 at 04:59 -0400, Amerigo Wang wrote:
>> + printk(KERN_INFO "%s: warning: whitespace"
>> + "is not allowed\n", np->name);
>
> Is it a warning or is it info? If it's a warning, then we probably need
> to add "netpoll" or whatever to the message so that people who've got a
> warning-level threshold will know what it's about.
>
If you mean KERN_INFO, yeah, I want to keep it in the same level
as other messages around.
Also, I already put "np->name" which will be "netconsole" when
we use netconsole.
Thanks.
^ permalink raw reply
* Re: [PATCH] can: bfin_can: switch to common Blackfin can header
From: David Miller @ 2010-03-23 1:48 UTC (permalink / raw)
To: vapier.adi-Re5JQEeQqe8AvxtiuMwx3w
Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w,
netdev-u79uwXL29TY76Z2rM5mHXA,
urs.thuermann-l29pVbxQd1IUtdQbppsyvg,
oliver.hartkopp-l29pVbxQd1IUtdQbppsyvg,
uclinux-dist-devel-ZG0+EudsQA8dtHy/vicBwGD2FQJk+8+b
In-Reply-To: <8bd0f97a1003221842k4e44bfccmfc7858b9c793b44f-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
From: Mike Frysinger <vapier.adi-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Date: Mon, 22 Mar 2010 21:42:43 -0400
> how about we skip to the end where we agree i'm wrong, you're right,
> and you just merge the patch that i already resent ...
Sure, if you wish to make it clear that you have zero
interest in understanding the substance of this issue.
What part of:
You only have to deal with me, and I have to deal with
hundreds of developers, so make my life as simple as possible
ok?
do you not understand?
Multiply every grievance you throw my way by however many other
patches, bug reports, and issues I deal with on a given day.
I think this is the part you don't get.
^ permalink raw reply
* Re: [RFC Patch 3/3] bonding: make bonding support netpoll
From: Cong Wang @ 2010-03-23 1:49 UTC (permalink / raw)
To: Andy Gospodarek
Cc: linux-kernel, netdev, bridge, Andy Gospodarek, Neil Horman,
Stephen Hemminger, bonding-devel, Jay Vosburgh, David Miller
In-Reply-To: <20100323005639.GH2108@gospo.rdu.redhat.com>
Andy Gospodarek wrote:
> On Mon, Mar 22, 2010 at 04:17:40AM -0400, Amerigo Wang wrote:
>> Based on Andy's work, but I modify a lot.
>>
>> Similar to the patch for bridge, this patch does:
>>
>> 1) implement the 4 methods to support netpoll for bonding;
>>
>> 2) modify netpoll during forwarding packets in bonding;
>>
>> 3) disable netpoll support of bridge when a netpoll-unabled device
>> is added to bonding;
>>
>> 4) enable netpoll support when all underlying devices support netpoll.
>>
>> Cc: Andy Gospodarek <gospo@redhat.com>
>> Cc: Neil Horman <nhorman@tuxdriver.com>
>> Cc: Jay Vosburgh <fubar@us.ibm.com>
>> Cc: David Miller <davem@davemloft.net>
>> Signed-off-by: WANG Cong <amwang@redhat.com>
>>
>
> How much testing was done on this?
>
> One of the potential problems with this code is how gracefully the
> system can handle tear-down of interfaces or removal of the bonding
> module when netconsole is active. Was that tested heavily?
>
For this case you mention, I did test it, but what I did is mainly basic
functionality testing, including bonding over bridge and bridge over bonding.
Thanks!
^ permalink raw reply
* Re: [Uclinux-dist-devel] [PATCH] can: bfin_can: switch to common Blackfin can header
From: Mike Frysinger @ 2010-03-23 1:50 UTC (permalink / raw)
To: David Miller
Cc: socketcan-core, netdev, uclinux-dist-devel, oliver.hartkopp,
urs.thuermann
In-Reply-To: <20100322.184830.93460099.davem@davemloft.net>
On Mon, Mar 22, 2010 at 21:48, David Miller wrote:
> From: Mike Frysinger
>> how about we skip to the end where we agree i'm wrong, you're right,
>> and you just merge the patch that i already resent ...
>
> Sure, if you wish to make it clear that you have zero
> interest in understanding the substance of this issue.
whatever makes you happy. you dont appear to have much interest in my
explanations.
-mike
^ permalink raw reply
* Re: kaweth: New USB ID 07c9:b010 Allied Telesyn AT-USB10
From: David Miller @ 2010-03-23 1:53 UTC (permalink / raw)
To: andi; +Cc: netdev
Your patch was corrupted by your email client, in particular
tab characters were transformed into spaces.
Please turn all text formatting options off in your email client
and resubmit your patch in a form that will actually apply.
If you are unsure, send yourself the patch and try to apply cleanly
the patch in the email you receive. You will save us a lot of time if
you make sure what you send is usable for us so please take the time
to do this right.
Thanks.
^ permalink raw reply
* Re: [Uclinux-dist-devel] [PATCH] can: bfin_can: switch to common Blackfin can header
From: David Miller @ 2010-03-23 1:57 UTC (permalink / raw)
To: vapier.adi
Cc: socketcan-core, netdev, uclinux-dist-devel, oliver.hartkopp,
urs.thuermann
In-Reply-To: <8bd0f97a1003221850p50fd9d1fq55a56265c19ca528@mail.gmail.com>
From: Mike Frysinger <vapier.adi@gmail.com>
Date: Mon, 22 Mar 2010 21:50:56 -0400
> On Mon, Mar 22, 2010 at 21:48, David Miller wrote:
>> From: Mike Frysinger
>>> how about we skip to the end where we agree i'm wrong, you're right,
>>> and you just merge the patch that i already resent ...
>>
>> Sure, if you wish to make it clear that you have zero
>> interest in understanding the substance of this issue.
>
> whatever makes you happy. you dont appear to have much interest in my
> explanations.
Your explanation is that "resubmit" doesn't really mean "resubmit"
even when mutliple people request it of you.
And you just think that "just do this" is a simple request when
it's thrown at a maintainer like me who has to deal with sometimes
hundreds of patch submissions per day.
It's _not_.
You do it if you don't believe me. I bet you'll have a different view
of the world and a different set of expectations after that.
^ permalink raw reply
* Re: [RFC Patch 2/3] bridge: make bridge support netpoll
From: Cong Wang @ 2010-03-23 2:03 UTC (permalink / raw)
To: Matt Mackall
Cc: linux-kernel, netdev, bridge, Andy Gospodarek, Neil Horman,
Stephen Hemminger, bonding-devel, Jay Vosburgh, David Miller
In-Reply-To: <1269297307.3552.23.camel@calx>
Matt Mackall wrote:
> On Mon, 2010-03-22 at 04:17 -0400, Amerigo Wang wrote:
>> Based on the previous patch, make bridge support netpoll by:
>>
>> 1) implement the 4 methods to support netpoll for bridge;
>>
>> 2) modify netpoll during forwarding packets in bridge;
>>
>> 3) disable netpoll support of bridge when a netpoll-unabled device
>> is added to bridge;
>
> Not sure if this is the right thing to do. Shouldn't we simply enable
> polling on all devices that support it and warn about the others (aka
> best effort)?
>
I don't think it's a good idea, because we check if a device
supports netpoll by checking if it has ndo_poll_controller method.
Thanks.
^ permalink raw reply
* Re: [RFC Patch 3/3] bonding: make bonding support netpoll
From: Cong Wang @ 2010-03-23 2:01 UTC (permalink / raw)
To: Jay Vosburgh
Cc: Matt Mackall, linux-kernel, netdev, bridge, Andy Gospodarek,
Neil Horman, Stephen Hemminger, bonding-devel, David Miller
In-Reply-To: <12061.1269301015@death.nxdomain.ibm.com>
Jay Vosburgh wrote:
> Matt Mackall <mpm@selenic.com> wrote:
>
>> On Mon, 2010-03-22 at 04:17 -0400, Amerigo Wang wrote:
>>> Based on Andy's work, but I modify a lot.
>>>
>>> Similar to the patch for bridge, this patch does:
>>>
>>> 1) implement the 4 methods to support netpoll for bonding;
>>>
>>> 2) modify netpoll during forwarding packets in bonding;
>>>
>>> 3) disable netpoll support of bridge when a netpoll-unabled device
>>> is added to bonding;
>>>
>>> 4) enable netpoll support when all underlying devices support netpoll.
>> Again, not sure if this is the right policy. Seems to me that on a
>> bonding device we should simply pick an interface to send netpoll
>> messages on, without reference to balancing, etc. Thus, if any of the
>> bonded devices supports polling, it should work.
>
> For some of the modes, the above is pretty straighforward.
> Others, 802.3ad and balance-alb, are a bit more complicated.
>
> The risk is that the network peers and switches may see the same
> MAC address on multiple ports, or different MAC addresses for the same
> IP address.
>
> To implement the above suggestion, I think a "current netpoll
> slave" would have to be tracked, and kept up to date (as slaves become
> active or inactive, etc). Reusing the existing "current active slave"
> won't work for the case that the active slave is not netpoll-capable,
> but a different slave is; also, not all modes use the current active
> slave.
>
> In 802.3ad, the "current netpoll slave" selector will have to
> poke into the aggregator status to choose the netpoll slave. It's not a
> simple matter of picking one and then sticking with it forever; if the
> aggregator containing the netpoll slave is deactivated, then peers may
> not receive the traffic, etc.
>
> In the active-backup mode, only the active slave can send or
> receive, so if it's not netpoll capable, but a backup slave is, you're
> still out of luck (unless netpoll awareness is added to the "best slave"
> selection logic, and even then it'd have to be a secondary criteria).
> Or, the inactive slave can be transmitted on, but if the same MAC comes
> out of the active and a backup slave, it can confuse switches.
>
> In one mode (balance-alb), slaves keep their own MAC addresses,
> and are matched with peers. Bypassing the balance algorithm could again
> confuse peers or switches, who could see two MAC addresses for the same
> IP address, if netpoll traffic goes out a different slave than the
> balance algorithm picks for the same destination.
>
> I think, then, the question becomes: is this extra complexity
> worth it to cover the cases of netpoll over bonding wherein one or more
> slaves don't support netpoll?
>
I see, thanks for your explanation, I overlooked the bonding case.
The current implementation will totally disable netpoll when at least one
slave doesn't support netpoll, so it looks like a safe choice. ;)
> How many network drivers don't support netpoll nowadays?
>
Only about 20% of network drivers support netpoll, a quick grep of 'ndo_poll_controller'
can show that.
Thanks a lot!
^ permalink raw reply
* Re: [RFC Patch 1/3] netpoll: add generic support for bridge and bonding devices
From: Cong Wang @ 2010-03-23 2:13 UTC (permalink / raw)
To: Matt Mackall
Cc: linux-kernel, netdev, bridge, Andy Gospodarek, Neil Horman,
Stephen Hemminger, bonding-devel, Jay Vosburgh, David Miller,
Jeff Moyer
In-Reply-To: <1269297081.3552.19.camel@calx>
Matt Mackall wrote:
> On Mon, 2010-03-22 at 04:17 -0400, Amerigo Wang wrote:
>> This whole patchset is for adding netpoll support to bridge and bonding
>> devices. I already tested it for bridge, bonding, bridge over bonding,
>> and bonding over bridge. It looks fine now.
>
> Ages ago, Jeff Moyer took a run at this, added him to the cc: on the off
> chance he still cares.
>
>> Please comment.
>>
>>
>> To make bridge and bonding support netpoll, we need to adjust
>> some netpoll generic code. This patch does the following things:
>>
>> 1) introduce two new priv_flags for struct net_device:
>> IFF_IN_NETPOLL which identifies we are processing a netpoll;
>> IFF_DISABLE_NETPOLL is used to disable netpoll support for a device
>> at run-time;
>
> This one is a little worrisome. I've tried to keep the netpoll code
> restricted to as tight an area as possible. Adding new flags like these
> that random drivers might try to fiddle with seems like a good way for a
> driver writer to get in trouble. Also flag space is filling up.
Somewhat, but currently I don't have other way to replace this.
Any suggestions?
>
>> 2) introduce three new methods for netdev_ops:
>> ->ndo_netpoll_setup() is used to setup netpoll for a device;
>> ->ndo_netpoll_xmit() is used to transmit netpoll requests;
>> ->ndo_netpoll_cleanup() is used to clean up netpoll when a device is
>> removed.
>
> Seems like a lot of interface for something to be used by only a couple
> core drivers. Hopefully Dave has an opinion here.
>
Yeah, I worry about this too, maybe we can group those methods
for netpoll together into another struct, and just put a pointer
here?
Thanks!
^ permalink raw reply
* Re: [PATCH] via-velocity: FLOW_CNTL_RX does not disable Asymmetric pause in set_mii_flow_control()
From: David Miller @ 2010-03-23 2:58 UTC (permalink / raw)
To: roel.kluin; +Cc: netdev, akpm, linux-kernel, davej
In-Reply-To: <4BA7D22A.6080300@gmail.com>
From: Roel Kluin <roel.kluin@gmail.com>
Date: Mon, 22 Mar 2010 21:25:14 +0100
> Asymmetric pause was not disabled in the RX case.
>
> Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
> ---
> I spotted this because In the FLOW_CNTL_RX and FLOW_CNTL_TX_RX
> cases the same code is executed, is the amendment below required?
The bit settings in the MII_ADVERTISE register should be:
1) RX and TX pause both requested
set only ADVERTISE_PAUSE_CAP
2) Only TX pause requested
set only ADVERTISE_PAUSE_ASYM
3) Only RX pause requested
set both ADVERTISE_PAUSE_CAP and ADVERTISE_PAUSE_ASYM
4) Neither TX nor RX pause requested
set nothing
(drivers/net/tg3.c:tg3_advert_flowctrl_1000t() is a good
reference point)
It would be easier to compare the VIA driver with the rest of
the world if it used the standard mii.h defines instead of
it's own private ones. That would be a good change to make at
some point.
Anyways, the correct fix is therefore something like:
diff --git a/drivers/net/via-velocity.c b/drivers/net/via-velocity.c
index 3a486f3..bc278d4 100644
--- a/drivers/net/via-velocity.c
+++ b/drivers/net/via-velocity.c
@@ -812,7 +812,7 @@ static void set_mii_flow_control(struct velocity_info *vptr)
case FLOW_CNTL_TX_RX:
MII_REG_BITS_ON(ANAR_PAUSE, MII_REG_ANAR, vptr->mac_regs);
- MII_REG_BITS_ON(ANAR_ASMDIR, MII_REG_ANAR, vptr->mac_regs);
+ MII_REG_BITS_OFF(ANAR_ASMDIR, MII_REG_ANAR, vptr->mac_regs);
break;
case FLOW_CNTL_DISABLE:
^ permalink raw reply related
* Re: [Patch v2] netpoll: warn when there are spaces in parameters
From: David Miller @ 2010-03-23 3:06 UTC (permalink / raw)
To: nhorman; +Cc: amwang, linux-kernel, netdev, elendil
In-Reply-To: <20100322123046.GA4288@hmsreliant.think-freely.org>
From: Neil Horman <nhorman@tuxdriver.com>
Date: Mon, 22 Mar 2010 08:30:46 -0400
> On Mon, Mar 22, 2010 at 04:59:58AM -0400, Amerigo Wang wrote:
>> v2: update according to Frans' comments.
>>
>> Currently, if we leave spaces before dst port,
>> netconsole will silently accept it as 0. Warn about this.
>>
>> Also, when spaces appear in other places, make them
>> visible in error messages.
>>
>> Signed-off-by: WANG Cong <amwang@redhat.com>
...
> Acked-by: Neil Horman <nhorman@tuxdriver.com>
Applied, thanks!
^ permalink raw reply
* Re: [PATCH] can: bfin_can: switch to common Blackfin can header
From: David Miller @ 2010-03-23 3:06 UTC (permalink / raw)
To: vapier-aBrp7R+bbdUdnm+yROfE0A
Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w,
netdev-u79uwXL29TY76Z2rM5mHXA,
uclinux-dist-devel-ZG0+EudsQA8dtHy/vicBwGD2FQJk+8+b,
oliver.hartkopp-l29pVbxQd1IUtdQbppsyvg,
urs.thuermann-l29pVbxQd1IUtdQbppsyvg
In-Reply-To: <1269241561-28956-1-git-send-email-vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
From: Mike Frysinger <vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
Date: Mon, 22 Mar 2010 03:06:01 -0400
> The MMR bits are being moved to this header, so include it.
>
> Signed-off-by: Mike Frysinger <vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>
Applied.
^ permalink raw reply
* Re: [PATCH] ixgbevf: Message formatting fixup
From: David Miller @ 2010-03-23 3:08 UTC (permalink / raw)
To: joe; +Cc: gregory.v.rose, netdev, jeffrey.t.kirsher
In-Reply-To: <1269285157.22616.55.camel@Joe-Laptop.home>
From: Joe Perches <joe@perches.com>
Date: Mon, 22 Mar 2010 12:12:37 -0700
> Commit 29b8dd024bd48c3d1d1e5140f5bbb683786f998e
> left a trailing ", " after a message.
> Fix it and make the text used a bit smaller when DEBUG is #defined
>
> Signed-off-by: Joe Perches <joe@perches.com>
Looks good, applied, thanks Joe.
^ permalink raw reply
* Re: [PATCH] netdev: don't always reset iflink when registering
From: David Miller @ 2010-03-23 3:20 UTC (permalink / raw)
To: Thomas.Goff; +Cc: kaber, netdev
In-Reply-To: <6AFD35BEDB60334C9A31FD694E13EF0D22676C2244@XCH-NW-18V.nw.nos.boeing.com>
From: "Goff, Thomas" <Thomas.Goff@boeing.com>
Date: Mon, 22 Mar 2010 12:06:46 -0700
> So maybe it makes more sense to change these if that's more consistent
> with other cases.
Yes, it does. The canonical place to set ->iflink by hand
is the ->ndo_init() method of the device.
^ permalink raw reply
* Re: [PATCH] netlink: use the appropriate namespace pid
From: David Miller @ 2010-03-23 3:25 UTC (permalink / raw)
To: thomas.goff; +Cc: netdev
In-Reply-To: <20100320013850.GA13239@boeing.com>
From: Tom Goff <thomas.goff@boeing.com>
Date: Fri, 19 Mar 2010 18:38:50 -0700
> This was included in OpenVZ kernels but wasn't integrated upstream.
>>From git://git.openvz.org/pub/linux-2.6.24-openvz:
>
> commit 5c69402f18adf7276352e051ece2cf31feefab02
> Author: Alexey Dobriyan <adobriyan@openvz.org>
> Date: Mon Dec 24 14:37:45 2007 +0300
>
> netlink: fixup ->tgid to work in multiple PID namespaces
>
> Signed-off-by: Tom Goff <thomas.goff@boeing.com>
BTW, we're ok without this.
The 'pid' in this code is just an arbitrary port number. We are
merely using the tgid to seed the port allocator, so it doesn't have
to be the real proper tgid.
I'm therefore dropping this for now. If Alexey can show why we
might need this, we can revisit the patch.
^ permalink raw reply
* Re: [PATCH] net_sched: make traffic control network namespace aware
From: David Miller @ 2010-03-23 3:26 UTC (permalink / raw)
To: kaber; +Cc: thomas.goff, netdev
In-Reply-To: <4BA7B13C.7020304@trash.net>
From: Patrick McHardy <kaber@trash.net>
Date: Mon, 22 Mar 2010 19:04:44 +0100
> Patrick McHardy wrote:
>> Tom Goff wrote:
>>> Mostly minor changes to add a net argument to various functions and
>>> remove initial network namespace checks.
>>>
>>> Make /proc/net/psched per network namespace.
>>
>> Looks fine from a qdisc POV. One thing that appears to be missing
>> though is teql master netdev registration in other than the initial
>> namespace.
>
> Actually we could take this opportunity and add rtnl_link support
> for teql device registration. I can look into this in a couple of
> days.
I've applied Tom's patch to net-next-2.6, thanks everyone.
^ permalink raw reply
* Re: Gianfar: RX Recycle skb->len error
From: David Miller @ 2010-03-23 3:30 UTC (permalink / raw)
To: ben; +Cc: avorontsov, netdev, Sandeep.Kumar
In-Reply-To: <A6A1774AFD79E346AE6D49A33CB294530DC19F18@EX-BE-017-SFO.shared.themessagecenter.com>
From: "Ben Menchaca (ben@bigfootnetworks.com)" <ben@bigfootnetworks.com>
Date: Mon, 22 Mar 2010 14:10:48 -0700
> It's yucky, but skb_reserve(skb, -alignamount) works,
I have no problem with people using that.
> It seems in both these cases for gianfar, the amount of the
> alignment is not immediately available to the code the
> recognizes that the skb_reset_reserved() is required. A bit
> larger rework appears to be needed.
There's no need to make this so complicated. Just remember the
value and then refer to it later, when needed.
struct gianfar_skb_cb {
int alignamount;
};
#define GIANFAR_CB(skb) ((struct gianfar_skb_cb *)((skb)->cb))
...
GIANFAR_CB(skb)->alignamount = alignamount;
...
skb_reserve(skb, -GIANFAR_CB(skb)->alignamount);
^ permalink raw reply
* Re: [PATCH] netxen: The driver doesn't work on NX_P3_B1 so cause probe to fail.
From: David Miller @ 2010-03-23 3:32 UTC (permalink / raw)
To: amit.salecha; +Cc: ebiederm, netdev, ameen.rahman
In-Reply-To: <99737F4847ED0A48AECC9F4A1974A4B80CFEFED1DB@MNEXMB2.qlogic.org>
From: Amit Salecha <amit.salecha@qlogic.com>
Date: Mon, 22 Mar 2010 01:35:20 -0500
> David,
> After discussing this issues with my team, this is fine with Qlogic to remove support of NX_P3_B1.
>
> You can go ahead with Eric's patch.
>
> Sorry for all these hiccups.
Ok, 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