From: James Cameron <quozl@laptop.org>
To: Avinash Patil <patila@marvell.com>
Cc: linville@tuxdriver.com, linux-wireless@vger.kernel.org,
akarwar@marvell.com, huxm@marvell.com, yangyang@marvell.com,
cluo@marvell.com, maithili@marvell.com
Subject: Re: [PATCH 17/17] mwifiex: add rx workqueue support
Date: Thu, 4 Sep 2014 13:47:58 +1000 [thread overview]
Message-ID: <20140904034758.GZ27002@us.netrek.org> (raw)
In-Reply-To: <1409576345-13717-18-git-send-email-patila@marvell.com>
On Mon, Sep 01, 2014 at 06:29:05PM +0530, Avinash Patil wrote:
> This patch adds RX work queue support to mwifiex.
> Packets received are queued to internal queue which are then
> processed by scheduling a work item for RX process.
>
> RX work is enabled only on SMP systems.
>
> Signed-off-by: Avinash Patil <patila@marvell.com>
> Signed-off-by: Marc Yang <yangyang@marvell.com>
> Signed-off-by: Cathy Luo <cluo@marvell.com>
> ---
> drivers/net/wireless/mwifiex/11n_rxreorder.c | 14 ++++
> drivers/net/wireless/mwifiex/init.c | 19 ++++++
> drivers/net/wireless/mwifiex/main.c | 95 ++++++++++++++++++++++++++++
> drivers/net/wireless/mwifiex/main.h | 14 ++++
> drivers/net/wireless/mwifiex/pcie.c | 12 +++-
> drivers/net/wireless/mwifiex/sdio.c | 11 +++-
> 6 files changed, 163 insertions(+), 2 deletions(-)
>
> [...]
> diff --git a/drivers/net/wireless/mwifiex/main.c b/drivers/net/wireless/mwifiex/main.c
> index fb38d81..2ed59a0 100644
> --- a/drivers/net/wireless/mwifiex/main.c
> +++ b/drivers/net/wireless/mwifiex/main.c
> [...]
> @@ -784,6 +858,15 @@ mwifiex_add_card(void *card, struct semaphore *sem,
> adapter->cmd_wait_q.status = 0;
> adapter->scan_wait_q_woken = false;
>
> + if (num_possible_cpus() > 1) {
> + adapter->rx_work_enabled = true;
> + pr_debug("info: %s rx work enabled, cpus %d :\n", __func__,
> + num_possible_cpus());
> + /*to ensure this is seen in dmesg logs*/
> + pr_err("info: %s rx work enabled, cpus %d :\n", __func__,
> + num_possible_cpus());
Either pr_debug or pr_err, but not both.
(Would love to test, but stuck on 3.5, and don't know how to
backport mwifiex.)
--
James Cameron
http://quozl.linux.org.au/
next prev parent reply other threads:[~2014-09-04 3:56 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-01 12:58 [PATCH 00/17] mwifiex updates for 3.17 Avinash Patil
2014-09-01 12:58 ` [PATCH 01/17] mwifiex: fix probable memory corruption while processing TDLS frame Avinash Patil
2014-09-01 12:58 ` [PATCH 02/17] mwifiex: avoid processing RX packets with invalid length Avinash Patil
2014-09-01 12:58 ` [PATCH 03/17] mwifiex: rework internal scan for association Avinash Patil
2014-09-01 12:58 ` [PATCH 04/17] mwifiex: support for event done interrupt Avinash Patil
2014-09-01 12:58 ` [PATCH 05/17] mwifiex: fix 5G association failure after leaving 2.4G IBSS Avinash Patil
2014-09-01 12:58 ` [PATCH 06/17] mwifiex: fix a bug in Tx multiport aggregation Avinash Patil
2014-09-01 12:58 ` [PATCH 07/17] mwifiex: minor cleanup in " Avinash Patil
2014-09-01 12:58 ` [PATCH 08/17] mwifiex: fix left_len calculation issue Avinash Patil
2014-09-01 12:58 ` [PATCH 09/17] mwifiex: rename macro and variables related to API revision Avinash Patil
2014-09-01 12:58 ` [PATCH 10/17] mwifiex: use firmware API revision from GET_HW_SPEC response Avinash Patil
2014-09-01 12:58 ` [PATCH 11/17] mwifiex: set passive scan type for scan requests with no ssid Avinash Patil
2014-09-01 12:59 ` [PATCH 12/17] mwifiex: bring in scan channel gap feature Avinash Patil
2014-09-01 12:59 ` [PATCH 13/17] mwifiex: remove restriction of single channel scan when connected Avinash Patil
2014-09-01 12:59 ` [PATCH 14/17] mwifiex: process TX even when scan is ongoing Avinash Patil
2014-09-01 12:59 ` [PATCH 15/17] mwifiex: remove redundant variable report_scan_result Avinash Patil
2014-09-01 12:59 ` [PATCH 16/17] mwifiex: remove low priority scan handling Avinash Patil
2014-09-01 12:59 ` [PATCH 17/17] mwifiex: add rx workqueue support Avinash Patil
2014-09-04 3:47 ` James Cameron [this message]
2014-09-05 8:08 ` Avinash Patil
2014-09-04 17:15 ` [PATCH 00/17] mwifiex updates for 3.17 John W. Linville
2014-09-04 17:22 ` Avinash Patil
2014-09-09 17:10 ` Avinash Patil
2014-09-09 18:25 ` John W. Linville
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=20140904034758.GZ27002@us.netrek.org \
--to=quozl@laptop.org \
--cc=akarwar@marvell.com \
--cc=cluo@marvell.com \
--cc=huxm@marvell.com \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=maithili@marvell.com \
--cc=patila@marvell.com \
--cc=yangyang@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 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).