All of lore.kernel.org
 help / color / mirror / Atom feed
* checkpatch.pl "uninitialized value"
@ 2013-01-14 13:42 Schrober
  2013-01-14 13:45 ` Bernd Petrovitsch
  0 siblings, 1 reply; 5+ messages in thread
From: Schrober @ 2013-01-14 13:42 UTC (permalink / raw)
  To: linux-kernel; +Cc: Andy Whitcroft, Joe Perches, Andrew Morton

Hi,

it seems the current version of checkpatch.pl in linux-next ist broken. I get
a lot of

Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.
Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3386.
Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.
Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3386.
Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.
Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3386.
-- 
Franz Schrober

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: checkpatch.pl "uninitialized value"
  2013-01-14 13:42 checkpatch.pl "uninitialized value" Schrober
@ 2013-01-14 13:45 ` Bernd Petrovitsch
  2013-01-14 13:50   ` Joe Perches
  2013-01-14 13:53   ` Schrober
  0 siblings, 2 replies; 5+ messages in thread
From: Bernd Petrovitsch @ 2013-01-14 13:45 UTC (permalink / raw)
  To: Schrober; +Cc: linux-kernel, Andy Whitcroft, Joe Perches, Andrew Morton

Hi!

On Mon, 2013-01-14 at 14:42 +0100, Schrober wrote:
[...]
> it seems the current version of checkpatch.pl in linux-next ist broken. I get
> a lot of
> 
> Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.
> Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3386.
> Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.
> Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3386.
> Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.
> Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3386.

on which file?
If you would have added the command line to trivially reproduce that (or
whatever you did to get it), people are more likely to look into it.

MfG,
	Bernd
-- 
Bernd Petrovitsch                  Email : bernd@petrovitsch.priv.at
                     LUGA : http://www.luga.at


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: checkpatch.pl "uninitialized value"
  2013-01-14 13:45 ` Bernd Petrovitsch
@ 2013-01-14 13:50   ` Joe Perches
  2013-01-14 13:59     ` Schrober
  2013-01-14 13:53   ` Schrober
  1 sibling, 1 reply; 5+ messages in thread
From: Joe Perches @ 2013-01-14 13:50 UTC (permalink / raw)
  To: Bernd Petrovitsch; +Cc: Schrober, linux-kernel, Andy Whitcroft, Andrew Morton

On Mon, 2013-01-14 at 14:45 +0100, Bernd Petrovitsch wrote:
> On Mon, 2013-01-14 at 14:42 +0100, Schrober wrote:
> [...]
> > it seems the current version of checkpatch.pl in linux-next ist broken. I get
> > a lot of
> > 
> > Use of uninitialized value $ms_size in pattern match (m//) at ./scripts/checkpatch.pl line 3383.

> on which file?

Any file with a usleep_range or a memset

> If you would have added the command line to trivially reproduce that (or
> whatever you did to get it), people are more likely to look into it.

Try this patch:
https://lkml.org/lkml/2013/1/10/444




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: checkpatch.pl "uninitialized value"
  2013-01-14 13:45 ` Bernd Petrovitsch
  2013-01-14 13:50   ` Joe Perches
@ 2013-01-14 13:53   ` Schrober
  1 sibling, 0 replies; 5+ messages in thread
From: Schrober @ 2013-01-14 13:53 UTC (permalink / raw)
  To: Bernd Petrovitsch
  Cc: linux-kernel, Andy Whitcroft, Joe Perches, Andrew Morton

On Monday 14 January 2013 14:45:06 Bernd Petrovitsch wrote:
> on which file?
> If you would have added the command line to trivially reproduce that (or
> whatever you did to get it), people are more likely to look into it.

All input I've tried. So I thought it is not relevant here. But of course, 
here is a simple example:

./scripts/checkpatch.pl -f lib/decompress_bunzip2.c
-- 
Franz Schrober

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: checkpatch.pl "uninitialized value"
  2013-01-14 13:50   ` Joe Perches
@ 2013-01-14 13:59     ` Schrober
  0 siblings, 0 replies; 5+ messages in thread
From: Schrober @ 2013-01-14 13:59 UTC (permalink / raw)
  To: Joe Perches
  Cc: Bernd Petrovitsch, linux-kernel, Andy Whitcroft, Andrew Morton

On Monday 14 January 2013 05:50:12 Joe Perches wrote:
> On Mon, 2013-01-14 at 14:45 +0100, Bernd Petrovitsch wrote:
> > On Mon, 2013-01-14 at 14:42 +0100, Schrober wrote:
> > [...]
> > 
> > > it seems the current version of checkpatch.pl in linux-next ist broken.
> > > I get a lot of
> > > 
> > > Use of uninitialized value $ms_size in pattern match (m//) at
> > > ./scripts/checkpatch.pl line 3383.> 
> > on which file?
> 
> Any file with a usleep_range or a memset
> 
> > If you would have added the command line to trivially reproduce that (or
> > whatever you did to get it), people are more likely to look into it.
> 
> Try this patch:
> https://lkml.org/lkml/2013/1/10/444

Yes, it works. Thanks
-- 
Franz Schrober

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-01-14 14:00 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-14 13:42 checkpatch.pl "uninitialized value" Schrober
2013-01-14 13:45 ` Bernd Petrovitsch
2013-01-14 13:50   ` Joe Perches
2013-01-14 13:59     ` Schrober
2013-01-14 13:53   ` Schrober

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.