public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: Artem Bityutskiy <dedekind1@gmail.com>
To: stefani@seibold.net
Cc: dwmw2@infradead.org, linux-mtd@lists.infradead.org,
	Artem.Bityutskiy@nokia.com
Subject: Re: [PATCH] New tuneubifs command to adjust tunable filesystem parameters on ubifs
Date: Tue, 18 Jan 2011 10:39:20 +0200	[thread overview]
Message-ID: <1295339960.2470.70.camel@koala> (raw)
In-Reply-To: <1295339326-1720-1-git-send-email-stefani@seibold.net>

On Tue, 2011-01-18 at 09:28 +0100, stefani@seibold.net wrote:
> From: Stefani Seibold <stefani@seibold.net>
> 
> ChangeLog:
> 31.12.2010 first proposal
> 18.01.2011 fixes suggested by Artem:
>            - fix help message
> 	   - rename the tool into tuneubifs
> 	   - rename mkfs.ubifs" directory into "ubifs-utils"
> 	   - move the tool to ubifs-utils/tuneubifs
> 	   - code cleanup
> 
> tuneubifs allows the system administrator to adjust various tunable filesystem
> parameters on Linux ubifs filesystems. The current values of these parameters
> can be displayed by using the -v or calling without any option.
> 
> Currently the tuneubifs supports the following parameters:
> 
> -x, --compr=<none|lzo|zlib>    compression type
> -R, --reserved=SIZE            how much space should be reserved for super-user
> 
> Maybe it will be nice to adjust also the uid and gid of the reserved owner,
> or the increase the max. size of the ubi filesystem. This will be very
> simple to implement.
> 
> Signed-off-by: Stefani Seibold <stefani@seibold.net>

Stefani, I'm sorry for bothering you with small things, but it is really
bead idea to combine the re-naming and the new utility in one patch.
These are really 2 separate steps: 1 - prepare things by re-naming, 2 -
add new utility.

Please, make it 2 separate patches, not one patch.

Also, it is very helpful when people add v2 v3 etc to the subject. This
is just another "friendly hacker's practice".

Thanks!

-- 
Best Regards,
Artem Bityutskiy (Битюцкий Артём)

      reply	other threads:[~2011-01-18  8:39 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-18  8:28 [PATCH] New tuneubifs command to adjust tunable filesystem parameters on ubifs stefani
2011-01-18  8:39 ` Artem Bityutskiy [this message]

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=1295339960.2470.70.camel@koala \
    --to=dedekind1@gmail.com \
    --cc=Artem.Bityutskiy@nokia.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=stefani@seibold.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