| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 3 Sep 2010 22:32:19 +0900
From: "Masayuki Ohtake" <masa-korg@....okisemi.com>
To: "Stephen Hemminger" <shemminger@...tta.com>,
"Eric Dumazet" <eric.dumazet@...il.com>
Cc: "Sam Ravnborg" <sam@...nborg.org>, "Joe Perches" <joe@...ches.com>,
"LKML" <linux-kernel@...r.kernel.org>,
"ML netdev" <netdev@...r.kernel.org>,
"Greg Rose" <gregory.v.rose@...el.com>,
"Maxime Bizon" <mbizon@...ebox.fr>,
"Kristoffer Glembo" <kristoffer@...sler.com>,
"Ralf Baechle" <ralf@...ux-mips.org>,
"John Linn" <john.linn@...inx.com>,
"Randy Dunlap" <randy.dunlap@...cle.com>,
"David S. Miller" <davem@...emloft.net>,
"MeeGo" <meego-dev@...go.com>, "Wang, Qi" <qi.wang@...el.com>,
"Wang, Yong Y" <yong.y.wang@...el.com>,
"Andrew" <andrew.chih.howe.khor@...el.com>,
"Intel OTC" <joel.clark@...el.com>,
"Foster, Margie" <margie.foster@...el.com>,
"Toshiharu Okada" <okada533@....okisemi.com>,
"Tomoya Morinaga" <morinaga526@....okisemi.com>,
"Takahiro Shimizu" <shimizu394@....okisemi.com>
Subject: Re: [PATCH] Gigabit Ethernet driver of Topcliff PCH
Hi Eric and Stephen
Thank you for your suggestion.
I tried modification of "dev->hard_header_len".
However, there was no changed to a format of SKB received by kernel.
[Header:14octet] + [payload]
So, I can not a single memcpy.
>Something like:
> skb = skb_realloc_headroom(skb, 2);
> if (!skb)
> goto drop;
>__skb_push(skb, 2);
> memmove(skb->data, skb->data + 2, ETH_HLEN);
>
>skb->data[ETH_HLEN] = 0;
>skb->data[ETH_HLEN+1] = 0;
>
> if (!IS_ALIGNED(skb->data, 16)) {
> nskb = netdev_alloc_skb(dev, skb->len + 16);
> if (!nskb)
> goto drop2;
> skb_reserve(nskb, PTR_ALIGN(skb->data, 16) - skb->data);
> skb_put(skb, skb->len);
> memcpy(nskb->data, skb->data, skb->len);
> dev_kfree_skb(skb);
> skb = nskb;
> }
I am going to try modification using "memmove()" and "IS_ALIGNED()".
Thanks, Ohtake(OKISemi)
----- Original Message -----
From: "Stephen Hemminger" <shemminger@...tta.com>
To: "Eric Dumazet" <eric.dumazet@...il.com>
Cc: "Masayuki Ohtake" <masa-korg@....okisemi.com>; "Sam Ravnborg" <sam@...nborg.org>; "Joe Perches" <joe@...ches.com>;
"LKML" <linux-kernel@...r.kernel.org>; "ML netdev" <netdev@...r.kernel.org>; "Greg Rose" <gregory.v.rose@...el.com>;
"Maxime Bizon" <mbizon@...ebox.fr>; "Kristoffer Glembo" <kristoffer@...sler.com>; "Ralf Baechle" <ralf@...ux-mips.org>;
"John Linn" <john.linn@...inx.com>; "Randy Dunlap" <randy.dunlap@...cle.com>; "David S. Miller" <davem@...emloft.net>;
"MeeGo" <meego-dev@...go.com>; "Wang, Qi" <qi.wang@...el.com>; "Wang, Yong Y" <yong.y.wang@...el.com>; "Andrew"
<andrew.chih.howe.khor@...el.com>; "Intel OTC" <joel.clark@...el.com>; "Foster, Margie" <margie.foster@...el.com>;
"Toshiharu Okada" <okada533@....okisemi.com>; "Tomoya Morinaga" <morinaga526@....okisemi.com>; "Takahiro Shimizu"
<shimizu394@....okisemi.com>
Sent: Friday, September 03, 2010 12:10 AM
Subject: Re: [PATCH] Gigabit Ethernet driver of Topcliff PCH
On Thu, 02 Sep 2010 15:40:47 +0200
Eric Dumazet <eric.dumazet@...il.com> wrote:
> Le jeudi 02 septembre 2010 à 21:39 +0900, Masayuki Ohtake a écrit :
> > Hi Eric
> >
> > Thank you for your comments.
> >
> > > I find hard to believe this driver needs to copy all outgoing frames on
> > > pre-allocated skbs.
> > >
> > > + /* [Header:14][payload] ---> [Header:14][paddong:2][payload] */
> > > + memcpy(tmp_skb->data, skb->data, ETH_HLEN);
> > > + tmp_skb->data[ETH_HLEN] = 0x00;
> > > + tmp_skb->data[ETH_HLEN + 1] = 0x00;
> > > + tmp_skb->len = skb->len;
> > > + memcpy(&tmp_skb->data[ETH_HLEN + 2], &skb->data[ETH_HLEN],
> > > + (skb->len - ETH_HLEN));
> > > + buffer_info->kernel_skb = skb;
> > > + skb = tmp_skb;
> > >
> > > Whats the deal here please ?
> >
> > This processing depends on hardware specification.
> >
> > At the time of transmission.
> > Hardware accepts a packet in the following format.
> > [Header: 14octet] + [padding: 2octet] + [payload]
> > Also, it is necessary to align the head of a [Header: 14octet] at 64byte.
> >
> > In my knowledge, SKB received by kernel are the following format.
> > [padding: 2octet] + [Header:14octet] + [payload]
> > Also, The head of [payload] has aligned at 16 byte.
> >
> > So, it has adjusted with the format of hardware by a copy.
>
> The two bytes padding can be handled by network stack, if you
> force in your setup phase :
> dev->hard_header_len = ETH_HLEN + 2;
>
> then, you can do a single memcpy:
>
> memcpy(tmp_skb->data, skb->data, skb->len);
>
> About the 64 byte alignement, it might be a bit more complex :)
With Eric's suggestion, you might find most packets are going
to be aligned but the code should not depend on it always being true.
Packets that get forwarded have header determined by the receiving
interface.
Something like:
skb = skb_realloc_headroom(skb, 2);
if (!skb)
goto drop;
__skb_push(skb, 2);
memmove(skb->data, skb->data + 2, ETH_HLEN);
skb->data[ETH_HLEN] = 0;
skb->data[ETH_HLEN+1] = 0;
if (!IS_ALIGNED(skb->data, 16)) {
nskb = netdev_alloc_skb(dev, skb->len + 16);
if (!nskb)
goto drop2;
skb_reserve(nskb, PTR_ALIGN(skb->data, 16) - skb->data);
skb_put(skb, skb->len);
memcpy(nskb->data, skb->data, skb->len);
dev_kfree_skb(skb);
skb = nskb;
}
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists