From: Johannes Berg <johannes@sipsolutions.net>
To: Thomas Huehn <thomas@net.t-labs.tu-berlin.de>
Cc: linux-wireless <linux-wireless@vger.kernel.org>, coelho <coelho@ti.com>
Subject: Re: [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path
Date: Thu, 26 Jul 2012 18:46:05 +0200 [thread overview]
Message-ID: <1343321165.4477.4.camel@jlt3.sipsolutions.net> (raw)
In-Reply-To: <501170F2.3050300@net.t-labs.tu-berlin.de>
On Thu, 2012-07-26 at 18:31 +0200, Thomas Huehn wrote:
> This is the call path
>
> in main.c ... INIT_WORK(&wl->tx_work, wl1271_tx_work);
> jumps to tx.c...wl1271_tx_work(struct work_struct *work)
> calls wl1271_tx_work(struct work_struct *work)
> calls wlcore_tx_work_locked(struct wl1271 *wl)
> calls wl1271_skb_queue_head()
> calls wl12xx_tx_get_hlid()
> calls wl12xx_tx_get_hlid_ap() .... where sta is needed.
>
> But I could not find any point in the call path where I could grab the
> sta pointer and pass it through. I will move the rcu into
> wl12xx_tx_get_hlid_ap(), but I do not see any cheaper way here as using rcu.
I don't know this driver, maybe it could determine the hlid before and
store that.
johannes
next prev parent reply other threads:[~2012-07-26 16:46 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-26 16:09 [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path Thomas Huehn
2012-07-26 16:09 ` [ath9k-devel] " Thomas Huehn
2012-07-26 16:17 ` Johannes Berg
2012-07-26 16:17 ` Johannes Berg
2012-07-26 16:17 ` [ath9k-devel] " Johannes Berg
2012-07-26 16:19 ` Johannes Berg
2012-07-26 16:19 ` Johannes Berg
2012-07-26 16:19 ` [ath9k-devel] " Johannes Berg
2012-07-26 16:31 ` Thomas Huehn
2012-07-26 16:31 ` Thomas Huehn
2012-07-26 16:31 ` [ath9k-devel] " Thomas Huehn
2012-07-26 16:46 ` Johannes Berg [this message]
2012-07-26 16:57 ` Arik Nemtsov
2012-07-26 16:57 ` Arik Nemtsov
2012-07-26 16:57 ` [ath9k-devel] " Arik Nemtsov
2012-07-26 17:04 ` Arik Nemtsov
2012-07-26 17:04 ` Arik Nemtsov
2012-07-26 17:04 ` [ath9k-devel] " Arik Nemtsov
2012-07-26 18:41 ` Arik Nemtsov
2012-07-26 18:41 ` Arik Nemtsov
2012-07-26 18:41 ` [ath9k-devel] " Arik Nemtsov
2012-07-26 20:02 ` Thomas Huehn
2012-07-26 20:02 ` Thomas Huehn
2012-07-26 20:02 ` [ath9k-devel] " Thomas Huehn
2012-07-27 7:26 ` Johannes Berg
2012-07-27 7:26 ` Johannes Berg
2012-07-27 7:26 ` [ath9k-devel] " Johannes Berg
2012-07-27 9:35 ` Thomas Huehn
2012-07-27 9:35 ` Thomas Huehn
2012-07-27 9:35 ` [ath9k-devel] " Thomas Huehn
2012-07-27 11:52 ` Johannes Berg
2012-07-27 11:52 ` Johannes Berg
2012-07-27 11:52 ` [ath9k-devel] " Johannes Berg
2012-07-27 13:28 ` Thomas Huehn
2012-07-27 13:28 ` Thomas Huehn
2012-07-27 13:28 ` [ath9k-devel] " Thomas Huehn
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=1343321165.4477.4.camel@jlt3.sipsolutions.net \
--to=johannes@sipsolutions.net \
--cc=coelho@ti.com \
--cc=linux-wireless@vger.kernel.org \
--cc=thomas@net.t-labs.tu-berlin.de \
/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.