From: Vladimir Oltean <email@example.com> To: Christian Eggers <firstname.lastname@example.org> Cc: Andrew Lunn <email@example.com>, Vivien Didelot <firstname.lastname@example.org>, Florian Fainelli <email@example.com>, Jakub Kicinski <firstname.lastname@example.org>, Kurt Kanzenbach <email@example.com>, "David S . Miller" <firstname.lastname@example.org>, Woojung Huh <email@example.com>, Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>, firstname.lastname@example.org, email@example.com Subject: Re: [PATCH net-next 1/3] net: dsa: don't pass cloned skb's to drivers xmit function Date: Sat, 17 Oct 2020 22:12:47 +0300 [thread overview] Message-ID: <20201017191247.ohslc77wkhbhffym@skbuf> (raw) In-Reply-To: <2130539.dlFve3NVyK@n95hx1g2> On Sat, Oct 17, 2020 at 08:53:19PM +0200, Christian Eggers wrote: > > Does 1588 work for you using this change, or you haven't finished > > implementing it yet? If you haven't, I would suggest finishing that > > part first. > Yes it does. Just after finishing this topic, I would to sent the patches for > PTP. Maybe I'll do it in parallel, anything but the combination of L2/E2E/SLOB > seems to work. 2 aspects: - net-next is closed for this week and the next one, due to the merge window. You'll have to wait until it reopens. - Actually I was asking you this because sja1105 PTP no longer works after this change, due to the change of txflags. > I don't like to touch the non-tail taggers, this is too much out of the scope > of my current work. Do you want me to try and send a version using pskb_expand_head and you can test if it works for your tail-tagging switch? > > Also, if the result is going to be longer than ~20 lines of code, I > > strongly suggest moving the reallocation to a separate function so you > > don't clutter dsa_slave_xmit. > As Florian requested I'll likely put the code into a separate function in > slave.c and call it from the individual tail-taggers in order not to put > extra conditionals in dsa_slave_xmit. I think it would be best to use the unlikely(tail_tag) approach though. The reallocation function should still be in the common code path. Even for a non-1588 switch, there are other code paths that clone packets on TX. For example, the bridge does that, when flooding packets. Currently, DSA ensures that the header area is writable by calling skb_cow_head, as far as I can see. But the point is, maybe we can do TX reallocation centrally.
next prev parent reply other threads:[~2020-10-17 19:12 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-10-16 20:02 [PATCH 0/3] net: dsa: move skb reallocation to dsa_slave_xmit Christian Eggers 2020-10-16 20:02 ` [PATCH net-next 1/3] net: dsa: don't pass cloned skb's to drivers xmit function Christian Eggers 2020-10-17 0:48 ` Vladimir Oltean 2020-10-17 18:53 ` Christian Eggers 2020-10-17 19:12 ` Vladimir Oltean [this message] 2020-10-17 20:56 ` Christian Eggers 2020-10-17 21:35 ` Vladimir Oltean 2020-10-17 2:35 ` Florian Fainelli 2020-10-16 20:02 ` [PATCH net-next 2/3] net: dsa: tag_ksz: don't allocate additional memory for padding/tagging Christian Eggers 2020-10-16 20:02 ` [PATCH net-next 3/3] net: dsa: trailer: " Christian Eggers 2020-10-17 2:44 ` [PATCH 0/3] net: dsa: move skb reallocation to dsa_slave_xmit Florian Fainelli
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=20201017191247.ohslc77wkhbhffym@skbuf \ --firstname.lastname@example.org \ --cc=UNGLinuxDriver@microchip.com \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --subject='Re: [PATCH net-next 1/3] net: dsa: don'\''t pass cloned skb'\''s to drivers xmit function' \ /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
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).