linux-stable.git/block/partitions, branch linux-3.19.y

block: Replace strnicmp with strncasecmp

2014-09-27T22:48:55+00:00

The kernel used to contain two functions for length-delimited,
case-insensitive string comparison, strnicmp with correct semantics
and a slightly buggy strncasecmp. The latter is the POSIX name, so
strnicmp was renamed to strncasecmp, and strnicmp made into a wrapper
for the new strncasecmp to avoid breaking existing users.

To allow the compat wrapper strnicmp to be removed at some point in
the future, and to avoid the extra indirection cost, do
s/strnicmp/strncasecmp/g.

Cc: Jens Axboe 
Signed-off-by: Rasmus Villemoes 
Signed-off-by: Jens Axboe

partitions: aix.c: off by one bug

2014-08-05T19:13:24+00:00

The lvip[] array has "state->limit" elements so the condition here
should be >= instead of >.

Fixes: 6ceea22bbbc8 ('partitions: add aix lvm partition support files')
Signed-off-by: Dan Carpenter 
Acked-by: Philippe De Muyter 
Signed-off-by: Jens Axboe

block/partitions/efi.c: kerneldoc fixing

2014-07-01T16:40:05+00:00

Adding function documentation and fixing kerneldoc warnings
('field: description' uniformization).

Cc: Davidlohr Bueso 
Cc: Jens Axboe 
Signed-off-by: Fabian Frederick 
Signed-off-by: Jens Axboe

block/partitions/msdos.c: code clean-up

2014-07-01T16:40:03+00:00

checkpatch fixing:
WARNING: Missing a blank line after declarations
WARNING: space prohibited between function name and open parenthesis '('
ERROR: spaces required around that '<' (ctx:VxV)

Cc: Jens Axboe 
Cc: Andrew Morton 
Signed-off-by: Fabian Frederick 
Signed-off-by: Jens Axboe

block/partitions/amiga.c: replace nolevel printk by pr_err

2014-07-01T16:40:02+00:00

Also add no prefix pr_fmt to avoid any future default format update

Cc: Jens Axboe 
Cc: Andrew Morton 
Signed-off-by: Fabian Frederick 
Signed-off-by: Jens Axboe

block/partitions/aix.c: replace count*size kzalloc by kcalloc

2014-07-01T16:40:00+00:00

kcalloc manages count*sizeof overflow.

Cc: Jens Axboe 
Cc: Andrew Morton 
Signed-off-by: Fabian Frederick 
Signed-off-by: Jens Axboe

block: Use macros from compiler.h instead of attribute((...))

2014-02-18T20:20:01+00:00

To increase compiler portability there are several macros defined
in  for various gcc __attribute((..)) constructs.
I've made sure gcc these specific were replaced with the right
macro and an #include  was placed where needed.

Signed-off-by: Gideon Israel Dsouza 
Signed-off-by: Jens Axboe

block/partitions/efi.c: fix bound check

2013-11-22T00:42:27+00:00

Use ARRAY_SIZE instead of sizeof to get proper max for label length.

Since this is just a read out of bounds it's not that bad, but the
problem becomes user-visible eg if one tries to use DEBUG_PAGEALLOC and
DEBUG_RODATA, at least with some enhancements from Hiroshi.  Of course
the destination array can contain garbage when we read beyond the end of
source array so that would be another user-visible problem.

Signed-off-by: Antti P Miettinen 
Reviewed-by: Hiroshi Doyu 
Tested-by: Hiroshi Doyu 
Cc: Will Drewry 
Cc: Matt Fleming 
Acked-by: Davidlohr Bueso 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

block/partitions/efi.c: treat size mismatch as a warning, not an error

2013-10-17T04:35:53+00:00

In commit 27a7c642174e ("partitions/efi: account for pmbr size in lba")
we started treating bad sizes in lba field of the partition that has the
0xEE (GPT protective) as errors.

However, we may run into these "bad sizes" in the real world if someone
uses dd to copy an image from a smaller disk to a bigger disk.  Since
this case used to work (even without using force_gpt), keep it working
and treat the size mismatch as a warning instead of an error.

Reported-by: Josh Triplett 
Reported-by: Sean Paul 
Signed-off-by: Doug Anderson 
Reviewed-by: Josh Triplett 
Acked-by: Davidlohr Bueso 
Tested-by: Artem Bityutskiy 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

block: change config option name for cmdline partition parsing

2013-09-30T21:31:02+00:00

Recently commit bab55417b10c ("block: support embedded device command
line partition") introduced CONFIG_CMDLINE_PARSER.  However, that name
is too generic and sounds like it enables/disables generic kernel boot
arg processing, when it really is block specific.

Before this option becomes a part of a full/final release, add the BLK_
prefix to it so that it is clear in absence of any other context that it
is block specific.

In addition, fix up the following less critical items:
 - help text was not really at all helpful.
 - index file for Documentation was not updated
 - add the new arg to Documentation/kernel-parameters.txt
 - clarify wording in source comments

Signed-off-by: Paul Gortmaker 
Cc: Jens Axboe 
Cc: Cai Zhiyong 
Cc: Wei Yongjun 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds