From: Marek Lindner <mareklindner@neomailbox.ch>
To: b.a.t.m.a.n@lists.open-mesh.org
Subject: Re: [B.A.T.M.A.N.] [PATCHv4 1/4] batman-adv: Remove explicit compat.h include and split compat.h
Date: Tue, 21 Apr 2015 11:41:48 +0800 [thread overview]
Message-ID: <1834679.g60VTMDq3l@voltaire> (raw)
In-Reply-To: <1429292430-30883-1-git-send-email-sven@narfation.org>
[-- Attachment #1: Type: text/plain, Size: 1054 bytes --]
On Friday, April 17, 2015 19:40:27 Sven Eckelmann wrote:
> The current approach for the compat layer is to include the compat.h file at
> exactly the right time to provide additional functionality and to live
> patch the batman-adv sources. This has the problem that the compat.h file
> becomes harder to read over time. Also live patching and adding of missing
> functionality partially conflict. This becomes obvious when the include
> order in some files is different compared to the one the author of an
> compat.h entry expected.
>
> A different approach is to inject intermediate header files which add
> additional features. This allows to reduce the size of compat.h and only
> leaves the live patching part in it. The compat.h can then added
> automatically to each compile run before anything else is included.
> compat.h has therefore the control which headers must be included before
> the live patching of the batman-adv source can begin.
>
> Signed-off-by: Sven Eckelmann <sven@narfation.org>
Applied in revision 8746f0f.
Thanks,
Marek
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
next prev parent reply other threads:[~2015-04-21 3:41 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-17 17:40 [B.A.T.M.A.N.] [PATCHv4 1/4] batman-adv: Remove explicit compat.h include and split compat.h Sven Eckelmann
2015-04-17 17:40 ` [B.A.T.M.A.N.] [PATCHv4 2/4] batman-adv: Add required to includes to all files Sven Eckelmann
2015-04-20 16:25 ` Sven Eckelmann
2015-04-21 3:46 ` Marek Lindner
2015-04-17 17:40 ` [B.A.T.M.A.N.] [PATCHv4 3/4] batman-adv: Remove unused IFF_BRIDGE_PORT live patching hack Sven Eckelmann
2015-04-21 3:48 ` Marek Lindner
2015-04-21 6:04 ` Sven Eckelmann
2015-04-21 8:53 ` Marek Lindner
2015-04-17 17:40 ` [B.A.T.M.A.N.] [PATCHv4 4/4] batman-adv: Automatically create nested kfree_rcu helper functions Sven Eckelmann
2015-04-21 3:52 ` Marek Lindner
2015-04-21 6:35 ` Sven Eckelmann
2015-04-21 9:01 ` Marek Lindner
2015-04-20 16:12 ` [B.A.T.M.A.N.] [PATCHv5 1/4] batman-adv: Remove explicit compat.h include and split compat.h Sven Eckelmann
2015-04-21 3:41 ` Marek Lindner [this message]
2015-04-21 6:19 ` [B.A.T.M.A.N.] [PATCHv4 " Sven Eckelmann
2015-04-21 9:02 ` 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=1834679.g60VTMDq3l@voltaire \
--to=mareklindner@neomailbox.ch \
--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 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.