From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <alex@iss-integration.com>
MIME-Version: 1.0
In-Reply-To: <20180413160334.5496-3-josef@toxicpanda.com>
References: <20180413160334.5496-1-josef@toxicpanda.com> <20180413160334.5496-3-josef@toxicpanda.com>
From: Alex Gorbachev <alex@iss-integration.com>
Date: Fri, 13 Apr 2018 23:01:28 -0400
Message-ID: <CAM1qzF7TSb57pjhSqtSq7wXUc7zR6SPa7tK-tXAdPNGet55T6A@mail.gmail.com>
Subject: Re: [PATCH 3/3] nbd: use bd_set_size when updating disk size
To: Josef Bacik <josef@toxicpanda.com>
Cc: axboe@kernel.dk, nbd@other.debian.org, linux-block@vger.kernel.org,
	kernel-team@fb.com, Josef Bacik <jbacik@fb.com>, stable@vger.kernel.org
Content-Type: multipart/alternative; boundary="00000000000087cd8f0569c6343c"
List-ID: <linux-block@vger.kernel.org>

--00000000000087cd8f0569c6343c
Content-Type: text/plain; charset="UTF-8"

On Fri, Apr 13, 2018 at 12:03 PM, Josef Bacik <josef@toxicpanda.com> wrote:

> From: Josef Bacik <jbacik@fb.com>
>
> When we stopped relying on the bdev everywhere I broke updating the
> block device size on the fly, which ceph relies on.  We can't just do
> set_capacity, we also have to do bd_set_size so things like parted will
> notice the device size change.
>

The three patches Josef submitted today fix the problem with the online
rbd-nbd expansion completely.


Fixes: 29eaadc ("nbd: stop using the bdev everywhere")

> cc: stable@vger.kernel.org
Signed-off-by: Josef Bacik <jbacik@fb.com>
Tested-by: Alex Gorbachev <ag@iss-integration.com>

> ---

>  drivers/block/nbd.c | 7 ++++++-

>  1 file changed, 6 insertions(+), 1 deletion(-)

>
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c

> index 1520383b12f6..61dd95aa47fa 100644

> --- a/drivers/block/nbd.c

> +++ b/drivers/block/nbd.c

> @@ -231,9 +231,15 @@ static void nbd_size_clear(struct nbd_device *nbd)

>  static void nbd_size_update(struct nbd_device *nbd)

>  {

>         struct nbd_config *config = nbd->config;

> +       struct block_device *bdev = bdget_disk(nbd->disk, 0);

> +

>         blk_queue_logical_block_size(nbd->disk->queue, config->blksize);

>         blk_queue_physical_block_size(nbd->disk->queue, config->blksize);

>         set_capacity(nbd->disk, config->bytesize >> 9);

> +       if (bdev) {

> +               bd_set_size(bdev, config->bytesize);

> +               bdput(bdev);

> +       }

>         kobject_uevent(&nbd_to_dev(nbd)->kobj, KOBJ_CHANGE);

>  }

>
@@ -1111,7 +1117,6 @@ static int nbd_start_device_ioctl(struct nbd_device
*nbd, struct block_device *b

>         if (ret)

>                 return ret;

>
-       bd_set_size(bdev, config->bytesize);

>         if (max_part)

>                 bdev->bd_invalidated = 1;

>         mutex_unlock(&nbd->config_lock);

> --

> 2.14.3

>
>

--00000000000087cd8f0569c6343c
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_extra"><div class=3D"gmail_quote">On F=
ri, Apr 13, 2018 at 12:03 PM, Josef Bacik <span dir=3D"ltr">&lt;<a href=3D"=
mailto:josef@toxicpanda.com" target=3D"_blank">josef@toxicpanda.com</a>&gt;=
</span> wrote:<br><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .=
8ex;border-left:1px #ccc solid;padding-left:1ex">From: Josef Bacik &lt;<a h=
ref=3D"mailto:jbacik@fb.com">jbacik@fb.com</a>&gt;<br>
<br>
When we stopped relying on the bdev everywhere I broke updating the<br>
block device size on the fly, which ceph relies on.=C2=A0 We can&#39;t just=
 do<br>
set_capacity, we also have to do bd_set_size so things like parted will<br>
notice the device size change.<br></blockquote><div><br></div><div>The thre=
e patches Josef submitted today fix the problem with the online rbd-nbd exp=
ansion completely.</div><div><br></div><div>=C2=A0</div>Fixes: 29eaadc (&qu=
ot;nbd: stop using the bdev everywhere&quot;)<br><blockquote class=3D"gmail=
_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bo=
rder-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"><=
/blockquote>cc: <a href=3D"mailto:stable@vger.kernel.org">stable@vger.kerne=
l.org</a><br>
Signed-off-by: Josef Bacik &lt;<a href=3D"mailto:jbacik@fb.com">jbacik@fb.c=
om</a>&gt;<br><div>Tested-by: Alex Gorbachev &lt;<a href=3D"mailto:ag@iss-i=
ntegration.com">ag@iss-integration.com</a>&gt;</div><blockquote class=3D"gm=
ail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204)=
;border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex=
"></blockquote>---<br><blockquote class=3D"gmail_quote" style=3D"margin:0px=
 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(20=
4,204,204);padding-left:1ex;padding-right:1ex"></blockquote>=C2=A0drivers/b=
lock/nbd.c | 7 ++++++-<br><blockquote class=3D"gmail_quote" style=3D"margin=
:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rg=
b(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>=C2=A01 fil=
e changed, 6 insertions(+), 1 deletion(-)<br><blockquote class=3D"gmail_quo=
te" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204=
);padding-left:1ex">
</blockquote><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8e=
x;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204=
,204);padding-left:1ex;padding-right:1ex"></blockquote>diff --git a/drivers=
/block/nbd.c b/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" sty=
le=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:=
1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote=
>index 1520383b12f6..61dd95aa47fa 100644<br><blockquote class=3D"gmail_quot=
e" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-=
right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></bloc=
kquote>--- a/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" style=
=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1p=
x solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=
++ b/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" style=3D"marg=
in:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid =
rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>@@ -231,9=
 +231,15 @@ static void nbd_size_clear(struct nbd_device *nbd)<br><blockquo=
