- 06 May, 2005 2 commits
-
-
Paul Mackerras authored
The code in unflatten_device_tree knows that get_property is written to only return with lenp equal to 1 when also returning a valid pointer. The gcc 3.3.3 compiler is not able to prove this to itself, so it warns about a possible uninitialized pointer dereference: .../arch/ppc64/kernel/prom.c: In function `unflatten_device_tree': .../arch/ppc64/kernel/prom.c:828: warning: `p' might be used uninitialized in this function Unless it is desired to rework the interaction between the two functions, this will keep the existing behavior but quiet the compiler. Signed-off-by:
Amos Waterland <apw@us.ibm.com> Signed-off-by:
Paul Mackerras <paulus@samba.org> Signed-off-by:
Linus Torvalds <torvalds@osdl.org>
-
Tobias Klauser authored
Replace a custom MIN() macro with the min() macro from kernel.h This patch removes 4 lines of redundant code. Signed-off-by:
Tobias Klauser <tklauser@nuerscht.ch> Signed-off-by:
-
- 05 May, 2005 38 commits
-
-
Adrian Bunk authored
The Coverity checker found that this for loop was wrong. This patch changes it to what seems to be intended. Signed-off-by:
Adrian Bunk <bunk@stusta.de> Cc: James Bottomley <James.Bottomley@steeleye.com> Cc: Matthew Wilcox <willy@debian.org> Signed-off-by:
Andrew Morton <akpm@osdl.org> Signed-off-by:
-
Adrian Bunk authored
This patch fixes an off by one error found by the Coverity checker. Signed-off-by:
-
walter harms authored
this clarifies the documentation on the behavier of strncpy(). Signed-off-by:
Domen Puncer <domen@coderock.org> Signed-off-by:
-
Domen Puncer authored
As per http://www.nist.gov/dads/HTML/shellsort.html, this should be referred to as a Shell sort. Shell-Metzner is a misnomer. Signed-off-by:
Daniel Dickman <didickman@yahoo.com> Signed-off-by:
-
Domen Puncer authored
The regression test in lib/sort.c is currently worthless because the array that is generated for sorting will be all zeros. This patch fixes things so that the array that is generated will contain unsorted integers (that are not all identical) as was probably intended. Signed-off-by Daniel Dickman <didickman@yahoo.com> Signed-off-by:
-
Adrian Bunk authored
drivers/scsi/NCR53C9x.c: In function `esp_do_data': drivers/scsi/NCR53C9x.c:1838: warning: unused variable `flags' Signed-off-by:
Adrian Bunk <bunk@fs.tum.de> Signed-off-by:
-
Kianusch Sayah Karadji authored
- Changed Name/defines from "Geode GX" to "Geode GX1" for clarification - Dropped "-march=i586" in favor of "-march=i486" - Dopped X86_OOSTORE support for Geode GX1 Signed-off-by:
Kianusch Sayah Karadji <kianusch@sk-tech.net> Signed-off-by:
-
Jesper Juhl authored
Ross moved. Remove the bad email address so people will find the correct one in ./CREDITS. Signed-off-by:
Jesper Juhl <juhl-lkml@dif.dk> Signed-off-by:
-
Domen Puncer authored
When I do a "diff -Nur arch/i386 arch/x86_64" to see what is different between these two architectures, I see some differences due to whitespace issues only. The attached patch removes some of the noise by fixing up the following files: - arch/i386/boot/bootsect.S - arch/i386/boot/video.S - arch/x86_64/boot/bootsect.S Signed-off-by:
-
Domen Puncer authored
Just a few small cleanups to make this coherent english. Signed-Off-By:
Martin Hicks <mort@wildopensource.com> Signed-off-by:
-
Domen Puncer authored
compile warning cleanup - suggested by Adrian Bunk; remove unmaintained rcs char strings from source and handle the occurrences of their use, make sure kernel-userspace issues taken care of; break out into separate patch Signed-off-by:
Stephen Biggs <yrgrknmxpzlk@gawab.com> Signed-off-by:
-
Jesper Juhl authored
Signed-off-by:
-
Paolo 'Blaisorblade' Giarrusso authored
Add some comments about task->comm, to explain what it is near its definition and provide some important pointers to its uses. Signed-off-by:
Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it> Signed-off-by:
-
Randy.Dunlap authored
Use NULL instead of 0 for pointer (sparse warning): fs/reiserfs/namei.c:611:50: warning: Using plain integer as NULL pointer Signed-off-by:
Randy Dunlap <rddunlap@osdl.org> Signed-off-by:
-
maximilian attems authored
Fix cyrix section references: convert __initdata to __devinitdata. Error: ./arch/i386/kernel/cpu/mtrr/cyrix.o .text refers to 00000379 R_386_32 .init.data Error: ./arch/i386/kernel/cpu/mtrr/cyrix.o .text refers to 00000399 R_386_32 .init.data Error: ./arch/i386/kernel/cpu/mtrr/cyrix.o .text refers to 000003b3 R_386_32 .init.data Error: ./arch/i386/kernel/cpu/mtrr/cyrix.o .text refers to 000003b9 R_386_32 .init.data Error: ./arch/i386/kernel/cpu/mtrr/cyrix.o .text refers to 000003bf R_386_32 .init.data Signed-of-by:
maximilian attems <janitor@sternwelten.at> Signed-off-by:
-
Adrian Bunk authored
This patch makes some needlessly global identifiers static. Signed-off-by:
Arjan van de Ven <arjanv@infradead.org> Acked-by:
Trond Myklebust <trond.myklebust@fys.uio.no> Signed-off-by:
-
Alasdair G Kergon authored
This patch makes some needlessly global code static. Signed-Off-By:
Alasdair G Kergon <agk@redhat.com> Signed-off-by:
-
Alasdair G Kergon authored
The dm emc hardware handler code memset the hardware handler structure to zero AFTER it had initialized the structure's spinlock field. Signed-Off-By:
-
Alasdair G Kergon authored
dm-mpath.c needs to use a private workqueue (like other dm targets already do) to avoid interfering with users of the default workqueue. Signed-Off-By:
Jens Axboe <axboe@suse.de> Signed-off-by:
Lars Marowsky-Bree <lmb@suse.de> Signed-off-by: <mikenc@us.ibm.com> Signed-off-by:
-
Alasdair G Kergon authored
Tidy dm_suspend. Signed-Off-By:
-
Alasdair G Kergon authored
Handle error from __lock_fs() Signed-Off-By:
-
Alasdair G Kergon authored
Allow freeze_bdev() to return an error. Signed-Off-By:
-
Alasdair G Kergon authored
Make __unlock_fs() void. From: Christoph Hellwig <hch@lst.de> Signed-off-by:
-
Alasdair G Kergon authored
Store the struct block_device while device is frozen, saving us one call to bdget_disk(). Signed-Off-By:
-
Andrew Morton authored
This had a fatal lock ranking bug: we do journal_start outside mpage_writepages()'s lock_page(). Revert the whole thing, think again. Credit-to: Jan Kara <jack@suse.cz> For identifying the bug. Cc: Badari Pulavarty <pbadari@us.ibm.com> Signed-off-by:
-
Pavel Roskin authored
Add mailing list addresses for Orinoco and update its homepage. Signed-off-by:
Pavel Roskin <proski@gnu.org> Cc: David Gibson <hermes@gibson.dropbear.id.au> Signed-off-by:
-
Manu Abraham authored
Attached is a patch to bttv which fixes the following problems. Affected cards and problems: ~~~~~~~~~~~~~~~~~~~~~~~~ o VP-1020 (200103A) Tuning problems, device detection. o VP-1020 (DST-MOT) Errors during tuning, device detection fails in a while. o VP-1030 (DST-CI) Tuning sometimes fails after CI commands. o VP-2031 (DCT-CI) Tuning problems The timeout happens before the actual timeout occured in the MCU on the board, and hence the problems. Changes: (bttv-i2c.diff) ~~~~~~~~~~~~~~~~~~~~~~~~ o Changed the custom wait queue to wait_event_interruptible_timeout() - Suggestion by Johannes Stezenbach. o Fixed the wait queue timeout problem - This fixes the timeout problem on various cards. - This problem was visible as many * Cannot tune to channels, when signal levels are very low. * app_info does not work in some conditions for CI based cards - Smaller values worked good for newer cards, but the older cards suffered, settled down to the worst case values that could happen in any eventuality. Signed-off-by:
Manu Abraham <manu@kromtek.com> Signed-off-by:
-
Christoph Hellwig authored
The only caller that ever sets it can call fsync_bdev itself easily. Also update some comments. Signed-off-by:
Christoph Hellwig <hch@lst.de> Cc: <viro@parcelfarce.linux.theplanet.co.uk> Signed-off-by:
-
Christoph Hellwig authored
Signed-off-by:
-
Christoph Hellwig authored
This patch adds support for a new class of DAC960 controllers. It's based on the GPLed idac320 driver from IBM for Linux 2.4.18. That driver is a fork of the 2.4.18 version of DAC960 that adds support for this new type of controllers (internally called "GEM Series"), that differ from other DAC960 V2 firmware controllers only in the register offsets and removes support for all others. This patch instead integrates support for these controllers into the DAC960 driver. Thanks to Anders Norrbring for pointing me to the idac320 driver and testing this patch. No Signed-Off: line because all code is either copy & pasted from IBM's idac320 driver or support for other controllers in the 2.6 DAC960 driver. Note: the really odd formating matches the rest of the DAC960 driver. Cc: Dave Olien <dmo@osdl.org> Signed-off-by:
-
Daniel Ritz authored
Make sure that if the INTRTIE bit is set both functions of the cardbus bridge use the same IRQ before doing any probing... [ yes i hate the TI bridges for the fact that they are very flexible so that so many BIOS vendors get it wrong. ] Signed-off-by:
Daniel Ritz <daniel.ritz@gmx.ch> Signed-off-by:
Dominik Brodowski <linux@dominikbrodowski.net> Signed-off-by:
-
Dominik Brodowski authored
Enable 32-bit memory windows on pd6729 PCI-PCMCIA bridges. Signed-off-by:
Jarkko Raja <jar@pcuf.fi> Signed-off-by:
-
Frederic CAND authored
This patch handles the VIDIOC_S_FMT and VIDIOC_G_FMT ioctls for the saa6752hs. As only 4 preset video formats are supported (SIF, 1/2D1, 2/3D1, D1), we compute to which the asked resolution is the nearest and apply it. Signed-off-by:
Frederic Cand <frederic.cand@anevia.com> Acked-by:
Gerd Knorr <kraxel@bytesex.org> Signed-off-by:
-
Matt Porter authored
Additions to the dontdiff list. Signed-off-by:
Matt Porter <mporter@kernel.crashing.org> Signed-off-by:
-
Adrian Bunk authored
There's no longer a reason to document the obsolete BK usage. Signed-off-by:
-
Andrew Morton authored
Has lots of callsites. Signed-off-by:
-
Adrian Bunk authored
These three functions are referenced from the __devinitdata sis5513_chipset. Signed-off-by:
-
Paulo Marques authored
It seems that the code responsible for this is in kernel/itimer.c:126: p->signal->real_timer.expires = jiffies + interval; add_timer(&p->signal->real_timer); If you request an interval of, lets say 900 usecs, the interval given by timeval_to_jiffies will be 1. If you request this when we are half-way between two timer ticks, the interval will only give 400 usecs. If we want to guarantee that we never ever give intervals less than requested, the simple solution would be to change that to: p->signal->real_timer.expires = jiffies + interval + 1; This however will produce pathological cases, like having a idle system being requested 1 ms timeouts will give systematically 2 ms timeouts, whereas currently it simply gives a few usecs less than 1 ms. The complex (and more computationally expensive) solution would be to check the gettimeofday time, and compute the correct number of jiffies. This way, if we request a 300 usecs timer 200 usecs inside the timer tick, we can wait just one tick, but not if we are 800 usecs inside the tick. This would also mean that we would have to lock preemption during these computations to avoid races, etc. I've searched the archives but couldn't find this particular issue being discussed before. Attached is a patch to do the simple solution, in case anybody thinks that it should be used. Signed-Off-By:
Paulo Marques <pmarques@grupopie.com> Signed-off-by:
-
