Message-ID: <1950665.2mvfrclfR5@fat-tyre>
Date:	Fri, 31 Jul 2015 14:48:53 +0200
From:	Philipp Reisner <philipp.reisner@...bit.com>
To:	Nick Wang <nwang@...e.com>
Cc:	drbd-dev@...ts.linbit.com, linux-kernel@...r.kernel.org,
	Lars Ellenberg <lars.ellenberg@...bit.com>
Subject: Re: [Drbd-dev] [Patch v2 00/10] Zero out devices instead of initial full sync

Hi Nick,

finally I have time to really review it:

* it uses blkdev_issue_zeroout() with the discard parameter
  set to false. I.e. it will completely allocate a thinly
  provided backing device. Please make this more generic.
  Maybe two one option --zeroout-device and --discard-device
  instread of --zap-device

* The patch-set is not very complex, I am fine with having
  this in a single patch

* It introduces a new packet into the protocol without 
  bumping the protocol version or introducing a protocol
  reature flag. Do that, and make sure to send the new 
  packets only when you know that the peer is recent enough
  to process them.
  See here for an example:
  http://git.drbd.org/gitweb.cgi?p=drbd-8.4.git;a=commitdiff;h=476039f699948155d71d6f86323a3b16e6d05f0c;hp=4c7521e19c6c2c046be6547490334294b6f190e4

Best regards,
phil

> Patch set is based on drbd-8.4 859e34a9, have
> already compiled/tested against SLES12.
> 
> If this feature can be merged into upstream, please
> ignore v1, since v1 may caused pingACK timeout when
> zeroing out large device.
> 
> Compare to v1, changes are as follow:
>  1. Using drbd_device_post_work to zero out device as background
>     task, so that it won't block pingACK when zeroing out
>     large device.
> 
>  2. Fix bug of won't update peer node status if it finished
>     zeroing out earier.
> 
>  3. Change some functions from file drbd_receive.c to
>     drbd_worker.c and reorder the patch set.
> 
> 
> Full sync for drbd initial usually take a long time, especically
> when network become the bottleneck of the syncing. Simply skip
> the full sync with "--clear-bitmap" may not the perfect solution
> for all the cases, like using the bare device(no filesystem) to
> work,etc database,vm... This patche set can be used to zero out
> devices locally instead of a full sync to make the consistent
> block device. This approach can be useful when lack of network
> bandwidth to sync.
> 
> The patches add one new option "--zap-devices" to "new-current-uuid"
> to zero out devices. It will start zeroing out devices of both
> side.
> 
> Nick Wang (10):
>   drbd: Fix the wrong logic of move history.
>   drbd: Add options zap_devices to new-current-uuid
>   drbd: Add a function to zero out drbd backing device.
>   drbd: New packet P_ZERO_OUT.
>   drbd: Functions to notify peer node to start
>         zeroing out and zero out finished.
>   drbd: Wapper for zeroing out device by worker.
>   drbd: Add flag for drbd device work.
>   drbd: Function to work with packet P_ZERO_OUT.
>   drbd: Receive zero out command from peer node.
>   drbd: Handle new-current-uuid --zap-devices.
> 
> Signed-off-by: Nick Wang <nwang@...e.com>
> CC: Philipp Reisner <philipp.reisner@...bit.com>
> CC: Lars Ellenberg <lars.ellenberg@...bit.com>
> CC: drbd-dev@...ts.linbit.com
> CC: linux-kernel@...r.kernel.org
> 
>  drbd/drbd_int.h        | 13 ++++++++
>  drbd/drbd_main.c       | 51 ++++++++++++++++++++++++++--
>  drbd/drbd_nl.c         | 21 +++++++++++-
>  drbd/drbd_protocol.h   |  1 +
>  drbd/drbd_receiver.c   | 69 ++++++++++++++++++++++++++++++++++++++
>  drbd/drbd_worker.c     | 90
> ++++++++++++++++++++++++++++++++++++++++++++++++++ drbd/linux/drbd_genl.h |
>  1 +
>  7 files changed, 243 insertions(+), 3 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists