From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Bryn M. Reeves" Subject: Re: [PATCH 2/4] dm stats: support precise timestamps Date: Mon, 27 Jul 2015 16:11:09 +0100 Message-ID: <20150727151108.GA18547@localhost.localdomain> References: Reply-To: device-mapper development Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: device-mapper development Cc: Mike Snitzer , Laurence Oberman , Tao Ma , Robin Dong , Coly Li , "Alasdair G. Kergon" List-Id: dm-devel.ids On Tue, Jun 09, 2015 at 05:21:39PM -0400, Mikulas Patocka wrote: > /* > * Input format: > - * [ []] > + * [ ] [ []] > */ Rather than adding optional parameters would it make sense to add a new create method? E.g. "@stats_create_precise". In some ways this is easier to deal with in userspace and there's alrady a precedent for it with @stats_print vs. @stats_print_clear. It's not a huge deal either way though - just simplifies formatting of the messages a little. Regards, Bryn.