Am Dienstag, 18. September 2012 schrieb Miao Xie:

 static const char * const cmd_subvol_list_usage[] = {
-       "btrfs subvolume list [-pu] [-s 0|1] <path>",
+       "btrfs subvolume list [-pur] [-s 0|1] <path>",
        "List subvolumes (and snapshots)",
        "",
        "-p           print parent ID",
        "-u           print the uuid of subvolumes (and snapshots)",
        "-s value     list snapshots with generation in ascending/descending order",
        "             (1: ascending, 0: descending)",
+       "-r           list readonly subvolumes(including snapshots)",

I would add an space between "subvolumes" and "(including snapshots)" here.

        NULL
 };

(No kernel developer, thus not commenting on the code.)

Thanks,
-- 
Martin 'Helios' Steigerwald - http://www.Lichtvoll.de
GPG: 03B0 0D6C 0040 0710 4AFA  B82F 991B EAAC A599 84C7