Build check errors found (B.A.T.M.A.N. V, patch batv_5): 2015-11-24

postmaster at open-mesh.org postmaster at open-mesh.org
Tue Nov 24 12:00:23 CET 2015


Name of failed tests
====================

 * bracket_align ecsv/batv_5 ./net/batman-adv/bat_v_elp.c
 * checkpatch ecsv/batv_5 ./net/batman-adv/bat_v_elp.c
 * checkpatch ecsv/batv_5 ./net/batman-adv/bat_v_ogm.c
 * headers ecsv/batv_5
 * kerneldoc ecsv/batv_5 ./net/batman-adv/bat_v_ogm.c
 * sparse ecsv/batv_5 linux-3.16 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-3.17 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-3.18 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-3.19 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-4.0 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-4.1 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-4.2 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * sparse ecsv/batv_5 linux-4.3 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-3.16 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-3.17 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-3.18 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-3.19 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-4.0 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-4.1 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-4.2 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
 * unused_symbols ecsv/batv_5 linux-4.3 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y

Output of different failed tests
================================

bracket_align ecsv/batv_5 ./net/batman-adv/bat_v_elp.c
------------------------------------------------------

    Found wrong alignment at ./net/batman-adv/bat_v_elp.c:237, was 1 but expected 37
    Found wrong alignment at ./net/batman-adv/bat_v_elp.c:239, was 1 but expected 28
    Found wrong alignment at ./net/batman-adv/bat_v_elp.c:240, was 1 but expected 28


checkpatch ecsv/batv_5 ./net/batman-adv/bat_v_elp.c
---------------------------------------------------

    ERROR: do not use C99 // comments
    #235: FILE: ./net/batman-adv/bat_v_elp.c:235:
    +//	TODO: host restarted ?
    
    WARNING: please, no space before tabs
    #236: FILE: ./net/batman-adv/bat_v_elp.c:236:
    +// ^Iif (batadv_window_protected(bat_priv, seqno_diff,$
    
    ERROR: do not use C99 // comments
    #236: FILE: ./net/batman-adv/bat_v_elp.c:236:
    +// 	if (batadv_window_protected(bat_priv, seqno_diff,
    
    WARNING: please, no space before tabs
    #237: FILE: ./net/batman-adv/bat_v_elp.c:237:
    +// ^I^I^I^I    &neigh->batman_seqno_reset)) {$
    
    ERROR: do not use C99 // comments
    #237: FILE: ./net/batman-adv/bat_v_elp.c:237:
    +// 				    &neigh->batman_seqno_reset)) {
    
    WARNING: please, no space before tabs
    #238: FILE: ./net/batman-adv/bat_v_elp.c:238:
    +// ^I^Ibatadv_dbg(BATADV_DBG_BATMAN, bat_priv,$
    
    ERROR: do not use C99 // comments
    #238: FILE: ./net/batman-adv/bat_v_elp.c:238:
    +// 		batadv_dbg(BATADV_DBG_BATMAN, bat_priv,
    
    WARNING: line over 80 characters
    #239: FILE: ./net/batman-adv/bat_v_elp.c:239:
    +// 			   "Drop ELP packet: packet within window protection time from %pM\n",
    
    WARNING: please, no space before tabs
    #239: FILE: ./net/batman-adv/bat_v_elp.c:239:
    +// ^I^I^I   "Drop ELP packet: packet within window protection time from %pM\n",$
    
    ERROR: do not use C99 // comments
    #239: FILE: ./net/batman-adv/bat_v_elp.c:239:
    +// 			   "Drop ELP packet: packet within window protection time from %pM\n",
    
    WARNING: please, no space before tabs
    #240: FILE: ./net/batman-adv/bat_v_elp.c:240:
    +// ^I^I^I   elp_packet->orig);$
    
    ERROR: do not use C99 // comments
    #240: FILE: ./net/batman-adv/bat_v_elp.c:240:
    +// 			   elp_packet->orig);
    
    WARNING: please, no space before tabs
    #241: FILE: ./net/batman-adv/bat_v_elp.c:241:
    +// ^I^Igoto out;$
    
    ERROR: do not use C99 // comments
    #241: FILE: ./net/batman-adv/bat_v_elp.c:241:
    +// 		goto out;
    
    WARNING: please, no space before tabs
    #242: FILE: ./net/batman-adv/bat_v_elp.c:242:
    +// ^I}$
    
    ERROR: do not use C99 // comments
    #242: FILE: ./net/batman-adv/bat_v_elp.c:242:
    +// 	}
    
    total: 8 errors, 8 warnings, 0 checks, 313 lines checked


