Re: [Qemu-devel] [PATCH 1/3] nbd: Drop BDS backpointer

2015-02-03 Thread Max Reitz
On 2015-02-03 at 03:37, Paolo Bonzini wrote: On 02/02/2015 22:40, Max Reitz wrote: Before this patch, the "opaque" pointer in an NBD BDS points to a BDRVNBDState, which contains an NbdClientSession object, which in turn contains a pointer to the BDS. This pointer may become invalid due to bdrv_

Re: [Qemu-devel] [PATCH 1/3] nbd: Drop BDS backpointer

2015-02-03 Thread Paolo Bonzini
On 02/02/2015 22:40, Max Reitz wrote: > Before this patch, the "opaque" pointer in an NBD BDS points to a > BDRVNBDState, which contains an NbdClientSession object, which in turn > contains a pointer to the BDS. This pointer may become invalid due to > bdrv_swap(), so drop it, and instead pass th

[Qemu-devel] [PATCH 1/3] nbd: Drop BDS backpointer

2015-02-02 Thread Max Reitz
Before this patch, the "opaque" pointer in an NBD BDS points to a BDRVNBDState, which contains an NbdClientSession object, which in turn contains a pointer to the BDS. This pointer may become invalid due to bdrv_swap(), so drop it, and instead pass the BDS directly to the nbd-client.c functions whi