19 Sep, 2006
5 commits
-
Signed-off-by: David Woodhouse
-
(And reset it on new thread creation)
It turns out that eflags is important to save and restore not just
because of iopl, but due to the magic bits like the NT bit, which we
don't want leaking between different threads.Tested-by: Mike Galbraith
Signed-off-by: Linus Torvalds -
* master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6:
[ATM] CLIP: Do not refer freed skbuff in clip_mkip().
[NET]: Drop tx lock in dev_watchdog_up
[PACKET]: Don't truncate non-linear skbs with mmaped IO
[NET]: Mark frame diverter for future removal.
[NETFILTER]: Add secmark headers to header-y
[ATM]: linux-atm-general mailing list is subscribers only
[ATM]: [he] when transmit fails, unmap the dma regions
[TCP] tcp-lp: update information to MAINTAINERS
[TCP] tcp-lp: bug fix for oops in 2.6.18-rc6
[BRIDGE]: random extra bytes on STP TCN packet
[IPV6]: Accept -1 for IPV6_TCLASS
[IPV6]: Fix tclass setting for raw sockets.
[IPVS]: remove the debug option go ip_vs_ftp
[IPVS]: Make sure ip_vs_ftp ports are valid
[IPVS]: auto-help for ip_vs_ftp
[IPVS]: Document the ports option to ip_vs_ftp in kernel-parameters.txt
[TCP]: Turn ABC off.
[NEIGH]: neigh_table_clear() doesn't free stats -
* master.kernel.org:/pub/scm/linux/kernel/git/davem/sparc-2.6:
[SPARC]: Fix regression in sys_getdomainname()
[OPENPROMIO]: Handle current_node being NULL correctly. -
* master.kernel.org:/home/rmk/linux-2.6-arm:
[ARM] 3815/1: headers_install support for ARM
[ARM] 3794/1: S3C24XX: do not defined set_irq_wake when no CONFIG_PM
[ARM] 3793/1: S3C2412: fix wrong serial info struct
[ARM] 3780/1: Fix iop321 cpuid
[ARM] 3786/1: pnx4008: update defconfig
[ARM] 3785/1: S3C2412: Fix idle code as default uses wrong clocks
[ARM] 3784/1: S3C2413: fix config for MACH_S3C2413/MACH_SMDK2413
18 Sep, 2006
23 commits
-
Move kernel-only #includes into #ifdef __KERNEL__, so that
headers_install target can be used on ARM.Signed-off-by: Ralph Siemsen
Signed-off-by: Russell King -
This patch corrects the buffer length checking in the
sys_getdomainname() implementation for sparc/sparc64.Signed-off-by: Andy Walker
Signed-off-by: David S. Miller -
In clip_mkip(), skb->dev is dereferenced after clip_push(),
which frees up skb.Advisory: AD_LAB-06009 ().
Signed-off-by: YOSHIFUJI Hideaki
Signed-off-by: David S. Miller -
Patch from Ben Dooks
Do not define set_irq_wake as a real function if
the CONFIG_PM option is not set.Fixes bug reported by Thomas Gleixner.
Signed-off-by: Ben Dooks
Signed-off-by: Russell King -
Patch from Ben Dooks
The S3C2440 serial info struct is being passed
through the S3C2412 serial info struct probe
routine.Signed-off-by: Ben Dooks
Signed-off-by: Thomas Glexiner
Signed-off-by: Russell King -
If the user tries to traverse to the next node of the
last node, we get NULL in current_node and a zero phandle
returned. That's fine, but if the user tries to obtain
properties in that state, we try to dereference a NULL
pointer in the downcall to the of_*() routines.So protect against that.
Signed-off-by: David S. Miller
-
Fix lockdep warning with GRE, iptables and Speedtouch ADSL, PPP over ATM.
On Sat, Sep 02, 2006 at 08:39:28PM +0000, Krzysztof Halasa wrote:
>
> =======================================================
> [ INFO: possible circular locking dependency detected ]
> -------------------------------------------------------
> swapper/0 is trying to acquire lock:
> (&dev->queue_lock){-+..}, at: [] dev_queue_xmit+0x56/0x290
>
> but task is already holding lock:
> (&dev->_xmit_lock){-+..}, at: [] dev_queue_xmit+0x224/0x290
>
> which lock already depends on the new lock.This turns out to be a genuine bug. The queue lock and xmit lock are
intentionally taken out of order. Two things are supposed to prevent
dead-locks from occuring:1) When we hold the queue_lock we're supposed to only do try_lock on the
tx_lock.2) We always drop the queue_lock after taking the tx_lock and before doing
anything else.>
> the existing dependency chain (in reverse order) is:
>
> -> #1 (&dev->_xmit_lock){-+..}:
> [] lock_acquire+0x76/0xa0
> [] _spin_lock_bh+0x31/0x40
> [] dev_activate+0x69/0x120This path obviously breaks assumption 1) and therefore can lead to ABBA
dead-locks.I've looked at the history and there seems to be no reason for the lock
to be held at all in dev_watchdog_up. The lock appeared in day one and
even there it was unnecessary. In fact, people added __dev_watchdog_up
precisely in order to get around the tx lock there.The function dev_watchdog_up is already serialised by rtnl_lock since
its only caller dev_activate is always called under it.So here is a simple patch to remove the tx lock from dev_watchdog_up.
In 2.6.19 we can eliminate the unnecessary __dev_watchdog_up and
replace it with dev_watchdog_up.Signed-off-by: Herbert Xu
Signed-off-by: David S. Miller -
Non-linear skbs are truncated to their linear part with mmaped IO.
Fix by using skb_copy_bits instead of memcpy.Signed-off-by: Patrick McHardy
Acked-by: Alexey Kuznetsov
Signed-off-by: David S. Miller -
The code for frame diverter is unmaintained and has bitrotted.
The number of users is very small and the code has lots of problems.
If anyone is using it, they maybe exposing themselves to bad packet attacks.Signed-off-by: Stephen Hemminger
Signed-off-by: David S. Miller -
This patch includes xt_SECMARK.h and xt_CONNSECMARK.h to the kernel
headers which are exported via 'make headers_install'. This is needed to
allow userland code to be built correctly with these features.Please apply, and consider for inclusion with 2.6.18 as a bugfix.
Signed-off-by: James Morris
Signed-off-by: David S. Miller -
As the automated reply I got to my last ATM patch shows, the
linux-atm-general mailing list is subscribers-only.Signed-off-by: Roland Dreier
Signed-off-by: David S. Miller -
Signed-off-by: Chas Williams
Signed-off-by: David S. Miller -
Signed-off-by: Wong Hoi Sing Edison
Signed-off-by: David S. Miller -
Sorry that the patch submited yesterday still contain a small bug.
This version have already been test for hours with BT connections. The
oops is now difficult to reproduce.Signed-off-by: Wong Hoi Sing Edison
Signed-off-by: David S. Miller -
We seem to send 3 extra bytes in a TCN, which will be whatever happens
to be on the stack. Thanks to Aji_Srinivas@emc.com for seeing.Signed-off-by: Stephen Hemminger
Signed-off-by: David S. Miller -
This patch should add support for -1 as "default" IPv6 traffic class,
as specified in IETF RFC3542 §6.5. Within the kernel, it seems tclass
< 0 is already handled, but setsockopt, getsockopt and recvmsg calls
won't accept it from userland.Signed-off-by: Remi Denis-Courmont
Acked-by: YOSHIFUJI Hideaki
Signed-off-by: David S. Miller -
np->cork.tclass is used only in cork'ed context.
Otherwise, np->tclass should be used.Bug#7096 reported by Remi Denis-Courmont .
Signed-off-by: YOSHIFUJI Hideaki
Signed-off-by: David S. Miller -
This patch makes the debuging behaviour of this code more consistent
with the rest of IPVS.Signed-Off-By: Simon Horman
Signed-off-by: David S. Miller -
I'm not entirely sure what happens in the case of a valid port,
at best it'll be silently ignored. This patch ignores them a little
more verbosely.Signed-Off-By: Simon Horman
Signed-off-by: David S. Miller -
Fill in a help message for the ports option to ip_vs_ftp
Signed-Off-By: Simon Horman
Signed-off-by: David S. Miller -
I'm not sure if documenting this here is appropriate, but
if it is, here is some text to put there.Signed-Off-By: Simon Horman
Signed-off-by: David S. Miller -
Turn Appropriate Byte Count off by default because it unfairly
penalizes applications that do small writes. Add better documentation
to describe what it is so users will understand why they might want to
turn it on.Signed-off-by: Stephen Hemminger
Signed-off-by: David S. Miller -
neigh_table_clear() doesn't free tbl->stats.
Found by Alexey Kuznetsov. Though Alexey considers this
leak minor for mainstream, I still believe that cleanup
code should not forget to free some of the resources :)At least, this is critical for OpenVZ with virtualized
neighbour tables.Signed-Off-By: Kirill Korotaev
Signed-off-by: David S. Miller
17 Sep, 2006
12 commits
-
[PATCH 9/9] s390: qeth driver fixes [6/6]
From: Frank Pavlic
- Hipersockets has no IPV6 support, thus prevent issueing
SETRTG_IPV6 control commands on Hipersockets devices.
- fixed error handling in qeth_sysfs_(un)registerSigned-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 8/9] s390: qeth driver fixes [5/6]
From: Frank Pavlic
fix kernel panic in qdio queue handling.
qeth_qdio_clear_card() could be invoked by 2 CPUs
simultaneously (for example reboot event and recovery).Signed-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 7/9] s390: qeth driver fixes [4/6]
From: Frank Pavlic
- fix kernel crash due to race,
set card->state to SOFTSETUP after
card and card->dev are initialized properly.
- remove CONFIG_QETH_PERF_STATS, use sysfs attribute instead,
as we want to have the ability to turn on/off the
statistics at runtime.Signed-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 6/9] s390: qeth driver fixes [3/6]
From: Frank Pavlic
fixed kernel panic caused by qeth driver:
Using a bonding device qeth driver will realloc
headroom for every skb coming from the bond device.
Once this happens qeth frees the original skb and
set the skb pointer to the new realloced skb.
Under heavy transmit workload (e.g.UDP streams) through bond
network device the qdio output queue might get full.
In this case we return with EBUSY from qeth_send_packet.
Returning to qeth_hard_start_xmit routine
the skb address on the stack still points to the old address,
which has been freed before.
Returning from qeth_hard_start_xmit with EBUSY results in
requeuing the skb. In this case it corrupts the qdisc queue
and results in kernel panic.Signed-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 5/9] s390: qeth driver fixes [2/6]
From: Frank Pavlic
- fixed error handling in create_device_attributes
- fixed some minor bugs in IPv4
and IPv6 address checkingSigned-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 4/9] s390: qeth driver fixes [1/6]
From: Frank Pavlic
- Drop incoming packets with vlan_tag set
if card->vlangrp is not set.
- use always vlan_hwaccel_rx to pass
vlan frames to the stack.
- fix recovery problem. Device was recovered
properly but still not working.
netif_carrier_on call right before
recovery start fixes it.Signed-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 3/9] s390: Makefile cleanup
From: Frank Pavlic
remove CONFIG_MPC from Makefile which was
introduced accidently in the past.Signed-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
[PATCH 2/9] s390: netiucv driver fixes
From: Frank Pavlic
- missing lock initialization added
- avoid duplicate iucv-interfaces to the same peer
- rw-lock added for manipulating the list of
defined iucv connectionsSigned-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
Hi Jeff,
this is a RESEND of the nine s390 network driver patches.
I finally found that my kmail corrupted almost every patch
I sent the last time. Please apply these 9 patches and forget
about my first attempt! Sorry for the delay, I had some fights
with sendmail, IMAP and mutt configuration.Frank
[RESEND PATCH 1/9] s390: minor s390 network driver fixes
From: Frank Pavlic
- iucv driver:
use do { } while (0) constructs
instead of empty defines to avoid compile bugs.
- ctc driver:
missing lock initialization added
- lcs driver:
BUG_ON usage was removed accidently
with the last lcs patch.
Put them back in place.Signed-off-by: Frank Pavlic
Signed-off-by: Jeff Garzik -
Add the CPU identification needed by oprofile for Intel (r) Core (tm) 2
CPUs.Signed-off-by: Benjamin LaHaise
Signed-off-by: Andi Kleen
Cc: "Arun Sharma"
Signed-off-by: Andrew Morton
Signed-off-by: Linus Torvalds -
* 'for-linus' of master.kernel.org:/pub/scm/linux/kernel/git/roland/infiniband:
RDMA/cma: Increase the IB CM retry count in CMA
IPoIB: Retry failed send-only multicast group joins
IB/srp: Don't schedule reconnect from srp
