Skip to content

s390/debug: Fine-tuning for several function implementations

SF Markus Elfring alsa-devel ath9k-devel ceph-devel dm-devel dri-devel intel-wired-lan kernel-janitors kvm kvm-ppc linux-acpi linux-arm-kernel linux-clk linux-crypto linux-gpio linux-i2c linux-ide linux-iio linux-input linux-m68k linux-media linux-mmc linux-mtd linux-omap linux-raid linux-rdma linux-renesas-soc linux-rtc linux-s390 linux-scsi linux-wireless linuxppc-dev lkml ltp lustre-devel netdev qemu-devel sparclinux virtualization xen-devel 1 version NAK

Status from this message: NAK.

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest17 patches 4 repliesNAK

Patches in v1

#Subject and filesSizeRepliesApplied
1/17s390/debug: Use kmalloc_array() in debug_areas_alloc()arch/s390/kernel/debug.c+4 -50 replies
2/17s390/debug: Delete unnecessary bracesarch/s390/kernel/debug.c+16 -320 replies
3/17s390/debug: Add some spaces for better code readabilityarch/s390/kernel/debug.c+93 -840 replies
4/17s390/debug: Rename jump labels in debug_areas_alloc()arch/s390/kernel/debug.c+5 -60 replies
5/17s390/debug: Fix jump targets in debug_info_alloc()arch/s390/kernel/debug.c+8 -90 replies
6/17s390/debug: Rename jump labels in debug_info_copy()arch/s390/kernel/debug.c+3 -30 replies
7/17s390/debug: Rename jump labels in debug_open()arch/s390/kernel/debug.c+6 -80 replies
8/17s390/debug: Fix a jump target in debug_register_mode()arch/s390/kernel/debug.c+5 -50 replies
9/17s390/debug: Return directly if a null pointer was passed to debug_unregister()arch/s390/kernel/debug.c+1 -41 reply
10/17s390/debug: Delete an unnecessary initialisation in debug_prolog_level_fn()arch/s390/kernel/debug.c+1 -10 replies
11/17s390/debug: Fix indentation in 13 functionsarch/s390/kernel/debug.c+59 -590 replies
12/17s390/debug: Use memdup_user() rather than duplicating its implementationarch/s390/kernel/debug.c+3 -70 replies
13/17s390/debug: Improve a size determination in debug_open()arch/s390/kernel/debug.c+1 -20 replies
14/17s390/debug: Improve a size determination in debug_sprintf_format_fn()arch/s390/kernel/debug.c+1 -10 replies
15/17s390/debug: Improve a size determination in debug_raw_header_fn()arch/s390/kernel/debug.c+2 -50 replies
16/17s390/debug: Improve determination of sizes in debug_info_alloc()arch/s390/kernel/debug.c+2 -30 replies
17/17s390/debug: Improve another size determination in debug_info_alloc()arch/s390/kernel/debug.c+1 -20 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 00/17] s390/debug: Fine-tuning for several function implementations

From: Markus Elfring <[email protected]>
Date: Sat, 3 Sep 2016 13:54:32 +0200

Several update suggestions were taken into account from static source code analysis.

Markus Elfring (17):
Use kmalloc_array() in debug_areas_alloc()
Delete unnecessary braces
Add some spaces for better code readability
Rename jump labels in debug_areas_alloc()
Fix jump targets in debug_info_alloc()
Rename jump labels in debug_info_copy()
Rename jump labels in debug_open()
Fix a jump target in debug_register_mode()
Return directly if a null pointer was passed to debug_unregister() Delete an unnecessary initialisation in debug_prolog_level_fn() Fix indentation in 13 functions
Use memdup_user() rather than duplicating its implementation Improve a size determination in debug_open()
Improve a size determination in debug_sprintf_format_fn() Improve a size determination in debug_raw_header_fn() Improve determination of sizes in debug_info_alloc() Improve another size determination in debug_info_alloc()

 arch/s390/kernel/debug.c | 433 ++++++++++++++++++++++-------------------------
 1 file changed, 204 insertions(+), 229 deletions(-)
Signature · 1 line
2.9.3