linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Eric Dumazet <eric.dumazet@gmail.com>
Cc: artem.bityutskiy@linux.intel.com,
	Eric Dumazet <edumazet@google.com>,
	linux-wireless@vger.kernel.org
Subject: Re: regression: tethering fails in 3.5 with iwlwifi
Date: Thu, 20 Sep 2012 17:22:55 +0200	[thread overview]
Message-ID: <1348154575.4161.39.camel@jlt4.sipsolutions.net> (raw)
In-Reply-To: <1348154294.31352.165.camel@edumazet-glaptop> (sfid-20120920_171820_134642_A4BA7EB1)

On Thu, 2012-09-20 at 17:18 +0200, Eric Dumazet wrote:

> > Note I think the failing packets are dhcp packets, and connman seems to
> > use an AF_INET, SOCK_DGRAM, IPPROTO_UDP socket for those, and binds it
> > to the right device. I'd be quite surprised though if UDP code had
> > issues with paged Rx??

> You told me : socket(PF_PACKET, SOCK_RAW, htons(ETH_P_EAPOL))
> 
> So I looked at raw code.

and even found an issue. Artem wasn't really sure, but looking at his
information again it seems that the EAPOL packets (those on the raw
sockets) do go through, and then DHCP fails. In the printk he did for
me, I can see that the EAPOL packets are small enough to get pulled in
completely in iwlwifi, and the bigger DHCP packets (~600 bytes) only go
through if we linearize them.

johannes


  reply	other threads:[~2012-09-20 15:22 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1347361823.26457.3.camel@sauron.fi.intel.com>
     [not found] ` <1DC40B07CD6EC041A66726C271A73AE6195AE9C8@IRSMSX102.ger.corp.intel.com>
2012-09-14 14:02   ` regression: tethering fails in 3.5 with iwlwifi Artem Bityutskiy
2012-09-14 16:39     ` Artem Bityutskiy
2012-09-17 14:41       ` Artem Bityutskiy
2012-09-17 15:15         ` Eric Dumazet
2012-09-17 15:17           ` Johannes Berg
2012-09-18  7:09         ` Johannes Berg
2012-09-18  8:40           ` Artem Bityutskiy
2012-09-18  8:44             ` Johannes Berg
2012-09-18 11:45               ` Johannes Berg
2012-09-18 13:20                 ` Artem Bityutskiy
2012-09-20 12:06         ` Artem Bityutskiy
2012-09-20 12:35           ` Johannes Berg
2012-09-20 12:40             ` Eric Dumazet
2012-09-20 12:45               ` Eric Dumazet
2012-09-20 12:46                 ` Eric Dumazet
2012-09-20 12:50                   ` Johannes Berg
2012-09-20 12:47                 ` Johannes Berg
2012-09-20 12:48                   ` Eric Dumazet
2012-09-20 12:50                     ` Johannes Berg
2012-09-20 12:51                       ` Eric Dumazet
2012-09-20 12:54                         ` Johannes Berg
2012-09-20 12:51                   ` Johannes Berg
2012-09-20 12:58                 ` Artem Bityutskiy
2012-09-20 13:04                   ` Eric Dumazet
2012-09-20 13:15                     ` Eric Dumazet
2012-09-20 13:22                     ` Artem Bityutskiy
2012-09-20 13:22                       ` Eric Dumazet
2012-09-20 13:39                         ` Artem Bityutskiy
2012-09-20 13:41                           ` Eric Dumazet
2012-09-20 14:25                       ` Eric Dumazet
2012-09-20 14:37                         ` Artem Bityutskiy
2012-09-20 14:56                           ` Eric Dumazet
2012-09-20 15:05                             ` Johannes Berg
2012-09-20 15:18                               ` Eric Dumazet
2012-09-20 15:22                                 ` Johannes Berg [this message]
2012-09-20 15:27                                   ` Eric Dumazet
2012-09-20 15:31                                     ` Eric Dumazet
2012-09-20 15:32                                       ` Eric Dumazet
2012-09-20 15:19                               ` Eric Dumazet
2012-09-20 15:26                                 ` Johannes Berg
2012-09-20 15:09                             ` Artem Bityutskiy
2012-09-21 17:24                         ` David Miller
2012-09-20 12:47               ` Johannes Berg
2013-03-21 19:34         ` Johannes Berg
2013-03-21 21:24           ` David Miller
2013-03-22  7:01           ` Artem Bityutskiy
2013-03-23 15:45           ` Luis Henriques

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=1348154575.4161.39.camel@jlt4.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=artem.bityutskiy@linux.intel.com \
    --cc=edumazet@google.com \
    --cc=eric.dumazet@gmail.com \
    --cc=linux-wireless@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).