How to create patch for checkpatch.pl cleanups

How to create patch for checkpatch.pl cleanups

[kashyap gada]

Hello Linux Newbies, I'm one too.
I was looking out for some clean up work and I ran the checkpatch.pl 
script. I came across a file with some 400 errors and 200 warnings. Now 
there were bunch of 80 char per line warning , braces and related 
things. I don't want to be sending patches for each and every error or 
warning I fix as it becomes tedious and If i send lots of correction 
patches in one go the kernel maintainers wont accept it. I want to know 
Can we correct all similar errors or warnings in one patch? or series of 
patches? 

Kashyap Gada gada.kashyap at gmail.com 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20120206/414fe5bb/attachment.html 

How to use /proc

[Surenkumar Nihalani]

Hello Guys, 

I am CS student trying to write a kernel module as a part of my class assignment. I wanted to know, from within kernel, how do I read and write data to my proc file? 
I need to store state from last call of my function. 

Thanks, 
Suren Nihalani. 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20120205/445fe425/attachment.html 

How to create patch for checkpatch.pl cleanups

[Daniel Baluta]

On Sun, Feb 5, 2012 at 11:00 PM, kashyap gada <kashyapgada@yahoo.com> wrote:
> Hello Linux Newbies, I'm one too. I was looking out for some clean up work
> and I ran the checkpatch.pl script. I came across a file with some 400
> errors and 200 warnings. Now there were bunch of 80 char per line warning ,
> braces and related things. I don't want to be sending patches for each and
> every error or warning I fix as it becomes tedious and If i send lots of
> correction patches in one go the kernel maintainers wont accept it. I want
> to know Can we correct all similar errors or warnings in one patch? or
> series of patches?

Regarding 80 chars per line issues, people are discussing right now
on increasing the limit to 100. See [1].

For the other issues, just send a patch with all the changes. It should
be fine since you are modifying one single file.

thanks,
Daniel.

[1] https://lkml.org/lkml/2012/2/5/141

How to use /proc

[Daniel Baluta]

On Sun, Feb 5, 2012 at 11:07 PM, Surenkumar Nihalani <suren@gatech.edu> wrote:
> Hello Guys,
>
> I am CS student trying to write a kernel module as a part of my class
> assignment. I wanted to know, from within kernel, how do I read and write
> data to my proc file?
> I need to store state from last call of my function.

Hello,

Have you tried searching on the internet? :D [1].

Another approach is to see how an existing proc
entry is handled. Look for example at /proc/sched_debug [2].

thanks,
Daniel.

[1] http://www.ibm.com/developerworks/linux/library/l-proc/index.html
[2] http://lxr.linux.no/linux+v3.2.4/kernel/sched_debug.c#L393

How to use /proc

[Surenkumar Nihalani]

Hey Daniel, 

Thanks for the quick reply.

The IBM tutorial helps me writing the function for handling user reads and writes to your corresponding proc file. It doesn't help in terms of opening
and writing from the kernel side.
the example you provided just creates a proc entry. It doesn't tell me how to write to it.

Please help.

Thanks,
Suren Nihalani.

----- Original Message -----
From: "Daniel Baluta" <daniel.baluta@gmail.com>
To: "Surenkumar Nihalani" <suren@gatech.edu>
Cc: Kernelnewbies at kernelnewbies.org
Sent: Sunday, February 5, 2012 4:20:28 PM
Subject: Re: How to use /proc

On Sun, Feb 5, 2012 at 11:07 PM, Surenkumar Nihalani <suren@gatech.edu> wrote:
> Hello Guys,
>
> I am CS student trying to write a kernel module as a part of my class
> assignment. I wanted to know, from within kernel, how do I read and write
> data to my proc file?
> I need to store state from last call of my function.

Hello,

Have you tried searching on the internet? :D [1].

Another approach is to see how an existing proc
entry is handled. Look for example at /proc/sched_debug [2].

thanks,
Daniel.

[1] http://www.ibm.com/developerworks/linux/library/l-proc/index.html
[2] http://lxr.linux.no/linux+v3.2.4/kernel/sched_debug.c#L393

How to use /proc

[Anuz Pratap Singh Tomar]

Read LDD, there are plenty of examples on using proc interface.

On Sun, Feb 5, 2012 at 10:14 PM, Surenkumar Nihalani <suren@gatech.edu>wrote:

> Hey Daniel,
>
> Thanks for the quick reply.
>
> The IBM tutorial helps me writing the function for handling user reads and
> writes to your corresponding proc file. It doesn't help in terms of opening
> and writing from the kernel side.
> the example you provided just creates a proc entry. It doesn't tell me how
> to write to it.
>
> Please help.
>
> Thanks,
> Suren Nihalani.
>
> ----- Original Message -----
> From: "Daniel Baluta" <daniel.baluta@gmail.com>
> To: "Surenkumar Nihalani" <suren@gatech.edu>
> Cc: Kernelnewbies at kernelnewbies.org
> Sent: Sunday, February 5, 2012 4:20:28 PM
> Subject: Re: How to use /proc
>
> On Sun, Feb 5, 2012 at 11:07 PM, Surenkumar Nihalani <suren@gatech.edu>
> wrote:
> > Hello Guys,
> >
> > I am CS student trying to write a kernel module as a part of my class
> > assignment. I wanted to know, from within kernel, how do I read and write
> > data to my proc file?
> > I need to store state from last call of my function.
>
> Hello,
>
> Have you tried searching on the internet? :D [1].
>
> Another approach is to see how an existing proc
> entry is handled. Look for example at /proc/sched_debug [2].
>
> thanks,
> Daniel.
>
> [1] http://www.ibm.com/developerworks/linux/library/l-proc/index.html
> [2] http://lxr.linux.no/linux+v3.2.4/kernel/sched_debug.c#L393
>
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies at kernelnewbies.org
> http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20120205/457c07c2/attachment.html 

