Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

btrfs-progs: doc: add a note on qgroup limit with inconsitent flag #939

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

adam900710
Copy link
Collaborator

Just like all qgroup functions, if qgroup is marked inconsistent, limit will not work as expected.
In fact with recent kernels, limit and qgroup number updating will be fully skipped if qgroup is already inconsistent.

Add one extra note on btrfs qgroup limit subcommand for it.

Reported-by: Vojtech Lacina [email protected]
Link: https://bugzilla.suse.com/show_bug.cgi?id=1235765

Just like all qgroup functions, if qgroup is marked inconsistent, limit
will not work as expected.
In fact with recent kernels, limit and qgroup number updating will be
fully skipped if qgroup is already inconsistent.

Add one extra note on `btrfs qgroup limit` subcommand for it.

Reported-by: Vojtech Lacina <[email protected]>
Link: https://bugzilla.suse.com/show_bug.cgi?id=1235765
Signed-off-by: Qu Wenruo <[email protected]>
Copy link
Owner

@kdave kdave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With some wording fixes.

@@ -31,6 +31,11 @@ ownership. For example a fresh snapshot shares almost all the blocks with the
original subvolume, new writes to either subvolume will raise towards the
exclusive limit.

.. note::
Qgroup limit only works when qgroup is in a consistent status.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Qgroup limit works only when a qgroup is in a consistent state."

@@ -31,6 +31,11 @@ ownership. For example a fresh snapshot shares almost all the blocks with the
original subvolume, new writes to either subvolume will raise towards the
exclusive limit.

.. note::
Qgroup limit only works when qgroup is in a consistent status.
If by some workload, qgroup is mark inconsistent, the limit will no longer
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"If some workload marks qgroup as inconsistent (for example <WHAT... >) the limits will not apply until the inconsistent flag is cleared but :command:btrfs quota rescan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants