All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Yogesh Ashok Powar <yogeshp@marvell.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"John W. Linville" <linville@tuxdriver.com>,
	Andreas Hartmann <andihartmann@01019freenet.de>
Subject: Re: [PATCH 2/2] mac80211: Fixing Races for skipping tailroom reservation
Date: Tue, 21 Jun 2011 15:43:34 +0200	[thread overview]
Message-ID: <1308663814.4276.3.camel@jlt3.sipsolutions.net> (raw)
In-Reply-To: <20110621130305.GB32464@hertz.marvell.com>

On Tue, 2011-06-21 at 18:33 +0530, Yogesh Ashok Powar wrote:

> Following is the implementation based on your design.
> 
> Let me know if I miss anything here.

Since your original code was reverted you should send a full patch. But
overall it looks fine. Came to think of it yesterday though, for the
0->1 case it's of course necessary to do the adjustment before the key
can be found, but also for the 1->0 case it'll be necessary to do the
counter adjustment after the key adjustments.

johannes


  reply	other threads:[~2011-06-21 13:43 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-16 10:21 [PATCH 0/2] mac80211: Fixing races for hw crypto skipping tailroom Yogesh Ashok Powar
2011-06-16 10:25 ` [PATCH 1/2] Revert "Revert "mac80211: Skip tailroom reservation for full HW-crypto devices"" Yogesh Ashok Powar
2011-06-16 10:27 ` [PATCH 2/2] mac80211: Fixing Races for skipping tailroom reservation Yogesh Ashok Powar
2011-06-16 15:36   ` Johannes Berg
2011-06-17 13:25     ` Yogesh Ashok Powar
2011-06-17 17:24       ` Johannes Berg
2011-06-20 14:30         ` Yogesh Ashok Powar
2011-06-20 15:29           ` Johannes Berg
2011-06-20 16:49             ` Yogesh Powar
2011-06-20 17:29               ` Johannes Berg
2011-06-21 13:03                 ` Yogesh Ashok Powar
2011-06-21 13:43                   ` Johannes Berg [this message]
2011-06-21 14:10                     ` Yogesh Ashok Powar
2011-06-21 14:40                       ` Johannes Berg
2011-06-21 16:33                         ` Yogesh Ashok Powar
2011-06-21 17:44                           ` Andreas Hartmann
2011-06-22  7:17                           ` Yogesh Ashok Powar
2011-06-22 12:31                             ` Yogesh Ashok Powar
2011-06-22 12:49                               ` Johannes Berg
2011-06-22 12:58                                 ` Yogesh Ashok Powar
2011-06-22 13:12                                   ` Johannes Berg
2011-06-23 11:52                                     ` Yogesh Powar
2011-06-24  9:04                                     ` yogeshp
2011-06-25 13:07                                       ` Johannes Berg
2011-06-27  6:02                                         ` [PATCH] nl80211: use netlink consistent dump feature for BSS dumps Walter Goldens

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=1308663814.4276.3.camel@jlt3.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=andihartmann@01019freenet.de \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=yogeshp@marvell.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.