linux-ext4 - Re: [PATCH 01/10] ext4: balloc: Fixed coding style issue

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LFD.2.00.1210171609040.30534@dhcp-1-104.brq.redhat.com>
Date:	Wed, 17 Oct 2012 16:27:39 +0200 (CEST)
From:	Lukáš Czerner <lczerner@...hat.com>
To:	Adil Mujeeb <mujeeb.adil@...il.com>
cc:	richard -rw- weinberger <richard.weinberger@...il.com>,
	Lukáš Czerner <lczerner@...hat.com>,
	linux-ext4@...r.kernel.org
Subject: Re: [PATCH 01/10] ext4: balloc: Fixed coding style issue

On Wed, 17 Oct 2012, Adil Mujeeb wrote:

> Date: Wed, 17 Oct 2012 19:35:40 +0530
> From: Adil Mujeeb <mujeeb.adil@...il.com>
> To: richard -rw- weinberger <richard.weinberger@...il.com>,
>     Lukáš Czerner <lczerner@...hat.com>
> Subject: Re: [PATCH 01/10] ext4: balloc: Fixed coding style issue
> 
> Hi,
> 
> On Wed, Oct 17, 2012 at 5:43 PM, richard -rw- weinberger
> <richard.weinberger@...il.com> wrote:
> > On Wed, Oct 17, 2012 at 2:05 PM, Lukáš Czerner <lczerner@...hat.com> wrote:
> >> On Wed, 17 Oct 2012, Adil Mujeeb wrote:
> >>
> >>> Date: Wed, 17 Oct 2012 00:42:56 +0530
> >>> From: Adil Mujeeb <mujeeb.adil@...il.com>
> >>> To: tytso@....edu, adilger.kernel@...ger.ca, linux-ext4@...r.kernel.org,
> >>>     linux-kernel@...r.kernel.org
> >>> Cc: Adil Mujeeb <mujeeb.adil@...il.com>
> >>> Subject: [PATCH 01/10] ext4: balloc: Fixed coding style issue
> >>>
> >>> Fixed checkpatch.pl reported ERRORs
> >>
> >> Hi Adil,
> >>
> >> let me ask you something. How useful do you think those changes are ?
> >> Have you learned anything by creating those patches ?
> 
> I am newbie. I just thought of starting with cleanup thing. I know
> that from functionality point of view, I am not adding anything but
> thought if it helps in coding guideline compliance and uniformity of
> code.
> Now I understand that this is not required. In future, I will avoid
> submitting such patches.

So you've learned something :). But seriously such patches are
causing more problems than it solves. It's perfectly fine to clean
up the code in the area you're changing anyway, but making coding
style patches just for a sake of it is not usually useful. Moreover
it is not useful for you as well, because you're not going to learn
anything.

There are plenty of real problem to be solved, but it actually
involves reading and understanding the code first.

Just for example:

 - get familiar with xfstests
 - there might still be problems with unified ext4 driver where
   we might use options not suited for respective file system
 - there are still issues with bigalloc
 - I suspect that there will be some problems with file system >16TB
 - recently there has been some reports from bigzilla.kernel.org
 - or you can just read the code, trying to understand how it works
   and soon or later you will find something to fix :) I am sure
   about that.

Maybe someone else have better suggestions...

Good luck!
-Lukas

> 
> >>
> >> Just to clarify why I am asking such weird questions. It's not one of
> >> those sneer questions, I would really like to know.
> >>>  linux-3.7-rc1/fs/ext4/balloc.c |    8 ++++----
> >
> > The really sad thing is that this patch does not even apply because
> > the path is malformed.
> 
> I am sorry, seems I am missing something. I tried with my local repo
> and it worked :(
> Its not the right place to ask, so I'll look what went wrong with the patches.
> 
> Rgds,
> Adil
> 
> >
> > --
> > Thanks,
> > //richard
>