te class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rg=
b(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;pad=
ding-right:1ex"></blockquote>=C2=A0static void nbd_size_update(struct nbd_d=
evice *nbd)<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;=
border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,2=
04);padding-left:1ex;padding-right:1ex"></blockquote>=C2=A0{<br><blockquote=
 class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(=
204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;paddi=
ng-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 struct nbd_config *c=
onfig =3D nbd-&gt;config;<br><blockquote class=3D"gmail_quote" style=3D"mar=
gin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid=
 rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=C2=A0 =
=C2=A0 =C2=A0 =C2=A0struct block_device *bdev =3D bdget_disk(nbd-&gt;disk, =
0);<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-l=
eft:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padd=
ing-left:1ex;padding-right:1ex"></blockquote>+<br><blockquote class=3D"gmai=
l_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);b=
order-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex">=
</blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 blk_queue_logical_block_size(<wbr>=
nbd-&gt;disk-&gt;queue, config-&gt;blksize);<br><blockquote class=3D"gmail_=
quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bor=
der-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></=
blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 blk_queue_physical_block_size(<wbr>n=
bd-&gt;disk-&gt;queue, config-&gt;blksize);<br><blockquote class=3D"gmail_q=
uote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bord=
er-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></b=
lockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 set_capacity(nbd-&gt;disk, config-&gt=
;bytesize &gt;&gt; 9);<br><blockquote class=3D"gmail_quote" style=3D"margin=
:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rg=
b(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=C2=A0 =C2=
=A0 =C2=A0 =C2=A0if (bdev) {<br><blockquote class=3D"gmail_quote" style=3D"=
margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px so=
lid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bd_set_size(bdev, confi=
g-&gt;bytesize);<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0=
.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,=
204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bdput(bdev);<br><blockquote class=
=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,20=
4,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-rig=
ht:1ex"></blockquote>+=C2=A0 =C2=A0 =C2=A0 =C2=A0}<br><blockquote class=3D"=
gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,20=
4);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1=
ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 kobject_uevent(&amp;nbd_to_dev=
(<wbr>nbd)-&gt;kobj, KOBJ_CHANGE);<br><blockquote class=3D"gmail_quote" sty=
le=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:=
1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote=
>=C2=A0}<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0=
.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
</blockquote><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8e=
x;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204=
,204);padding-left:1ex;padding-right:1ex"></blockquote>@@ -1111,7 +1117,6 @=
@ static int nbd_start_device_ioctl(struct nbd_device *nbd, struct block_de=
vice *b<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;bord=
er-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);=
padding-left:1ex;padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=
=A0 if (ret)<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex=
;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,=
204);padding-left:1ex;padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 return ret;<br><blockquote class=3D"gmai=
l_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,20=
4,204);padding-left:1ex">
</blockquote><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8e=
x;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204=
,204);padding-left:1ex;padding-right:1ex"></blockquote>-=C2=A0 =C2=A0 =C2=
=A0 =C2=A0bd_set_size(bdev, config-&gt;bytesize);<br><blockquote class=3D"g=
mail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204=
);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1e=
x"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 if (max_part)<br><blockquote cl=
ass=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204=
,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-=
right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 bdev-&gt;bd_invalidated =3D 1;<br><blockquote class=3D"gmail_quote" =
style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-rig=
ht:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockqu=
ote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 mutex_unlock(&amp;nbd-&gt;config_<wbr>lock)=
;<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bo=
rder-left:1px solid rgb(204,204,204);padding-left:1ex">
</blockquote><span style=3D"color:rgb(136,136,136)">--=C2=A0</span><br><blo=
ckquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px sol=
id rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1e=
x;padding-right:1ex"></blockquote><span style=3D"color:rgb(136,136,136)">2.=
14.3</span><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0p=
x 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span clas=
s=3D"HOEnZb"><font color=3D"#888888">
<br>
</font></span></blockquote></div><br></div></div>

--00000000000087cd8f0569c6343c--

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <josef@toxicpanda.com>
Return-Path: <josef@toxicpanda.com>
From: Josef Bacik <josef@toxicpanda.com>
To: axboe@kernel.dk,
	nbd@other.debian.org,
	linux-block@vger.kernel.org,
	kernel-team@fb.com,
	alex@iss-integration.com
Cc: Josef Bacik <jbacik@fb.com>,
	stable@vger.kernel.org
Subject: [PATCH 3/3] nbd: use bd_set_size when updating disk size
Date: Fri, 13 Apr 2018 12:03:34 -0400
Message-Id: <20180413160334.5496-3-josef@toxicpanda.com>
In-Reply-To: <20180413160334.5496-1-josef@toxicpanda.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
List-ID: <linux-block@vger.kernel.org>

From: Josef Bacik <jbacik@fb.com>

When we stopped relying on the bdev everywhere I broke updating the
block device size on the fly, which ceph relies on.  We can't just do
set_capacity, we also have to do bd_set_size so things like parted will
notice the device size change.

Fixes: 29eaadc ("nbd: stop using the bdev everywhere")
cc: stable@vger.kernel.org
Signed-off-by: Josef Bacik <jbacik@fb.com>
---
 drivers/block/nbd.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index 1520383b12f6..61dd95aa47fa 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -231,9 +231,15 @@ static void nbd_size_clear(struct nbd_device *nbd)
 static void nbd_size_update(struct nbd_device *nbd)
 {
 	struct nbd_config *config = nbd->config;
+	struct block_device *bdev = bdget_disk(nbd->disk, 0);
+
 	blk_queue_logical_block_size(nbd->disk->queue, config->blksize);
 	blk_queue_physical_block_size(nbd->disk->queue, config->blksize);
 	set_capacity(nbd->disk, config->bytesize >> 9);
+	if (bdev) {
+		bd_set_size(bdev, config->bytesize);
+		bdput(bdev);
+	}
 	kobject_uevent(&nbd_to_dev(nbd)->kobj, KOBJ_CHANGE);
 }
 
