[RFC,5/5] batman-adv: ELP - add configurable minimum ELP packet length (def: 300B)
Commit Message
From: Linus Luessing <linus.luessing@web.de>
With the current setting of no minimum ELP packet length, BATMAN very
often choses too short paths. These paths are often only usable for small
packets, but not for e.g. larger TCP downloads.
Therefore this commit introduces a minimum ELP packet length. If the ELP
packet size is smaller than the specified minimum size the packet will be
padded with zeroes up to this specified size.
Developed by Linus during a 6 months trainee study period in Ascom
(Switzerland) AG.
Signed-off-by: Linus Luessing <linus.luessing@web.de>
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
---
bat_sysfs.c | 2 ++
bat_v_elp.c | 9 ++++++++-
hard-interface.c | 1 +
sysfs-class-net-batman-adv | 8 +++++++-
types.h | 1 +
5 files changed, 19 insertions(+), 2 deletions(-)
Comments
> + if (skb->len < min_len) {
> + memset(elp_neigh_entry, 0, min_len - skb->len);
> + skb_put(skb, min_len - skb->len);
> + }
> +
Hi Marek
I don't know the skbuf code too well, maybe you know it better than
me.
Is this memset needed? The skb is a copy of the template ELP packet
created then the soft interface is created. So if the contents of the
template is set to 0, should we be able to skip this zeroing here?
Thanks
Andrew
On Saturday, March 24, 2012 22:39:45 Andrew Lunn wrote:
> > + if (skb->len < min_len) {
> > + memset(elp_neigh_entry, 0, min_len - skb->len);
> > + skb_put(skb, min_len - skb->len);
> > + }
> > +
>
> I don't know the skbuf code too well, maybe you know it better than
> me.
>
> Is this memset needed? The skb is a copy of the template ELP packet
> created then the soft interface is created. So if the contents of the
> template is set to 0, should we be able to skip this zeroing here?
skb_copy() does not initialize the skb with zeros as far as I can tell.
bat_v_elp_iface_enable() only writes zeros into the ELP header
(BATMAN_ELP_HLEN), so zeroing the rest of the skb seems correct.
We could zero the entire skb in bat_v_elp_iface_enable() to save the memset.
Regards,
Marek
@@ -632,6 +632,7 @@ static BAT_ATTR(mesh_iface, S_IRUGO | S_IWUSR,
static BAT_ATTR(iface_status, S_IRUGO, show_iface_status, NULL);
#ifdef CONFIG_BATMAN_ADV_BATMAN_V
BAT_ATTR_HIF_UINT(elp_interval, S_IRUGO | S_IWUSR, 2 * JITTER, INT_MAX, NULL);
+BAT_ATTR_HIF_UINT(elp_min_len, S_IRUGO | S_IWUSR, 0, ETH_DATA_LEN, NULL);
#endif
static struct bat_attribute *batman_attrs[] = {
@@ -639,6 +640,7 @@ static struct bat_attribute *batman_attrs[] = {
&bat_attr_iface_status,
#ifdef CONFIG_BATMAN_ADV_BATMAN_V
&bat_attr_elp_interval,
+ &bat_attr_elp_min_len,
#endif
NULL,
};
@@ -108,7 +108,7 @@ static void bat_v_elp_send_outstanding(struct work_struct *work)
struct neigh_node *neigh_node;
struct hlist_node *node;
struct sk_buff *skb;
- unsigned int max_len;
+ unsigned int max_len, min_len;
hard_iface = container_of(work, struct hard_iface, elp_wq.work);
bat_priv = netdev_priv(hard_iface->soft_iface);
@@ -126,6 +126,8 @@ static void bat_v_elp_send_outstanding(struct work_struct *work)
goto restart_timer;
max_len = min_t(unsigned int, ETH_DATA_LEN, hard_iface->net_dev->mtu);
+ min_len = min_t(unsigned int, atomic_read(&hard_iface->elp_min_len),
+ max_len);
skb = skb_copy(hard_iface->elp_skb, GFP_ATOMIC);
if (!skb)
@@ -159,6 +161,11 @@ static void bat_v_elp_send_outstanding(struct work_struct *work)
}
rcu_read_unlock();
+ if (skb->len < min_len) {
+ memset(elp_neigh_entry, 0, min_len - skb->len);
+ skb_put(skb, min_len - skb->len);
+ }
+
bat_dbg(DBG_BATMAN, bat_priv,
"Sending elp packet on interface %s, seqno %d\n",
hard_iface->net_dev->name, atomic_read(&hard_iface->elp_seqno));
@@ -452,6 +452,7 @@ static struct hard_iface *hardif_add_interface(struct net_device *net_dev)
#ifdef CONFIG_BATMAN_ADV_BATMAN_V
atomic_set(&hard_iface->elp_interval, 500);
+ atomic_set(&hard_iface->elp_min_len, 300);
atomic_set(&hard_iface->elp_seqno, 1);
hard_iface->elp_skb = NULL;
#endif /* CONFIG_BATMAN_ADV_BATMAN_V */
@@ -18,4 +18,10 @@ Date: Mar 2012
Contact: Linus Lüssing <linus.luessing@web.de>
Description:
Defines the interval in milliseconds in which batman
- sends its probing packets for link quality measurements.
\ No newline at end of file
+ sends its probing packets for link quality measurements.
+
+What: /sys/class/net/<mesh_iface>/batman-adv/elp_min_len
+Date: Mar 2012
+Contact: Linus Lüssing <linus.luessing@web.de>
+Description:
+ Defines the minimum ELP packet size.
@@ -60,6 +60,7 @@ struct hard_iface {
struct rcu_head rcu;
#ifdef CONFIG_BATMAN_ADV_BATMAN_V
atomic_t elp_interval;
+ atomic_t elp_min_len;
atomic_t elp_seqno;
struct hlist_head neigh_list;
spinlock_t neigh_list_lock;