[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20161206095958.GA9387@kroah.com>
Date: Tue, 6 Dec 2016 10:59:58 +0100
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: James Simmons <jsimmons@...radead.org>
Cc: devel@...verdev.osuosl.org,
Andreas Dilger <andreas.dilger@...el.com>,
Oleg Drokin <oleg.drokin@...el.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Alexander Boyko <alexander.boyko@...gate.com>,
Lustre Development List <lustre-devel@...ts.lustre.org>
Subject: Re: [PATCH 14/22] staging: lustre: obd: add callback for
llog_cat_process_or_fork
On Fri, Dec 02, 2016 at 07:53:21PM -0500, James Simmons wrote:
> From: Alexander Boyko <alexander.boyko@...gate.com>
>
> Currently llog_process_or_fork() is hard coded to
> always pass the function pointer llog_cat_process_cb().
> Change llog_cat_process_or_fork() to pass in any
> function pointer which will allow us more options
> for llog_cat callback routines in the future.
Don't change this until you have an actual user for this. Adding
infrastructure that is not used is not ok for a staging driver, or
really any kernel driver at all.
sorry,
greg k-h
Powered by blists - more mailing lists