checkpatch ecsv/batv_5 ./net/batman-adv/bat_v_ogm.c
---------------------------------------------------

    WARNING: line over 80 characters
    #266: FILE: ./net/batman-adv/bat_v_ogm.c:266:
    +	/* TODO: also switch route if this seqno is X newer than the last received seqno from our best next hop */
    
    WARNING: line over 80 characters
    #271: FILE: ./net/batman-adv/bat_v_ogm.c:271:
    +		    router_ifinfo->bat_v.throughput >= neigh_ifinfo->bat_v.throughput)
    
    total: 0 errors, 2 warnings, 0 checks, 669 lines checked


headers ecsv/batv_5
-------------------

    diff --git a/net/batman-adv/bat_v_ogm.c b/net/batman-adv/bat_v_ogm.c
    index e7ad0e3..dbd4049 100644
    --- a/net/batman-adv/bat_v_ogm.c
    +++ b/net/batman-adv/bat_v_ogm.c
    @@ -26,7 +26,6 @@
     #include <linux/if_ether.h>
     #include <linux/jiffies.h>
     #include <linux/kernel.h>
    -#include <linux/list.h>
     #include <linux/netdevice.h>
     #include <linux/random.h>
     #include <linux/rculist.h>


kerneldoc ecsv/batv_5 ./net/batman-adv/bat_v_ogm.c
--------------------------------------------------

    ./net/batman-adv/bat_v_ogm.c:392: warning: No description found for parameter 'bat_priv'
    ./net/batman-adv/bat_v_ogm.c:392: warning: No description found for parameter 'ethhdr'
    ./net/batman-adv/bat_v_ogm.c:392: warning: No description found for parameter 'ogm2'
    ./net/batman-adv/bat_v_ogm.c:392: warning: No description found for parameter 'orig_node'
    ./net/batman-adv/bat_v_ogm.c:392: warning: No description found for parameter 'neigh_node'
    ./net/batman-adv/bat_v_ogm.c:392: warning: Excess function parameter 'skb' description in 'batadv_v_ogm_process_per_outif'


sparse ecsv/batv_5 linux-4.3 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
---------------------------------------------------------------------------

    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:411:38: warning: incorrect type in argument 3 (different base types)
    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:411:38:    expected signed int [signed] [usertype] [explicitly-signed] seq_old_max_diff
    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:411:38:    got unsigned long *<noident>
    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:410:36: error: not enough arguments for function batadv_window_protected
    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c: In function 'batadv_v_ogm_process_per_outif':
    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:411:9: warning: passing argument 3 of 'batadv_window_protected' makes integer from pointer without a cast
             &orig_ifinfo->batman_seqno_reset)) {
             ^
    In file included from /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:45:0:
    /home/build_test/build_env/tmp/net/batman-adv/routing.h:54:5: note: expected 's32' but argument is of type 'long unsigned int *'
     int batadv_window_protected(struct batadv_priv *bat_priv, s32 seq_num_diff,
         ^
    /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:410:6: error: too few arguments to function 'batadv_window_protected'
          batadv_window_protected(bat_priv, seq_diff,
          ^
    In file included from /home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.c:45:0:
    /home/build_test/build_env/tmp/net/batman-adv/routing.h:54:5: note: declared here
     int batadv_window_protected(struct batadv_priv *bat_priv, s32 seq_num_diff,
         ^
    make[2]: *** [/home/build_test/build_env/tmp/net/batman-adv/bat_v_ogm.o] Error 1
    make[1]: *** [_module_/home/build_test/build_env/tmp/net/batman-adv] Error 2
    make: *** [all] Error 2


unused_symbols ecsv/batv_5 linux-4.3 cfg: BLA=n DAT=n DEBUG=n NC=y MCAST=n BATMAN_V=y
---------------------------------------------------------------------------

    batadv_iv_init
    batadv_v_init
    batadv_v_mesh_free
    batadv_v_mesh_init


More information about the linux-merge mailing list