linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: Eric Sandeen <sandeen@sandeen.net>
Cc: linux-xfs@vger.kernel.org
Subject: [GIT PULL] xfsprogs: mkfs refactor
Date: Tue, 3 Oct 2017 19:06:07 +1100	[thread overview]
Message-ID: <20171003080607.GM15067@dastard> (raw)

Hi Eric,

I've put the latest mkfs refactor code that I have up in place
you can pull it from. I've rebased it against the current for-next
tree (4.13.1 release) and fixed all the problems that xfstests
exposes. The only thing I haven't fixed is xfs/191 that does mkfs
command line behaviour verification because the refactored version
fixes several problems that the old mkfs didn't handle correctly
(e.g. being able to specify certain things like agsize in blocks or
sectors).

There's a small filter patch needed for xfstests that I'll post in
a reply to this pull request that will filter out the new "defaults
sourced from ..." output and so prevent spurious xfstests failures.

If you want I can tag the branch with a signed tag for you to pull
from (same process as Linus prefers) rather than just a branch in a
tree. If you'd prefer that I post this as patches instead, then let
me know and I'll bomb the list instead.

Cheers,

Dave.

The following changes since commit d4a36331dc383c7c7747e244b3ae20155ae92c98:

  xfsprogs: Release v4.13.1 (2017-09-26 20:45:05 -0500)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/dgc/xfsprogs-dev mkfs-refactor

for you to fetch changes up to a4bc6d3c7bb5babc51f7341039dafcff5fcc6c7e:

  mkfs: tidy up definitions (2017-09-29 08:44:30 +1000)

----------------------------------------------------------------
Dave Chinner (42):
      mkfs: can't specify sector size of internal log
      mkfs: make subopt table const
      mkfs: introduce a structure to hold CLI options
      mkfs: add generic subopt parsing table
      mkfs: factor block subopts parser
      mkfs: factor data subopts parser
      mkfs: factor inode subopts parser
      mkfs: factor log subopts parser
      mkfs: factor meta subopts parser
      mkfs: factor naming subopts parser
      mkfs: factor rt subopts parser
      mkfs: factor sector subopts parser
      mkfs: Introduce mkfs configuration structure
      mkfs: factor printing of mkfs config
      mkfs: factor in memory superblock setup
      mkfs: factor out device preparation
      mkfs: factor writing AG headers
      mkfs: factor secondary superblock updates
      mkfs: introduce default configuration structure
      mkfs: rename top level CLI parameters
      mkfs: factor sectorsize validation
      mkfs: factor blocksize validation
      mkfs: factor log sector size validation
      mkfs: factor superblock feature validation
      mkfs: factor directory blocksize validation
      mkfs: factor inode size validation
      mkfs: factor out device size calculations
      mkfs: fix hidden parameter in DTOBT()
      mkfs: factor rtdev extent size validation
      mkfs: rework stripe calculations
      mkfs: factor device opening
      mkfs: factor data device validation
      mkfs: factor log device validation
      mkfs: factor rt device validation
      mkfs: factor AG geometry calculations
      mkfs: factor AG alignment
      mkfs: rework imaxpct calculation
      mkfs: factor initial mount setup
      mkfs: factor log size calculations
      mkfs: cleanup redundant temporary code
      mkfs: move error functions
      mkfs: tidy up definitions

 include/libxfs.h |    2 +-
 mkfs/xfs_mkfs.c  | 4645 ++++++++++++++++++++++++++++++------------------------
 2 files changed, 2602 insertions(+), 2045 deletions(-)
-- 
Dave Chinner
david@fromorbit.com

             reply	other threads:[~2017-10-03  8:06 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-03  8:06 Dave Chinner [this message]
2017-10-03  8:21 ` fstests: update mkfs.xfs filters for new refactoring Dave Chinner
2017-10-03 17:16 ` [GIT PULL] xfsprogs: mkfs refactor Darrick J. Wong
2017-10-03 20:07   ` Dave Chinner
2017-10-03 20:14     ` Darrick J. Wong
2017-10-06 18:01 ` Eric Sandeen
2017-10-06 18:18   ` Eric Sandeen
2017-10-09  0:42   ` Dave Chinner
2017-10-09  3:11     ` Eric Sandeen

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=20171003080607.GM15067@dastard \
    --to=david@fromorbit.com \
    --cc=linux-xfs@vger.kernel.org \
    --cc=sandeen@sandeen.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).