Re: [RFC PATCH v2 0/2] block: fix backing_dev_info lifetime

2017-01-26 Thread Dan Williams
On Thu, Jan 26, 2017 at 5:17 AM, Christoph Hellwig wrote: > On Thu, Jan 26, 2017 at 11:06:53AM +0100, Jan Kara wrote: >> Yeah, so my patches (and I suspect your as well), have a problem when the >> backing_device_info stays around because blkdev inode still exists, device >> gets

Re: [RFC PATCH v2 0/2] block: fix backing_dev_info lifetime

2017-01-26 Thread Jan Kara
On Wed 25-01-17 13:43:58, Dan Williams wrote: > On Mon, Jan 23, 2017 at 1:17 PM, Thiago Jung Bauermann > wrote: > > Hello Dan, > > > > Am Freitag, 6. Januar 2017, 17:02:51 BRST schrieb Dan Williams: > >> v1 of these changes [1] was a one line change to

Re: [RFC PATCH v2 0/2] block: fix backing_dev_info lifetime

2017-01-25 Thread Dan Williams
On Mon, Jan 23, 2017 at 1:17 PM, Thiago Jung Bauermann wrote: > Hello Dan, > > Am Freitag, 6. Januar 2017, 17:02:51 BRST schrieb Dan Williams: >> v1 of these changes [1] was a one line change to bdev_get_queue() to >> prevent a shutdown crash when del_gendisk() races

Re: [RFC PATCH v2 0/2] block: fix backing_dev_info lifetime

2017-01-23 Thread Thiago Jung Bauermann
Hello Dan, Am Freitag, 6. Januar 2017, 17:02:51 BRST schrieb Dan Williams: > v1 of these changes [1] was a one line change to bdev_get_queue() to > prevent a shutdown crash when del_gendisk() races the final > __blkdev_put(). > > While it is known at del_gendisk() time that the queue is still

[RFC PATCH v2 0/2] block: fix backing_dev_info lifetime

2017-01-06 Thread Dan Williams
v1 of these changes [1] was a one line change to bdev_get_queue() to prevent a shutdown crash when del_gendisk() races the final __blkdev_put(). While it is known at del_gendisk() time that the queue is still alive, Jan Kara points to other paths [2] that are racing __blkdev_put() where the