Re: [Nbd] [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Pranay Srivastava
On Wed, Jun 15, 2016 at 12:30 PM, Wouter Verhelst wrote: > On Wed, Jun 15, 2016 at 08:30:45AM +0200, Markus Pargmann wrote: >> Thanks for the explanations. I think my understanding was off by one ;).. >> I didn't realize that the DO_IT thread from the userspace has the block >>

Re: [Nbd] [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Pranay Srivastava
On Wed, Jun 15, 2016 at 12:30 PM, Wouter Verhelst wrote: > On Wed, Jun 15, 2016 at 08:30:45AM +0200, Markus Pargmann wrote: >> Thanks for the explanations. I think my understanding was off by one ;).. >> I didn't realize that the DO_IT thread from the userspace has the block >> device open as

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Pranay Srivastava
Hey Markus, On Wed, Jun 15, 2016 at 12:00 PM, Markus Pargmann wrote: > Hi Pranay, > > On Tuesday 14 June 2016 15:03:40 Pranay Srivastava wrote: >> Hi Markus, >> >> On Tue, Jun 14, 2016 at 2:29 PM, Markus Pargmann wrote: >> > >> > On Thursday 02 June

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Pranay Srivastava
Hey Markus, On Wed, Jun 15, 2016 at 12:00 PM, Markus Pargmann wrote: > Hi Pranay, > > On Tuesday 14 June 2016 15:03:40 Pranay Srivastava wrote: >> Hi Markus, >> >> On Tue, Jun 14, 2016 at 2:29 PM, Markus Pargmann wrote: >> > >> > On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: >>

Re: [Nbd] [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Wouter Verhelst
On Wed, Jun 15, 2016 at 08:30:45AM +0200, Markus Pargmann wrote: > Thanks for the explanations. I think my understanding was off by one ;).. > I didn't realize that the DO_IT thread from the userspace has the block > device open as well. Obviously, otherwise it couldn't do an ioctl() to it :-) >

Re: [Nbd] [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Wouter Verhelst
On Wed, Jun 15, 2016 at 08:30:45AM +0200, Markus Pargmann wrote: > Thanks for the explanations. I think my understanding was off by one ;).. > I didn't realize that the DO_IT thread from the userspace has the block > device open as well. Obviously, otherwise it couldn't do an ioctl() to it :-) >

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Markus Pargmann
Hi Pranay, On Tuesday 14 June 2016 15:03:40 Pranay Srivastava wrote: > Hi Markus, > > On Tue, Jun 14, 2016 at 2:29 PM, Markus Pargmann wrote: > > > > On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: > > > When a timeout occurs or a recv fails, then > > >

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-15 Thread Markus Pargmann
Hi Pranay, On Tuesday 14 June 2016 15:03:40 Pranay Srivastava wrote: > Hi Markus, > > On Tue, Jun 14, 2016 at 2:29 PM, Markus Pargmann wrote: > > > > On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: > > > When a timeout occurs or a recv fails, then > > > instead of abruplty

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-14 Thread Pranay Srivastava
Hi Markus, On Tue, Jun 14, 2016 at 2:29 PM, Markus Pargmann wrote: > > On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: > > When a timeout occurs or a recv fails, then > > instead of abruplty killing nbd block device > > wait for it's users to finish. > > > >

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-14 Thread Pranay Srivastava
Hi Markus, On Tue, Jun 14, 2016 at 2:29 PM, Markus Pargmann wrote: > > On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: > > When a timeout occurs or a recv fails, then > > instead of abruplty killing nbd block device > > wait for it's users to finish. > > > > This is more required

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-14 Thread Markus Pargmann
On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: > When a timeout occurs or a recv fails, then > instead of abruplty killing nbd block device > wait for it's users to finish. > > This is more required when filesystem(s) like > ext2 or ext3 don't expect their buffer heads to >

Re: [PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-14 Thread Markus Pargmann
On Thursday 02 June 2016 13:25:00 Pranay Kr. Srivastava wrote: > When a timeout occurs or a recv fails, then > instead of abruplty killing nbd block device > wait for it's users to finish. > > This is more required when filesystem(s) like > ext2 or ext3 don't expect their buffer heads to >

[PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-02 Thread Pranay Kr. Srivastava
When a timeout occurs or a recv fails, then instead of abruplty killing nbd block device wait for it's users to finish. This is more required when filesystem(s) like ext2 or ext3 don't expect their buffer heads to disappear while the filesystem is mounted. Each open of a nbd device is

[PATCH v2 4/5]nbd: make nbd device wait for its users.

2016-06-02 Thread Pranay Kr. Srivastava
When a timeout occurs or a recv fails, then instead of abruplty killing nbd block device wait for it's users to finish. This is more required when filesystem(s) like ext2 or ext3 don't expect their buffer heads to disappear while the filesystem is mounted. Each open of a nbd device is