aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2010-03-12fbdev: bf54x-lq043fb/bfin-t350mcqb-fb: drop custom mmap() handlerMichael Hennerich2-59/+0
The common Blackfin code implements the get_fb_unmapped_area() function, so we no longer need to have our own mmap() handler. The common fb code will take care of everything for us now. Signed-off-by: Michael Hennerich <[email protected]> Signed-off-by: Mike Frysinger <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12asiliantfb: fix test of unsigned in asiliant_calc_dclk2()Roel Kluin1-2/+2
Ftarget, Fret, n and m are unsigned so the tests did not work. Signed-off-by: Roel Kluin <[email protected]> Cc: Andres Salomon <[email protected]> Cc: Krzysztof Helt <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12coredump: plug a memory leak situation on dump_seek()André Goddard Rosa1-3/+7
After having started writing the coredump, if filesystem reports an error anytime while writing part of the core file, we would leak a memory page when bailing out. Signed-off-by: AndrĂ© Goddard Rosa <[email protected]> Acked-by: KOSAKI Motohiro <[email protected]> Cc: Alexander Viro <[email protected]> Cc: Roland McGrath <[email protected]> Cc: WANG Cong <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12Add generic sys_olduname()Christoph Hellwig19-284/+66
Add generic implementations of the old and really old uname system calls. Note that sh only implements sys_olduname but not sys_oldolduname, but I'm not going to bother with another ifdef for that special case. m32r implemented an old uname but never wired it up, so kill it, too. Signed-off-by: Christoph Hellwig <[email protected]> Cc: Ralf Baechle <[email protected]> Cc: Benjamin Herrenschmidt <[email protected]> Cc: Paul Mundt <[email protected]> Cc: Jeff Dike <[email protected]> Cc: Hirokazu Takata <[email protected]> Cc: Thomas Gleixner <[email protected]> Cc: Ingo Molnar <[email protected]> Cc: H. Peter Anvin <[email protected]> Cc: Al Viro <[email protected]> Cc: Arnd Bergmann <[email protected]> Cc: Heiko Carstens <[email protected]> Cc: Martin Schwidefsky <[email protected]> Cc: "Luck, Tony" <[email protected]> Cc: James Morris <[email protected]> Cc: Andreas Schwab <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12improve sys_newuname() for compat architecturesChristoph Hellwig28-123/+36
On an architecture that supports 32-bit compat we need to override the reported machine in uname with the 32-bit value. Instead of doing this separately in every architecture introduce a COMPAT_UTS_MACHINE define in <asm/compat.h> and apply it directly in sys_newuname(). Signed-off-by: Christoph Hellwig <[email protected]> Cc: Ralf Baechle <[email protected]> Cc: Benjamin Herrenschmidt <[email protected]> Cc: Paul Mundt <[email protected]> Cc: Jeff Dike <[email protected]> Cc: Hirokazu Takata <[email protected]> Cc: Thomas Gleixner <[email protected]> Cc: Ingo Molnar <[email protected]> Cc: H. Peter Anvin <[email protected]> Cc: Al Viro <[email protected]> Cc: Arnd Bergmann <[email protected]> Cc: Heiko Carstens <[email protected]> Cc: Martin Schwidefsky <[email protected]> Cc: "Luck, Tony" <[email protected]> Cc: James Morris <[email protected]> Cc: Andreas Schwab <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12Add generic sys_ipc wrapperChristoph Hellwig41-1259/+124
Add a generic implementation of the ipc demultiplexer syscall. Except for s390 and sparc64 all implementations of the sys_ipc are nearly identical. There are slight differences in the types of the parameters, where mips and powerpc as the only 64-bit architectures with sys_ipc use unsigned long for the "third" argument as it gets casted to a pointer later, while it traditionally is an "int" like most other paramters. frv goes even further and uses unsigned long for all parameters execept for "ptr" which is a pointer type everywhere. The change from int to unsigned long for "third" and back to "int" for the others on frv should be fine due to the in-register calling conventions for syscalls (we already had a similar issue with the generic sys_ptrace), but I'd prefer to have the arch maintainers looks over this in details. Except for that h8300, m68k and m68knommu lack an impplementation of the semtimedop sub call which this patch adds, and various architectures have gets used - at least on i386 it seems superflous as the compat code on x86-64 and ia64 doesn't even bother to implement it. [[email protected]: add sys_ipc to sys_ni.c] Signed-off-by: Christoph Hellwig <[email protected]> Cc: Ralf Baechle <[email protected]> Cc: Benjamin Herrenschmidt <[email protected]> Cc: Paul Mundt <[email protected]> Cc: Jeff Dike <[email protected]> Cc: Hirokazu Takata <[email protected]> Cc: Thomas Gleixner <[email protected]> Cc: Ingo Molnar <[email protected]> Reviewed-by: H. Peter Anvin <[email protected]> Cc: Al Viro <[email protected]> Cc: Arnd Bergmann <[email protected]> Cc: Heiko Carstens <[email protected]> Cc: Martin Schwidefsky <[email protected]> Cc: "Luck, Tony" <[email protected]> Cc: James Morris <[email protected]> Cc: Andreas Schwab <[email protected]> Acked-by: Jesper Nilsson <[email protected]> Acked-by: Russell King <[email protected]> Acked-by: David Howells <[email protected]> Acked-by: Kyle McMartin <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12Add generic sys_old_mmap()Christoph Hellwig31-258/+79
Add a generic implementation of the old mmap() syscall, which expects its argument in a memory block and switch all architectures over to use it. Signed-off-by: Christoph Hellwig <[email protected]> Cc: Ralf Baechle <[email protected]> Cc: Benjamin Herrenschmidt <[email protected]> Cc: Paul Mundt <[email protected]> Cc: Jeff Dike <[email protected]> Cc: Hirokazu Takata <[email protected]> Cc: Thomas Gleixner <[email protected]> Cc: Ingo Molnar <[email protected]> Reviewed-by: H. Peter Anvin <[email protected]> Cc: Al Viro <[email protected]> Cc: Arnd Bergmann <[email protected]> Cc: Heiko Carstens <[email protected]> Cc: Martin Schwidefsky <[email protected]> Cc: "Luck, Tony" <[email protected]> Cc: James Morris <[email protected]> Cc: Andreas Schwab <[email protected]> Acked-by: Jesper Nilsson <[email protected]> Acked-by: Russell King <[email protected]> Acked-by: Greg Ungerer <[email protected]> Acked-by: David Howells <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12Add generic sys_old_select()Christoph Hellwig27-152/+52
Add a generic implementation of the old select() syscall, which expects its argument in a memory block and switch all architectures over to use it. Signed-off-by: Christoph Hellwig <[email protected]> Cc: Ralf Baechle <[email protected]> Cc: Benjamin Herrenschmidt <[email protected]> Cc: Paul Mundt <[email protected]> Cc: Jeff Dike <[email protected]> Cc: Hirokazu Takata <[email protected]> Cc: Thomas Gleixner <[email protected]> Cc: Ingo Molnar <[email protected]> Reviewed-by: H. Peter Anvin <[email protected]> Cc: Al Viro <[email protected]> Cc: Arnd Bergmann <[email protected]> Cc: Heiko Carstens <[email protected]> Cc: Martin Schwidefsky <[email protected]> Cc: "Luck, Tony" <[email protected]> Cc: James Morris <[email protected]> Acked-by: Andreas Schwab <[email protected]> Acked-by: Russell King <[email protected]> Acked-by: Greg Ungerer <[email protected]> Acked-by: David Howells <[email protected]> Cc: Andreas Schwab <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove unnecessary casting in fd_ioctlJoe Perches1-9/+8
Convert outparam to const void *. Cast outparam to const char * for strlen(). Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove misleading, used once FD_IOCTL_ALLOWED macroJoe Perches1-3/+1
Just code the test directly Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove obfuscating CODE2SIZE macroJoe Perches1-3/+2
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: add __func__ to debugtJoe Perches1-15/+15
Make debugt messages a little neater. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: convert raw_cmd_copyin from while(1) to label: gotoJoe Perches1-35/+38
Reduces indent. Makes a bit more readable and intelligible. Return value now at bottom of function. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove some unnecessary castingJoe Perches1-6/+5
Remove char/void __user * use. Remove kmalloc cast. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: use %pf in logging messagesJoe Perches1-8/+8
Print the function name not the pointer address where useful and possible Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: use __func__ where appropriateJoe Perches1-11/+11
Add and use __func__ to is_alive. Use __func__ in some DPRINTs. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: DPRINT neateningJoe Perches1-9/+7
Move DPRINT macro definition above 1st use Consolidate a format string (>80 columns) Add a newline to an unterminated message Comment neatened Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove #define FLOPPY_SANITY_CHECKJoe Perches1-34/+1
The code could not be compiled without the #define, so just remove it and the #ifdef/#endif lines. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove unnecessary argument from [__]reschedule_timeoutJoe Perches1-10/+12
Prior to patch "drivers/block/floppy.c: Use pr_<level>" only reschedule_timeout(,"request done"...) printed a numeric value after a reschedule_timeout event message. Restore that behavior and remove the now unnecessary argument. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: unclutter redo_fd_request logicJoe Perches1-53/+52
Change for(;;) with continue; to label: goto label Reduces indentation and adds a bit of clarity. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove REPEAT macroJoe Perches1-4/+4
Macros with hidden flow changes aren't nice. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove unnecessary return and bracesJoe Perches1-3/+1
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: add function is_ready_stateJoe Perches1-4/+15
Used a couple of times, might simplify the code a bit. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: convert int initialising to bool initializedJoe Perches1-11/+10
Don't initialize initialized either. Default is false. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove #define DEVICE_NAME "floppy"Joe Perches1-3/+1
Use it directly in the one place it's used. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: move leading && and || to preceding lineJoe Perches1-4/+4
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: convert int 1/0 to bool true/falseJoe Perches1-30/+30
Various functions use int where bool is appropriate lock_fdc, wait_til_done, poll_drive, user_reset_fdc Convert to bool. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove macros CALL, WAIT and IWAITJoe Perches1-10/+8
Obfuscating macros with embedded returns are not nice Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove [_]COPYIN [_]COPYOUT and ECALL macrosJoe Perches1-11/+6
Remove these obfuscating macros with hidden returns Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove most uses of CALL and ECALL macrosJoe Perches1-16/+35
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove [U]CLEARF, [U]SETF, and [U]TESTF macrosJoe Perches1-47/+47
Use clear_bit, set_bit, and test_bit functions directly Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: add debug_dcl(...) macroJoe Perches1-52/+31
Converted #ifdef DCL_DEBUG if (test) DPRINTK(...); #endif to debug_dcl(test, ...); Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove macro LOCK_FDCJoe Perches1-17/+26
Macros with hidden returns aren't nice. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove a few spaces from function castsJoe Perches1-4/+4
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove IN/OUT macros, indent switch/caseJoe Perches1-111/+107
Remove ugly IN/OUT macros, use direct case and code Add missing semicolon after ECALL Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: indent a commentJoe Perches1-1/+1
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove CLEARSTRUCT macro, use memsetJoe Perches1-7/+5
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: comment neatening and remove naked ;Joe Perches1-25/+30
Spacing, column alignment and a for loop with a naked semicolon converted to an assign and while Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove LAST_OUT macroJoe Perches1-4/+8
Macros with hidden returns are not nice. Convert the 2 uses to use direct code. Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: hoist assigns from if()s, neateningJoe Perches1-24/+29
Move assigns above if()s Remove unnecessary parentheses from returns Use a temporary for a duplicated test Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove used once CHECK_READY macroJoe Perches1-2/+4
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: remove unnecessary bracesJoe Perches1-27/+17
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: use pr_<level>Joe Perches1-178/+175
Convert bare printk to pr_info and pr_cont Convert printk(KERN_ERR to pr_err Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: #define space and column neateningJoe Perches1-83/+97
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/block/floppy.c: convert some #include <asm/ to #include <linux/Joe Perches1-2/+2
Signed-off-by: Joe Perches <[email protected]> Cc: Stephen Hemminger <[email protected]> Cc: Jens Axboe <[email protected]> Cc: Marcin Slusarz <[email protected]> Cc: Bartlomiej Zolnierkiewicz <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12mm: introduce dump_page() and print symbolic flag namesWu Fengguang4-13/+86
- introduce dump_page() to print the page info for debugging some error condition. - convert three mm users: bad_page(), print_bad_pte() and memory offline failure. - print an extra field: the symbolic names of page->flags Example dump_page() output: [ 157.521694] page:ffffea0000a7cba8 count:2 mapcount:1 mapping:ffff88001c901791 index:0x147 [ 157.525570] page flags: 0x100000000100068(uptodate|lru|active|swapbacked) Signed-off-by: Wu Fengguang <[email protected]> Cc: Ingo Molnar <[email protected]> Cc: Alex Chiang <[email protected]> Cc: Rik van Riel <[email protected]> Cc: Andi Kleen <[email protected]> Cc: Mel Gorman <[email protected]> Cc: Christoph Lameter <[email protected]> Cc: KOSAKI Motohiro <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12drivers/scsi/ses.c: eliminate double freeJulia Lawall1-2/+2
The few lines below the kfree of hdr_buf may go to the label err_free which will also free hdr_buf. The most straightforward solution seems to be to just move the kfree of hdr_buf after these gotos. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r@ identifier E; expression E1; iterator I; statement S; @@ *kfree(E); ... when != E = E1 when != I(E,...) S when != &E *kfree(E); // </smpl> Signed-off-by: Julia Lawall <[email protected]> Cc: James Bottomley <[email protected]> Cc: <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12mm: do not iterate over NR_CPUS in __zone_pcp_update()Thomas Gleixner1-1/+1
__zone_pcp_update() iterates over NR_CPUS instead of limiting the access to the possible cpus. This might result in access to uninitialized areas as the per cpu allocator only populates the per cpu memory for possible cpus. This problem was created as a result of the dynamic allocation of pagesets from percpu memory that went in during the merge window - commit 99dcc3e5a94ed491fbef402831d8c0bbb267f995 ("this_cpu: Page allocator conversion"). Signed-off-by: Thomas Gleixner <[email protected]> Acked-by: Pekka Enberg <[email protected]> Acked-by: Tejun Heo <[email protected]> Acked-by: Christoph Lameter <[email protected]> Acked-by: Mel Gorman <[email protected]> Reviewed-by: KOSAKI Motohiro <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12nommu: fix build breakageKAMEZAWA Hiroyuki3-4/+7
Commit 34e55232e59f7b19050267a05ff1226e5cd122a5 ("mm: avoid false sharing of mm_counter") added sync_mm_rss() for syncing loosely accounted rss counters. It's for CONFIG_MMU but sync_mm_rss is called even in NOMMU enviroment (kerne/exit.c, fs/exec.c). Above commit doesn't handle it well. This patch changes SPLIT_RSS_COUNTING depends on SPLIT_PTLOCKS && CONFIG_MMU And for avoid unnecessary function calls, sync_mm_rss changed to be inlined noop function in header file. Reported-by: David Howells <[email protected]> Signed-off-by: KAMEZAWA Hiroyuki <[email protected]> Signed-off-by: Mike Frysinger <[email protected]> Signed-off-by: Michal Simek <[email protected]> Signed-off-by: David Howells <[email protected]> Cc: Greg Ungerer <[email protected]> Cc: Geert Uytterhoeven <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
2010-03-12MAINTAINERS: add bfin_sdh driverMike Frysinger1-0/+7
Signed-off-by: Mike Frysinger <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>