All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sven Eckelmann <sven@narfation.org>
To: b.a.t.m.a.n@lists.open-mesh.org
Cc: Antonio Quartulli <a@unstable.cc>, mareklindner@neomailbox.ch
Subject: Re: [B.A.T.M.A.N.] [PATCH 1/7] batman-adv: remove useless inline attribute for sysfs helper function
Date: Thu, 05 May 2016 22:17:18 +0200	[thread overview]
Message-ID: <2675515.pbCZCtK98O@sven-edge> (raw)
In-Reply-To: <1462474003-25279-1-git-send-email-a@unstable.cc>


[-- Attachment #1.1: Type: text/plain, Size: 10298 bytes --]

=2D-nextPart14925138.mhnyUsvyfy
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"

On Friday 06 May 2016 02:46:37 Antonio Quartulli wrote:
> the compiler can optimize functions within the same C file and therefore
> there is no need to make it explicit.
> 
> Remove the useless inline attribute for __batadv_store_uint_attr()
> 
> Signed-off-by: Antonio Quartulli <a@unstable.cc>
> ---
>  net/batman-adv/sysfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)

This is not about this particular patch but for the patch series.

Please check the attached test results of the daily build tests.

Kind regards,
	Sve
=2D-nextPart14925138.mhnyUsvyfy
Content-Disposition: attachment; filename="gw_per_algo_test.mbox"
Content-Transfer-Encoding: 7Bit
Content-Type: application/mbox; name="gw_per_algo_test.mbox"

Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Build check errors found: 2016-05-05
From: postmaster@open-mesh.org
Date: Thu,  5 May 2016 21:59:37 +0200 (CEST)

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

 * headers master
 * kerneldoc master ./net/batman-adv/types.h
 * unused_symbols master linux-3.10 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.10 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.10 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.10 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.11 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.11 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.11 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.11 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.12 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.12 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.12 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.12 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.13 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.13 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.13 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.13 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.14 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.14 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.14 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.14 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.15 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.15 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.15 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.15 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.16 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.16 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.16 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.16 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.17 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.17 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.17 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.17 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.18 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.18 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.18 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.18 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.19 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.19 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.19 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.19 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.2 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.2 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.2 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.2 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.3 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.3 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.3 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.3 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.4 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.4 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.4 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.4 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.5 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.5 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.5 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.5 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.6 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.6 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.6 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.6 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.7 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.7 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.7 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.7 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.8 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.8 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.8 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.8 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.9 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.9 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-3.9 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-3.9 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.0 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.0 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.0 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.0 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.1 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.1 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.1 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.1 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.2 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.2 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.2 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.2 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.3 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.3 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.3 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.3 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.4 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.4 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.4 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.4 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.5 cfg: BLA=n DAT=y DEBUG=n NC=n MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.5 cfg: BLA=n DAT=y DEBUG=n NC=y MCAST=y BATMAN_V=n
 * unused_symbols master linux-4.5 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
 * unused_symbols master linux-4.5 cfg: BLA=y DAT=y DEBUG=n NC=n MCAST=n BATMAN_V=n

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

headers master
=2D-------------

    diff --git a/net/batman-adv/bat_v.c b/net/batman-adv/bat_v.c
    index 02359d0..cb09ae5 100644
    --- a/net/batman-adv/bat_v.c
    +++ b/net/batman-adv/bat_v.c
    @@ -19,6 +19,10 @@
     #include "main.h" // IWYU pragma: keep
     
     #include <linux/atomic.h>
    +#include <linux/errno.h>
    +#include <linux/kernel.h>
    +#include <linux/kref.h>
    +#include <linux/stddef.h>
     #include <linux/bug.h>
     #include <linux/cache.h>
     #include <linux/init.h>


kerneldoc master ./net/batman-adv/types.h
=2D----------------------------------------

    ./net/batman-adv/types.h:1338: warning: No description found for parameter 'gw'


unused_symbols master linux-4.5 cfg: BLA=n DAT=y DEBUG=y NC=n MCAST=n BATMAN_V=n
=2D--------------------------------------------------------------------------

    batadv_gw_node_get


=2D-nextPart14925138.mhnyUsvyfy--
This is a multi-part message in MIME format.

[-- Attachment #1.2: Type: text/plain, Size: 550 bytes --]

On Friday 06 May 2016 02:46:37 Antonio Quartulli wrote:
> the compiler can optimize functions within the same C file and therefore
> there is no need to make it explicit.
> 
> Remove the useless inline attribute for __batadv_store_uint_attr()
> 
> Signed-off-by: Antonio Quartulli <a@unstable.cc>
> ---
>  net/batman-adv/sysfs.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)

This is not about this particular patch but for the patch series.

Please check the attached test results of the daily build tests.

Kind regards,
	Sve

[-- Attachment #1.3: gw_per_algo_test.mbox --]
[-- Type: application/mbox, Size: 9359 bytes --]

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  parent reply	other threads:[~2016-05-05 20:17 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-05 18:46 [B.A.T.M.A.N.] [PATCH 1/7] batman-adv: remove useless inline attribute for sysfs helper function Antonio Quartulli
2016-05-05 18:46 ` [B.A.T.M.A.N.] [PATCH 2/7] batman-adv: move GW mode and selection class to private data structure Antonio Quartulli
2016-05-06  1:06   ` Marek Lindner
2016-05-05 18:46 ` [B.A.T.M.A.N.] [PATCH 3/7] batman-adv: make the GW selection class algorithm specific Antonio Quartulli
2016-05-05 18:46 ` [B.A.T.M.A.N.] [PATCH 4/7] batman-adv: split routing API data structure in subobjects Antonio Quartulli
2016-05-06  1:10   ` Marek Lindner
2016-05-06  1:20   ` Marek Lindner
2016-05-05 18:46 ` [B.A.T.M.A.N.] [PATCH 5/7] batman-adv: statically print MAX TQ value in gateway table header Antonio Quartulli
2016-05-05 18:46 ` [B.A.T.M.A.N.] [PATCH 6/7] batman-adv: make GW election code protocol specific Antonio Quartulli
2016-05-05 18:46 ` [B.A.T.M.A.N.] [PATCH 7/7] batman-adv: B.A.T.M.A.N. V - implement GW selection logic Antonio Quartulli
2016-05-05 20:17 ` Sven Eckelmann [this message]
2016-05-06  2:34   ` [B.A.T.M.A.N.] [PATCH 1/7] batman-adv: remove useless inline attribute for sysfs helper function Antonio Quartulli
2016-05-06  7:23     ` Sven Eckelmann
2016-05-06  7:53       ` Sven Eckelmann
2016-05-08 16:33         ` Antonio Quartulli
2016-05-08 16:59           ` Sven Eckelmann
2016-05-15  9:01             ` Sven Eckelmann
2016-05-06  1:03 ` Marek Lindner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2675515.pbCZCtK98O@sven-edge \
    --to=sven@narfation.org \
    --cc=a@unstable.cc \
    --cc=b.a.t.m.a.n@lists.open-mesh.org \
    --cc=mareklindner@neomailbox.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.