All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Sakai <msakai@redhat.com>
To: dm-devel@lists.linux.dev
Cc: Matthew Sakai <msakai@redhat.com>
Subject: [PATCH 00/10] dm vdo: standardize on VDO_SUCCESS
Date: Thu, 29 Feb 2024 23:38:43 -0500	[thread overview]
Message-ID: <cover.1709267597.git.msakai@redhat.com> (raw)

Clean up code to consistently return and check for appropriate
return values, usually VDO_SUCCESS.

Matthew Sakai (2):
  dm vdo errors: remove unused error codes
  dm vdo indexer: update ASSERT and ASSERT_LOG_ONLY usage

Mike Snitzer (8):
  dm vdo memory-alloc: return VDO_SUCCESS on success
  dm vdo: check for VDO_SUCCESS return value from memory-alloc functions
  dm vdo int-map: return VDO_SUCCESS on success
  dm vdo thread-utils: return VDO_SUCCESS on vdo_create_thread success
  dm-vdo funnel-workqueue: return VDO_SUCCESS from
    make_simple_work_queue
  dm vdo permassert: audit all of ASSERT to test for VDO_SUCCESS
  dm vdo encodings: update some stale comments
  dm vdo target: eliminate inappropriate uses of UDS_SUCCESS

 drivers/md/dm-vdo/action-manager.c            |   8 +-
 drivers/md/dm-vdo/block-map.c                 | 128 +++++++-------
 drivers/md/dm-vdo/completion.c                |  10 +-
 drivers/md/dm-vdo/completion.h                |   6 +-
 drivers/md/dm-vdo/data-vio.c                  | 110 ++++++------
 drivers/md/dm-vdo/data-vio.h                  |  68 ++++----
 drivers/md/dm-vdo/dedupe.c                    | 165 +++++++++---------
 drivers/md/dm-vdo/dm-vdo-target.c             |  70 ++++----
 drivers/md/dm-vdo/encodings.c                 | 162 +++++++++--------
 drivers/md/dm-vdo/errors.c                    |   9 +-
 drivers/md/dm-vdo/errors.h                    |  13 +-
 drivers/md/dm-vdo/flush.c                     |  22 +--
 drivers/md/dm-vdo/funnel-queue.c              |   2 +-
 drivers/md/dm-vdo/funnel-workqueue.c          |  30 ++--
 drivers/md/dm-vdo/indexer/chapter-index.c     |  18 +-
 drivers/md/dm-vdo/indexer/config.c            |  18 +-
 drivers/md/dm-vdo/indexer/delta-index.c       | 100 +++++------
 .../md/dm-vdo/indexer/funnel-requestqueue.c   |   4 +-
 drivers/md/dm-vdo/indexer/geometry.c          |   2 +-
 drivers/md/dm-vdo/indexer/index-layout.c      |  23 +--
 drivers/md/dm-vdo/indexer/index-page-map.c    |   8 +-
 drivers/md/dm-vdo/indexer/index-session.c     |  14 +-
 drivers/md/dm-vdo/indexer/index.c             |  18 +-
 drivers/md/dm-vdo/indexer/io-factory.c        |   6 +-
 drivers/md/dm-vdo/indexer/open-chapter.c      |   4 +-
 drivers/md/dm-vdo/indexer/radix-sort.c        |   2 +-
 drivers/md/dm-vdo/indexer/sparse-cache.c      |   8 +-
 drivers/md/dm-vdo/indexer/volume-index.c      |  38 ++--
 drivers/md/dm-vdo/indexer/volume.c            |  48 ++---
 drivers/md/dm-vdo/int-map.c                   |  38 ++--
 drivers/md/dm-vdo/io-submitter.c              |  12 +-
 drivers/md/dm-vdo/logical-zone.c              |  22 +--
 drivers/md/dm-vdo/memory-alloc.c              |  36 ++--
 drivers/md/dm-vdo/memory-alloc.h              |   8 +-
 drivers/md/dm-vdo/message-stats.c             |   2 +-
 drivers/md/dm-vdo/packer.c                    |  12 +-
 drivers/md/dm-vdo/permassert.c                |   2 +-
 drivers/md/dm-vdo/permassert.h                |  15 +-
 drivers/md/dm-vdo/physical-zone.c             |  48 ++---
 drivers/md/dm-vdo/priority-table.c            |   4 +-
 drivers/md/dm-vdo/recovery-journal.c          |  60 +++----
 drivers/md/dm-vdo/repair.c                    |  12 +-
 drivers/md/dm-vdo/slab-depot.c                | 118 ++++++-------
 drivers/md/dm-vdo/status-codes.c              |  10 --
 drivers/md/dm-vdo/status-codes.h              |  32 +---
 drivers/md/dm-vdo/thread-registry.c           |   4 +-
 drivers/md/dm-vdo/thread-utils.c              |   4 +-
 drivers/md/dm-vdo/uds-sysfs.c                 |   2 +-
 drivers/md/dm-vdo/vdo.c                       |  34 ++--
 drivers/md/dm-vdo/vio.c                       |  40 ++---
 drivers/md/dm-vdo/vio.h                       |   8 +-
 51 files changed, 794 insertions(+), 843 deletions(-)

-- 
2.42.0


             reply	other threads:[~2024-03-01  4:38 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-01  4:38 Matthew Sakai [this message]
2024-03-01  4:38 ` [PATCH 01/10] dm vdo errors: remove unused error codes Matthew Sakai
2024-03-01  4:38 ` [PATCH 02/10] dm vdo memory-alloc: return VDO_SUCCESS on success Matthew Sakai
2024-03-01  4:38 ` [PATCH 03/10] dm vdo: check for VDO_SUCCESS return value from memory-alloc functions Matthew Sakai
2024-03-01  4:38 ` [PATCH 04/10] dm vdo int-map: return VDO_SUCCESS on success Matthew Sakai
2024-03-01  4:38 ` [PATCH 05/10] dm vdo thread-utils: return VDO_SUCCESS on vdo_create_thread success Matthew Sakai
2024-03-01  4:38 ` [PATCH 06/10] dm-vdo funnel-workqueue: return VDO_SUCCESS from make_simple_work_queue Matthew Sakai
2024-03-01  4:38 ` [PATCH 07/10] dm vdo permassert: audit all of ASSERT to test for VDO_SUCCESS Matthew Sakai
2024-03-01  4:38 ` [PATCH 08/10] dm vdo encodings: update some stale comments Matthew Sakai
2024-03-01  4:38 ` [PATCH 09/10] dm vdo indexer: update ASSERT and ASSERT_LOG_ONLY usage Matthew Sakai
2024-03-01  4:38 ` [PATCH 10/10] dm vdo target: eliminate inappropriate uses of UDS_SUCCESS Matthew Sakai

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=cover.1709267597.git.msakai@redhat.com \
    --to=msakai@redhat.com \
    --cc=dm-devel@lists.linux.dev \
    /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.