@@ -1111,7 +1117,6 @@ static int nbd_start_device_ioctl(struct nbd_device *nbd, struct block_device *b
 	if (ret)
 		return ret;
 
-	bd_set_size(bdev, config->bytesize);
 	if (max_part)
 		bdev->bd_invalidated = 1;
 	mutex_unlock(&nbd->config_lock);
-- 
2.14.3

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <alex@iss-integration.com>
MIME-Version: 1.0
In-Reply-To: <20180413160334.5496-1-josef@toxicpanda.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
From: Alex Gorbachev <alex@iss-integration.com>
Date: Fri, 13 Apr 2018 23:03:12 -0400
Message-ID: <CAM1qzF5DOT1-RwpkwtG4X4MHtTgttz1mLLmXvhg_qyvN_h5=2Q@mail.gmail.com>
Subject: Re: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
To: Josef Bacik <josef@toxicpanda.com>
Cc: axboe@kernel.dk, nbd@other.debian.org, linux-block@vger.kernel.org,
	kernel-team@fb.com, Josef Bacik <jbacik@fb.com>, stable@vger.kernel.org
Content-Type: multipart/alternative; boundary="f4f5e80ecc14bbd6470569c63aa4"
List-ID: <linux-block@vger.kernel.org>

--f4f5e80ecc14bbd6470569c63aa4
Content-Type: text/plain; charset="UTF-8"

On Fri, Apr 13, 2018 at 12:03 PM, Josef Bacik <josef@toxicpanda.com> wrote:

> From: Josef Bacik <jbacik@fb.com>
>
> This fixes a use after free bug, we need to do the del_gendisk after we
> cleanup the queue on the device.
>

The three patches Josef submitted today fix the problem with the online
rbd-nbd expansion completely.

Fixes: c6a4759ea0c9 ("nbd: add device refcounting")

> cc: stable@vger.kernel.org

> Signed-off-by: Josef Bacik <jbacik@fb.com>
Tested-by: Alex Gorbachev <ag@iss-integration.com>

> ---

>  drivers/block/nbd.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

>
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c

> index 86258b00a1d4..e33da3e6aa20 100644

> --- a/drivers/block/nbd.c

> +++ b/drivers/block/nbd.c

> @@ -174,8 +174,8 @@ static void nbd_dev_remove(struct nbd_device *nbd)

>  {

>         struct gendisk *disk = nbd->disk;

>         if (disk) {

> -               del_gendisk(disk);

>                 blk_cleanup_queue(disk->queue);

> +               del_gendisk(disk);

>                 blk_mq_free_tag_set(&nbd->tag_set);

>                 disk->private_data = NULL;

>                 put_disk(disk);

> --

> 2.14.3

>
>

--f4f5e80ecc14bbd6470569c63aa4
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_extra"><div><div class=3D"gmail_signat=
ure" data-smartmail=3D"gmail_signature"><div dir=3D"ltr"><div dir=3D"ltr"><=
div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><div dir=
=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr=
"><div dir=3D"ltr">On Fri, Apr 13, 2018 at 12:03 PM, Josef Bacik <span dir=
=3D"ltr">&lt;<a href=3D"mailto:josef@toxicpanda.com" target=3D"_blank">jose=
f@toxicpanda.com</a>&gt;</span> wrote:<br></div></div></div></div></div></d=
iv></div></div></div></div></div></div></div></div><div class=3D"gmail_quot=
e"><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left=
:1px #ccc solid;padding-left:1ex">From: Josef Bacik &lt;<a href=3D"mailto:j=
bacik@fb.com">jbacik@fb.com</a>&gt;<br>
<br>
This fixes a use after free bug, we need to do the del_gendisk after we<br>
cleanup the queue on the device.<br></blockquote><div><br></div><div>

<span style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;font-size:1=
2.8px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:nor=
mal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;=
text-transform:none;white-space:normal;word-spacing:0px;background-color:rg=
b(255,255,255);text-decoration-style:initial;text-decoration-color:initial;=
float:none;display:inline">The three patches Josef submitted today fix the =
problem with the online rbd-nbd expansion completely.</span>=C2=A0</div><di=
v>=C2=A0
=C2=A0</div>Fixes: c6a4759ea0c9 (&quot;nbd: add device refcounting&quot;)<b=
r><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1=
px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-l=
eft:1ex;padding-right:1ex"></blockquote>cc: <a href=3D"mailto:stable@vger.k=
ernel.org">stable@vger.kernel.org</a><br><blockquote class=3D"gmail_quote" =
style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-rig=
ht:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockqu=
ote>Signed-off-by: Josef Bacik &lt;<a href=3D"mailto:jbacik@fb.com">jbacik@=
fb.com</a>&gt;</div><div class=3D"gmail_quote">

<span style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;font-size:1=
2.8px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:nor=
mal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;=
text-transform:none;white-space:normal;word-spacing:0px;background-color:rg=
b(255,255,255);text-decoration-style:initial;text-decoration-color:initial;=
float:none;display:inline">Tested-by: Alex Gorbachev &lt;</span><a href=3D"=
mailto:ag@iss-integration.com" target=3D"_blank" style=3D"color:rgb(17,85,2=
04);font-family:arial,sans-serif;font-size:12.8px;font-style:normal;font-va=
riant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spac=
ing:normal;text-align:start;text-indent:0px;text-transform:none;white-space=
:normal;word-spacing:0px;background-color:rgb(255,255,255)">ag@iss-integrat=
ion.com</a><span style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;=
font-size:12.8px;font-style:normal;font-variant-ligatures:normal;font-varia=
nt-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-=
indent:0px;text-transform:none;white-space:normal;word-spacing:0px;backgrou=
nd-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-col=
or:initial;float:none;display:inline">&gt;</span>

<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left=
:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding=
-left:1ex;padding-right:1ex"></blockquote>---<br><blockquote class=3D"gmail=
_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bo=
rder-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"><=
/blockquote>=C2=A0drivers/block/nbd.c | 2 +-<br><blockquote class=3D"gmail_=
quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bor=
der-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></=
blockquote>=C2=A01 file changed, 1 insertion(+), 1 deletion(-)<br><blockquo=
te class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px =
solid rgb(204,204,204);padding-left:1ex">
</blockquote><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8e=
x;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204=
,204);padding-left:1ex;padding-right:1ex"></blockquote>diff --git a/drivers=
/block/nbd.c b/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" sty=
le=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:=
1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote=
>index 86258b00a1d4..e33da3e6aa20 100644<br><blockquote class=3D"gmail_quot=
e" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-=
right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></bloc=
kquote>--- a/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" style=
=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1p=
x solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=
++ b/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" style=3D"marg=
in:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid =
rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>@@ -174,8=
 +174,8 @@ static void nbd_dev_remove(struct nbd_device *nbd)<br><blockquot=
e class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb=
(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;padd=
ing-right:1ex"></blockquote>=C2=A0{<br><blockquote class=3D"gmail_quote" st=
yle=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right=
:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquot=
e>=C2=A0 =C2=A0 =C2=A0 =C2=A0 struct gendisk *disk =3D nbd-&gt;disk;<br><bl=
ockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px so=
lid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1=
ex;padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 if (disk) {<=
br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:=
1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-=
left:1ex;padding-right:1ex"></blockquote>-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0del_gendisk(disk);<br><blockquote class=3D"gmail_qu=
ote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);borde=
r-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></bl=
ockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 blk_cleanu=
p_queue(disk-&gt;queue)<wbr>;<br><blockquote class=3D"gmail_quote" style=3D=
"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px s=
olid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0del_gendisk(disk);<br><=
blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px =
solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left=
:1ex;padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 blk_mq_free_tag_set(&amp;nbd-&gt;tag_<wbr>set);<br><block=
quote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid=
 rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;=
padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 disk-&gt;private_data =3D NULL;<br><blockquote class=3D"gmail=
_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bo=
rder-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"><=
/blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 put_dis=
k(disk);<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0=
.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
</blockquote><span style=3D"color:rgb(136,136,136)">--=C2=A0</span><br><blo=
ckquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px sol=
id rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1e=
x;padding-right:1ex"></blockquote><span style=3D"color:rgb(136,136,136)">2.=
14.3</span><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0p=
x 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span clas=
s=3D"HOEnZb"><font color=3D"#888888">
<br>
</font></span></blockquote></div><br></div></div>

--f4f5e80ecc14bbd6470569c63aa4--

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <prvs=6348fa43e=Bart.VanAssche@wdc.com>
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "josef@toxicpanda.com" <josef@toxicpanda.com>
CC: "kernel-team@fb.com" <kernel-team@fb.com>, "jbacik@fb.com"
	<jbacik@fb.com>, "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"alex@iss-integration.com" <alex@iss-integration.com>, "nbd@other.debian.org"
	<nbd@other.debian.org>, "axboe@kernel.dk" <axboe@kernel.dk>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
Date: Fri, 13 Apr 2018 16:25:33 +0000
Message-ID: <1f0ebe74a96c6515716a25b49ea588f9ee28e137.camel@wdc.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
	 <0a8b3a85910e7bbe0103120e85844e96341d73ec.camel@wdc.com>
	 <20180413162132.xudowhvxyijbzqqa@destiny>
In-Reply-To: <20180413162132.xudowhvxyijbzqqa@destiny>
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
List-ID: <linux-block@vger.kernel.org>

T24gRnJpLCAyMDE4LTA0LTEzIGF0IDEyOjIxIC0wNDAwLCBKb3NlZiBCYWNpayB3cm90ZToNCj4g
T24gRnJpLCBBcHIgMTMsIDIwMTggYXQgMDQ6MTY6MThQTSArMDAwMCwgQmFydCBWYW4gQXNzY2hl
IHdyb3RlOg0KPiA+IE9uIEZyaSwgMjAxOC0wNC0xMyBhdCAxMjowMyAtMDQwMCwgSm9zZWYgQmFj
aWsgd3JvdGU6DQo+ID4gPiBUaGlzIGZpeGVzIGEgdXNlIGFmdGVyIGZyZWUgYnVnLCB3ZSBuZWVk
IHRvIGRvIHRoZSBkZWxfZ2VuZGlzayBhZnRlciB3ZQ0KPiA+ID4gY2xlYW51cCB0aGUgcXVldWUg
b24gdGhlIGRldmljZS4NCj4gPiANCj4gPiBIZWxsbyBKb3NlZiwNCj4gPiANCj4gPiBXaGljaCB1
c2UtYWZ0ZXItZnJlZSBidWcgZG9lcyB0aGlzIHBhdGNoIGZpeD8gQXJlIHlvdSBhd2FyZSB0aGF0
IG1vc3QgYmxvY2sNCj4gPiBkcml2ZXJzIGNhbGwgYmxrX2NsZWFudXBfcXVldWUoKSBiZWZvcmUg
Y2FsbGluZyBkZWxfZ2VuZGlzaygpPyBXaHkgZG8geW91DQo+ID4gdGhpbmsgdGhhdCB0aGUgbmJk
IGRyaXZlciBzaG91bGQgdXNlIHRoZSBvcHBvc2l0ZSBvcmRlcj8NCj4gDQo+IEknbSBjb25mdXNl
ZCwgdGhhdCdzIHdoYXQgdGhpcyBwYXRjaCBkb2VzLiAgQmVmb3JlIEkgaGFkIGRlbF9nZW5kaXNr
KCkgZmlyc3QgYW5kDQo+IHRoZW4gdGhlIGJsa19jbGVhbnVwX3F1ZXVlKCksIHdoaWNoIHdhcyBi
dWdnaW5nIG91dCB3aGVuIEkgd2FzIHRlc3Rpbmcgc3R1ZmYNCj4gd2l0aCBhIG51bGwgcG9pbnRl
ciBkZXJlZiB3aGVuZXZlciBJIHJtbW9kJ2VkIHRoZSBuYmQuICBTd2FwcGluZyBpdCB0byB0aGUg
d2F5DQo+IGV2ZXJ5Ym9keSBlbHNlIGRpZCBpdCBmaXhlZCB0aGUgcHJvYmxlbS4gIFRoYW5rcywN
Cg0KSGVsbG8gSm9zZWYsDQoNCk9vcHMsIEkgc3dhcHBlZCAiYmxrX2NsZWFudXBfcXVldWUoKSIg
YW5kICJkZWxfZ2VuZGlzaygpIiBpbiBteSBlLW1haWwuDQoNCkNhbiB5b3Ugc2hhcmUgdGhlIGNh
bGwgc3RhY2sgb2YgdGhlIE5VTEwgcG9pbnRlciBkZXJlZiBhbmQgYWxzbyB0aGUgdHJhbnNsYXRp
b24NCm9mIHRoZSBjcmFzaCBhZGRyZXNzIGludG8gYSBzb3VyY2UgY29kZSBsaW5lPw0KDQpUaGFu
a3MsDQoNCkJhcnQuDQoNCg0K

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <josef@toxicpanda.com>
Return-Path: <josef@toxicpanda.com>
Date: Fri, 13 Apr 2018 12:21:33 -0400
From: Josef Bacik <josef@toxicpanda.com>
To: Bart Van Assche <Bart.VanAssche@wdc.com>
Cc: "kernel-team@fb.com" <kernel-team@fb.com>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"josef@toxicpanda.com" <josef@toxicpanda.com>,
	"nbd@other.debian.org" <nbd@other.debian.org>,
	"axboe@kernel.dk" <axboe@kernel.dk>,
	"alex@iss-integration.com" <alex@iss-integration.com>,
	"jbacik@fb.com" <jbacik@fb.com>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
Message-ID: <20180413162132.xudowhvxyijbzqqa@destiny>
References: <20180413160334.5496-1-josef@toxicpanda.com>
 <0a8b3a85910e7bbe0103120e85844e96341d73ec.camel@wdc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
In-Reply-To: <0a8b3a85910e7bbe0103120e85844e96341d73ec.camel@wdc.com>
List-ID: <linux-block@vger.kernel.org>

On Fri, Apr 13, 2018 at 04:16:18PM +0000, Bart Van Assche wrote:
> On Fri, 2018-04-13 at 12:03 -0400, Josef Bacik wrote:
> > This fixes a use after free bug, we need to do the del_gendisk after we
> > cleanup the queue on the device.
> 
> Hello Josef,
> 
> Which use-after-free bug does this patch fix? Are you aware that most block
> drivers call blk_cleanup_queue() before calling del_gendisk()? Why do you
> think that the nbd driver should use the opposite order?
> 

I'm confused, that's what this patch does.  Before I had del_gendisk() first and
then the blk_cleanup_queue(), which was bugging out when I was testing stuff
with a null pointer deref whenever I rmmod'ed the nbd.  Swapping it to the way
everybody else did it fixed the problem.  Thanks,

Josef

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <prvs=6348fa43e=Bart.VanAssche@wdc.com>
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "kernel-team@fb.com" <kernel-team@fb.com>, "linux-block@vger.kernel.org"
	<linux-block@vger.kernel.org>, "josef@toxicpanda.com" <josef@toxicpanda.com>,
	"nbd@other.debian.org" <nbd@other.debian.org>, "axboe@kernel.dk"
	<axboe@kernel.dk>, "alex@iss-integration.com" <alex@iss-integration.com>
CC: "jbacik@fb.com" <jbacik@fb.com>, "stable@vger.kernel.org"
	<stable@vger.kernel.org>
Subject: Re: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
Date: Fri, 13 Apr 2018 16:16:18 +0000
Message-ID: <0a8b3a85910e7bbe0103120e85844e96341d73ec.camel@wdc.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
In-Reply-To: <20180413160334.5496-1-josef@toxicpanda.com>
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
List-ID: <linux-block@vger.kernel.org>

T24gRnJpLCAyMDE4LTA0LTEzIGF0IDEyOjAzIC0wNDAwLCBKb3NlZiBCYWNpayB3cm90ZToNCj4g
VGhpcyBmaXhlcyBhIHVzZSBhZnRlciBmcmVlIGJ1Zywgd2UgbmVlZCB0byBkbyB0aGUgZGVsX2dl
bmRpc2sgYWZ0ZXIgd2UNCj4gY2xlYW51cCB0aGUgcXVldWUgb24gdGhlIGRldmljZS4NCg0KSGVs
bG8gSm9zZWYsDQoNCldoaWNoIHVzZS1hZnRlci1mcmVlIGJ1ZyBkb2VzIHRoaXMgcGF0Y2ggZml4
PyBBcmUgeW91IGF3YXJlIHRoYXQgbW9zdCBibG9jaw0KZHJpdmVycyBjYWxsIGJsa19jbGVhbnVw
X3F1ZXVlKCkgYmVmb3JlIGNhbGxpbmcgZGVsX2dlbmRpc2soKT8gV2h5IGRvIHlvdQ0KdGhpbmsg
dGhhdCB0aGUgbmJkIGRyaXZlciBzaG91bGQgdXNlIHRoZSBvcHBvc2l0ZSBvcmRlcj8NCg0KVGhh
bmtzLA0KDQpCYXJ0Lg0KDQo=

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <josef@toxicpanda.com>
Return-Path: <josef@toxicpanda.com>
From: Josef Bacik <josef@toxicpanda.com>
To: axboe@kernel.dk,
	nbd@other.debian.org,
	linux-block@vger.kernel.org,
	kernel-team@fb.com,
	alex@iss-integration.com
Cc: Josef Bacik <jbacik@fb.com>,
	stable@vger.kernel.org
Subject: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
Date: Fri, 13 Apr 2018 12:03:32 -0400
Message-Id: <20180413160334.5496-1-josef@toxicpanda.com>
List-ID: <linux-block@vger.kernel.org>

From: Josef Bacik <jbacik@fb.com>

This fixes a use after free bug, we need to do the del_gendisk after we
cleanup the queue on the device.

Fixes: c6a4759ea0c9 ("nbd: add device refcounting")
cc: stable@vger.kernel.org
Signed-off-by: Josef Bacik <jbacik@fb.com>
---
 drivers/block/nbd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index 86258b00a1d4..e33da3e6aa20 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -174,8 +174,8 @@ static void nbd_dev_remove(struct nbd_device *nbd)
 {
 	struct gendisk *disk = nbd->disk;
 	if (disk) {
-		del_gendisk(disk);
 		blk_cleanup_queue(disk->queue);
+		del_gendisk(disk);
 		blk_mq_free_tag_set(&nbd->tag_set);
 		disk->private_data = NULL;
 		put_disk(disk);
-- 
2.14.3

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <alex@iss-integration.com>
MIME-Version: 1.0
In-Reply-To: <20180413160334.5496-2-josef@toxicpanda.com>
References: <20180413160334.5496-1-josef@toxicpanda.com> <20180413160334.5496-2-josef@toxicpanda.com>
From: Alex Gorbachev <alex@iss-integration.com>
Date: Fri, 13 Apr 2018 23:02:33 -0400
Message-ID: <CAM1qzF7dMiFcNXnmd+PB0w29EnQ2KTEHAvtU=PdhPnHav3rpUQ@mail.gmail.com>
Subject: Re: [PATCH 2/3] nbd: update size when connected
To: Josef Bacik <josef@toxicpanda.com>
Cc: axboe@kernel.dk, nbd@other.debian.org, linux-block@vger.kernel.org,
	kernel-team@fb.com, Josef Bacik <jbacik@fb.com>, stable@vger.kernel.org
Content-Type: multipart/alternative; boundary="0000000000006172df0569c63832"
List-ID: <linux-block@vger.kernel.org>

--0000000000006172df0569c63832
Content-Type: text/plain; charset="UTF-8"

On Fri, Apr 13, 2018 at 12:03 PM, Josef Bacik <josef@toxicpanda.com> wrote:

> From: Josef Bacik <jbacik@fb.com>
>
> I messed up changing the size of an NBD device while it was connected by
> not actually updating the device or doing the uevent.  Fix this by
> updating everything if we're connected and we change the size.
>

The three patches Josef submitted today fix the problem with the online
rbd-nbd expansion completely.


cc: stable@vger.kernel.org

> Fixes: 639812a ("nbd: don't set the device size until we're connected")

> Signed-off-by: Josef Bacik <jbacik@fb.com>
Tested-by: Alex Gorbachev <ag@iss-integration.com>

> ---

>  drivers/block/nbd.c | 2 ++

>  1 file changed, 2 insertions(+)

>
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c

> index e33da3e6aa20..1520383b12f6 100644

> --- a/drivers/block/nbd.c

> +++ b/drivers/block/nbd.c

> @@ -243,6 +243,8 @@ static void nbd_size_set(struct nbd_device *nbd,
loff_t blocksize,

>         struct nbd_config *config = nbd->config;

>         config->blksize = blocksize;

>         config->bytesize = blocksize * nr_blocks;

> +       if (nbd->task_recv != NULL)

> +               nbd_size_update(nbd);

>  }

>
 static void nbd_complete_rq(struct request *req)

> --

> 2.14.3

>
>

--0000000000006172df0569c63832
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_extra"><div class=3D"gmail_quote">On F=
ri, Apr 13, 2018 at 12:03 PM, Josef Bacik <span dir=3D"ltr">&lt;<a href=3D"=
mailto:josef@toxicpanda.com" target=3D"_blank">josef@toxicpanda.com</a>&gt;=
</span> wrote:<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px=
 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: J=
osef Bacik &lt;<a href=3D"mailto:jbacik@fb.com">jbacik@fb.com</a>&gt;<br>
<br>
I messed up changing the size of an NBD device while it was connected by<br=
>
not actually updating the device or doing the uevent.=C2=A0 Fix this by<br>
updating everything if we&#39;re connected and we change the size.<br></blo=
ckquote><div><br></div><div>

<span style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;font-size:1=
2.8px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:nor=
mal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;=
text-transform:none;white-space:normal;word-spacing:0px;background-color:rg=
b(255,255,255);text-decoration-style:initial;text-decoration-color:initial;=
float:none;display:inline">The three patches Josef submitted today fix the =
problem with the online rbd-nbd expansion completely.</span>

<br></div><div>=C2=A0</div><br><blockquote class=3D"gmail_quote" style=3D"m=
argin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px sol=
id rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>cc: <a=
 href=3D"mailto:stable@vger.kernel.org">stable@vger.kernel.org</a><br><bloc=
kquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px soli=
d rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex=
;padding-right:1ex"></blockquote>Fixes: 639812a (&quot;nbd: don&#39;t set t=
he device size until we&#39;re connected&quot;)<br><blockquote class=3D"gma=
il_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);=
border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"=
></blockquote>Signed-off-by: Josef Bacik &lt;<a href=3D"mailto:jbacik@fb.co=
m">jbacik@fb.com</a>&gt;</div><div class=3D"gmail_quote">

<span style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;font-size:s=
mall;font-style:normal;font-variant-ligatures:normal;font-variant-caps:norm=
al;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;t=
ext-transform:none;white-space:normal;word-spacing:0px;background-color:rgb=
(255,255,255);text-decoration-style:initial;text-decoration-color:initial;f=
loat:none;display:inline">Tested-by: Alex Gorbachev &lt;<a href=3D"mailto:a=
g@iss-integration.com">ag@iss-integration.com</a>&gt;</span>

<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left=
:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding=
-left:1ex;padding-right:1ex"></blockquote>---<br><blockquote class=3D"gmail=
_quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bo=
rder-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"><=
/blockquote>=C2=A0drivers/block/nbd.c | 2 ++<br><blockquote class=3D"gmail_=
quote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);bor=
der-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></=
blockquote>=C2=A01 file changed, 2 insertions(+)<br><blockquote class=3D"gm=
ail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,=
204,204);padding-left:1ex">
</blockquote><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8e=
x;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204=
,204);padding-left:1ex;padding-right:1ex"></blockquote>diff --git a/drivers=
/block/nbd.c b/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" sty=
le=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:=
1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote=
>index e33da3e6aa20..1520383b12f6 100644<br><blockquote class=3D"gmail_quot=
e" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-=
right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></bloc=
kquote>--- a/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" style=
=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1p=
x solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>+=
++ b/drivers/block/nbd.c<br><blockquote class=3D"gmail_quote" style=3D"marg=
in:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid =
rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>@@ -243,6=
 +243,8 @@ static void nbd_size_set(struct nbd_device *nbd, loff_t blocksiz=
e,<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-le=
ft:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);paddi=
ng-left:1ex;padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 str=
uct nbd_config *config =3D nbd-&gt;config;<br><blockquote class=3D"gmail_qu=
ote" style=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);borde=
r-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></bl=
ockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 config-&gt;blksize =3D blocksize;<br><=
blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px =
solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left=
:1ex;padding-right:1ex"></blockquote>=C2=A0 =C2=A0 =C2=A0 =C2=A0 config-&gt=
;bytesize =3D blocksize * nr_blocks;<br><blockquote class=3D"gmail_quote" s=
tyle=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-righ=
t:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquo=
te>+=C2=A0 =C2=A0 =C2=A0 =C2=A0if (nbd-&gt;task_recv !=3D NULL)<br><blockqu=
ote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px solid r=
gb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;pa=
dding-right:1ex"></blockquote>+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0nbd_size_update(nbd);<br><blockquote class=3D"gmail_quote" sty=
le=3D"margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:=
1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote=
>=C2=A0}<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0=
.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
</blockquote><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0.8e=
x;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204=
,204);padding-left:1ex;padding-right:1ex"></blockquote>=C2=A0static void nb=
d_complete_rq(struct request *req)<br><blockquote class=3D"gmail_quote" sty=
le=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);paddi=
ng-left:1ex">
</blockquote><span style=3D"color:rgb(136,136,136)">--=C2=A0</span><br><blo=
ckquote class=3D"gmail_quote" style=3D"margin:0px 0.8ex;border-left:1px sol=
id rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1e=
x;padding-right:1ex"></blockquote><span style=3D"color:rgb(136,136,136)">2.=
14.3</span><br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0p=
x 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span clas=
s=3D"HOEnZb"><font color=3D"#888888">
<br>
</font></span></blockquote></div><br></div></div>

