All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sasha Levin <sashal@kernel.org>
To: gregkh@linuxfoundation.org
Cc: mchristi@redhat.com, axboe@kernel.dk, jbacik@fb.com,
	josef@toxicpanda.com, stable@vger.kernel.org
Subject: Re: FAILED: patch "[PATCH] nbd: fix max number of supported devs" failed to apply to 4.14-stable tree
Date: Tue, 8 Oct 2019 18:31:52 -0400	[thread overview]
Message-ID: <20191008223152.GG1396@sasha-vm> (raw)
In-Reply-To: <157051955816884@kroah.com>

On Tue, Oct 08, 2019 at 09:25:58AM +0200, gregkh@linuxfoundation.org wrote:
>
>The patch below does not apply to the 4.14-stable tree.
>If someone wants it applied there, or to any other stable or longterm
>tree, then please email the backport, including the original git commit
>id to <stable@vger.kernel.org>.
>
>thanks,
>
>greg k-h
>
>------------------ original commit in Linus's tree ------------------
>
>From e9e006f5fcf2bab59149cb38a48a4817c1b538b4 Mon Sep 17 00:00:00 2001
>From: Mike Christie <mchristi@redhat.com>
>Date: Sun, 4 Aug 2019 14:10:06 -0500
>Subject: [PATCH] nbd: fix max number of supported devs
>
>This fixes a bug added in 4.10 with commit:
>
>commit 9561a7ade0c205bc2ee035a2ac880478dcc1a024
>Author: Josef Bacik <jbacik@fb.com>
>Date:   Tue Nov 22 14:04:40 2016 -0500
>
>    nbd: add multi-connection support
>
>that limited the number of devices to 256. Before the patch we could
>create 1000s of devices, but the patch switched us from using our
>own thread to using a work queue which has a default limit of 256
>active works.
>
>The problem is that our recv_work function sits in a loop until
>disconnection but only handles IO for one connection. The work is
>started when the connection is started/restarted, but if we end up
>creating 257 or more connections, the queue_work call just queues
>connection257+'s recv_work and that waits for connection 1 - 256's
>recv_work to be disconnected and that work instance completing.
>
>Instead of reverting back to kthreads, this has us allocate a
>workqueue_struct per device, so we can block in the work.
>
>Cc: stable@vger.kernel.org
>Reviewed-by: Josef Bacik <josef@toxicpanda.com>
>Signed-off-by: Mike Christie <mchristi@redhat.com>
>Signed-off-by: Jens Axboe <axboe@kernel.dk>

I've queued up 553768d1169a4 ("nbd: fix crash when the blksize is zero")
and 2189c97cdbed6 ("block/ndb: add WQ_UNBOUND to the knbd-recv
workqueue") to resolve this on 4.14.

-- 
Thanks,
Sasha

      reply	other threads:[~2019-10-08 22:31 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-08  7:25 FAILED: patch "[PATCH] nbd: fix max number of supported devs" failed to apply to 4.14-stable tree gregkh
2019-10-08 22:31 ` Sasha Levin [this message]

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=20191008223152.GG1396@sasha-vm \
    --to=sashal@kernel.org \
    --cc=axboe@kernel.dk \
    --cc=gregkh@linuxfoundation.org \
    --cc=jbacik@fb.com \
    --cc=josef@toxicpanda.com \
    --cc=mchristi@redhat.com \
    --cc=stable@vger.kernel.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 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.