Kai Mäkisara [Tue, 30 Dec 2003 05:17:13 +0000 (23:17 -0600)]
[PATCH] Add char_devs to st
This patch adds support for cdevs to the st driver. The changes are based on
Doug Gilbert's corresponding changes to the sg driver. Using cdevs brings the
following advantanges:
- support for many drives, currently the maximum is set to 128; the minor
number assignment is explained in the patch to Documentation/scsi/README.st
- the tape devices appear in /sys/cdev/major/st*, as an example here are
the entries for the first drive (4 modes, rewind and non-rewind
devices):
/sys/cdev/major/st0m0 /sys/cdev/major/st0m1 /sys/cdev/major/st0m2
/sys/cdev/major/st0m3
/sys/cdev/major/st0m0n /sys/cdev/major/st0m1n /sys/cdev/major/st0m2n
/sys/cdev/major/st0m3n
Daniele Venzano [Mon, 29 Dec 2003 17:59:18 +0000 (12:59 -0500)]
[netdrvr sis900] add suspend/resume support
The attached patch adds support for suspend/resume to the sis900 driver.
With this patch on resume the NIC is fully configured and operational,
before a module reload was needed because of the complete lack of
suspend/resume callbacks.
I added two functions, sis900_suspend and sis900_resume, with their
pointers in struct pci_driver. A vector of 16 u32 was then needed to the
to keep PCI data during suspend. I added it in struct sis900_private.
I updated the revision number to reflect my changes.
Looking at the code I also killed three typos.
The patch doesn't touch any other code.
Since I don't know anything on ethernet drivers the rule 'works for me'
is fully valid.
Andrew Morton [Mon, 29 Dec 2003 13:55:33 +0000 (05:55 -0800)]
[PATCH] dm: fix block device resizing
From: Joe Thornber <thornber@sistina.com>
When setting the size of a Device-Mapper device in the gendisk entry, also
try to set the size of the corresponding block_device entry's inode. This is
necessary to allow online device/filesystem resizing to work correctly.
[Kevin Corry]
Andrew Morton [Mon, 29 Dec 2003 13:55:25 +0000 (05:55 -0800)]
[PATCH] name_to_dev_t() fix
From: viro@parcelfarce.linux.theplanet.co.uk
When we register disks, we mangle the disk names that contain slashes (e.g.
cciss/c0d0) replacing them with '!' in corresponding sysfs names. So
name_to_dev_t() should mangle the name in the same way before looking for it
in /sys/block.
Andrew Morton [Mon, 29 Dec 2003 13:55:17 +0000 (05:55 -0800)]
[PATCH] Kernel Locking Documentation update
From: Rusty Russell <rusty@rustcorp.com.au>
Entirely revised, and largely rewritten. Has a continuing example now, which
I think makes things clearer. Also covers Read Copy Update. This version
further deprecates rwlock_t, shuffles sections for better organization.
Andrew Morton [Mon, 29 Dec 2003 13:55:09 +0000 (05:55 -0800)]
[PATCH] VIA audio fixes
From: Alan Cox <alan@redhat.com>
VIA audio had a fix from 2.4 missing so any user could spam the system log. Also
include a fix for a bug which is pending 2.4 fixing too and causes a bogus
warning to be displayed on close of audio file handle.
Andrew Morton [Mon, 29 Dec 2003 13:54:21 +0000 (05:54 -0800)]
[PATCH] Rename legacy_bus to platform_bus
From: Jeff Garzik <jgarzik@pobox.com>
I've seen this patch floating around. Not sure the origin, but it's
surfaced on lkml and also when I was poking around handhelds.org CVS for
iPAQ patches: on non-PCs, particularly system-on-chip devices but not
just there, you have a custom "platform bus" that is the root of pretty
much all other devices and buses.
It's something I wanted to make sure people didn't forget; to make sure
the legacy_bus didn't get "legacied out of existence." ;-)
Andrew Morton [Mon, 29 Dec 2003 13:54:13 +0000 (05:54 -0800)]
[PATCH] synchronize use of mm->core_waiters
From: Roland McGrath <roland@redhat.com>
I believe I have identified a failure mode that Linus saw a couple weeks
back when tracking down some other fork/exit sorts of races. We saw this
come up on rare occasions with the RHEL3 kernel's backport of the new code
(while trying to track down other race failure modes we have yet to fix, sigh).
I am talking about the following scenario:
> Btw, even with the fix, doing a "while : ; ./crash t 10 ; done" will
> eventually result in a stuck process:
>
> 1415 tty1 D 0:00 ./crash
>
> This is some kind of deadlock: most of the fifty threads are in "D"
> state, with a trace something like
>
> [<c011fbe3>] schedule+0x360/0x7f8
> [<c0120539>] wait_for_completion+0xd4/0x1c3
> [<c0128c9e>] do_exit+0x627/0x6a4
> [<c0128ddd>] do_group_exit+0x3d/0x177
> [<c0130c13>] dequeue_signal+0x2d/0x84
> [<c0133911>] get_signal_to_deliver+0x390/0x575
> [<c010a541>] do_signal+0x6c/0xf1
> [<c01200be>] default_wake_function+0x0/0x12
> [<c01200be>] default_wake_function+0x0/0x12
> [<c013d50f>] do_futex+0x6d/0x7d
> [<c013d635>] sys_futex+0x116/0x12f
> [<c010a601>] do_notify_resume+0x3b/0x3d
> [<c010a82e>] work_notifysig+0x13/0x15
>
> except for one that is trying to core-dump:
>
> [<c0120539>] wait_for_completion+0xd4/0x1c3
> [<c01200be>] default_wake_function+0x0/0x12
> [<c01200be>] default_wake_function+0x0/0x12
> [<c02101aa>] rwsem_wake+0x86/0x12d
> [<c01738af>] coredump_wait+0xa8/0xaa
> [<c0173a26>] do_coredump+0x175/0x26c
>
> and three that are just doing a regular "exit()" system call:
>
> [<c011fbe3>] schedule+0x360/0x7f8
> [<c011e19a>] recalc_task_prio+0x90/0x1aa
> [<c0120539>] wait_for_completion+0xd4/0x1c3
> [<c01200be>] default_wake_function+0x0/0x12
> [<c01200be>] default_wake_function+0x0/0x12
> [<c0210207>] rwsem_wake+0xe3/0x12d
> [<c0128c9e>] do_exit+0x627/0x6a4
> [<c0128d4d>] next_thread+0x0/0x53
> [<c010a7e3>] syscall_call+0x7/0xb
>
> However, the rest of the system is totally unaffected by this deadlock:
> it's only deadlocked withing the thread group itself, nobody else cares.
What happens here is a race between an exiting thread checking
mm->core_waiters in __exit_mm, and the thread taking the core-dump signal
(in coredump_wait) examining the first thread's ->mm pointer and
incrementing mm->core_waiters to account for it. There is no
synchronization at all in __exit_mm's use of mm->core_waiters. If the
coredump_wait thread reads tsk->mm when tsk is in __exit_mm between
checking mm->core_waiters and clearing tsk->mm, then it will increment
mm->core_waiters and the total count will later exceed the number of
threads that will ever decrement it and synchronize. Hence it blocks forever.
The following patch fixes the problem by using mm->mmap_sem in __exit_mm.
The read lock must be held around checking mm->core_waiters and clearing
tsk->mm so that coredump_wait (which gets the write lock) cannot come in
between and do bogus bookkeeping.
Andrew Morton [Mon, 29 Dec 2003 13:54:05 +0000 (05:54 -0800)]
[PATCH] DAC960 request queue per disk
From: Dave Olien <dmo@osdl.org>
Here's a patch that changes the DAC960 driver from having one request
queue for ALL disks on the controller, to having a request queue for
each logical disk. This turns out to make little difference for deadline
scheduler, nor for AS scheduler under light IO load. But under AS
scheduler with heavy IO, it makes about a 40% difference on dbt2
workload. Here are the measured numbers:
The 2.6.0-test11-D kernel version includes this mutli-queue patch to the
DAC960 driver.
For non-cached dbt2 workload (heavy IO load)
Scheduler kernel/driver NOTPM(bigger is better)
AS 2.6.0-test11-D 1598
AS 2.6.0-test11 973
deadline 2.6.0-test11 1640
deadline 2.6.0-test11-D 1645
For cached dbt2 workload (lighter IO load)
AS 2.6.0-test11-D 4993
AS 2.6.-test6-mm4 4976, 4890, 4972
deadline 2.6.0-test11-D 4998
Can this be included in 2.6.0? I know it's not a "critical patch"
in the sense that something won't work without it. On the other hand,
the change is isolated to a driver.
Andrew Morton [Mon, 29 Dec 2003 13:53:57 +0000 (05:53 -0800)]
[PATCH] fix userspace compiles with nbd.h
From: Paul Clements <Paul.Clements@SteelEye.com>
A previous "cleanup" on the nbd.h header file broke userspace compiles.
I've added an #ifdef __KERNEL__ so that userspace doesn't need to worry
about the nbd_device structure, which is only used in-kernel. The patch
allows me to compile my nbd tools with the 2.6 nbd.h.
Andrew Morton [Mon, 29 Dec 2003 13:53:42 +0000 (05:53 -0800)]
[PATCH] nr_slab accounting fix
From: Manfred Spraul <manfred@colorfullife.com>
if alloc_slabmgmt fails, then kmem_freepages() calls sub_page_state(),
altough nr_slab was not yet increased. The attached patch fixes that by
moving the inc_page_state into kmem_getpages().
Andrew Morton [Mon, 29 Dec 2003 13:53:02 +0000 (05:53 -0800)]
[PATCH] Allow unimap change on non fg console
From: Kurt Garloff <garloff@suse.de>
The comment in front of vt_ioctl() reads
/*
* We handle the console-specific ioctl's here. We allow the
* capability to modify any console, not just the fg_console.=20
*/
Unfortunately, this does not apply to PIO_UNIMAPCLR, nor
GIO_/PIO_UNIMAP. They always operate on the current foreground
console, which is inconsistent at least. For most ioctls, the
comment is applicable.
It also causes problems, as setfont can't do the full job on
the non-fg consoles. (OK, our setfont is slightly changed to
even try it ... as you know.)
The attached patch does fix this.
I have a similar patch for 2.4, but it never got merged :-(
because not many people seem to care and I submitted in the middle
of the 2.4 series ...
It has been in UnitedLinux/SUSE kernels for ages, though.
Andrew Morton [Mon, 29 Dec 2003 13:52:48 +0000 (05:52 -0800)]
[PATCH] list_empty_careful() documentation.
From: Ingo Molnar <mingo@elte.hu>
I'd also suggest the following patch below, to clarify the use of
unsynchronized list_empty(). list_empty_careful() can only be safe in the
very specific case of "one-shot" list entries which might be removed by
another CPU. (but nothing else can happen to them and this is their only
final state.) list_empty_careful() is otherwise completely unsynchronized
on both the compiler and CPU level and is not 'SMP safe' in any way.
Andrew Morton [Mon, 29 Dec 2003 13:52:31 +0000 (05:52 -0800)]
[PATCH] more correct get_compat_timespec interface
From: Joe Korty <joe.korty@ccur.com>
The API for get_compat_timespec / put_compat_timespec is incorrect, it
forces a caller with const args to (incorrectly) cast. The posix message
queue patch is one such caller.
Andrew Morton [Mon, 29 Dec 2003 13:52:15 +0000 (05:52 -0800)]
[PATCH] JBD: b_committed_data locking fix
The locking rules say that b_committed_data is covered by
jbd_lock_bh_state(), so implement that during the start of commit, while
throwing away unused shadow buffers.
I don't expect that there is really a race here, but them's the rules.
Andrew Morton [Mon, 29 Dec 2003 13:52:07 +0000 (05:52 -0800)]
[PATCH] O_DIRECT memory leak fix
From: Badari Pulavarty <pbadari@us.ibm.com>
I found the problem with O_DIRECT memory leak.
The problem is, when we are doing DIO read and crossed the end of file - we
don't release referencess on all the pages we got from get_user_pages().
(since it is a success case).
The fix is to call dio_cleanup() even for sucess cases.
Andrew Morton [Mon, 29 Dec 2003 13:52:00 +0000 (05:52 -0800)]
[PATCH] fix ELF exec with huge bss
From: Roland McGrath <roland@redhat.com>
The following test program will crash every time if dynamically linked.
I think this bites all 32-bit platforms, including 32-bit executables on
64-bit platforms that support them (and could in theory bite 64-bit
platforms with bss sizes beyond the bounds of comprehension).
The problem is that the kernel maps ld.so at 0x40000000 or some such place,
before it maps the bss. Here the bss is so large that it overlaps and
clobbers that mapping. I've changed it to map the bss before it loads the
interpreter, so that part of the address space is reserved before ld.so's
mapping (which doesn't really care where it goes) is done.
This patch also adds error checking to the bss setup (and interpreter's bss
setup). With the aforementioned change but no error checking, "ulimit -v
65536; ./hugebss" will crash in the store after the `system' call, because
the kernel will have failed to allocate the bss and ignored the error, so
the program runs without those pages being mapped at all. With this change
it dies with a SIGKILL as for a failure to set up stack pages. It might be
even better to try to detect the case earlier so that execve can return an
error before it has wiped out the address space. But that seems like it
would always be fragile and miss some corner cases, so I did not try to add
such complexity.
Andrew Morton [Mon, 29 Dec 2003 13:51:35 +0000 (05:51 -0800)]
[PATCH] md: Limit max_sectors on md when merge_bvec_fn defined on underlying device.
From: NeilBrown <neilb@cse.unsw.edu.au>
As no md personalities honour the merge_bvec_fn of underlying devices,
we must make sure never to submit a bio larger than 1 page when a
merge_bvec_fn is defined.
raid5 already does this (it never submits bios larger than one page).
With this patch, all other raid personalities limit their
max_sectors when a merge_bvec_fn is present.
Andrew Morton [Mon, 29 Dec 2003 13:50:23 +0000 (05:50 -0800)]
[PATCH] Merge i386 fix for page fault to x86-64
From: Andi Kleen <ak@muc.de>
Merge the i386 fix for the page fault from Linus to x86-64
(I'm not actually sure what it fixes, but if it's good for 32bit
it is likely good for 64bit too)
Andrew Morton [Mon, 29 Dec 2003 13:49:45 +0000 (05:49 -0800)]
[PATCH] Critical x86-64 IOMMU fixes for 2.6.0
From: Andi Kleen <ak@muc.de>
Please consider applying this patch, I would consider it critical for x86-64.
The 2.6.0 x86-64 IOMMU code unfortunately had a few problems, leading
to non booting systems and in a few cases to data corruption.
It fixes a two serious bugs in handling special kinds of scatter gather
lists in pci_map_sg.
AGP was completely broken with IOMMU because of a wrong #ifdef.
Fix that.
One TLB flush optimization I did a long time ago seems to break on
some 3ware boards (who require IOMMU because they don't support 64bit
addresses). The breakage lead to data corruption. This patch diables
the optimization for now and fixes a potential SMP race in the flush
code too. The TLB flush is done in a slower, but more reliable way
now too.
This patch fixes them. Please consider applying, because some of these
problems hit quite many people.
This also disables the IOMMU_DEBUG in the defconfig. A lot of people
were using the IOMMU when they didn't need to, which multiplied the
problems.
IOMMU merge is disabled for now. This was an experimental optimization
which helped with some block devices, but for production it seems to
be better to disable it for now because there are some questionable
corner cases when the IOMMU aperture fragments. The same is done
for IOMMU SAC force, which was related to that.
i386 has quite broken semantics for pci_alloc_consistent(). It uses
the standard device DMA mask instead of the consistent mask. Make us
bug-to-bug compatible here. This fixes problems with some sound
drivers that don't support full 32bit addressing.
Andrew Morton [Mon, 29 Dec 2003 13:49:36 +0000 (05:49 -0800)]
[PATCH] Add a.out support for x86-64
From: Andi Kleen <ak@muc.de>
Add 32bit a.out support for x86-64.
Not exactly an important bug fix, but maybe it will help someone. This
should increase the current 98% compatibility to i386 to perhaps 98.1% @)
I tested an old a.out SuSE 4.2 installation in chroot and it worked. It
also ran some very old linux binaries from '92 found on ftp.funet.fi. The
only program that didn't was the SuSE a.out GNU emacs, but I was too lazy
to track that down. Core dumps are not supported.
Andrew Morton [Mon, 29 Dec 2003 13:49:29 +0000 (05:49 -0800)]
[PATCH] statfs64 fix
From: Andi Kleen <ak@muc.de>
It fixes the statfs64 emulation on x86-64. The problem is that x86-64
needs an __attribute__((aligned)) on the compat_statfs64 structure. The
conclusion last time this was discussed was that the structure should be
duplicated.
Essentially it is the old shared structure copied to every user and x86-64
uses __attribute__((packed)).
I believe what is happening, is that the dm __clone_and_map function is
generating bio structures with the bi_idx field non-zero. When
__blk_queue_bounce creates a new bio with bounce pages, it sets the bi_idx
field to 0 rather than the bi_idx of the original. This causes trouble since
bv_page pointers will be dereferenced later that are zero. The following
uses the original bio structure's bi_idx in the new bio structure and in
copy_to_high_bio_irq and bounce_end_io.
This has cleared up the panic when using the volume.
Andrew Morton [Mon, 29 Dec 2003 13:49:13 +0000 (05:49 -0800)]
[PATCH] ext3: bd_claim for journal device
From: Neil Brown <neilb@cse.unsw.edu.au>
Change ext3 to run bd_claim() against external journal devices. It is
significant only for those who have ext3 journals on a separate device, and
gets exclusive access to that device.
Andrew Morton [Mon, 29 Dec 2003 13:48:49 +0000 (05:48 -0800)]
[PATCH] fix oops in proc_kill_inodes()
proc_kill_inodes() walks the s_files list, playing with ->f_dentry.
But there is a window in which __fput() will leave a file on that list with a
null f_dentry and f_vfsmnt.
I'm not sure it was ever confirmed that this fixed the reported oops, but it
seems much better to set those fields to null _after_ removing the filp from
the list.
(Actually, there's no need to null those pointers out at all. But whatever;
it caught a bug).
Andrew Morton [Mon, 29 Dec 2003 13:48:33 +0000 (05:48 -0800)]
[PATCH] Remove CLONE_FILES from init kernel thread creation
From: James Morris <jmorris@redhat.com>
The patch below removes the CLONE_FILES flag from the kernel_thread() call
which starts init.
This is to prevent other kernel threads from sharing file descriptors
opened by init (try 'lsof /dev/initctl' on a 2.6 system :-).
The reason this patch is being proposed is so that usermode helper apps
launched via kernel threads (e.g. modprobe, hotplug) do not then inherit
any such file descriptors. This is not a problem in itself so far (other
than being messy), but it is a problem for SELinux, which will otherwise
need to grant access to /dev/initctl by modprobe and hotplug, a somewhat
undesirable scenario.
As far as I can tell, there is no reason why init needs to be spawned with
CLONE_FILES. Please let me know if there are any objections to the
change, which I would like to propose for 2.6.0+ as a cleanup.
Andrew Morton [Mon, 29 Dec 2003 13:48:16 +0000 (05:48 -0800)]
[PATCH] new /proc/irq cpumask format; consolidate cpumask display and input code
From: Paul Jackson <pj@sgi.com>
This patch is a followup to one from Bill Irwin. On Nov
17, he had consolidated the half-dozen chunks of code
that displayed cpumasks in /proc/irq/prof_cpu_mask and
/proc/irq/<pid>/smp_affinity into a single routine, which he
called format_cpumask().
I believe that Andrew Morton has accepted Bill's patch into
his 2.6.0-test10-mm1 patch set as the "format_cpumask" patch.
I hope that the following patch will replace Bill's patch.
I look forward to Bill's feedback on this patch.
The following patch carries Bill's work further:
1) It also consolidates the input side (write syscalls).
2) It adapts a new format, same on input and output.
3) The core routines work for any multi-word bitmask,
not just cpumasks.
4) The core routines avoid overrunning their output
buffers.
Note esp. for David Mosberger:
The small patch I sent you and the linux-ia64 list
yesterday entitled: "check user access ok writing
/proc/irq/<pid>/smp_affinity" for arch ia64 only is
_separate_ from the following patch. Neither presumes the
other. However, they do collide on one line. Last one in
is a Monkey's Uncle and will need an updated patch from me
(or otherwise need to resolve the one obvious collision).
Details of the following patch:
Both the display and input of cpumasks on 9 arch's are
consolidated into a single pair of routines, which use the
same format for input and output, as recommended by Tony
Luck. The two common routines work on any multi-word bitmask
(array of unsigned longs). A pair of trivial inline wrappers
cpumask_snprintf() and cpumask_parse() hide this generality
for the common case of cpumask input and output.
My real motivation for consolidating this code will become
visible later - when I seek to add a nodemask_t that resembles
cpumask_t (just a different length). These common underlying
routines will be used there as well, following up on a suggestion
of Christoph Hellwig that I investigate implementing nodemask_t
as an ADT sharing infrastructure with cpumask_t. However, I
believe that this patch stands on its own merit, consolidating
a couple hundred lines of duplicated code, and making the
cpumask display format usable on very large systems.
There are two exceptions to the consolidation - the alpha and
sparc64 arch's manipulate bare unsigned longs, not cpumask_t's,
on input (write syscall), and do stuff that was more funky than
I could make sense of. So the input side of these two arch's
was left as-is. I'd welcome someone with access to either of
these systems to provide additional patches.
The new format consists of multiple 32 bit words, separated by
commas, displayed and input in hex. The following comment from
this patch describes this format further:
* The ascii representation of multi-word bit masks displays each
* 32bit word in hex (not zero filled), and for masks longer than
* one word, uses a comma separator between words. Words are
* displayed in big-endian order most significant first. And hex
* digits within a word are also in big-endian order, of course.
*
* Examples:
* A mask with just bit 0 set displays as "1".
* A mask with just bit 127 set displays as "80000000,0,0,0".
* A mask with just bit 64 set displays as "1,0,0".
* A mask with bits 0, 1, 2, 4, 8, 16, 32 and 64 set displays
* as "1,1,10117". The first "1" is for bit 64, the second
* for bit 32, the third for bit 16, and so forth, to the
* "7", which is for bits 2, 1 and 0.
* A mask with bits 32 through 39 set displays as "ff,0".
The essential reason for adding the comma breaks was to make
the long masks from our (SGI's) big 512 CPU systems parsable by
humans. An unbroken string of 128 hex digits is pretty difficult
to read. For those who are compiling systems with CONFIG_NR_CPUS
of 32 or less, there should be no visible change in format.
There are of course a thousand possible output formats that
meet similar criteria. If someone wants to lobby for and seek
consensus behind another such format, that's fine. Now that
the format is consolidated into a single pair of routines,
it should be easy to adapt whatever we choose.
Internally, the display routine uses snprintf to track the
remaining space in its output buffer, to avoid the risk of
overrunning it.
A new file, lib/mask.c, is added to the lib directory, to
hold the two common routines. I anticipate adding a few more
common routines for generic support of multi-word bit masks to
lib/mask.c, in subsequent patches that will add a nodemask_t
type as an ADT sharing implementation with cpumask_t.
Andrew Morton [Mon, 29 Dec 2003 13:48:07 +0000 (05:48 -0800)]
[PATCH] cpumask.h reorg
From: Paul Jackson <pj@sgi.com>
Push the cpumask implementation from linux/cpumask.h into asm/cpumask.h, so
that ia64 can do special things without breaking sparc64.
1) Each arch has its own include/asm-<arch>/cpumask.h file
2) That arch-specific header file can include <asm-generic/cpumask.h>,
if it wants to make use of the generic cpumask implementation.
3) Using code should continue to include linux/cpumask.h, which
in turn includes asm/cpumask.h. Some common implementation
independent cpumask related items, such as the cpu_online_map,
are declared directly in linux/cpumask.h.
Andrew Morton [Mon, 29 Dec 2003 13:47:43 +0000 (05:47 -0800)]
[PATCH] IDE MMIO fix
From: Alan Cox <alan@redhat.com>
IDE core code had the mmio==2 (ioremap) mode supported but two small changes
had been missed for ide-dma.c. Without this fix mmio IDE controllers bomb if
you have plenty of memory as it uses request_mem_region on an ioremap return.
Andrew Morton [Mon, 29 Dec 2003 13:47:28 +0000 (05:47 -0800)]
[PATCH] PIIX5 Doesn't work on IA64
From: Peter Chubb <peterc@gelato.unsw.edu.au>
The PIIX5 IDE controller on I2000 IA64 boxen using the 460GX chipset will
hang on startup if an ordinary harddrive is plugged into it (it seems to
workj for the LSI120 and the CDROM drives).
This is because the 460GX chipset contains a PCI expanssion bridge that
works like the 450NX PXB, and has the same PCI ID (but a later revision).
The PIIX driver, to work around interactions between PIIX4 and the 450NX
PXB, tries to disable DMA.
Unfortunately, the way it tries to disable DMA doesn't work, and the higher
layers think that DMA is still on, and so timeout waiting for DMA, and then
hang on bootup.
A simple workaround is to tighten the check for the buggy chipset, as in
the attached patch. However, someone with more time (and who actually
*understands* the IDE subsystem) needs to fix the real bug as well.
Andrew Morton [Mon, 29 Dec 2003 13:47:20 +0000 (05:47 -0800)]
[PATCH] ide-tape update
From: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>,
Stuart Hayes <stuart_hayes@dell.com>
- Check drive's write protect bit, try to return appropriate
errors when attempting to write a write-protected tape.
- Moved "idetape_read_position" call in idetape_chrdev_open
after the "wait_ready" call.
- Added IDETAPE_MEDIUM_PRESENT flag so driver would know
not to rewind tape after ejecting it.
- Fixed bug with ide_abort_pipeline (it was deleting stages
from tape->next_stage to end, instead of from
new_last_stage->next (tape->next_stage was set to NULL
by idetape_discard_read_pipeline before calling!).
- Made improvements to idetape_wait_ready.
- Added a few comments here and there.
- Made MTOFFL unlock tape drive door before attempting to eject.
- Added fixes to get Seagate STT3401A Travan working:
Handle drives that don't support 0-length reads/writes increased timeout
(retension takes ~10 minutes before irq is returned).
Fixed request mode page packet command byte 3.
Also remove code depending on NO_LONGER_REQUIRED to match 2.4.x (me).
Andrew Morton [Mon, 29 Dec 2003 13:47:04 +0000 (05:47 -0800)]
[PATCH] watchdog write() return value fixes
From: gleb@nbase.co.il (Gleb Natapov)
There is inconsistency in fops->write() implementation in different
watchdog drivers. Some of them return number of bytes written while others
return 1.
I think the correct implementation should always return number of bytes
written (we examine all the buffer after all) otherwise "echo V >
/dev/watchdog" doesn't work as expected (it doesn't stop watchdog).
Andrew Morton [Mon, 29 Dec 2003 13:46:40 +0000 (05:46 -0800)]
[PATCH] support centrino 1GHz
From: Jeremy Fitzhardinge <jeremy@goop.org>
I've been getting quite a lot of people mailing me about this CPU. It
seems Toshiba has released a machine with it. It would be nice if this
patch gets into a kernel soonish. It's very low-impact.
Andrew Morton [Mon, 29 Dec 2003 13:46:23 +0000 (05:46 -0800)]
[PATCH] seq_file version of /proc/interrupts
From: corbet@lwn.net (Jonathan Corbet)
This converts all architectures' /proc/interrupts implementation over to
seq_file. We need this for SMP machines with ridiculous numbers of CPUs and
if you convert one arch, you have to convert them all...
Andrew Morton [Mon, 29 Dec 2003 13:46:16 +0000 (05:46 -0800)]
[PATCH] eicon/ and hardware/eicon/ drivers using the same symbols
From: Adrian Bunk <bunk@fs.tum.de>
The legacy eicon driver in drivers/isdn/eicon is the old one and will be
removed as soon as all features went to the new driver. Anyway this old
driver was never meant to be non-module.
Andrew Morton [Mon, 29 Dec 2003 13:45:53 +0000 (05:45 -0800)]
[PATCH] use alloc_percpu in percpu_counters
From: Martin Hicks <mort@wildopensource.com>
Once NR_CPUS exceeds about 300 ext2 and ext3 will not compile, because the
percpu counters in the superblocks are so huge that they cannot be kmalloced.
Fix this by converting the percpu_counter mechanism to use alloc_percpu()
rather than an NR_CPUS-sized array.
Andrew Morton [Mon, 29 Dec 2003 13:45:44 +0000 (05:45 -0800)]
[PATCH] lockless semop
From: Manfred Spraul <manfred@colorfullife.com>
attached is the lockless semop patch. I did another test run with
idle=poll on an pentium III, and it remained unchanged: 99.9% direct
fast path, 0.1% race with wakeup against writing the final result code:
Current writev() of pipe/fifo can be interleaved with data from other
processes doing writes even when the requests size is <= PIPE_BUF. These
writes should in fact be atomic.
The readv() side is also supported for same behavior with read(). And it
is faster.
Andrew Morton [Mon, 29 Dec 2003 13:45:30 +0000 (05:45 -0800)]
[PATCH] optimize ia32 memmove
From: Manfred Spraul <manfred@colorfullife.com>
The memmove implementation of i386 is not optimized: it uses movsb, which is
far slower than movsd. The optimization is trivial: if dest is less than
source, then call memcpy(). markw tried it on a 4xXeon with dbt2, it saved
around 300 million cpu ticks in cache_flusharray():
Andrew Morton [Mon, 29 Dec 2003 13:45:14 +0000 (05:45 -0800)]
[PATCH] Fix for more than 256 CPUs
From: Paul Jackson <pj@sgi.com>
The patch is needed to build NR_CPUS > 256.
Without this fix, you get compile errors:
include/linux/cpumask.h: In function `next_online_cpu':
include/linux/cpumask.h:56: structure has no member named `val'
Andrew Morton [Mon, 29 Dec 2003 13:44:58 +0000 (05:44 -0800)]
[PATCH] Restore /proc/pid/maps formatting
The seq_file conversion of /proc/pid/maps caused altered behaviour with
respect to 2.4.22. Before the conversion, spaces and tabs in filenames were
displayed verbatim. After the conversion they are escaped as \040, etc.
Also, if the mmapped file has been unlinked the output appears as
Andrew Morton [Mon, 29 Dec 2003 13:44:34 +0000 (05:44 -0800)]
[PATCH] vmscan: reset refill_counter after refilling the inactive list
zone->refill_counter is only there to provide decent levels of work batching:
don't call refill_inactive_zone() just for a couple of pages.
But the logic in there allows it to build up to huge values and it can
overflow (go negative) which will disable refilling altogether until it wraps
positive again.
Just reset it to zero whenever we decide to do some refilling.