public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
To: Bhaktipriya Shridhar
	<bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Yishai Hadas <yishaih-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Sean Hefty <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Hal Rosenstock
	<hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Jason Gunthorpe
	<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>,
	Chuck Lever <chuck.lever-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>,
	Mark Bloch <markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Hans Westgaard Ry
	<hans.westgaard.ry-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>,
	Christoph Lameter <cl-vYTEC60ixJUAvxtiuMwx3w@public.gmane.org>,
	Erez Shitrit <erezsh-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Florian Westphal <fw-HFFVJYpyMKqzQB+pC5nmwQ@public.gmane.org>,
	Guy Shapiro <guysh-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Eli Cohen <eli-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Faisal Latif
	<faisal.latif-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Chien Tin Tung
	<chien.tin.tung-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Mustafa Ismail
	<mustafa.ismail-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Shiraz Saleem <shiraz.saleem@i>
Subject: Re: [PATCH 18/22] IB/mlx4/mcg: Remove deprecated create_singlethread_workqueue
Date: Fri, 19 Aug 2016 18:50:55 +0300	[thread overview]
Message-ID: <20160819155055.GK18515@leon.nu> (raw)
In-Reply-To: <4045f5014733b227e085c76f00f9ecf7f6266803.1471283718.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 1896 bytes --]

On Mon, Aug 15, 2016 at 11:43:35PM +0530, Bhaktipriya Shridhar wrote:
> alloc_ordered_workqueue() with WQ_MEM_RECLAIM set, replaces
> deprecated create_singlethread_workqueue(). This is the identity
> conversion.
> 
> The workqueue "mcg_wq" queues work items &group->work
> and &group->timeout_work.
> 
> The workqueue "clean_wq" queues work item mcg_clean_task.
> 
> Both have been identity converted.
> 
> WQ_MEM_RECLAIM has been set to ensure forward progress under
> memory pressure.
> 
> Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---

Thanks,
Reviewed-by: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>

