Re: [PATCH -rt] Fix initialization of spinlock in irttp_dup()

2007-03-20 Thread Ingo Molnar
* Deepak Saxena <[EMAIL PROTECTED]> wrote: > This was found around the 2.6.10 timeframe when testing with the -rt > patch and I believe is still is an issue. irttp_dup() does a memcpy() > of the tsap_cb structure causing the spinlock protecting various > fields in the structure to be duped. T

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Andrew Morton
On Tue, 20 Mar 2007 21:23:52 +0100 Andi Kleen <[EMAIL PROTECTED]> wrote: > Well it causes additional problems. We had some cases where it was really > hard to distingush garbage and the true call chain. yes, for some reason the naive backtraces seem to have got messier and messier over the years

Re: [PATCH 5/5] [NETLINK]: Ignore control messages directly in netlink_run_queue()

2007-03-20 Thread Patrick McHardy
Thomas Graf wrote: > Changes netlink_rcv_skb() to skip netlink controll messages and don't > pass them on to the message handler. > > Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> > > Index: net-2.6.22/net/netlink/af_netlink.c > ===

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Zachary Amsden
Linus Torvalds wrote: On Tue, 20 Mar 2007, Zachary Amsden wrote: Actually, I was thinking the irq handlers would just not mess around with eflags on the stack, just call the chip to ack the interrupt and re-enable hardware interrupts when they left, since that is free anyway with the iret.

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Linus Torvalds
On Tue, 20 Mar 2007, Zachary Amsden wrote: > > Actually, I was thinking the irq handlers would just not mess around with > eflags on the stack, just call the chip to ack the interrupt and re-enable > hardware interrupts when they left, since that is free anyway with the iret. No can do. Think l

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Zachary Amsden
Linus Torvalds wrote: On Tue, 20 Mar 2007, Zachary Amsden wrote: void local_irq_restore(int enabled) { pda.intr_mask = enabled; /* * note there is a window here where softirqs are not processed by * the interrupt handler, but that is not a problem, since it will * get done h

Re: [PATCH 10/12] [IPv6]: Use rtnl registration interface

2007-03-20 Thread YOSHIFUJI Hideaki / 吉藤英明
In article <[EMAIL PROTECTED]> (at Wed, 21 Mar 2007 01:06:03 +0100), Thomas Graf <[EMAIL PROTECTED]> says: > -static int > -inet6_rtm_deladdr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) > +static int nl_addr_del(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) > { > struct i

[PATCH 01/12] [RTNL]: Message handler registration interface

2007-03-20 Thread Thomas Graf
This patch adds a new interface to register rtnetlink message handlers replacing the exported rtnl_links[] array which required many message handlers to be exported unnecessarly. Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/include/net/rtnetlink.h =

[PATCH 11/12] [BRIDGE]: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/bridge/br.c === --- net-2.6.22.orig/net/bridge/br.c 2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/net/bridge/br.c 2007-03-21 00:52:42.0 +0100 @@ -4

[PATCH 04/12] [NET] rules: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/core/fib_rules.c === --- net-2.6.22.orig/net/core/fib_rules.c2007-03-20 23:53:21.0 +0100 +++ net-2.6.22/net/core/fib_rules.c 2007-03-21 00:52:30

[PATCH 03/12] [NEIGH]: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/include/net/neighbour.h === --- net-2.6.22.orig/include/net/neighbour.h 2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/include/net/neighbour.h 2007-03-21 00:52

[PATCH 10/12] [IPv6]: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/include/net/ip6_fib.h === --- net-2.6.22.orig/include/net/ip6_fib.h 2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/include/net/ip6_fib.h2007-03-21 00:52:4

[PATCH 07/12] [PKT_SCHED] cls: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/sched/cls_api.c === --- net-2.6.22.orig/net/sched/cls_api.c 2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/net/sched/cls_api.c 2007-03-21 00:52:36.0

[PATCH 08/12] [PKT_SCHED] act: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/sched/act_api.c === --- net-2.6.22.orig/net/sched/act_api.c 2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/net/sched/act_api.c 2007-03-21 00:52:37.0

[PATCH 09/12] [DECNet]: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/include/net/dn_fib.h === --- net-2.6.22.orig/include/net/dn_fib.h2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/include/net/dn_fib.h 2007-03-21 00:52:38

[PATCH 02/12] [NET] link: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/core/rtnetlink.c === --- net-2.6.22.orig/net/core/rtnetlink.c2007-03-21 00:52:26.0 +0100 +++ net-2.6.22/net/core/rtnetlink.c 2007-03-21 00:52:28

[PATCH 12/12] [RTNL]: Use rtnl registration interface for dump-all aliases

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/core/rtnetlink.c === --- net-2.6.22.orig/net/core/rtnetlink.c2007-03-21 00:52:30.0 +0100 +++ net-2.6.22/net/core/rtnetlink.c 2007-03-21 00:52:43

[PATCH 05/12] [IPv4]: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/include/net/ip_fib.h === --- net-2.6.22.orig/include/net/ip_fib.h2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/include/net/ip_fib.h 2007-03-21 00:52:33

[PATCH 06/12] [PKT_SCHED] qdisc: Use rtnl registration interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/include/net/sch_generic.h === --- net-2.6.22.orig/include/net/sch_generic.h 2007-03-20 23:53:20.0 +0100 +++ net-2.6.22/include/net/sch_generic.h2007-0

[PATCH 00/12] [PATCHSET] RTNetlink message handler registration interface

2007-03-20 Thread Thomas Graf
Introduces an interface to register rtnetlink message handlers and converts all users of rtnl_links[]. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Linus Torvalds
On Tue, 20 Mar 2007, Zachary Amsden wrote: > > void local_irq_restore(int enabled) > { >pda.intr_mask = enabled; >/* > * note there is a window here where softirqs are not processed by > * the interrupt handler, but that is not a problem, since it will > * get done here in th

[PATCH 2/5] [TCP] westwood: Use type safe netlink interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/ipv4/tcp_westwood.c === --- net-2.6.22.orig/net/ipv4/tcp_westwood.c 2007-03-20 23:53:19.0 +0100 +++ net-2.6.22/net/ipv4/tcp_westwood.c 2007-03-21 01:10

[PATCH 3/5] [NETLINK]: Remove unused groups variable

2007-03-20 Thread Thomas Graf
Leftover from dynamic multicast groups allocation work. Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6/net/netlink/af_netlink.c === --- net-2.6.orig/net/netlink/af_netlink.c 2007-01-23 14:09:34.0 +0100 +

[PATCH 4/5] [NETLINK]: Ignore !NLM_F_REQUEST messages directly in netlink_run_queue()

2007-03-20 Thread Thomas Graf
netlink_rcv_skb() is changed to skip messages which don't have the NLM_F_REQUEST bit to avoid every netlink family having to perform this check on their own. Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/netlink/af_netlink.c =

[PATCH 5/5] [NETLINK]: Ignore control messages directly in netlink_run_queue()

2007-03-20 Thread Thomas Graf
Changes netlink_rcv_skb() to skip netlink controll messages and don't pass them on to the message handler. Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/netlink/af_netlink.c === --- net-2.6.22.orig/net/netlink/

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Jeremy Fitzhardinge
Matt Mackall wrote: > On Tue, Mar 20, 2007 at 09:31:58AM -0700, Jeremy Fitzhardinge wrote: > >> Linus Torvalds wrote: >> >>> On Tue, 20 Mar 2007, Eric W. Biederman wrote: >>> >>> If that is the case. In the normal kernel what would the "the oops, we got an interrupt cod

[PATCH 0/5] [PATCHSET] Netlink Patches

2007-03-20 Thread Thomas Graf
Converts westwood and vegas netlink code to use the typesafe interface, removes an unused varaible, and move some netlink queue management code into the generic layer. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo i

[PATCH 1/5] [TCP] vegas: Use type safe netlink interface

2007-03-20 Thread Thomas Graf
Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/ipv4/tcp_vegas.c === --- net-2.6.22.orig/net/ipv4/tcp_vegas.c2007-03-20 23:53:19.0 +0100 +++ net-2.6.22/net/ipv4/tcp_vegas.c 2007-03-21 01:10:15

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Zachary Amsden
Jeremy Fitzhardinge wrote: Zachary Amsden wrote: I think Jeremy's idea was to have interrupt handlers leave interrupts disabled on exit if pda.intr_mask was set. In which case, they would bypass all work and we could never get preempted. Yes, I was worried that if we left the isr with

[NET] rules: Unified rules dumping

2007-03-20 Thread Thomas Graf
Implements a unified, protocol independant rules dumping function which is capable of both, dumping a specific protocol family or all of them. This speeds up dumping as less lookups are required. Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6.22/net/core/fib_rules.c ===

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Paul Mackerras
Linus Torvalds writes: > We should just do this natively. There's been several tests over the years > saying that it's much more efficient to do sti/cli as a simple store, and > handling the "oops, we got an interrupt while interrupts were disabled" as > a special case. > > I have this dim mem

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Matt Mackall
On Tue, Mar 20, 2007 at 03:08:19PM -0800, Zachary Amsden wrote: > Matt Mackall wrote: > >I don't know that you need an xchg there. If you're still on the same > >CPU, it should all be nice and causal even across an interrupt handler. > >So it could be: > > > > pda.intr_mask = 0; /* intr_pending c

Re: [Xen-devel] Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Linus Torvalds
On Tue, 20 Mar 2007, Andi Kleen wrote: > > Linus is worried about the unwinder crashing -- that wouldn't help with that. And to make it clear: this is not a theoretical worry. It happened many times over the months the unwinder was in. It was supposed to help debugging, but it made bugs that

Possible solution to those doubly created SAs in ipsec

2007-03-20 Thread Joy Latten
A while back I reported that I sometimes saw double and triple SAs being created. The patch to check for protocol when deleting larval SA removed one obstacle in that I no longer see triple SAs. Now, once in a while double SAs. I think I have figured out the second obstacle. The initiator instal

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Linus Torvalds
On Tue, 20 Mar 2007, Andi Kleen wrote: > On Tue, Mar 20, 2007 at 11:49:39AM -0700, Linus Torvalds wrote: > > > > the thing is, I'd rather see a long backtrace that is hard to decipher but > > that *never* *ever* causes any additional problems, over a pretty one. > > Well it causes additional

[PATCH] mv643xx_eth: add mv643xx_eth_shutdown function

2007-03-20 Thread Dale Farnsworth
From: Dale Farnsworth <[EMAIL PROTECTED]> mv643xx_eth_shutdown is needed for kexec. Signed-off-by: Dale Farnsworth <[EMAIL PROTECTED]> --- drivers/net/mv643xx_eth.c | 14 ++ 1 file changed, 14 insertions(+) Index: linux-2.6-powerpc-df/drivers/net/mv643xx_eth.c ===

Re: dst_ifdown breaks infiniband?

2007-03-20 Thread David Miller
From: "Michael S. Tsirkin" <[EMAIL PROTECTED]> Date: Tue, 20 Mar 2007 18:02:17 +0200 > David, Alexey, what do you think about this patch? Is it right? > Could this patch be considered for 2.6.21? > > Acked-by: Michael S. Tsirkin <[EMAIL PROTECTED]> I plan to apply it and merge. - To unsubscribe

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Jeremy Fitzhardinge
Zachary Amsden wrote: > I think Jeremy's idea was to have interrupt handlers leave interrupts > disabled on exit if pda.intr_mask was set. In which case, they would > bypass all work and we could never get preempted. Yes, I was worried that if we left the isr without actually handling the interru

Re: ping6 to own link-local address

2007-03-20 Thread YOSHIFUJI Hideaki / 吉藤英明
In article <[EMAIL PROTECTED]> (at Tue, 20 Mar 2007 15:16:40 -0700), Sridhar Samudrala <[EMAIL PROTECTED]> says: > On Tue, 2007-03-20 at 10:19 +0100, YOSHIFUJI Hideaki / 吉藤英明 wrote: > > Hello. > > > > Recent 2.6.21-git kernels do not respond to ping6 queries > > to our own (local) link-local add

[PATCH 01/31] [NETLINK]: Introduce nlmsg_hdr() helper

2007-03-20 Thread Arnaldo Carvalho de Melo
For the common "(struct nlmsghdr *)skb->data" sequence, so that we reduce the number of direct accesses to skb->data and for consistency with all the other cast skb member helpers. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- drivers/connector/connector.c |2 +- drive

[PATCH 01/31] [NETLINK]: Introduce nlmsg_hdr() helper

2007-03-20 Thread acme
For the common "(struct nlmsghdr *)skb->data" sequence, so that we reduce the number of direct accesses to skb->data and for consistency with all the other cast skb member helpers. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- drivers/connector/connector.c |2 +- drive

[PATCH 31/31] [CCID3]: Use initial RTT sample from SYN exchange

2007-03-20 Thread Arnaldo Carvalho de Melo
The patch follows the following recommendation made in an erratum to RFC 4342: "Senders MAY additionally make use of other available RTT measurements, including those from the initial Request-Response packet exchange." It implements larger initial windows with regard to this inital RTT measu

[PATCH 30/31] [DCCP]: Sample RTT from SYN exchange

2007-03-20 Thread Arnaldo Carvalho de Melo
Function: - This patch acquires the RTT of the initial SYN (Request/Response) exchange and stores it in an unused field of dccp_sock. Purpose: The purpose is primarily to provide an initial RTT sample for CCIDs. CCID 3 in particular benefits from such a sample; using the RTT o

[PATCH 25/31] [CCID3]: Remove build warnings for 64bit

2007-03-20 Thread Arnaldo Carvalho de Melo
This clears the following sparc64 build warnings: 1) warning: format "%ld" expects type "long int", but argument 3 has type "suseconds_t" 2) warning: format "%llu" expects type "long long unsigned int", but argument 3 has type "__u64" Fixed by using typecast to unsigned. This is argued to be sa

[PATCH 26/31] [CCID3]: Wrap computation of RFC3390-initial rate into separate function

2007-03-20 Thread Arnaldo Carvalho de Melo
The CCID 3 and TFRC specs (RFC 4342, RFC 3448, draft-3448bis) make frequent reference to the computation of the RFC-3390 initial sending rate: 1. Initial sending rate when RTT is known (RFC 4342, p. 6) 2. Response to Idle/Application-Limited periods (RFC 4342, 5.1) This warrants putting the c

[PATCH 27/31] [CCID3]: Handle Idle and Application-Limited periods

2007-03-20 Thread Arnaldo Carvalho de Melo
This updates the code with regard to handling idle and application-limited periods as specified in [RFC 4342, 5.1]. Background: -- The current code does not conform with this, it implements TFRC as per RFC 3448, 4.4: "If the sender has been idle since this nofeedback timer has been se

[PATCH 28/31] [DCCP]: Provide function for RTT sampling

2007-03-20 Thread Arnaldo Carvalho de Melo
A recurring problem, in particular in the CCID code, is that RTT samples from packets with timestamp echo and elapsed time options need to be taken. This service is provided via a new function dccp_sample_rtt in this patch. Furthermore, to protect against `insane' RTT samples, the sampled value is

[PATCH 29/31] [CCID3]: Use function for RTT sampling

2007-03-20 Thread Arnaldo Carvalho de Melo
This replaces the existing occurrences of RTT sampling with the use of the new function dccp_sample_rtt. Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Signed-off-by: Ian McDonald <[EMAIL PROTECTED]> Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- net/dccp/ccids/ccid3.c | 42 +

[PATCH 22/31] [DCCP]: More debug information for dccp_wait_for_ccid

2007-03-20 Thread Arnaldo Carvalho de Melo
This adds more detail in the wait_for_ccid packet scheduling loop. In particular, it informs about (i) when delay is used and (ii) why a packet is discarded. Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Signed-off-by: Ian McDonald <[EMAIL PROTECTED]> Signed-off-by: Arnaldo Carvalho de Melo <[E

[PATCH 23/31] [CCID3]: Add documentation for socket options

2007-03-20 Thread Arnaldo Carvalho de Melo
This updates the documentation on CCID3-specific options. Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Acked-by: Ian McDonald <[EMAIL PROTECTED]> Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- Documentation/networking/dccp.txt | 10 ++ 1 files changed, 10 insertions

[PATCH 24/31] [CCID3]: More to see in dccp_probe

2007-03-20 Thread Arnaldo Carvalho de Melo
This adds a few more fields of interest to /proc/net/dccpprobe, the following output ensues: 1 2 3 4 5 6 7 8910 11 sec.usec src:sport dst:dport size s rtt p X_calc X_recv X t_ipi Also made the formatting consistent. S

[PATCH 21/31] [DCCP]: Always use debug-toggle parameters

2007-03-20 Thread Arnaldo Carvalho de Melo
Currently debugging output (when configured) is automatically enabled when DCCP modules are compiled into the kernel rather than built as loadable modules. This is not necessary, since the module parameters in this case become kernel commandline parameters, e.g. DCCP or CCID3 debug output can be en

[PATCH 18/31] [CCID3]: Fix use of invalid loss intervals

2007-03-20 Thread Arnaldo Carvalho de Melo
This fixes a bug which uses an invalid comparison. The bug resulted in the use of invalid loss intervals. Signed-off-by: Ian McDonald <[EMAIL PROTECTED]> Acked-by: Gerrit Renker <[EMAIL PROTECTED]> Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- net/dccp/ccids/lib/loss_interval.c

[PATCH 19/31] [CCID3]: More verbose debugging

2007-03-20 Thread Arnaldo Carvalho de Melo
This adds a few debugging statements to ccid3.c Signed-off-by: Ian McDonald <[EMAIL PROTECTED]> Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- net/dccp/ccids/ccid3.c | 13 - 1 files changed, 12 insertions(+), 1 deleti

[PATCH 20/31] [CCID3]: Remove race condition and update t_ipi when `s' changes

2007-03-20 Thread Arnaldo Carvalho de Melo
This: 1. removes a race condition in the access to the scheduled send time t_nom which results from allowing asynchronous r/w access to t_nom without locks; 2. updates the inter-packet interval t_ipi = s/X when `s' changes, following a suggestion by Ian McDonald. Signed-off-by: Gerrit

[PATCH 15/31] [CCID3]: Remove redundant `len' test

2007-03-20 Thread Gerrit Renker
Since CCID3 avoids sending 0-byte data packets (cf. ccid3_hc_tx_send_packet), testing for zero-payload length, as performed by ccid3_hc_tx_update_s, is redundant - hence removed by this patch. Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Acked-by: Ian McDonald <[EMAIL PROTECTED]> Signed-off-b

[PATCH 16/31] [CCID3]: Re-order CCID 3 source file

2007-03-20 Thread Gerrit Renker
No code change at all. This splits ccid3.c into a RX and a TX section, so that the file has an organisation similar to the other ones (e.g. packet_history.{h,c}). Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Acked-by: Ian McDonald <[EMAIL PROTECTED]> Signed-off-by: Arnaldo Carvalho de Melo <[E

[PATCH 17/31] [CCID3]: Use MSS for larger initial windows

2007-03-20 Thread Gerrit Renker
This improves the slow-start phase by using the MSS (as suggested in RFC 4342, sec. 5) instead of the packet size s. Also figured out that __u32 is ample resource enough. After applying, I got the following in the logs: ccid3_hc_tx_packet_recv: client(f7421700), s=6, MSS=1424, w_init=4380, R_s

[PATCH 13/31] [DCCP]: Fix for follows48

2007-03-20 Thread Gerrit Renker
The follows48 relation identifies whether 48-bit sequence number x is the direct successor of y. Currently, it does not handle cases of the following type correctly: follows48(0x(prefix)1LL, 0x(prefix)0LL) where prefix is an arbitrary hex sequence of up to 7 digits. This is fixed

[PATCH 14/31] [DCCP]: Remove ambiguity in the way before48 is used

2007-03-20 Thread Gerrit Renker
This removes two ambiguities in employing the new definition of before48, following the analysis on http://www.mail-archive.com/dccp@vger.kernel.org/msg01295.html (1) Updating GSR when P.seqno >= S.SWL With the old definition we did not update when P.seqno and S.SWL are 2^47 apart. To

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Zachary Amsden
Matt Mackall wrote: I don't know that you need an xchg there. If you're still on the same CPU, it should all be nice and causal even across an interrupt handler. So it could be: pda.intr_mask = 0; /* intr_pending can't get set after this */ Why not? Oh, I see. intr_mask is inverted for

[PATCH 10/31] [DCCP]: 48-bit sequence number arithmetic

2007-03-20 Thread Gerrit Renker
This patch * organizes the sequence arithmetic functions into one corner of dccp.h * performs a small modification of dccp_set_seqno to make it more widely reusable (now it is safe to use any number, since it performs modulo-2^48 assignment) * adds functions and generic macros for 48-bit seq

[PATCH 11/31] [DCCP]: Make dccp_delta_seqno return signed numbers

2007-03-20 Thread Gerrit Renker
Problem: Using dccp_delta_seqno(a, b) produces unusable results when -- by accident or coincidence -- sequence number b precedes a. If e.g. a and b are merely reordered and have a distance 1, their delta_seqno is 2^48-1, which would indicate a loss of 2^48-2 packets. Fix: The f

[PATCH 12/31] [DCCP]: Make `before' relation unambiguous

2007-03-20 Thread Gerrit Renker
Problem: before48(a, b) returns the same as before48(b, a) for all a, b which are 2^47 apart (modulo-2^48). The reason is that the difference a-b is used and `true' is returned whenever 2^47 <= a-b <= 2^48-1. This has the disadvantage that a positive difference a-b = 2^47 can not be di

[PATCH 09/31] [FORCEDETH]: Use skb_tailroom where appropriate

2007-03-20 Thread Arnaldo Carvalho de Melo
Reducing the number of skb->data direct accesses. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- drivers/net/forcedeth.c | 18 +++--- 1 files changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/net/forcedeth.c b/drivers/net/forcedeth.c index 442f10e..423

[PATCH 10/31] [DCCP]: 48-bit sequence number arithmetic

2007-03-20 Thread Gerrit Renker
This patch * organizes the sequence arithmetic functions into one corner of dccp.h * performs a small modification of dccp_set_seqno to make it more widely reusable (now it is safe to use any number, since it performs modulo-2^48 assignment) * adds functions and generic macros for 48-bit seq

[PATCH 1/1] [SK_BUFF]: Adjust the zeroing up to tail in __alloc_skb too

2007-03-20 Thread Arnaldo Carvalho de Melo
I did it just in alloc_skb_from_cache, forgot __alloc_skb, fixed now. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- net/core/skbuff.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index c274ea3..d7d2dc3 100644

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Matt Mackall
On Tue, Mar 20, 2007 at 09:31:58AM -0700, Jeremy Fitzhardinge wrote: > Linus Torvalds wrote: > > On Tue, 20 Mar 2007, Eric W. Biederman wrote: > > > >> If that is the case. In the normal kernel what would > >> the "the oops, we got an interrupt code do?" > >> I assume it would leave interrupts

[PATCH 07/31] [ATM] idt77252: Fix double kfree_skb on failure in push_rx_skb

2007-03-20 Thread Arnaldo Carvalho de Melo
Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- drivers/atm/idt77252.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/atm/idt77252.c b/drivers/atm/idt77252.c index 20f2a3a..057efbc 100644 --- a/drivers/atm/idt77252.c +++ b/drivers/atm/idt77252.c @@

[PATCH 08/31] [LMC]: lmc_main wants to use skb_tailroom

2007-03-20 Thread Arnaldo Carvalho de Melo
At that point it is equivalent to what was being used, skb->end - skb->data, and the need is clearly the one skb_tailroom satisfies. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- drivers/net/wan/lmc/lmc_main.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --g

[PATCH 04/31] [NETLINK]: Use nlmsg_trim() where appropriate

2007-03-20 Thread Arnaldo Carvalho de Melo
Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- include/net/netlink.h|2 +- net/core/wireless.c |2 +- net/decnet/dn_route.c|3 ++- net/decnet/dn_table.c|3 ++- net/ipv4/inet_diag.c |

[PATCH 05/31] [SK_BUFF] ipv6: Use skb_network_offset in some more places

2007-03-20 Thread Arnaldo Carvalho de Melo
So that we reduce the number of direct accesses to skb->data. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- net/ipv6/netfilter/ip6_tables.c |2 +- net/ipv6/netfilter/nf_conntrack_reasm.c |5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/net

[PATCH 06/31] [NETLINK]: Add missing include net/netlink.h to sch_atm.c

2007-03-20 Thread Arnaldo Carvalho de Melo
Leftover from the nlmsg_trim cset. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- net/sched/sch_atm.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/net/sched/sch_atm.c b/net/sched/sch_atm.c index 53060e6..0cc3c9b 100644 --- a/net/sched/sch_atm.c +++ b/n

[PATCH 03/31] [NETLINK]: Remove NLMSG_{NEW_ANSWER,CANCEL,END}

2007-03-20 Thread Arnaldo Carvalho de Melo
Not used anywhere and defined inside __KERNEL__, Thomas acked this on irc. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- include/linux/netlink.h | 12 1 files changed, 0 insertions(+), 12 deletions(-) diff --git a/include/linux/netlink.h b/include/linux/netlink.h

[PATCH 02/31] [SK_BUFF]: Remove skb_add_mtu() leftovers

2007-03-20 Thread Arnaldo Carvalho de Melo
Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- include/linux/skbuff.h |1 - net/core/skbuff.c | 14 -- 2 files changed, 0 insertions(+), 15 deletions(-) diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 4803e4d..155f0e6 100644 --- a/include

[PATCH 02/31] [SK_BUFF]: Remove skb_add_mtu() leftovers

2007-03-20 Thread Arnaldo Carvalho de Melo
Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> --- include/linux/skbuff.h |1 - net/core/skbuff.c | 14 -- 2 files changed, 0 insertions(+), 15 deletions(-) diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 4803e4d..155f0e6 100644 --- a/include

[0/31] DCCP and skb offset stuff

2007-03-20 Thread Arnaldo Carvalho de Melo
Hi David, Sorry to disturb you in your spam mining journey, Zelda is nice at times, but real life is better another times, spam aside, take some ham: please consider pulling from: master.kernel.org:/pub/scm/linux/kernel/git/acme/net-2.6.22 As soon as the new AC-adapter comes to

Re: many sockets, slow sendto

2007-03-20 Thread Eric Dumazet
Zacco a écrit : Hi, David Miller wrote: From: Zacco <[EMAIL PROTECTED]> Date: Tue, 20 Mar 2007 00:10:19 +0100 As you recommended, I used oprofile and it turned out that the __udp4_lib_lookup function spent most of the time. There is a udp hash table and the sockets are sought based on the

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Rusty Russell
On Tue, 2007-03-20 at 09:58 -0600, Eric W. Biederman wrote: > Looking at the above code snippet. I guess it is about time to > merge our per_cpu and pda variables... Indeed, thanks for the prod. Now 2.6.21-rc4-mm1 is out, I'll resend the patches. Cheers, Rusty. - To unsubscribe from this list

Re: [PKT_SCHED] atm: Fix missing net/netlink.h include after nlmsg_trim() cleanup

2007-03-20 Thread Arnaldo Carvalho de Melo
On 3/20/07, Thomas Graf <[EMAIL PROTECTED]> wrote: sch_atm.c:(.text+0x380ed): undefined reference to `nlmsg_trim' Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> diff --git a/net/sched/sch_atm.c b/net/sched/sch_atm.c index 53060e6..daa1644 100644 --- a/net/sched/sch_atm.c +++ b/net/sched/sch_atm

Re: [PATCH 0/5] [RFC] AF_RXRPC socket family implementation [try #3]

2007-03-20 Thread David Howells
Alan Cox <[EMAIL PROTECTED]> wrote: > - recvmsg not supporting MSG_TRUNC is rather weird and really ought to be > fixed one day as its useful to find out the sizeof message pending when > combined with MSG_PEEK Hmmm... I hadn't considered that. I assumed MSG_TRUNC not to be useful as arbitraril

Re: ping6 to own link-local address

2007-03-20 Thread Sridhar Samudrala
On Tue, 2007-03-20 at 10:19 +0100, YOSHIFUJI Hideaki / 吉藤英明 wrote: > Hello. > > Recent 2.6.21-git kernels do not respond to ping6 queries > to our own (local) link-local address. Now bisecting... The following patch seems to be the cause for this regression. [IPV6] ROUTE: Do not route packets t

Re: [PATCH 0/5] [RFC] AF_RXRPC socket family implementation [try #3]

2007-03-20 Thread David Howells
Alan Cox <[EMAIL PROTECTED]> wrote: > > (*) SOCK_RPC has been removed. AF_RXRPC sockets now simply ignore the > > "type" argument to socket(). > > This is also incorrect Sigh. And what would you have me do? There *isn't* an appropriate SOCK_xxx constant available, and you won't let me ad

Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Zachary Amsden
Jeremy Fitzhardinge wrote: Yeah, disable interrupts, and set a flag that the fake "sti" can test, and just return without doing anything. (You may or may not also need to do extra work to Ack the hardware interrupt etc, which may be irq-controller specific. Once the CPU has accepted the inte

[PKT_SCHED] atm: Fix missing net/netlink.h include after nlmsg_trim() cleanup

2007-03-20 Thread Thomas Graf
sch_atm.c:(.text+0x380ed): undefined reference to `nlmsg_trim' Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> diff --git a/net/sched/sch_atm.c b/net/sched/sch_atm.c index 53060e6..daa1644 100644 --- a/net/sched/sch_atm.c +++ b/net/sched/sch_atm.c @@ -12,8 +12,8 @@ #include #include #include

Re: many sockets, slow sendto

2007-03-20 Thread Zacco
Hi, David Miller wrote: From: Zacco <[EMAIL PROTECTED]> Date: Tue, 20 Mar 2007 00:10:19 +0100 As you recommended, I used oprofile and it turned out that the __udp4_lib_lookup function spent most of the time. There is a udp hash table and the sockets are sought based on the 7 LSBs of the de

Re: [PATCH -rt] Fix initialization of spinlock in irttp_dup()

2007-03-20 Thread Samuel Ortiz
Hi Deepak, On Tue, Mar 20, 2007 at 12:22:33PM -0700, Deepak Saxena wrote: > This was found around the 2.6.10 timeframe when testing with the -rt patch > and I believe is still is an issue. irttp_dup() does a memcpy() of the > tsap_cb > structure causing the spinlock protecting various fields in

[NET] SAA9730: Fix large pile of warnings

2007-03-20 Thread Ralf Baechle
The SAA9730 driver doesn't quite grok what the difference between an ioport and memory mapped I/O is. It just happened to work on the one Linux system the SAA9730 happens to spend it's misserable existence on. drivers/net/saa9730.c: In function 'evm_saa9730_enable_lan_int': drivers/net/saa9730.c:

Re: [PATCH] Netpoll support for Sibyte MAC

2007-03-20 Thread Ralf Baechle
On Mon, Mar 19, 2007 at 03:43:11PM -0700, Deepak Saxena wrote: > +#ifdef CONFIG_SBMAC_COALESCE Not your patch's fault but ... CONFIG_SBMAC_COALESCE is being defined in sb1250-mac.c but the CONFIG_* namespace is reserved for kconfig. Ralf - To unsubscribe from this list: send the line "unsubsc

[PATCH 3/3] bridge: simpler hash with salt

2007-03-20 Thread Stephen Hemminger
Instead of hashing the whole Ethernet address, it should be faster to just use the last 4 bytes. Add a random salt value to the hash to make it more difficult to construct worst case DoS hash chains. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> --- net/bridge/br_fdb.c | 17

[PATCH 2/3] bridge: don't route packets while learning

2007-03-20 Thread Stephen Hemminger
While in the STP learning state, don't route packets; wait until forwarding delay has expired. The purpose of the forwarding delay is to detect loops in the network, and if a brouter started up and started forwarding, it could cause a flood. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> --

[PATCH 1/3] bridge: eliminate call by reference

2007-03-20 Thread Stephen Hemminger
Change the bridging hook to be simple function with return value rather than modifying the skb argument. This could generate better code and is cleaner. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> --- include/linux/if_bridge.h |3 ++- net/bridge/br_input.c | 20 +-

Re: [Xen-devel] Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Andi Kleen
On Tue, Mar 20, 2007 at 09:39:18PM +, Alan Cox wrote: > > > Because that's really the issue: do you want a "pretty" backtrace, or do > > > you want one that is rock solid but has some crud in it. > > > > I just want an as exact backtrace as possible. I also think > > that we can make the unwi

Re: [Xen-devel] Re: [patch 13/26] Xen-paravirt_ops: Consistently wrap paravirt ops callsites to make them patchable

2007-03-20 Thread Alan Cox
> > Because that's really the issue: do you want a "pretty" backtrace, or do > > you want one that is rock solid but has some crud in it. > > I just want an as exact backtrace as possible. I also think > that we can make the unwinder robust enough. Any reason you can't put the exact back trace i

Re: [PATCH 0/5] [RFC] AF_RXRPC socket family implementation [try #3]

2007-03-20 Thread Alan Cox
Ok quickly going over the code that hasn't made the list - recvmsg not supporting MSG_TRUNC is rather weird and really ought to be fixed one day as its useful to find out the sizeof message pending when combined with MSG_PEEK - RXRPC_MIN_SECURITY_LEVEL reads into rx->min_sec_level and then if it

[PATCH 2/5] AF_RXRPC: Move generic skbuff stuff from XFRM code to generic code [try #3]

2007-03-20 Thread David Howells
Move generic skbuff stuff from XFRM code to generic code so that AF_RXRPC can use it too. The kdoc comments I've attached to the functions needs to be checked by whoever wrote them as I had to make some guesses about the workings of these functions. Signed-Off-By: David Howells <[EMAIL PROTECTED]

Re: [PATCH 0/5] [RFC] AF_RXRPC socket family implementation [try #3]

2007-03-20 Thread David Howells
Alan Cox <[EMAIL PROTECTED]> wrote: > Some of them don't seem to be making it through to the list and are > dropped each time btw Yeah. It's a size issue. The fifth patch is a third of a megabyte. The patches at the URLs should now be updated (I'd forgotten to do that before sending the emails

[PATCH -rt] Fix initialization of spinlock in irttp_dup()

2007-03-20 Thread Deepak Saxena
This was found around the 2.6.10 timeframe when testing with the -rt patch and I believe is still is an issue. irttp_dup() does a memcpy() of the tsap_cb structure causing the spinlock protecting various fields in the structure to be duped. This works OK in the non-RT case but in the RT case we

Re: [PATCH 0/5] [RFC] AF_RXRPC socket family implementation [try #3]

2007-03-20 Thread Alan Cox
> These patches together supply secure client-side RxRPC connectivity as a Linux > kernel socket family. Only the transport/session side is supplied - the > presentation side (marshalling the data) is left to the client. Copies of the > patches can be found here: Some of them don't seem to be ma

Re: [PATCH 0/5] [RFC] AF_RXRPC socket family implementation [try #3]

2007-03-20 Thread Alan Cox
> (*) SOCK_RPC has been removed. AF_RXRPC sockets now simply ignore the "type" > argument to socket(). This is also incorrect NAK - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.or

  1   2   >