Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1646582 > unrolled thread

Re: kernel-trace: Fine-tuning for seven function implementations

Started bySF Markus Elfring <elfring@users.sourceforge.net>
First post2017-05-22 09:50 +0200
Last post2017-05-23 07:20 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: kernel-trace: Fine-tuning for seven function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2017-05-22 09:50 +0200
    Re: kernel-trace: Fine-tuning for seven function implementations Steven Rostedt <rostedt@goodmis.org> - 2017-05-23 07:20 +0200

#1646582 — Re: kernel-trace: Fine-tuning for seven function implementations

FromSF Markus Elfring <elfring@users.sourceforge.net>
Date2017-05-22 09:50 +0200
SubjectRe: kernel-trace: Fine-tuning for seven function implementations
Message-ID<tJQgi-2Cc-27@gated-at.bofh.it>
> I wont be touching or even looking at these until after 4.12-rc1 is released.
> Feel free to reply to this email with a ping in a week or two.

*ping*

How do you think about to give these update suggestions another look?

Regards,
Markus

[toc] | [next] | [standalone]


#1647637

FromSteven Rostedt <rostedt@goodmis.org>
Date2017-05-23 07:20 +0200
Message-ID<tKaoF-7aM-15@gated-at.bofh.it>
In reply to#1646582
On Mon, 22 May 2017 09:45:27 +0200
SF Markus Elfring <elfring@users.sourceforge.net> wrote:

> > I wont be touching or even looking at these until after 4.12-rc1 is
> > released. Feel free to reply to this email with a ping in a week or
> > two.  
> 
> *ping*
> 
> How do you think about to give these update suggestions another look?
>

The conversions of the seq_printf()s and friends are fine, but the rest
are not worth the churn. Yes checkpatch may complain about the NULL
compares and the kallocs(), but that's for new code, not for code that
has already been accepted.

-- Steve

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web