>  drivers/infiniband/hw/mlx4/mcg.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/infiniband/hw/mlx4/mcg.c b/drivers/infiniband/hw/mlx4/mcg.c
> index 8f7ad07..7d30be0 100644
> --- a/drivers/infiniband/hw/mlx4/mcg.c
> +++ b/drivers/infiniband/hw/mlx4/mcg.c
> @@ -1045,7 +1045,7 @@ int mlx4_ib_mcg_port_init(struct mlx4_ib_demux_ctx *ctx)
> 
>  	atomic_set(&ctx->tid, 0);
>  	sprintf(name, "mlx4_ib_mcg%d", ctx->port);
> -	ctx->mcg_wq = create_singlethread_workqueue(name);
> +	ctx->mcg_wq = alloc_ordered_workqueue(name, WQ_MEM_RECLAIM);
>  	if (!ctx->mcg_wq)
>  		return -ENOMEM;
> 
> @@ -1246,7 +1246,7 @@ void clean_vf_mcast(struct mlx4_ib_demux_ctx *ctx, int slave)
> 
>  int mlx4_ib_mcg_init(void)
>  {
> -	clean_wq = create_singlethread_workqueue("mlx4_ib_mcg");
> +	clean_wq = alloc_ordered_workqueue("mlx4_ib_mcg", WQ_MEM_RECLAIM);
>  	if (!clean_wq)
>  		return -ENOMEM;
> 
> --
> 2.1.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  parent reply	other threads:[~2016-08-19 15:50 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-15 17:57 [PATCH 00/22] Remove deprecated create_workqueue instances Bhaktipriya Shridhar
2016-08-15 17:57 ` [PATCH 01/22] IB/sa : Remove deprecated create_singlethread_workqueue Bhaktipriya Shridhar
2016-08-15 17:59 ` [PATCH 05/22] IB/cma: " Bhaktipriya Shridhar
     [not found]   ` <a56fa1378894125beed6a679b9d8ee14ae3b601f.1471283717.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-16 18:21     ` Steve Wise
2016-08-15 18:07 ` [PATCH 07/22] IB/iwcm: " Bhaktipriya Shridhar
2016-08-16 18:20   ` Steve Wise
2016-08-15 18:10 ` [PATCH 12/22] i40iw_main: " Bhaktipriya Shridhar
2016-08-15 18:10 ` [PATCH 13/22] i40iw_cm: " Bhaktipriya Shridhar
2016-08-15 18:11 ` [PATCH 14/22] IB/mlx5: " Bhaktipriya Shridhar
     [not found]   ` <7d8b7737365694e925e50cef5d673aba0b9dbc04.1471283717.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-19 15:44     ` Leon Romanovsky
2016-08-19 15:46   ` Leon Romanovsky
2016-08-15 18:12 ` [PATCH 16/22] IB/mlx4: " Bhaktipriya Shridhar
2016-08-19 15:43   ` Leon Romanovsky
2016-08-15 18:13 ` [PATCH 17/22] IB/mlx4/mad: " Bhaktipriya Shridhar
     [not found]   ` <18c4505e94d23ab7a34c2aaa9c4ee9f7ace09cef.1471283718.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-19 15:50     ` Leon Romanovsky
2016-08-15 18:13 ` [PATCH 18/22] IB/mlx4/mcg: " Bhaktipriya Shridhar
     [not found]   ` <4045f5014733b227e085c76f00f9ecf7f6266803.1471283718.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-19 15:50     ` Leon Romanovsky [this message]
2016-08-15 18:14 ` [PATCH 19/22] IB/nes: " Bhaktipriya Shridhar
2016-08-15 18:14 ` [PATCH 21/22] IB/ipoib_verbs: " Bhaktipriya Shridhar
2016-08-15 18:15 ` [PATCH 22/22] IB/mlx4/alias_GUID: " Bhaktipriya Shridhar
2016-08-19 15:51   ` Leon Romanovsky
     [not found] ` <cover.1471283717.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-15 17:58   ` [PATCH 02/22] IB/mad: " Bhaktipriya Shridhar
2016-08-15 17:58   ` [PATCH 03/22] IB/multicast: " Bhaktipriya Shridhar
2016-08-15 17:59   ` [PATCH 04/22] IB/ucma: " Bhaktipriya Shridhar
2016-08-15 18:00   ` [PATCH 06/22] IB/addr: " Bhaktipriya Shridhar
2016-08-15 18:08   ` [PATCH 08/22] iw_cxgb3: " Bhaktipriya Shridhar
2016-08-16 18:19     ` Steve Wise
2016-08-15 18:08   ` [PATCH 09/22] IB/qib: " Bhaktipriya Shridhar
     [not found]     ` <e0310e6be2420c4dc27d752c2b7465cd359cdca9.1471283717.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-16 17:32       ` Marciniszyn, Mike
2016-08-15 18:09   ` [PATCH 10/22] iw_cxgb4: " Bhaktipriya Shridhar
2016-08-16 18:19     ` Steve Wise
2016-08-15 18:09   ` [PATCH 11/22] IB/mthca: " Bhaktipriya Shridhar
2016-08-15 18:12   ` [PATCH 15/22] IB/mlx5/odp: " Bhaktipriya Shridhar
     [not found]     ` <fdc8a637db52a5ea829309bf52afec7e1825bafd.1471283717.git.bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-08-19 15:49       ` Leon Romanovsky
2016-08-15 18:14   ` [PATCH 20/22] IB/ipoib: " Bhaktipriya Shridhar
2016-08-15 23:11   ` [PATCH 00/22] Remove deprecated create_workqueue instances Tejun Heo
2016-08-19 15:48   ` Leon Romanovsky
2016-09-16 17:26 ` Doug Ledford

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=20160819155055.GK18515@leon.nu \
    --to=leonro-vpraknaxozvwk0htik3j/w@public.gmane.org \
    --cc=bhaktipriya96-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=chien.tin.tung-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=chuck.lever-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org \
    --cc=cl-vYTEC60ixJUAvxtiuMwx3w@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=eli-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=erezsh-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=faisal.latif-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=fw-HFFVJYpyMKqzQB+pC5nmwQ@public.gmane.org \
    --cc=guysh-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=hans.westgaard.ry-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org \
    --cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
    --cc=markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=mustafa.ismail-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=shiraz.saleem@i \
    --cc=yishaih-VPRAkNaXOzVWk0Htik3J/w@public.gmane.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