--0000000000006172df0569c63832--

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <josef@toxicpanda.com>
Return-Path: <josef@toxicpanda.com>
From: Josef Bacik <josef@toxicpanda.com>
To: axboe@kernel.dk,
	nbd@other.debian.org,
	linux-block@vger.kernel.org,
	kernel-team@fb.com,
	alex@iss-integration.com
Cc: Josef Bacik <jbacik@fb.com>,
	stable@vger.kernel.org
Subject: [PATCH 2/3] nbd: update size when connected
Date: Fri, 13 Apr 2018 12:03:33 -0400
Message-Id: <20180413160334.5496-2-josef@toxicpanda.com>
In-Reply-To: <20180413160334.5496-1-josef@toxicpanda.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
List-ID: <linux-block@vger.kernel.org>

From: Josef Bacik <jbacik@fb.com>

I messed up changing the size of an NBD device while it was connected by
not actually updating the device or doing the uevent.  Fix this by
updating everything if we're connected and we change the size.

cc: stable@vger.kernel.org
Fixes: 639812a ("nbd: don't set the device size until we're connected")
Signed-off-by: Josef Bacik <jbacik@fb.com>
---
 drivers/block/nbd.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index e33da3e6aa20..1520383b12f6 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -243,6 +243,8 @@ static void nbd_size_set(struct nbd_device *nbd, loff_t blocksize,
 	struct nbd_config *config = nbd->config;
 	config->blksize = blocksize;
 	config->bytesize = blocksize * nr_blocks;
+	if (nbd->task_recv != NULL)
+		nbd_size_update(nbd);
 }
 
 static void nbd_complete_rq(struct request *req)
