Re: [PATCH V1 for-next 0/2] HW Device hot-removal support

Or Gerlitz
2014-11-18T14:43:44Z

On 11/18/2014 2:11 PM, Yishai Hadas wrote:
> Changes from V0:
> patch #1: ib_uverbs_close, reduced mutex scope to enable tasks run in parallel.

Yishai, still some things to sort out here...1st, there must be away for
us to come up with two patches that do not introduce 55 "line over 80
characters" new checkaptch warnings on the code. Next, there are few
more checkpatch warnings, please fix them up.

# ./scripts/checkpatch.pl --strict /tmp/disc_v1/* | grep WARN | wc -l
59

# ./scripts/checkpatch.pl --strict /tmp/disc_v1/* | grep WARN | grep
"line over" | wc -l
55

# ./scripts/checkpatch.pl --strict /tmp/disc_v1/* | grep WARN | sort -u
WARNING: line over 80 characters
WARNING: msleep < 20ms can sleep for up to 20ms; see
Documentation/timers/timers-howto.txt
WARNING: sizeof *context should be sizeof(*context)
WARNING: sizeof *ev_file should be sizeof(*ev_file)
WARNING: sizeof *file should be sizeof(*file)