linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jiri Slaby <jirislaby@gmail.com>
To: Bob Copeland <me@bobcopeland.com>
Cc: Sitsofe Wheeler <sitsofe@yahoo.com>,
	Frederic Weisbecker <fweisbec@gmail.com>,
	linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org,
	ath5k-devel@venema.h4ckr.net,
	Nick Kossifidis <mickflemm@gmail.com>,
	"Luis R. Rodriguez" <lrodriguez@atheros.com>
Subject: Re: [TIP] BUG kmalloc-4096: Poison overwritten (ath5k_rx_skb_alloc)
Date: Mon, 23 Feb 2009 23:22:50 +0100	[thread overview]
Message-ID: <49A321BA.2040500@gmail.com> (raw)
In-Reply-To: <20090223152724.M82409@bobcopeland.com>

On 23.2.2009 16:35, Bob Copeland wrote:
> As I understand it, yes, we don't do the right thing when the more flag
> is set.  We're supposed to keep processing packets until we get one with
> the done flag, and then all of that is supposed to be merged into a single
> packet.  Other flags such as the rx rate are only valid on the final
> packet.

What I don't understand is how we can proceed further, if we should 
never get done flag set according to the specs for the buffer which has 
more flag set. We should hit (unlikely(ret == -EINPROGRESS)) test everytime.

> So I am not sure if the jumbo packets are causing bad things to happen,
> or if they are an indication that something bad has already happened.

I don't know, it was just an idea. I tried to lower rxbufsize to 1000 
and see no problems so far (except unsupported jumbo warnings and packet 
loss indeed).

  parent reply	other threads:[~2009-02-23 22:22 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-22 11:18 [TIP] BUG kmalloc-4096: Poison overwritten (ath5k_rx_skb_alloc) Sitsofe Wheeler
2009-02-22 12:01 ` Jiri Slaby
2009-02-22 12:20   ` Sitsofe Wheeler
2009-02-22 12:47     ` Jiri Slaby
2009-02-22 14:47     ` Frederic Weisbecker
2009-02-22 17:02       ` Sitsofe Wheeler
2009-02-22 17:10         ` Frederic Weisbecker
2009-02-22 19:27           ` Jiri Slaby
2009-02-22 19:42             ` Frederic Weisbecker
2009-02-22 20:18               ` Sitsofe Wheeler
2009-02-22 20:27                 ` Jiri Slaby
2009-02-22 20:30                 ` Frederic Weisbecker
2009-02-22 21:56         ` Jiri Slaby
2009-02-22 22:21           ` Sitsofe Wheeler
2009-02-22 23:20           ` Jiri Slaby
2009-02-23 15:35             ` Bob Copeland
2009-02-23 16:03               ` Nick Kossifidis
2009-02-23 16:15                 ` Nick Kossifidis
2009-02-23 16:21                   ` Bob Copeland
2009-02-23 16:27                     ` Nick Kossifidis
2009-02-23 16:30                       ` Bob Copeland
2009-02-23 16:41                         ` Nick Kossifidis
2009-02-23 16:44                           ` Bob Copeland
2009-02-23 16:16                 ` pat-lkml
2009-02-23 16:20                   ` Nick Kossifidis
2009-02-23 22:22               ` Jiri Slaby [this message]
2009-02-23 22:43                 ` Jiri Slaby
2009-02-23 23:08                   ` Nick Kossifidis
2009-02-24 13:58                     ` Bob Copeland
2009-02-24 21:47                       ` Jiri Slaby
2009-02-25 14:01                         ` Sitsofe Wheeler
2009-02-26  1:06                           ` Bob Copeland
2009-02-26 20:53                             ` Jiri Slaby
2009-02-26 21:05                               ` Bob Copeland
2009-02-26 13:59                           ` Bob Copeland
2009-02-26 17:03                             ` Sitsofe Wheeler
2009-03-02 17:34                               ` [ath5k-devel] " Bob Copeland
2009-03-03  4:12                               ` Bob Copeland
2009-03-03 20:03                                 ` Sitsofe Wheeler
2009-03-04 12:07                                   ` Bob Copeland
2009-03-06  9:42                                     ` Sitsofe Wheeler
2009-03-07  4:47                                       ` Bob Copeland
2009-03-07  8:04                                         ` Sitsofe Wheeler
2009-03-07 13:34                                           ` Bob Copeland
2009-03-08  3:09                                       ` Bob Copeland
2009-03-08  9:28                                         ` Jiri Slaby
2009-03-08 16:10                                           ` Bob Copeland
2009-03-10  0:43                                           ` Bob Copeland
2009-03-10  8:19                                             ` Sitsofe Wheeler
2009-03-12  6:10                                             ` Sitsofe Wheeler
2009-03-13  9:52                                               ` Sitsofe Wheeler
2009-03-13 12:28                                                 ` Bob Copeland
2009-03-20 13:14                                                 ` Bob Copeland
2009-03-29 14:24                                                   ` Sitsofe Wheeler
2009-03-29 15:14                                                     ` Bob Copeland
2009-03-31  8:30                                                       ` Sitsofe Wheeler
2009-05-13 21:44                                                         ` Sitsofe Wheeler
2009-05-15  4:09                                                           ` Bob Copeland
2009-05-18 10:05                                                             ` Sitsofe Wheeler
2009-05-22  9:39                                                               ` Sitsofe Wheeler
2009-05-22 12:06                                                                 ` Bob Copeland
2009-05-26 21:10                                                                 ` Sitsofe Wheeler
2009-06-28 20:23                                                                   ` Sitsofe Wheeler
2009-07-14  2:24                                                                     ` Bob Copeland
2009-02-26  1:11                         ` Bob Copeland
2009-02-22 20:17       ` Bob Copeland

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=49A321BA.2040500@gmail.com \
    --to=jirislaby@gmail.com \
    --cc=ath5k-devel@venema.h4ckr.net \
    --cc=fweisbec@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=lrodriguez@atheros.com \
    --cc=me@bobcopeland.com \
    --cc=mickflemm@gmail.com \
    --cc=sitsofe@yahoo.com \
    /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).