How to create patch for checkpatch.pl cleanups

[Greg KH]

On Sun, Feb 05, 2012 at 11:10:29PM +0200, Daniel Baluta wrote:
> On Sun, Feb 5, 2012 at 11:00 PM, kashyap gada <kashyapgada@yahoo.com> wrote:
> > Hello Linux Newbies, I'm one too. I was looking out for some clean up work
> > and I ran the checkpatch.pl script. I came across a file with some 400
> > errors and 200 warnings. Now there were bunch of 80 char per line warning ,
> > braces and related things. I don't want to be sending patches for each and
> > every error or warning I fix as it becomes tedious and If i send lots of
> > correction patches in one go the kernel maintainers wont accept it. I want
> > to know Can we correct all similar errors or warnings in one patch? or
> > series of patches?
> 
> Regarding 80 chars per line issues, people are discussing right now
> on increasing the limit to 100. See [1].
> 
> For the other issues, just send a patch with all the changes. It should
> be fine since you are modifying one single file.

No, please break it up into "one thing per patch" to make it easy to
review, otherwise it will be ignored and/or rejected.

Also realize that a lot of subsystem maintainers do not like these types
of patches, outside of the drivers/staging/ area, so you might want to
ask the subsystem developers if they will take this type of work before
you do it.

greg k-h

How to use /proc

[Greg KH]

A: No.
Q: Should I include quotations after my reply?

http://daringfireball.net/2007/07/on_top

On Sun, Feb 05, 2012 at 05:14:02PM -0500, Surenkumar Nihalani wrote:
> Hey Daniel, 
> 
> Thanks for the quick reply.
> 
> The IBM tutorial helps me writing the function for handling user reads
> and writes to your corresponding proc file. It doesn't help in terms
> of opening and writing from the kernel side.  the example you provided
> just creates a proc entry. It doesn't tell me how to write to it.

You do not write/read proc files from within the kernel, that makes no
sense.  This is a virtual file, just modify the value that the file is
showing.

And please, never create new proc files, unless it really is for process
information, use debugfs for "playing around" type things, that is what
it is there for.

greg k-h

How to create patch for checkpatch.pl cleanups

[kashyap gada]

By one thing per patch you mean that I can correct say for example all the 80 char per line warnings in a single file as a patch?

Kashyap Gada



________________________________
 From: Greg KH <greg@kroah.com>
To: Daniel Baluta <daniel.baluta@gmail.com> 
Cc: kashyap gada <kashyapgada@yahoo.com>; "kernelnewbies at kernelnewbies.org" <kernelnewbies@kernelnewbies.org> 
Sent: Sunday, 5 February 2012 10:51 PM
Subject: Re: How to create patch for checkpatch.pl cleanups
 
On Sun, Feb 05, 2012 at 11:10:29PM +0200, Daniel Baluta wrote:
> On Sun, Feb 5, 2012 at 11:00 PM, kashyap gada <kashyapgada@yahoo.com> wrote:
> > Hello Linux Newbies, I'm one too. I was looking out for some clean up work
> > and I ran the checkpatch.pl script. I came across a file with some 400
> > errors and 200 warnings. Now there were bunch of 80 char per line warning ,
> > braces and related things. I don't want to be sending patches for each and
> > every error or warning I fix as it becomes tedious and If i send lots of
> > correction patches in one go the kernel maintainers wont accept it. I want
> > to know Can we correct all similar errors or warnings in one patch? or
> > series of patches?
> 
> Regarding 80 chars per line issues, people are discussing right now
> on increasing the limit to 100. See [1].
> 
> For the other issues, just send a patch with all the changes. It should
> be fine since you are modifying one single file.

No, please break it up into "one thing per patch" to make it easy to
review, otherwise it will be ignored and/or rejected.

Also realize that a lot of subsystem maintainers do not like these types
of patches, outside of the drivers/staging/ area, so you might want to
ask the subsystem developers if they will take this type of work before
you do it.

greg k-h
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20120206/cf27f85d/attachment-0001.html 

How to create patch for checkpatch.pl cleanups

[Greg KH]

A: No.
Q: Should I include quotations after my reply?

http://daringfireball.net/2007/07/on_top

On Mon, Feb 06, 2012 at 04:56:40AM +0530, kashyap gada wrote:
> By one thing per patch you mean that I can correct say for example all the 80
> char per line warnings in a single file as a patch?

Yes.

How to create patch for checkpatch.pl cleanups

[Bernd Petrovitsch]

On Mon, 2012-02-06 at 04:56 +0530, kashyap gada wrote:
> By one thing per patch you mean that I can correct say for example all
> the 80 char per line warnings in a single file as a patch?

For beginners, don't even try to work on these warnings - it is far more
important to keep the source easily readable (even if these rule is
broken) than to adhere some formatting rules.

[ Full-quote deleted ]

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