From: Sven Eckelmann <sven@narfation.org>
To: b.a.t.m.a.n@lists.open-mesh.org
Subject: Re: [B.A.T.M.A.N.] [PATCHv4 00/12] netns and netlink support
Date: Tue, 10 May 2016 11:25:42 +0200 [thread overview]
Message-ID: <1919523.EiXNAC6LDj@bentobox> (raw)
In-Reply-To: <12399409.fEQLp0ttQ1@bentobox>
[-- Attachment #1: Type: text/plain, Size: 1599 bytes --]
On Tuesday 10 May 2016 09:00:49 Sven Eckelmann wrote:
> On Monday 09 May 2016 21:06:08 Andrew Lunn wrote:
> > > I hope this helps to reduce the conflicts (even when not resolving it
> > > completely) between the tp_meter and the netlink patchset.
> >
> > Hi Sven
> >
> > Do you want to take over my patches and merge them all into one set?
>
> No, this is the last thing I want to do :)
>
> I only want to get the "big" conflicts solved and mark the parts which most
> likely will needs to be changed before Marek will accept them.
>
> I will have not much time in the next days to do any private work on it. So it
> most likely not the best idea to give me the patchset with a lot of TODOs in
> it ;)
>
> I will try to poke Antonio and Marek some more so we get a common starting
> point (the two patches from batctl/batman-adv) which we then can use prepare
> our patchsets with a lot less conflicts.
Some reports which I forgot to mention yesterday and which I've just remembered:
bracket_align master ./net/batman-adv/bat_iv_ogm.c
--------------------------------------------------
Found wrong alignment at ./net/batman-adv/bat_iv_ogm.c:2371, was 37 but expected 61
Found wrong alignment at ./net/batman-adv/bat_iv_ogm.c:2372, was 37 but expected 61
checkpatch master ./net/batman-adv/bat_v.c
------------------------------------------
CHECK: Please don't use multiple blank lines
#450: FILE: ./net/batman-adv/bat_v.c:450:
+
+
total: 0 errors, 0 warnings, 1 checks, 732 lines checked
Kind regards,
Sven
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
prev parent reply other threads:[~2016-05-10 9:25 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-09 18:07 [B.A.T.M.A.N.] [PATCHv4 00/12] netns and netlink support Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 01/12] batman-adv: Handle parent interfaces in a different netns Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 02/12] batman-adv: compat.h: Add workaround for get_link_net() Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 03/12] batman-adv: Suppress debugfs entries for netns's Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 04/12] batman-adv: add generic netlink query API to replace debugfs files Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 05/12] batman-adv: compat: Workarounds for previous patch Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 06/12] batman-adv: netlink: add translation table query Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 07/12] batman-adv: netlink: add originator and neighbor table queries Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 08/12] batman-adv: add B.A.T.M.A.N. IV bat_{orig, neigh}_dump implementations Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 09/12] batman-adv: add B.A.T.M.A.N. V " Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 10/12] batman-adv: Indicate netlink socket can be used with netns Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 11/12] batman-adv: add B.A.T.M.A.N. Dump gateways via netlink Andrew Lunn
2016-05-09 18:07 ` [B.A.T.M.A.N.] [PATCHv4 12/12] batman-adv: add B.A.T.M.A.N. Dump BLA claims " Andrew Lunn
2016-05-09 18:29 ` [B.A.T.M.A.N.] [PATCHv4 00/12] netns and netlink support Sven Eckelmann
2016-05-09 18:50 ` Sven Eckelmann
2016-05-09 19:06 ` Andrew Lunn
2016-05-10 7:00 ` Sven Eckelmann
2016-05-10 9:25 ` Sven Eckelmann [this message]
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=1919523.EiXNAC6LDj@bentobox \
--to=sven@narfation.org \
--cc=b.a.t.m.a.n@lists.open-mesh.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox