Re: [Nbd] [patch] fix "copy on write" wrong code
- To: Maxim Syrchin <syrchin@...62...>
- Cc: nbd-general@lists.sourceforge.net
- Subject: Re: [Nbd] [patch] fix "copy on write" wrong code
- From: Wouter Verhelst <wouter@...3...>
- Date: Fri, 21 Oct 2005 10:43:22 +0200
- Message-id: <20051021084322.GA4352@...39...>
- In-reply-to: <4358744C.7080709@...62...>
- References: <4356475C.6060308@...62...> <20051020142608.GA13752@...39...> <4358744C.7080709@...62...>
On Fri, Oct 21, 2005 at 08:53:32AM +0400, Maxim Syrchin wrote:
> Wouter Verhelst wrote:
> >>- wrong place for initialisation of diffmaps
> >>
> >>
> >
> >Well -- what's "wrong" is just a matter of opinion here, I'd say. It's
> >both initialization code...
> >
> At does not work before you deternine size of exported media/file. So,
> IMHO it should be moved in separate function and called later.
Ah, that way. Right, misunderstood you then.
> >Hm. Do you mean that without this change, using nbd-server with -c
> >doesn't support different blocksizes?
> >
> Yes. functuin will return after reading first "page" of block. Other
> pages are not readed :(.
Okay.
> I've test this patch ;)
Sure :-)
I just wanted to check before I'd apply, that's all. And it's been a
while since I had a look at 2.8, so I didn't recall all the details
anymore...
--
The amount of time between slipping on the peel and landing on the
pavement is precisely one bananosecond
Reply to: