| To: | Wanlong Gao <gaowanlong@xxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH V7] xfs: cleanup the mount options |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Mon, 30 Jul 2012 08:09:09 +1000 |
| Cc: | xfs@xxxxxxxxxxx, Ben Myers <bpm@xxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, Zach Brown <zab@xxxxxxxxx> |
| In-reply-to: | <1343178697-3479-1-git-send-email-gaowanlong@xxxxxxxxxxxxxx> |
| References: | <20120724211857.GO23387@dastard> <1343178697-3479-1-git-send-email-gaowanlong@xxxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Wed, Jul 25, 2012 at 09:11:37AM +0800, Wanlong Gao wrote: > remove the mount options macro, use tokens instead. > > change from V6: (Dave) > 1. switch GFP_NOFS to GFP_KERNEL > > change from V5: (Dave) > 1. fix a string leak. > 2. leave the show options to raw numbers. > > change from V4: (as Dave suggested) > 1. suffix_match_int avoid overflow. > 2. convert the return style to be consistent with others. > 3. move the "intarg" init above the switch. > 4. move match_strdup to a help function. > 5. collapsed Opt_biosize and Opt_allocsize. > 6. make a consistent deprecated message. Looks good now. Thanks for takingthe time to get this right! Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [XFS updates] XFS development tree branch, for-next, updated. v3.5-rc1-44-g9a57fa8, xfs |
|---|---|
| Next by Date: | Re: realtime section bugs still around, Dave Chinner |
| Previous by Thread: | [PATCH V7] xfs: cleanup the mount options, Wanlong Gao |
| Next by Thread: | [PATCH V6] xfs: cleanup the mount options, Wanlong Gao |
| Indexes: | [Date] [Thread] [Top] [All Lists] |