Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-31 Thread Peter Lieven
On 29.01.2014 17:19, Benoît Canet wrote: Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : This patch adds native support for accessing images on NFS shares without the requirement to actually mount the entire NFS share on the host. NFS Images can simply be specified by an url

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-31 Thread Peter Lieven
On 31.01.2014 11:46, Stefan Hajnoczi wrote: On Fri, Jan 31, 2014 at 10:11 AM, Peter Lieven wrote: On 31.01.2014 09:57, Stefan Hajnoczi wrote: On Thu, Jan 30, 2014 at 10:35 PM, Peter Lieven wrote: Am 30.01.2014 um 15:22 schrieb Stefan Hajnoczi : On Wed, Jan 29, 2014 at 05:19:59PM +0100, Ben

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-31 Thread Stefan Hajnoczi
On Fri, Jan 31, 2014 at 10:11 AM, Peter Lieven wrote: > On 31.01.2014 09:57, Stefan Hajnoczi wrote: >> >> On Thu, Jan 30, 2014 at 10:35 PM, Peter Lieven wrote: >>> >>> Am 30.01.2014 um 15:22 schrieb Stefan Hajnoczi : >>> On Wed, Jan 29, 2014 at 05:19:59PM +0100, Benoīt Canet wrote: > >>>

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-31 Thread Peter Lieven
On 31.01.2014 09:57, Stefan Hajnoczi wrote: On Thu, Jan 30, 2014 at 10:35 PM, Peter Lieven wrote: Am 30.01.2014 um 15:22 schrieb Stefan Hajnoczi : On Wed, Jan 29, 2014 at 05:19:59PM +0100, Benoît Canet wrote: Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : +static int nf

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-31 Thread Stefan Hajnoczi
On Thu, Jan 30, 2014 at 10:35 PM, Peter Lieven wrote: > > Am 30.01.2014 um 15:22 schrieb Stefan Hajnoczi : > >> On Wed, Jan 29, 2014 at 05:19:59PM +0100, Benoît Canet wrote: >>> Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : +static int nfs_file_open(BlockDriverState *bs,

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-30 Thread Peter Lieven
Am 30.01.2014 um 15:22 schrieb Stefan Hajnoczi : > On Wed, Jan 29, 2014 at 05:19:59PM +0100, Benoît Canet wrote: >> Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : > > Hi Peter, > If I read your reply to Benoit correctly, you only addressed the > questions about nfs_client_cl

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-30 Thread Peter Lieven
Am 30.01.2014 um 15:23 schrieb Stefan Hajnoczi : > On Thu, Jan 30, 2014 at 10:12:35AM +0100, Peter Lieven wrote: >> >> Am 30.01.2014 um 10:05 schrieb Stefan Hajnoczi : >> >>> On Wed, Jan 29, 2014 at 05:38:29PM +0100, Peter Lieven wrote: On 29.01.2014 17:19, Benoît Canet wrote: > Le Wed

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-30 Thread Stefan Hajnoczi
On Thu, Jan 30, 2014 at 10:12:35AM +0100, Peter Lieven wrote: > > Am 30.01.2014 um 10:05 schrieb Stefan Hajnoczi : > > > On Wed, Jan 29, 2014 at 05:38:29PM +0100, Peter Lieven wrote: > >> On 29.01.2014 17:19, Benoît Canet wrote: > >>> Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a éc

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-30 Thread Stefan Hajnoczi
On Wed, Jan 29, 2014 at 05:19:59PM +0100, Benoît Canet wrote: > Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : Hi Peter, If I read your reply to Benoit correctly, you only addressed the questions about nfs_client_close(). Here are Benoits other comments and my thoughts on the

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-30 Thread Peter Lieven
Am 30.01.2014 um 10:05 schrieb Stefan Hajnoczi : > On Wed, Jan 29, 2014 at 05:38:29PM +0100, Peter Lieven wrote: >> On 29.01.2014 17:19, Benoît Canet wrote: >>> Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : This patch adds native support for accessing images on NFS

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-30 Thread Stefan Hajnoczi
On Wed, Jan 29, 2014 at 05:38:29PM +0100, Peter Lieven wrote: > On 29.01.2014 17:19, Benoît Canet wrote: > >Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : > >>This patch adds native support for accessing images on NFS > >>shares without the requirement to actually mount the ent

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-29 Thread Peter Lieven
On 29.01.2014 17:19, Benoît Canet wrote: Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : This patch adds native support for accessing images on NFS shares without the requirement to actually mount the entire NFS share on the host. NFS Images can simply be specified by an url

Re: [Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-29 Thread Benoît Canet
Le Wednesday 29 Jan 2014 à 09:50:21 (+0100), Peter Lieven a écrit : > This patch adds native support for accessing images on NFS > shares without the requirement to actually mount the entire > NFS share on the host. > > NFS Images can simply be specified by an url of the form: > nfs:[?param=va

[Qemu-devel] [PATCHv7 1/5] block: add native support for NFS

2014-01-29 Thread Peter Lieven
This patch adds native support for accessing images on NFS shares without the requirement to actually mount the entire NFS share on the host. NFS Images can simply be specified by an url of the form: nfs:[?param=value[¶m2=value2[&...]]] For example: qemu-img create -f qcow2 nfs://10.0.0.1/qem