-- 
2.14.3

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from esa5.hgst.iphmx.com ([216.71.153.144]:50695 "EHLO
        esa5.hgst.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1752339AbeDMQZk (ORCPT
        <rfc822;stable@vger.kernel.org>); Fri, 13 Apr 2018 12:25:40 -0400
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "josef@toxicpanda.com" <josef@toxicpanda.com>
CC: "kernel-team@fb.com" <kernel-team@fb.com>,
        "jbacik@fb.com" <jbacik@fb.com>,
        "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
        "alex@iss-integration.com" <alex@iss-integration.com>,
        "nbd@other.debian.org" <nbd@other.debian.org>,
        "axboe@kernel.dk" <axboe@kernel.dk>,
        "stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
Date: Fri, 13 Apr 2018 16:25:33 +0000
Message-ID: <1f0ebe74a96c6515716a25b49ea588f9ee28e137.camel@wdc.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
         <0a8b3a85910e7bbe0103120e85844e96341d73ec.camel@wdc.com>
         <20180413162132.xudowhvxyijbzqqa@destiny>
In-Reply-To: <20180413162132.xudowhvxyijbzqqa@destiny>
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
Content-ID: <C13FC41EE5EAC346B456FDA770B09213@namprd04.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

T24gRnJpLCAyMDE4LTA0LTEzIGF0IDEyOjIxIC0wNDAwLCBKb3NlZiBCYWNpayB3cm90ZToNCj4g
T24gRnJpLCBBcHIgMTMsIDIwMTggYXQgMDQ6MTY6MThQTSArMDAwMCwgQmFydCBWYW4gQXNzY2hl
IHdyb3RlOg0KPiA+IE9uIEZyaSwgMjAxOC0wNC0xMyBhdCAxMjowMyAtMDQwMCwgSm9zZWYgQmFj
aWsgd3JvdGU6DQo+ID4gPiBUaGlzIGZpeGVzIGEgdXNlIGFmdGVyIGZyZWUgYnVnLCB3ZSBuZWVk
IHRvIGRvIHRoZSBkZWxfZ2VuZGlzayBhZnRlciB3ZQ0KPiA+ID4gY2xlYW51cCB0aGUgcXVldWUg
b24gdGhlIGRldmljZS4NCj4gPiANCj4gPiBIZWxsbyBKb3NlZiwNCj4gPiANCj4gPiBXaGljaCB1
c2UtYWZ0ZXItZnJlZSBidWcgZG9lcyB0aGlzIHBhdGNoIGZpeD8gQXJlIHlvdSBhd2FyZSB0aGF0
IG1vc3QgYmxvY2sNCj4gPiBkcml2ZXJzIGNhbGwgYmxrX2NsZWFudXBfcXVldWUoKSBiZWZvcmUg
Y2FsbGluZyBkZWxfZ2VuZGlzaygpPyBXaHkgZG8geW91DQo+ID4gdGhpbmsgdGhhdCB0aGUgbmJk
IGRyaXZlciBzaG91bGQgdXNlIHRoZSBvcHBvc2l0ZSBvcmRlcj8NCj4gDQo+IEknbSBjb25mdXNl
ZCwgdGhhdCdzIHdoYXQgdGhpcyBwYXRjaCBkb2VzLiAgQmVmb3JlIEkgaGFkIGRlbF9nZW5kaXNr
KCkgZmlyc3QgYW5kDQo+IHRoZW4gdGhlIGJsa19jbGVhbnVwX3F1ZXVlKCksIHdoaWNoIHdhcyBi
dWdnaW5nIG91dCB3aGVuIEkgd2FzIHRlc3Rpbmcgc3R1ZmYNCj4gd2l0aCBhIG51bGwgcG9pbnRl
ciBkZXJlZiB3aGVuZXZlciBJIHJtbW9kJ2VkIHRoZSBuYmQuICBTd2FwcGluZyBpdCB0byB0aGUg
d2F5DQo+IGV2ZXJ5Ym9keSBlbHNlIGRpZCBpdCBmaXhlZCB0aGUgcHJvYmxlbS4gIFRoYW5rcywN
Cg0KSGVsbG8gSm9zZWYsDQoNCk9vcHMsIEkgc3dhcHBlZCAiYmxrX2NsZWFudXBfcXVldWUoKSIg
YW5kICJkZWxfZ2VuZGlzaygpIiBpbiBteSBlLW1haWwuDQoNCkNhbiB5b3Ugc2hhcmUgdGhlIGNh
bGwgc3RhY2sgb2YgdGhlIE5VTEwgcG9pbnRlciBkZXJlZiBhbmQgYWxzbyB0aGUgdHJhbnNsYXRp
b24NCm9mIHRoZSBjcmFzaCBhZGRyZXNzIGludG8gYSBzb3VyY2UgY29kZSBsaW5lPw0KDQpUaGFu
a3MsDQoNCkJhcnQuDQoNCg0K

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from esa3.hgst.iphmx.com ([216.71.153.141]:54888 "EHLO
        esa3.hgst.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1752262AbeDMQQY (ORCPT
        <rfc822;stable@vger.kernel.org>); Fri, 13 Apr 2018 12:16:24 -0400
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "kernel-team@fb.com" <kernel-team@fb.com>,
        "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
        "josef@toxicpanda.com" <josef@toxicpanda.com>,
        "nbd@other.debian.org" <nbd@other.debian.org>,
        "axboe@kernel.dk" <axboe@kernel.dk>,
        "alex@iss-integration.com" <alex@iss-integration.com>
CC: "jbacik@fb.com" <jbacik@fb.com>,
        "stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH 1/3] nbd: del_gendisk after we cleanup the queue
Date: Fri, 13 Apr 2018 16:16:18 +0000
Message-ID: <0a8b3a85910e7bbe0103120e85844e96341d73ec.camel@wdc.com>
References: <20180413160334.5496-1-josef@toxicpanda.com>
In-Reply-To: <20180413160334.5496-1-josef@toxicpanda.com>
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
Content-ID: <BCB223BAAEB305418C6574F8889AD96A@namprd04.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

T24gRnJpLCAyMDE4LTA0LTEzIGF0IDEyOjAzIC0wNDAwLCBKb3NlZiBCYWNpayB3cm90ZToNCj4g
VGhpcyBmaXhlcyBhIHVzZSBhZnRlciBmcmVlIGJ1Zywgd2UgbmVlZCB0byBkbyB0aGUgZGVsX2dl
bmRpc2sgYWZ0ZXIgd2UNCj4gY2xlYW51cCB0aGUgcXVldWUgb24gdGhlIGRldmljZS4NCg0KSGVs
bG8gSm9zZWYsDQoNCldoaWNoIHVzZS1hZnRlci1mcmVlIGJ1ZyBkb2VzIHRoaXMgcGF0Y2ggZml4
PyBBcmUgeW91IGF3YXJlIHRoYXQgbW9zdCBibG9jaw0KZHJpdmVycyBjYWxsIGJsa19jbGVhbnVw
X3F1ZXVlKCkgYmVmb3JlIGNhbGxpbmcgZGVsX2dlbmRpc2soKT8gV2h5IGRvIHlvdQ0KdGhpbmsg
dGhhdCB0aGUgbmJkIGRyaXZlciBzaG91bGQgdXNlIHRoZSBvcHBvc2l0ZSBvcmRlcj8NCg0KVGhh
bmtzLA0KDQpCYXJ0Lg0KDQo=