All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Snitzer <snitzer@redhat.com>
To: dm-devel@redhat.com
Subject: [PATCH 00/14] mikulas' shared snapshot patches
Date: Mon,  1 Mar 2010 19:23:44 -0500	[thread overview]
Message-ID: <1267489438-6271-1-git-send-email-snitzer@redhat.com> (raw)

Mikulas,

This is just the full submit of your shared snapshot patches from:
http://people.redhat.com/mpatocka/patches/kernel/new-snapshots/r15/

I think the next phase of review should possibly be driven through the
dm-devel mailing list.  I'd at least like the option of exchanging
mail on aspects of some of these patches.

The first patch has one small cleanup in do_origin_write(): I
eliminated the 'midcycle' goto.

But the primary difference with this submission (when compared to your
r15 patches) is I editted the patches for whitespace and typos.  I'm
_really_ not trying to step on your hard work by doing this
superficial stuff.  But while reviewing the code the insanely long
lines really were distracting.  I tried very hard to preserve the
intent of the DM_MULTISNAP_SET_ERROR/DM_ERR messages by still having
grep'able content (on a single line).

I also didn't go crazy like a checkpatch.pl zealot.. I didn't even run
these patches through checkpatch!

I know how sensitive you are about allowing the editor do the wrapping
but I trully think the length of some lines would never get past
Alasdair (or Linus) -- even though they have relaxed the rules for
line length.

I'll respond to this cover-letter with a single incremental patch that
shows my edits.

All my edits aside; I must say I'm impressed at the amount/complexity
of code you've cranked out for this shared snapshot support.  It is
going to take me many more review iterations of these patches before
I'll be able to say I understand all that these patches achieve.

I think drivers/md/dm-bufio.c will be controversial (to the greater
upstream community) but I understand that it enabled you to focus on
the problem of shared snapshots without having to concern yourself
with core VM and block changes to accomplish the same.

Mikulas Patocka (14):
  dm-multisnap-common
  dm-bufio
  dm-multisnap-mikulas-headers
  dm-multisnap-mikulas-alloc
  dm-multisnap-mikulas-blocks
  dm-multisnap-mikulas-btree
  dm-multisnap-mikulas-commit
  dm-multisnap-mikulas-delete
  dm-multisnap-mikulas-freelist
  dm-multisnap-mikulas-io
  dm-multisnap-mikulas-snaps
  dm-multisnap-mikulas-common
  dm-multisnap-mikulas-config
  dm-multisnap-daniel

 Documentation/device-mapper/dm-multisnapshot.txt |   77 +
 drivers/md/Kconfig                               |   33 +
 drivers/md/Makefile                              |   10 +
 drivers/md/dm-bufio.c                            |  987 +++++++++++
 drivers/md/dm-bufio.h                            |   35 +
 drivers/md/dm-multisnap-alloc.c                  |  590 +++++++
 drivers/md/dm-multisnap-blocks.c                 |  333 ++++
 drivers/md/dm-multisnap-btree.c                  |  838 +++++++++
 drivers/md/dm-multisnap-commit.c                 |  245 +++
 drivers/md/dm-multisnap-daniel.c                 | 1711 ++++++++++++++++++
 drivers/md/dm-multisnap-delete.c                 |  137 ++
 drivers/md/dm-multisnap-freelist.c               |  296 ++++
 drivers/md/dm-multisnap-io.c                     |  209 +++
 drivers/md/dm-multisnap-mikulas-struct.h         |  380 ++++
 drivers/md/dm-multisnap-mikulas.c                |  760 ++++++++
 drivers/md/dm-multisnap-mikulas.h                |  247 +++
 drivers/md/dm-multisnap-private.h                |  161 ++
 drivers/md/dm-multisnap-snaps.c                  |  636 +++++++
 drivers/md/dm-multisnap.c                        | 2007 ++++++++++++++++++++++
 drivers/md/dm-multisnap.h                        |  183 ++
 20 files changed, 9875 insertions(+), 0 deletions(-)
 create mode 100644 Documentation/device-mapper/dm-multisnapshot.txt
 create mode 100644 drivers/md/dm-bufio.c
 create mode 100644 drivers/md/dm-bufio.h
 create mode 100644 drivers/md/dm-multisnap-alloc.c
 create mode 100644 drivers/md/dm-multisnap-blocks.c
 create mode 100644 drivers/md/dm-multisnap-btree.c
 create mode 100644 drivers/md/dm-multisnap-commit.c
 create mode 100644 drivers/md/dm-multisnap-daniel.c
 create mode 100644 drivers/md/dm-multisnap-delete.c
 create mode 100644 drivers/md/dm-multisnap-freelist.c
 create mode 100644 drivers/md/dm-multisnap-io.c
 create mode 100644 drivers/md/dm-multisnap-mikulas-struct.h
 create mode 100644 drivers/md/dm-multisnap-mikulas.c
 create mode 100644 drivers/md/dm-multisnap-mikulas.h
 create mode 100644 drivers/md/dm-multisnap-private.h
 create mode 100644 drivers/md/dm-multisnap-snaps.c
 create mode 100644 drivers/md/dm-multisnap.c
 create mode 100644 drivers/md/dm-multisnap.h

             reply	other threads:[~2010-03-02  0:23 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-02  0:23 Mike Snitzer [this message]
2010-03-02  0:23 ` [PATCH 01/14] dm-multisnap-common Mike Snitzer
2010-03-02  0:23 ` [PATCH 02/14] dm-bufio Mike Snitzer
2010-03-02  0:23 ` [PATCH 03/14] dm-multisnap-mikulas-headers Mike Snitzer
2010-03-05 22:46   ` Mike Snitzer
2010-03-06  1:54     ` Mike Snitzer
2010-03-09  3:08     ` Mikulas Patocka
2010-03-09  3:30       ` Mike Snitzer
2010-03-02  0:23 ` [PATCH 04/14] dm-multisnap-mikulas-alloc Mike Snitzer
2010-03-02  0:23 ` [PATCH 05/14] dm-multisnap-mikulas-blocks Mike Snitzer
2010-03-02  0:23 ` [PATCH 06/14] dm-multisnap-mikulas-btree Mike Snitzer
2010-03-02  0:23 ` [PATCH 07/14] dm-multisnap-mikulas-commit Mike Snitzer
2010-03-02  0:23 ` [PATCH 08/14] dm-multisnap-mikulas-delete Mike Snitzer
2010-03-02  0:23 ` [PATCH 09/14] dm-multisnap-mikulas-freelist Mike Snitzer
2010-03-02  0:23 ` [PATCH 10/14] dm-multisnap-mikulas-io Mike Snitzer
2010-03-02  0:23 ` [PATCH 11/14] dm-multisnap-mikulas-snaps Mike Snitzer
2010-03-02  0:23 ` [PATCH 12/14] dm-multisnap-mikulas-common Mike Snitzer
2010-03-02  0:23 ` [PATCH 13/14] dm-multisnap-mikulas-config Mike Snitzer
2010-03-02  0:23 ` [PATCH 14/14] dm-multisnap-daniel Mike Snitzer
2010-03-02  0:57   ` FUJITA Tomonori
2010-03-02  0:32 ` [PATCH 00/14] mikulas' shared snapshot patches Mike Snitzer
2010-03-02 14:56 ` Mike Snitzer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1267489438-6271-1-git-send-email-snitzer@redhat.com \
    --to=snitzer@redhat.com \
    --cc=dm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.