From: Omar Sandoval <osandov@osandov.com>
To: linux-btrfs@vger.kernel.org
Cc: kernel-team@fb.com, Misono Tomohiro <misono.tomohiro@jp.fujitsu.com>
Subject: [PATCH 00/10] btrfs-progs: my libbtrfsutil patch queue
Date: Tue, 13 Nov 2018 23:46:55 -0800 [thread overview]
Message-ID: <cover.1542181521.git.osandov@fb.com> (raw)
From: Omar Sandoval <osandov@fb.com>
Hi,
This series contains my backlog of libbtrfsutil changes which I've been
collecting over the past few weeks.
Patches 1-4 are fixes. Patches 5-6 add functionality to the unit tests
which is needed for patches 7-8. Patches 7-8 add support for the
unprivileged ioctls added in Linux 4.18; more on those below. Patch 9
bumps the library version. Patch 10 adds documentation for the available
API along with examples.
Patches 7-8 are based on Misono Tomohiro's previous patch series [1],
with a few important changes.
- Both subvolume_info() and create_subvolume_iterator() now have unit
tests for the unprivileged case.
- Both no longer explicitly check that top == 0 in the unprivileged
case, since that will already fail with a clear permission error.
- Unprivileged iteration is much simpler: it uses openat() instead of
fchdir() and is based more closely on the original tree search
variant. This fixes a bug in post-order iteration in Misono's version.
- Unprivileged iteration does _not_ support passing in a non-subvolume
path; if this behavior is desired, I'd like it to be a separate change
with an explicit flag.
Please take a look.
Thanks!
1: https://www.spinics.net/lists/linux-btrfs/msg79285.html
Omar Sandoval (10):
libbtrfsutil: use top=0 as default for SubvolumeIterator()
libbtrfsutil: change async parameters to async_ in Python bindings
libbtrfsutil: document qgroup_inherit parameter in Python bindings
libbtrfsutil: use SubvolumeIterator as context manager in tests
libbtrfsutil: add test helpers for dropping privileges
libbtrfsutil: allow tests to create multiple Btrfs instances
libbtrfsutil: relax the privileges of subvolume_info()
libbtrfsutil: relax the privileges of subvolume iterator
libbtrfsutil: bump version to 1.1.0
libbtrfsutil: document API in README
libbtrfsutil/README.md | 422 +++++++++++++++++++-
libbtrfsutil/btrfsutil.h | 21 +-
libbtrfsutil/errors.c | 8 +
libbtrfsutil/python/module.c | 17 +-
libbtrfsutil/python/subvolume.c | 6 +-
libbtrfsutil/python/tests/__init__.py | 66 ++-
libbtrfsutil/python/tests/test_subvolume.py | 215 +++++++---
libbtrfsutil/subvolume.c | 407 ++++++++++++++++---
8 files changed, 1029 insertions(+), 133 deletions(-)
--
2.19.1
next reply other threads:[~2018-11-14 7:47 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-14 7:46 Omar Sandoval [this message]
2018-11-14 7:46 ` [PATCH 01/10] libbtrfsutil: use top=0 as default for SubvolumeIterator() Omar Sandoval
2018-11-14 7:46 ` [PATCH 02/10] libbtrfsutil: change async parameters to async_ in Python bindings Omar Sandoval
2018-11-14 7:46 ` [PATCH 03/10] libbtrfsutil: document qgroup_inherit parameter " Omar Sandoval
2018-11-14 7:46 ` [PATCH 04/10] libbtrfsutil: use SubvolumeIterator as context manager in tests Omar Sandoval
2018-11-14 7:47 ` [PATCH 05/10] libbtrfsutil: add test helpers for dropping privileges Omar Sandoval
2018-11-14 7:47 ` [PATCH 06/10] libbtrfsutil: allow tests to create multiple Btrfs instances Omar Sandoval
2018-11-14 7:47 ` [PATCH 07/10] libbtrfsutil: relax the privileges of subvolume_info() Omar Sandoval
2018-11-14 7:47 ` [PATCH 08/10] libbtrfsutil: relax the privileges of subvolume iterator Omar Sandoval
2018-11-14 7:47 ` [PATCH 09/10] libbtrfsutil: bump version to 1.1.0 Omar Sandoval
2018-11-14 7:47 ` [PATCH 10/10] libbtrfsutil: document API in README Omar Sandoval
2018-11-26 16:18 ` [PATCH 00/10] btrfs-progs: my libbtrfsutil patch queue David Sterba
2018-11-26 17:15 ` Omar Sandoval
2018-11-27 2:51 ` misono.tomohiro
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.1542181521.git.osandov@fb.com \
--to=osandov@osandov.com \
--cc=kernel-team@fb.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=misono.tomohiro@jp.fujitsu.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.