Bug 1058608
| Summary: | [RFE] btrfs-progs: btrfs resize doesn't support T/P/E suffix | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Linux 7 | Reporter: | Eryu Guan <eguan> |
| Component: | kernel | Assignee: | Eric Sandeen <esandeen> |
| kernel sub component: | Btrfs | QA Contact: | XuWang <xuw> |
| Status: | CLOSED ERRATA | Docs Contact: | |
| Severity: | medium | ||
| Priority: | medium | CC: | eguan, esandeen, wangsl.fnst, xzhou |
| Version: | 7.0 | Keywords: | FutureFeature, Reopened |
| Target Milestone: | rc | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | kernel-3.10.0-167.el7 | Doc Type: | Enhancement |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2015-03-05 11:39:22 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 1113520 | ||
|
Description
Eryu Guan
2014-01-28 06:54:04 UTC
ok, thanks for reporting, we will fix it. I'll take this, odds are Ric isn't going to fix it ;) However, it's not super-trivial to fix, and in fact requires something kernelside, due to the tortured, hacky implementation of the resize ioctl, which requires parsing strings from userspace in the kernel driver. Today (as Eryu pointed out), that parsing is done by a function intended for RAM-scale numbers, so it only goes to gigabytes. The patch has been sent to the btrfs & kernel lists:
[PATCH v4] lib: add size unit t/p/e to memparse
and akpm has noticed, so I suppose it'll get merged soon.
Development Management has reviewed and declined this request. You may appeal this decision by reopening this request. Hohum, I misfired on dev_ack. FWIW, this is actually a kernel bug. Note, the patch to fix this has still not made it upstream. From e004f3c7780de32fa822f292ebadd985bcadb1e0 Mon Sep 17 00:00:00 2001 From: Gui Hecheng <guihc.fnst.com> Date: Wed, 6 Aug 2014 16:09:29 -0700 Subject: lib/cmdline.c: add size unit t/p/e to memparse For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng <guihc.fnst.com> Acked-by: David Rientjes <rientjes> Reviewed-by: Satoru Takeuchi <takeuchi_satoru.com> Signed-off-by: Andrew Morton <akpm> Signed-off-by: Linus Torvalds <torvalds> Patch(es) available on kernel-3.10.0-167.el7 run btrfs-progs regression test case "/btrfs-progs/Regression/bz1058608-resize-support-T_P_E" succesfully in RHEL-7.1-20141204.2 in following job: https://beaker.engineering.redhat.com/jobs/821711. Also run some regressions for RHEL7.1(man kernel versions, but one btrfs-progs version): J:803063 xfstests-btrfs: RHEL-7.1-20141113.0,s390x J:803061 xfstests-btrfs: RHEL-7.1-20141113.0-ppc64 J:801031 xfstests-btrfs: RHEL-7.1-20141111.0 J:798051 ltp-aiodio-btrfs: RHEL-LE-7.1-20141105.n.2 J:796795 ltp-btrfs: RHEL-7.1-20141107.n.0, kernel-3.10.0-199.el7 Also I run some cases munually: /kernel/filesystems/btrfs/degraded-mount-replace--panic for kernel reason /kernel/filesystems/btrfs/profile-conversion--good /kernel/filesystems/btrfs/online-resize--good /kernel/filesystems/btrfs/regression--good /kernel/filesystems/btrfs/mkfs--good /kernel/filesystems/btrfs/mount--good /kernel/filesystems/btrfs/clone--good /kernel/filesystemd/btrfs/compress--good /kernel/filesystem/btrfs/defragment--good /kernel/filesystem/btrfs/online-device-add-delete-balance--good So I think I can change this but status to verified. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://rhn.redhat.com/errata/RHSA-2015-0290.html |