linux-kernel - Re: [PATCH 17/22] IB/mlx4/mad: Remove deprecated create_singlethread

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160819155025.GJ18515@leon.nu>
Date:   Fri, 19 Aug 2016 18:50:25 +0300
From:   Leon Romanovsky <leonro@...lanox.com>
To:     Bhaktipriya Shridhar <bhaktipriya96@...il.com>
Cc:     Yishai Hadas <yishaih@...lanox.com>,
        Doug Ledford <dledford@...hat.com>,
        Sean Hefty <sean.hefty@...el.com>,
        Hal Rosenstock <hal.rosenstock@...il.com>,
        Jason Gunthorpe <jgunthorpe@...idianresearch.com>,
        Chuck Lever <chuck.lever@...cle.com>,
        Mark Bloch <markb@...lanox.com>,
        Hans Westgaard Ry <hans.westgaard.ry@...cle.com>,
        Christoph Lameter <cl@...ux.com>,
        Erez Shitrit <erezsh@...lanox.com>,
        Haggai Eran <haggaie@...lanox.com>,
        Florian Westphal <fw@...len.de>,
        Guy Shapiro <guysh@...lanox.com>, Eli Cohen <eli@...lanox.com>,
        Or Gerlitz <ogerlitz@...lanox.com>,
        Faisal Latif <faisal.latif@...el.com>,
        Matan Barak <matanb@...lanox.com>,
        Chien Tin Tung <chien.tin.tung@...el.com>,
        Mustafa Ismail <mustafa.ismail@...el.com>,
        Shiraz Saleem <shiraz.saleem@...el.com>,
        Tatyana Nikolova <tatyana.e.nikolova@...el.com>,
        Steve Wise <swise@...lsio.com>,
        Mike Marciniszyn <infinipath@...el.com>,
        Bart Van Assche <bart.vanassche@...disk.com>,
        Yotam Kenneth <yotamke@...lanox.com>,
        Somnath Kotur <Somnath.Kotur@...gotech.Com>,
        Moni Shoua <monis@...lanox.com>,
        Shachar Raindel <raindel@...lanox.com>,
        Parav Pandit <pandit.parav@...il.com>,
        Sagi Grimberg <sagi@...mberg.me>,
        Christoph Hellwig <hch@...radead.org>,
        Dean Luick <dean.luick@...el.com>,
        Ira Weiny <ira.weiny@...el.com>,
        Kaike Wan <kaike.wan@...el.com>, linux-rdma@...r.kernel.org,
        linux-kernel@...r.kernel.org, Tejun Heo <tj@...nel.org>
Subject: Re: [PATCH 17/22] IB/mlx4/mad: Remove deprecated
 create_singlethread_workqueue

On Mon, Aug 15, 2016 at 11:43:10PM +0530, Bhaktipriya Shridhar wrote:
> alloc_ordered_workqueue() with WQ_MEM_RECLAIM set, replaces
> deprecated create_singlethread_workqueue(). This is the identity
> conversion.
> 
> The workqueue "wq" queues work item &ctx->work and the workqueue "ud_wq"
> queues work item &dm[i]->work.
> 
> Both the workqueues have been identity converted.
> 
> WQ_MEM_RECLAIM has been set to ensure forward progress under
> memory pressure.
> 
> Signed-off-by: Bhaktipriya Shridhar <bhaktipriya96@...il.com>

Thanks,
Reviewed-by: Leon Romanovsky <leonro@...lanox.com>

> ---
>  drivers/infiniband/hw/mlx4/mad.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/infiniband/hw/mlx4/mad.c b/drivers/infiniband/hw/mlx4/mad.c
> index 9c2e53d..35b1260 100644
> --- a/drivers/infiniband/hw/mlx4/mad.c
> +++ b/drivers/infiniband/hw/mlx4/mad.c
> @@ -2070,7 +2070,7 @@ static int mlx4_ib_alloc_demux_ctx(struct mlx4_ib_dev *dev,
>  	}
> 
>  	snprintf(name, sizeof name, "mlx4_ibt%d", port);
> -	ctx->wq = create_singlethread_workqueue(name);
> +	ctx->wq = alloc_ordered_workqueue(name, WQ_MEM_RECLAIM);
>  	if (!ctx->wq) {
>  		pr_err("Failed to create tunnelling WQ for port %d\n", port);
>  		ret = -ENOMEM;
> @@ -2078,7 +2078,7 @@ static int mlx4_ib_alloc_demux_ctx(struct mlx4_ib_dev *dev,
>  	}
> 
>  	snprintf(name, sizeof name, "mlx4_ibud%d", port);
> -	ctx->ud_wq = create_singlethread_workqueue(name);
> +	ctx->ud_wq = alloc_ordered_workqueue(name, WQ_MEM_RECLAIM);
>  	if (!ctx->ud_wq) {
>  		pr_err("Failed to create up/down WQ for port %d\n", port);
>  		ret = -ENOMEM;
> --
> 2.1.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)