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

Groups > linux.kernel > #1585121

Re: [PATCH] drm: drm_printer: add __printf validation

Show all headers | View raw

On Mon, 20 Feb 2017, Joe Perches <joe@perches.com> wrote:
> On Mon, 2017-02-20 at 12:17 +0000, Eric Engestrom wrote:
>> On Wednesday, 2017-02-15 15:33:18 -0800, Joe Perches wrote:
>> > drm_printf does not currently use the compiler to verify
>> > format and arguments.  Make it do so.
>> > 
>> > Miscellanea:
>> > 
>> > o Add appropriate #include files for __printf and struct va_format
>> > o Convert dev_printk to dev_info
>> 
>> I think these unrelated changes should be in 4 patches:
>> 1 - add annotation to check the format string against the arguments
>>     (linux/compiler.h should be added here)
>> 2 - add missing linux/printk.h header for struct va_format
>>     Note that I think a forward declaration is more appropriate here, as
>>     we only use pointers to this struct in this file, we never try to
>>     look inside. On the other hand:
>> 3 - drm_print.c needs the header in drm_printf(), but as a separate
>>     patch
>> 4 - convert dev_printk to dev_info (you need to include linux/device.h
>>     there)
>
> I am not a big fan of making trivial patches into a series.

It's standard procedure in kernel development to split out unrelated
changes into individual patches, regardless of whether you think they
are trivial or not. Four is probably excessive, but you get the idea.

>> You can add my r-b on all four patches when you send them to the list :)
>
> If you want to break it up, go ahead.

You know how this stuff works, please split it up to get the stuff
merged.


BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] drm: drm_printer: add __printf validation Joe Perches <joe@perches.com> - 2017-02-16 00:40 +0100
  Re: [PATCH] drm: drm_printer: add __printf validation Eric Engestrom <eric.engestrom@imgtec.com> - 2017-02-20 13:40 +0100
    Re: [PATCH] drm: drm_printer: add __printf validation Joe Perches <joe@perches.com> - 2017-02-20 18:20 +0100
      Re: [PATCH] drm: drm_printer: add __printf validation Jani Nikula <jani.nikula@linux.intel.com> - 2017-02-21 09:30 +0100
        Re: [PATCH] drm: drm_printer: add __printf validation Joe Perches <joe@perches.com> - 2017-02-21 09:40 +0100
          Re: [PATCH] drm: drm_printer: add __printf validation Jani Nikula <jani.nikula@linux.intel.com> - 2017-02-21 10:10 +0100
            Re: [PATCH] drm: drm_printer: add __printf validation Joe Perches <joe@perches.com> - 2017-02-21 10:20 +0100
              Re: [PATCH] drm: drm_printer: add __printf validation Daniel Vetter <daniel@ffwll.ch> - 2017-02-26 21:50 +0100

csiph-web