Build check errors found: 2016-04-01
by postmaster＠open-mesh.org
Name of failed tests
====================
* checkpatch master ./net/batman-adv/fragmentation.c
* checkpatch next ./net/batman-adv/fragmentation.c
* difference between net-next and batadv next
Output of different failed tests
================================
checkpatch master ./net/batman-adv/fragmentation.c
--------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 1 warnings, 0 checks, 516 lines checked
checkpatch next ./net/batman-adv/fragmentation.c
------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#410: FILE: ./net/batman-adv/fragmentation.c:410:
+ unsigned header_size = sizeof(*frag_head);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#411: FILE: ./net/batman-adv/fragmentation.c:411:
+ unsigned fragment_size = mtu - header_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#447: FILE: ./net/batman-adv/fragmentation.c:447:
+ unsigned mtu = neigh_node->if_incoming->net_dev->mtu;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#448: FILE: ./net/batman-adv/fragmentation.c:448:
+ unsigned header_size = sizeof(frag_header);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#449: FILE: ./net/batman-adv/fragmentation.c:449:
+ unsigned max_fragment_size, max_packet_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 6 warnings, 0 checks, 516 lines checked
difference between net-next and batadv next
-------------------------------------------
net/batman-adv/bat_iv_ogm.c | 9 +++++----
net/batman-adv/hard-interface.c | 3 ++-
net/batman-adv/originator.c | 1 -
net/batman-adv/routing.c | 9 ---------
net/batman-adv/soft-interface.c | 8 ++------
5 files changed, 9 insertions(+), 21 deletions(-)
6 years, 10 months
Noticed difference in packet.h in maint.diff
by postmaster＠open-mesh.org
--- batman-adv_maint.packet.h 2016-04-01 00:05:01.916185203 +0200
+++ batctl_maint.packet.h 2016-04-01 00:05:01.924185706 +0200
@@ -1,4 +1,4 @@
-/* Copyright (C) 2007-2015 B.A.T.M.A.N. contributors:
+/* Copyright (C) 2007-2016 B.A.T.M.A.N. contributors:
*
* Marek Lindner, Simon Wunderlich
*
6 years, 10 months
Build check errors found: 2016-03-31
by postmaster＠open-mesh.org
Name of failed tests
====================
* checkpatch master ./net/batman-adv/fragmentation.c
* checkpatch next ./net/batman-adv/fragmentation.c
* difference between net-next and batadv next
Output of different failed tests
================================
checkpatch master ./net/batman-adv/fragmentation.c
--------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 1 warnings, 0 checks, 516 lines checked
checkpatch next ./net/batman-adv/fragmentation.c
------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#410: FILE: ./net/batman-adv/fragmentation.c:410:
+ unsigned header_size = sizeof(*frag_head);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#411: FILE: ./net/batman-adv/fragmentation.c:411:
+ unsigned fragment_size = mtu - header_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#447: FILE: ./net/batman-adv/fragmentation.c:447:
+ unsigned mtu = neigh_node->if_incoming->net_dev->mtu;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#448: FILE: ./net/batman-adv/fragmentation.c:448:
+ unsigned header_size = sizeof(frag_header);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#449: FILE: ./net/batman-adv/fragmentation.c:449:
+ unsigned max_fragment_size, max_packet_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 6 warnings, 0 checks, 516 lines checked
difference between net-next and batadv next
-------------------------------------------
net/batman-adv/bat_iv_ogm.c | 9 +++++----
net/batman-adv/hard-interface.c | 3 ++-
net/batman-adv/originator.c | 1 -
net/batman-adv/routing.c | 9 ---------
net/batman-adv/soft-interface.c | 8 ++------
5 files changed, 9 insertions(+), 21 deletions(-)
6 years, 10 months
Noticed difference in packet.h in maint.diff
by postmaster＠open-mesh.org
--- batman-adv_maint.packet.h 2016-03-31 00:05:01.501498719 +0200
+++ batctl_maint.packet.h 2016-03-31 00:05:01.509499254 +0200
@@ -1,4 +1,4 @@
-/* Copyright (C) 2007-2015 B.A.T.M.A.N. contributors:
+/* Copyright (C) 2007-2016 B.A.T.M.A.N. contributors:
*
* Marek Lindner, Simon Wunderlich
*
6 years, 10 months
Build check errors found: 2016-03-30
by postmaster＠open-mesh.org
Name of failed tests
====================
* checkpatch master ./net/batman-adv/fragmentation.c
* checkpatch next ./net/batman-adv/fragmentation.c
* difference between net-next and batadv next
Output of different failed tests
================================
checkpatch master ./net/batman-adv/fragmentation.c
--------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 1 warnings, 0 checks, 516 lines checked
checkpatch next ./net/batman-adv/fragmentation.c
------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#410: FILE: ./net/batman-adv/fragmentation.c:410:
+ unsigned header_size = sizeof(*frag_head);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#411: FILE: ./net/batman-adv/fragmentation.c:411:
+ unsigned fragment_size = mtu - header_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#447: FILE: ./net/batman-adv/fragmentation.c:447:
+ unsigned mtu = neigh_node->if_incoming->net_dev->mtu;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#448: FILE: ./net/batman-adv/fragmentation.c:448:
+ unsigned header_size = sizeof(frag_header);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#449: FILE: ./net/batman-adv/fragmentation.c:449:
+ unsigned max_fragment_size, max_packet_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 6 warnings, 0 checks, 516 lines checked
difference between net-next and batadv next
-------------------------------------------
net/batman-adv/bat_iv_ogm.c | 9 +++++----
net/batman-adv/hard-interface.c | 3 ++-
net/batman-adv/originator.c | 1 -
net/batman-adv/routing.c | 9 ---------
net/batman-adv/soft-interface.c | 8 ++------
5 files changed, 9 insertions(+), 21 deletions(-)
6 years, 10 months
Noticed difference in packet.h in maint.diff
by postmaster＠open-mesh.org
--- batman-adv_maint.packet.h 2016-03-30 00:05:01.316163718 +0200
+++ batctl_maint.packet.h 2016-03-30 00:05:01.324164252 +0200
@@ -1,4 +1,4 @@
-/* Copyright (C) 2007-2015 B.A.T.M.A.N. contributors:
+/* Copyright (C) 2007-2016 B.A.T.M.A.N. contributors:
*
* Marek Lindner, Simon Wunderlich
*
6 years, 10 months
Build check errors found: 2016-03-29
by postmaster＠open-mesh.org
Name of failed tests
====================
* checkpatch master ./net/batman-adv/fragmentation.c
* checkpatch next ./net/batman-adv/fragmentation.c
* difference between net-next and batadv next
Output of different failed tests
================================
checkpatch master ./net/batman-adv/fragmentation.c
--------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 1 warnings, 0 checks, 516 lines checked
checkpatch next ./net/batman-adv/fragmentation.c
------------------------------------------------
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#410: FILE: ./net/batman-adv/fragmentation.c:410:
+ unsigned header_size = sizeof(*frag_head);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#411: FILE: ./net/batman-adv/fragmentation.c:411:
+ unsigned fragment_size = mtu - header_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#447: FILE: ./net/batman-adv/fragmentation.c:447:
+ unsigned mtu = neigh_node->if_incoming->net_dev->mtu;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#448: FILE: ./net/batman-adv/fragmentation.c:448:
+ unsigned header_size = sizeof(frag_header);
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#449: FILE: ./net/batman-adv/fragmentation.c:449:
+ unsigned max_fragment_size, max_packet_size;
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
#455: FILE: ./net/batman-adv/fragmentation.c:455:
+ mtu = min_t(unsigned, mtu, BATADV_FRAG_MAX_FRAG_SIZE);
total: 0 errors, 6 warnings, 0 checks, 516 lines checked
difference between net-next and batadv next
-------------------------------------------
net/batman-adv/bat_iv_ogm.c | 9 +++++----
net/batman-adv/hard-interface.c | 3 ++-
net/batman-adv/originator.c | 1 -
net/batman-adv/routing.c | 9 ---------
net/batman-adv/soft-interface.c | 8 ++------
5 files changed, 9 insertions(+), 21 deletions(-)
6 years, 10 months
Noticed difference in packet.h in master.diff
by postmaster＠open-mesh.org
--- batman-adv_master.packet.h 2016-03-29 00:05:01.121091676 +0200
+++ batctl_master.packet.h 2016-03-29 00:05:01.129092211 +0200
@@ -501,7 +501,7 @@
#pragma pack()
/**
- * struct batadv_unicast_tvlv_packet - generic unicast packet with tvlv payload
+ * struct batadv_unicast_tvlv - generic unicast packet with tvlv payload
* @packet_type: batman-adv packet type, part of the general header
* @version: batman-adv protocol version, part of the genereal header
* @ttl: time to live for this packet, part of the genereal header
6 years, 10 months
Noticed difference in packet.h in maint.diff
by postmaster＠open-mesh.org
--- batman-adv_maint.packet.h 2016-03-29 00:05:01.125091943 +0200
+++ batctl_maint.packet.h 2016-03-29 00:05:01.161094347 +0200
@@ -1,4 +1,4 @@
-/* Copyright (C) 2007-2015 B.A.T.M.A.N. contributors:
+/* Copyright (C) 2007-2016 B.A.T.M.A.N. contributors:
*
* Marek Lindner, Simon Wunderlich
*
6 years, 10 months
