diff options
author | 2010-03-15 12:46:51 +0100 | |
---|---|---|
committer | 2010-03-15 12:47:59 +0100 | |
commit | 2cda2728aa1c8c006418a24f867b25e5eb7a32e2 (patch) | |
tree | c23ec7c38d74237646c5453d96bfbbea4cadfee7 /lib/Makefile | |
parent | vfs: improve writeback_inodes_wb() (diff) | |
download | linux-2cda2728aa1c8c006418a24f867b25e5eb7a32e2.tar.gz linux-2cda2728aa1c8c006418a24f867b25e5eb7a32e2.tar.bz2 linux-2cda2728aa1c8c006418a24f867b25e5eb7a32e2.zip |
block: Fix overrun in lcm() and move it to lib
lcm() was defined to take integer-sized arguments. The supplied
arguments are multiplied, however, causing us to overflow given
sufficiently large input. That in turn led to incorrect optimal I/O
size reporting in some cases (RAID over RAID).
Switch lcm() over to unsigned long similar to gcd() and move the
function from blk-settings.c to lib.
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'lib/Makefile')
-rw-r--r-- | lib/Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Makefile b/lib/Makefile index 2e152aed7198..0d4015205c64 100644 --- a/lib/Makefile +++ b/lib/Makefile @@ -21,7 +21,7 @@ lib-y += kobject.o kref.o klist.o obj-y += bcd.o div64.o sort.o parser.o halfmd4.o debug_locks.o random32.o \ bust_spinlocks.o hexdump.o kasprintf.o bitmap.o scatterlist.o \ - string_helpers.o gcd.o list_sort.o + string_helpers.o gcd.o lcm.o list_sort.o ifeq ($(CONFIG_DEBUG_KOBJECT),y) CFLAGS_kobject.o += -DDEBUG |