All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] edac: Remove fixmes in e7xxx_edac.c
@ 2014-07-22  4:53 Nicholas Krause
  2014-07-22 14:04 ` Bjørn Mork
  0 siblings, 1 reply; 16+ messages in thread
From: Nicholas Krause @ 2014-07-22  4:53 UTC (permalink / raw)
  To: dougthompson; +Cc: bp, m.chehab, linux-edac, linux-kernel

This removes two Page shift fixs me in this file and not checking
if row is -1 in process_ce as it cannot be this value or be must
exit this function by returning.

Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
---
 drivers/edac/e7xxx_edac.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/edac/e7xxx_edac.c b/drivers/edac/e7xxx_edac.c
index 3cda79b..563983f 100644
--- a/drivers/edac/e7xxx_edac.c
+++ b/drivers/edac/e7xxx_edac.c
@@ -211,12 +211,13 @@ static void process_ce(struct mem_ctl_info *mci, struct e7xxx_error_info *info)
 	edac_dbg(3, "\n");
 	/* read the error address */
 	error_1b = info->dram_celog_add;
-	/* FIXME - should use PAGE_SHIFT */
-	page = error_1b >> 6;	/* convert the address to 4k page */
+	page = error_1b >> PAGE_SHIFT;	/* convert the address to 4k page */
 	/* read the syndrome */
 	syndrome = info->dram_celog_syndrome;
-	/* FIXME - check for -1 */
 	row = edac_mc_find_csrow_by_page(mci, page);
+	if (row == -1)
+		return;
 	/* convert syndrome to channel */
 	channel = e7xxx_find_channel(syndrome);
 	edac_mc_handle_error(HW_EVENT_ERR_CORRECTED, mci, 1, page, 0, syndrome,
@@ -238,8 +239,7 @@ static void process_ue(struct mem_ctl_info *mci, struct e7xxx_error_info *info)
 	edac_dbg(3, "\n");
 	/* read the error address */
 	error_2b = info->dram_uelog_add;
-	/* FIXME - should use PAGE_SHIFT */
-	block_page = error_2b >> 6;	/* convert to 4k address */
+	block_page = error_2b >> PAGE_SHIFT;	/* convert to 4k address */
 	row = edac_mc_find_csrow_by_page(mci, block_page);
 
 	edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 1, block_page, 0, 0,
-- 
1.9.1


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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22  4:53 [PATCH] edac: Remove fixmes in e7xxx_edac.c Nicholas Krause
@ 2014-07-22 14:04 ` Bjørn Mork
  2014-07-22 17:25   ` Tony Luck
  0 siblings, 1 reply; 16+ messages in thread
From: Bjørn Mork @ 2014-07-22 14:04 UTC (permalink / raw)
  To: Nicholas Krause; +Cc: dougthompson, bp, m.chehab, linux-edac, linux-kernel

Nicholas Krause <xerofoify@gmail.com> writes:

> This removes two Page shift fixs me in this file and not checking
> if row is -1 in process_ce as it cannot be this value or be must
> exit this function by returning.
>
> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
> ---
>  drivers/edac/e7xxx_edac.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/edac/e7xxx_edac.c b/drivers/edac/e7xxx_edac.c
> index 3cda79b..563983f 100644
> --- a/drivers/edac/e7xxx_edac.c
> +++ b/drivers/edac/e7xxx_edac.c
> @@ -211,12 +211,13 @@ static void process_ce(struct mem_ctl_info *mci, struct e7xxx_error_info *info)
>  	edac_dbg(3, "\n");
>  	/* read the error address */
>  	error_1b = info->dram_celog_add;
> -	/* FIXME - should use PAGE_SHIFT */
> -	page = error_1b >> 6;	/* convert the address to 4k page */
> +	page = error_1b >> PAGE_SHIFT;	/* convert the address to 4k page */

I just have to bite...  Where did you get the idea that you can go
around and "fix" FIMXE's like they were spelling errors?


Bjørn

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 14:04 ` Bjørn Mork
@ 2014-07-22 17:25   ` Tony Luck
  2014-07-22 17:40     ` Nick Krause
  0 siblings, 1 reply; 16+ messages in thread
From: Tony Luck @ 2014-07-22 17:25 UTC (permalink / raw)
  To: Bjørn Mork
  Cc: Nicholas Krause, Doug Thompson, Borislav Petkov, m.chehab,
	Linux Edac Mailing List, Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 7:04 AM, Bjørn Mork <bjorn@mork.no> wrote:
>> -     /* FIXME - should use PAGE_SHIFT */
>> -     page = error_1b >> 6;   /* convert the address to 4k page */
>> +     page = error_1b >> PAGE_SHIFT;  /* convert the address to 4k page */
>
> I just have to bite...  Where did you get the idea that you can go
> around and "fix" FIMXE's like they were spelling errors?

Or apparently without thinking? Since "PAGE_SHIFT" does not have
the numerical value "6" your new code is obviously not equivalent to
the old.

-Tony

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 17:25   ` Tony Luck
@ 2014-07-22 17:40     ` Nick Krause
  2014-07-22 17:46       ` Borislav Petkov
  2014-07-22 18:14       ` Levente Kurusa
  0 siblings, 2 replies; 16+ messages in thread
From: Nick Krause @ 2014-07-22 17:40 UTC (permalink / raw)
  To: Tony Luck
  Cc: Bjørn Mork, Doug Thompson, Borislav Petkov, m.chehab,
	Linux Edac Mailing List, Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 1:25 PM, Tony Luck <tony.luck@gmail.com> wrote:
> On Tue, Jul 22, 2014 at 7:04 AM, Bjørn Mork <bjorn@mork.no> wrote:
>>> -     /* FIXME - should use PAGE_SHIFT */
>>> -     page = error_1b >> 6;   /* convert the address to 4k page */
>>> +     page = error_1b >> PAGE_SHIFT;  /* convert the address to 4k page */
>>
>> I just have to bite...  Where did you get the idea that you can go
>> around and "fix" FIMXE's like they were spelling errors?
>
> Or apparently without thinking? Since "PAGE_SHIFT" does not have
> the numerical value "6" your new code is obviously not equivalent to
> the old.
>
> -Tony

Tony,

What is the value of Page shift then ? If you can tell me it would be
much easier for
me to fix this.
Cheers Nick

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 17:40     ` Nick Krause
@ 2014-07-22 17:46       ` Borislav Petkov
  2014-07-22 18:03         ` Nick Krause
  2014-07-22 18:14       ` Levente Kurusa
  1 sibling, 1 reply; 16+ messages in thread
From: Borislav Petkov @ 2014-07-22 17:46 UTC (permalink / raw)
  To: Nick Krause
  Cc: Tony Luck, Bjørn Mork, Doug Thompson, m.chehab,
	Linux Edac Mailing List, Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 01:40:42PM -0400, Nick Krause wrote:
> What is the value of Page shift then ? If you can tell me it would be
> much easier for
> me to fix this.

I'll give you a rat's ass if you find that out on your own.

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 17:46       ` Borislav Petkov
@ 2014-07-22 18:03         ` Nick Krause
  2014-07-22 18:08           ` Borislav Petkov
  0 siblings, 1 reply; 16+ messages in thread
From: Nick Krause @ 2014-07-22 18:03 UTC (permalink / raw)
  To: Borislav Petkov
  Cc: Tony Luck, Bjørn Mork, Doug Thompson, m.chehab,
	Linux Edac Mailing List, Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 1:46 PM, Borislav Petkov <bp@alien8.de> wrote:
> On Tue, Jul 22, 2014 at 01:40:42PM -0400, Nick Krause wrote:
>> What is the value of Page shift then ? If you can tell me it would be
>> much easier for
>> me to fix this.
>
> I'll give you a rat's ass if you find that out on your own.
>
> --
> Regards/Gruss,
>     Boris.
>
> Sent from a fat crate under my desk. Formatting is fine.
> --
Page shift is 12 for 4k pages.
Nick

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 18:03         ` Nick Krause
@ 2014-07-22 18:08           ` Borislav Petkov
  0 siblings, 0 replies; 16+ messages in thread
From: Borislav Petkov @ 2014-07-22 18:08 UTC (permalink / raw)
  To: Nick Krause
  Cc: Tony Luck, Bjørn Mork, Doug Thompson, m.chehab,
	Linux Edac Mailing List, Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 02:03:38PM -0400, Nick Krause wrote:
> Page shift is 12 for 4k pages.

So you can find stuff out on your own after all, good. Hold on to that
thought like your life depended on it!

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 17:40     ` Nick Krause
  2014-07-22 17:46       ` Borislav Petkov
@ 2014-07-22 18:14       ` Levente Kurusa
  2014-07-22 18:25         ` Nick Krause
  1 sibling, 1 reply; 16+ messages in thread
From: Levente Kurusa @ 2014-07-22 18:14 UTC (permalink / raw)
  To: Nick Krause
  Cc: Tony Luck, Bjørn Mork, Doug Thompson, Borislav Petkov,
	m.chehab, Linux Edac Mailing List, Linux Kernel Mailing List

2014-07-22 19:40 GMT+02:00 Nick Krause <xerofoify@gmail.com>:
> [...]
>
> Tony,
>
> What is the value of Page shift then ? If you can tell me it would be
> much easier for
> me to fix this.
> Cheers Nick

Nick,

Help is always welcome. However, what you are doing is effectively a
waste of time for you and for the maintainers as well, and hence cannot
be called help. You are writing emails that make absolutely no sense at all.
If you want to help us out, please consider learning three things:

* The language that the kernel is written in: C
* The language that most kernel developers understand: English
* The environment the kernel is (in most cases) written in: UNIX

If you had known these, then:

* ... you would have already known that FIXMEs exist for a reason which is
   not at all simple.

* ... you would have understood that fixing FIXMEs without ANY reasoning and
   asking about it is completely useless and wastes precious maintainer time.

* ... you could have used grep for finding out PAGE_SHIFT and not need to ask
   for it.

Nick. Look, it is good that you are enthusiastic. However, this kind
of 'work' gives
you an undesirable reputation, which is {IF FROM:xerofoify@* THEN TRASH}...

If you really want to help out, I suggest you to find yourself a piece
of hardware from
the store and try to write a driver for it. That way, you will learn
how it all works, and if
you are intelligent enough then you might get a "Kernel Job - Payed"
like you asked us
for them in November.

Until then, please stop trying for FIXMEs. They will not get you a
commit in the kernel
99% of the time.

Hope you understand,

Levente Kurusa

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 18:14       ` Levente Kurusa
@ 2014-07-22 18:25         ` Nick Krause
  2014-07-22 19:17           ` Randy Dunlap
  2014-07-22 19:33           ` Steven Rostedt
  0 siblings, 2 replies; 16+ messages in thread
From: Nick Krause @ 2014-07-22 18:25 UTC (permalink / raw)
  To: Levente Kurusa
  Cc: Tony Luck, Bjørn Mork, Doug Thompson, Borislav Petkov,
	m.chehab, Linux Edac Mailing List, Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 2:14 PM, Levente Kurusa <lkurusa@redhat.com> wrote:
> 2014-07-22 19:40 GMT+02:00 Nick Krause <xerofoify@gmail.com>:
>> [...]
>>
>> Tony,
>>
>> What is the value of Page shift then ? If you can tell me it would be
>> much easier for
>> me to fix this.
>> Cheers Nick
>
> Nick,
>
> Help is always welcome. However, what you are doing is effectively a
> waste of time for you and for the maintainers as well, and hence cannot
> be called help. You are writing emails that make absolutely no sense at all.
> If you want to help us out, please consider learning three things:
>
> * The language that the kernel is written in: C
> * The language that most kernel developers understand: English
> * The environment the kernel is (in most cases) written in: UNIX
>
> If you had known these, then:
>
> * ... you would have already known that FIXMEs exist for a reason which is
>    not at all simple.
>
> * ... you would have understood that fixing FIXMEs without ANY reasoning and
>    asking about it is completely useless and wastes precious maintainer time.
>
> * ... you could have used grep for finding out PAGE_SHIFT and not need to ask
>    for it.
>
> Nick. Look, it is good that you are enthusiastic. However, this kind
> of 'work' gives
> you an undesirable reputation, which is {IF FROM:xerofoify@* THEN TRASH}...
>
> If you really want to help out, I suggest you to find yourself a piece
> of hardware from
> the store and try to write a driver for it. That way, you will learn
> how it all works, and if
> you are intelligent enough then you might get a "Kernel Job - Payed"
> like you asked us
> for them in November.
>
> Until then, please stop trying for FIXMEs. They will not get you a
> commit in the kernel
> 99% of the time.
>
> Hope you understand,
>
> Levente Kurusa


Levente,

I understand what your saying and I should have searched for Page shift.
In addition I am already got commits in the kernel for fix mes so I feel
that your comment on fix mes is incorrect.
Nick

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 18:25         ` Nick Krause
@ 2014-07-22 19:17           ` Randy Dunlap
  2014-07-22 19:33           ` Steven Rostedt
  1 sibling, 0 replies; 16+ messages in thread
From: Randy Dunlap @ 2014-07-22 19:17 UTC (permalink / raw)
  To: Nick Krause, Levente Kurusa
  Cc: Tony Luck, Bjørn Mork, Doug Thompson, Borislav Petkov,
	m.chehab, Linux Edac Mailing List, Linux Kernel Mailing List

On 07/22/2014 11:25 AM, Nick Krause wrote:
> On Tue, Jul 22, 2014 at 2:14 PM, Levente Kurusa <lkurusa@redhat.com> wrote:
>> 2014-07-22 19:40 GMT+02:00 Nick Krause <xerofoify@gmail.com>:
>>> [...]
>>>
>>> Tony,
>>>
>>> What is the value of Page shift then ? If you can tell me it would be
>>> much easier for
>>> me to fix this.
>>> Cheers Nick
>>
>> Nick,
>>
>> Help is always welcome. However, what you are doing is effectively a
>> waste of time for you and for the maintainers as well, and hence cannot
>> be called help. You are writing emails that make absolutely no sense at all.
>> If you want to help us out, please consider learning three things:
>>
>> * The language that the kernel is written in: C
>> * The language that most kernel developers understand: English
>> * The environment the kernel is (in most cases) written in: UNIX
>>
>> If you had known these, then:
>>
>> * ... you would have already known that FIXMEs exist for a reason which is
>>    not at all simple.
>>
>> * ... you would have understood that fixing FIXMEs without ANY reasoning and
>>    asking about it is completely useless and wastes precious maintainer time.
>>
>> * ... you could have used grep for finding out PAGE_SHIFT and not need to ask
>>    for it.
>>
>> Nick. Look, it is good that you are enthusiastic. However, this kind
>> of 'work' gives
>> you an undesirable reputation, which is {IF FROM:xerofoify@* THEN TRASH}...
>>
>> If you really want to help out, I suggest you to find yourself a piece
>> of hardware from
>> the store and try to write a driver for it. That way, you will learn
>> how it all works, and if
>> you are intelligent enough then you might get a "Kernel Job - Payed"
>> like you asked us
>> for them in November.
>>
>> Until then, please stop trying for FIXMEs. They will not get you a
>> commit in the kernel
>> 99% of the time.
>>
>> Hope you understand,
>>
>> Levente Kurusa
> 
> 
> Levente,
> 
> I understand what your saying and I should have searched for Page shift.

so you know that PAGE_SHIFT is not always 12?

> In addition I am already got commits in the kernel for fix mes so I feel
> that your comment on fix mes is incorrect.

what is your goal?


-- 
~Randy

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 18:25         ` Nick Krause
  2014-07-22 19:17           ` Randy Dunlap
@ 2014-07-22 19:33           ` Steven Rostedt
  2014-07-22 20:56             ` Nick Krause
  1 sibling, 1 reply; 16+ messages in thread
From: Steven Rostedt @ 2014-07-22 19:33 UTC (permalink / raw)
  To: Nick Krause
  Cc: Levente Kurusa, Tony Luck, Bjørn Mork, Doug Thompson,
	Borislav Petkov, m.chehab, Linux Edac Mailing List,
	Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 02:25:24PM -0400, Nick Krause wrote:
> 
> I understand what your saying and I should have searched for Page shift.
> In addition I am already got commits in the kernel for fix mes so I feel
> that your comment on fix mes is incorrect.

After seeing your other solutions to "fix mes" around the kernel, I now have to
investigate the patches that were accepted, and see if they did not cause
any new bugs. If any of them do, I will ask to have all your patches reverted.

You are not helping. You are actually doing quite the opposite. Who do you work
for? Microsoft or Apple?

-- Steve


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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 19:33           ` Steven Rostedt
@ 2014-07-22 20:56             ` Nick Krause
  2014-07-23 14:37               ` Steven Rostedt
  0 siblings, 1 reply; 16+ messages in thread
From: Nick Krause @ 2014-07-22 20:56 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Levente Kurusa, Tony Luck, Bjørn Mork, Doug Thompson,
	Borislav Petkov, m.chehab, Linux Edac Mailing List,
	Linux Kernel Mailing List

On Tue, Jul 22, 2014 at 3:33 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
> On Tue, Jul 22, 2014 at 02:25:24PM -0400, Nick Krause wrote:
>>
>> I understand what your saying and I should have searched for Page shift.
>> In addition I am already got commits in the kernel for fix mes so I feel
>> that your comment on fix mes is incorrect.
>
> After seeing your other solutions to "fix mes" around the kernel, I now have to
> investigate the patches that were accepted, and see if they did not cause
> any new bugs. If any of them do, I will ask to have all your patches reverted.
>
> You are not helping. You are actually doing quite the opposite. Who do you work
> for? Microsoft or Apple?
>
> -- Steve
>

I don't work for any companies. And here is one of them that has been accepted.
I am removing two fix mes in this file as after dicussing then it  seems
there is no reason to check against Null for usb_device as it can never
be NULL and this is check is therefore not needed.

Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
---
 drivers/usb/core/hcd.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
index bec31e2..487abcf 100644
--- a/drivers/usb/core/hcd.c
+++ b/drivers/usb/core/hcd.c
@@ -855,8 +855,6 @@ static ssize_t authorized_default_show(struct device *dev,
        struct usb_bus *usb_bus = rh_usb_dev->bus;
        struct usb_hcd *usb_hcd;

-       if (usb_bus == NULL)    /* FIXME: not sure if this case is possible */
-               return -ENODEV;
        usb_hcd = bus_to_hcd(usb_bus);
        return snprintf(buf, PAGE_SIZE, "%u\n", usb_hcd->authorized_default);
 }
@@ -871,8 +869,6 @@ static ssize_t authorized_default_store(
struct device *dev,
        struct usb_bus *usb_bus = rh_usb_dev->bus;
        struct usb_hcd *usb_hcd;

-       if (usb_bus == NULL)    /* FIXME: not sure if this case is possible */
-               return -ENODEV;
        usb_hcd = bus_to_hcd(usb_bus);
        result = sscanf(buf, "%u\n", &val);
        if (result == 1) {
Nick

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-22 20:56             ` Nick Krause
@ 2014-07-23 14:37               ` Steven Rostedt
  2014-07-23 15:35                 ` Nick Krause
  0 siblings, 1 reply; 16+ messages in thread
From: Steven Rostedt @ 2014-07-23 14:37 UTC (permalink / raw)
  To: Nick Krause
  Cc: Levente Kurusa, Tony Luck, Bjørn Mork, Doug Thompson,
	Borislav Petkov, m.chehab, Linux Edac Mailing List,
	Linux Kernel Mailing List

On Tue, 22 Jul 2014 16:56:06 -0400
Nick Krause <xerofoify@gmail.com> wrote:

> On Tue, Jul 22, 2014 at 3:33 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
> > On Tue, Jul 22, 2014 at 02:25:24PM -0400, Nick Krause wrote:
> >>
> >> I understand what your saying and I should have searched for Page shift.
> >> In addition I am already got commits in the kernel for fix mes so I feel
> >> that your comment on fix mes is incorrect.
> >
> > After seeing your other solutions to "fix mes" around the kernel, I now have to
> > investigate the patches that were accepted, and see if they did not cause
> > any new bugs. If any of them do, I will ask to have all your patches reverted.
> >
> > You are not helping. You are actually doing quite the opposite. Who do you work
> > for? Microsoft or Apple?
> >
> > -- Steve
> >
> 
> I don't work for any companies. And here is one of them that has been accepted.
> I am removing two fix mes in this file as after dicussing then it  seems
> there is no reason to check against Null for usb_device as it can never
> be NULL and this is check is therefore not needed.

http://marc.info/?l=linux-m68k&m=140612440420712&w=2

And your patches seem to be getting reverted. Wow, I think you broke a
new record in having the percentage of patches added to the kernel and
then reverted. Even before they made it to mainline.

You may be the only developer that has all their commits reverted
before they ever make it into the mainline tree!

Congratulations! I think it is wise now for all maintainers to add your
email to their /dev/null folder. It will help them from wasting any
more of their precious time on you.

-- Steve



> 
> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
> ---
>  drivers/usb/core/hcd.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
> index bec31e2..487abcf 100644
> --- a/drivers/usb/core/hcd.c
> +++ b/drivers/usb/core/hcd.c
> @@ -855,8 +855,6 @@ static ssize_t authorized_default_show(struct device *dev,
>         struct usb_bus *usb_bus = rh_usb_dev->bus;
>         struct usb_hcd *usb_hcd;
> 
> -       if (usb_bus == NULL)    /* FIXME: not sure if this case is possible */
> -               return -ENODEV;
>         usb_hcd = bus_to_hcd(usb_bus);
>         return snprintf(buf, PAGE_SIZE, "%u\n", usb_hcd->authorized_default);
>  }
> @@ -871,8 +869,6 @@ static ssize_t authorized_default_store(
> struct device *dev,
>         struct usb_bus *usb_bus = rh_usb_dev->bus;
>         struct usb_hcd *usb_hcd;
> 
> -       if (usb_bus == NULL)    /* FIXME: not sure if this case is possible */
> -               return -ENODEV;
>         usb_hcd = bus_to_hcd(usb_bus);
>         result = sscanf(buf, "%u\n", &val);
>         if (result == 1) {
> Nick


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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-23 14:37               ` Steven Rostedt
@ 2014-07-23 15:35                 ` Nick Krause
  2014-07-23 16:07                   ` Steven Rostedt
  0 siblings, 1 reply; 16+ messages in thread
From: Nick Krause @ 2014-07-23 15:35 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Levente Kurusa, Tony Luck, Bjørn Mork, Doug Thompson,
	Borislav Petkov, m.chehab, Linux Edac Mailing List,
	Linux Kernel Mailing List

On Wed, Jul 23, 2014 at 10:37 AM, Steven Rostedt <rostedt@goodmis.org> wrote:
> On Tue, 22 Jul 2014 16:56:06 -0400
> Nick Krause <xerofoify@gmail.com> wrote:
>
>> On Tue, Jul 22, 2014 at 3:33 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
>> > On Tue, Jul 22, 2014 at 02:25:24PM -0400, Nick Krause wrote:
>> >>
>> >> I understand what your saying and I should have searched for Page shift.
>> >> In addition I am already got commits in the kernel for fix mes so I feel
>> >> that your comment on fix mes is incorrect.
>> >
>> > After seeing your other solutions to "fix mes" around the kernel, I now have to
>> > investigate the patches that were accepted, and see if they did not cause
>> > any new bugs. If any of them do, I will ask to have all your patches reverted.
>> >
>> > You are not helping. You are actually doing quite the opposite. Who do you work
>> > for? Microsoft or Apple?
>> >
>> > -- Steve
>> >
>>
>> I don't work for any companies. And here is one of them that has been accepted.
>> I am removing two fix mes in this file as after dicussing then it  seems
>> there is no reason to check against Null for usb_device as it can never
>> be NULL and this is check is therefore not needed.
>
> http://marc.info/?l=linux-m68k&m=140612440420712&w=2
>
> And your patches seem to be getting reverted. Wow, I think you broke a
> new record in having the percentage of patches added to the kernel and
> then reverted. Even before they made it to mainline.
>
> You may be the only developer that has all their commits reverted
> before they ever make it into the mainline tree!
>
> Congratulations! I think it is wise now for all maintainers to add your
> email to their /dev/null folder. It will help them from wasting any
> more of their precious time on you.
>
> -- Steve
Steve, I  have make a few mistakes. That doesn't give you the right to
put me under a waste of time yet.
Nick

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-23 15:35                 ` Nick Krause
@ 2014-07-23 16:07                   ` Steven Rostedt
  2014-07-23 16:19                     ` Nick Krause
  0 siblings, 1 reply; 16+ messages in thread
From: Steven Rostedt @ 2014-07-23 16:07 UTC (permalink / raw)
  To: Nick Krause
  Cc: Levente Kurusa, Tony Luck, Bjørn Mork, Doug Thompson,
	Borislav Petkov, m.chehab, Linux Edac Mailing List,
	Linux Kernel Mailing List

On Wed, 23 Jul 2014 11:35:21 -0400
Nick Krause <xerofoify@gmail.com> wrote:

> Steve, I  have make a few mistakes. That doesn't give you the right to
> put me under a waste of time yet.

Nobody is upset at you for making a few mistakes. But ignoring advice
from people is something quite different, and that is what tends to
piss people off.

You were told time and time again to stop this fixme craze because you
obviously didn't know the details of what you were fixing, but you
continued to care on. The issue is with these patches that you probably
caught a maintainer off guard, and they just accepted it thinking you
had some clue to the code you were fixing. Thus, not only are you
removing crucial comments about areas of code that needs more
attention, you end up making things worse. Your actions are dangerous to
the kernel and the fact you ignoring everyone telling you to stop it,
makes you dangerous to the kernel. That gives me the right to put you
under the "waste of time" folder.

Now it may seem that I'm being rather harsh to you, and I may be. The
Linux kernel is a serious project and it can not afford having to deal
with those that do not listen.

That said...

I see you stated in another email:

"Very well then I guess the community wants me to listen better to the
maintainers and not do stupid things"


Finally, you are listening.

You obviously are very enthusiastic, and I would like you not to waste
your own time and energy trying to help in a not so helpful way. If you
really want to help the Linux community, here's what you do.

Find a single area to focus on. Not a key word throughout the kernel.
Look at the USB stack, or find some driver in staging that is for some
really cheap hardware that you can afford. And get it cleaned up and
working nicely. Start simple. Try to understand that area completely.

Remember, nobody understand every aspect of the kernel. All the main
kernel developers are experts in select parts. I wouldn't even try
fixing "fixmes" in parts of the kernel that I'm unfamiliar with.

Find a part of the kernel, and learn every aspect of that part inside
and out. Study the code. Boot the kernel. Make a modification of that
code and see what happens. Yes! Break it (but don't submit a patch that
does ;). Sometimes breaking stuff is the best way to understand why it
worked in the first place. Use function graph tracing to see the code
flow.

After you have done that, and you really have a good understanding of
that part of the kernel, you should in the mean time see a way to
enhance it, or better yet, find a real bug and fix it. Then you can
submit a patch and that will be something of use.

Moral of the story: Learn the kernel before submitting a patch, do not
submit a patch to help you learn the kernel.

-- Steve

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

* Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c
  2014-07-23 16:07                   ` Steven Rostedt
@ 2014-07-23 16:19                     ` Nick Krause
  0 siblings, 0 replies; 16+ messages in thread
From: Nick Krause @ 2014-07-23 16:19 UTC (permalink / raw)
  To: Steven Rostedt
  Cc: Levente Kurusa, Tony Luck, Bjørn Mork, Doug Thompson,
	Borislav Petkov, m.chehab, Linux Edac Mailing List,
	Linux Kernel Mailing List

On Wed, Jul 23, 2014 at 12:07 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
> On Wed, 23 Jul 2014 11:35:21 -0400
> Nick Krause <xerofoify@gmail.com> wrote:
>
>> Steve, I  have make a few mistakes. That doesn't give you the right to
>> put me under a waste of time yet.
>
> Nobody is upset at you for making a few mistakes. But ignoring advice
> from people is something quite different, and that is what tends to
> piss people off.
>
> You were told time and time again to stop this fixme craze because you
> obviously didn't know the details of what you were fixing, but you
> continued to care on. The issue is with these patches that you probably
> caught a maintainer off guard, and they just accepted it thinking you
> had some clue to the code you were fixing. Thus, not only are you
> removing crucial comments about areas of code that needs more
> attention, you end up making things worse. Your actions are dangerous to
> the kernel and the fact you ignoring everyone telling you to stop it,
> makes you dangerous to the kernel. That gives me the right to put you
> under the "waste of time" folder.
>
> Now it may seem that I'm being rather harsh to you, and I may be. The
> Linux kernel is a serious project and it can not afford having to deal
> with those that do not listen.
>
> That said...
>
> I see you stated in another email:
>
> "Very well then I guess the community wants me to listen better to the
> maintainers and not do stupid things"
>
>
> Finally, you are listening.
>
> You obviously are very enthusiastic, and I would like you not to waste
> your own time and energy trying to help in a not so helpful way. If you
> really want to help the Linux community, here's what you do.
>
> Find a single area to focus on. Not a key word throughout the kernel.
> Look at the USB stack, or find some driver in staging that is for some
> really cheap hardware that you can afford. And get it cleaned up and
> working nicely. Start simple. Try to understand that area completely.
>
> Remember, nobody understand every aspect of the kernel. All the main
> kernel developers are experts in select parts. I wouldn't even try
> fixing "fixmes" in parts of the kernel that I'm unfamiliar with.
>
> Find a part of the kernel, and learn every aspect of that part inside
> and out. Study the code. Boot the kernel. Make a modification of that
> code and see what happens. Yes! Break it (but don't submit a patch that
> does ;). Sometimes breaking stuff is the best way to understand why it
> worked in the first place. Use function graph tracing to see the code
> flow.
>
> After you have done that, and you really have a good understanding of
> that part of the kernel, you should in the mean time see a way to
> enhance it, or better yet, find a real bug and fix it. Then you can
> submit a patch and that will be something of use.
>
> Moral of the story: Learn the kernel before submitting a patch, do not
> submit a patch to help you learn the kernel.
>
> -- Steve


Steve ,
Thanks for your help.
Cheers Nick

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

end of thread, other threads:[~2014-07-23 16:19 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-22  4:53 [PATCH] edac: Remove fixmes in e7xxx_edac.c Nicholas Krause
2014-07-22 14:04 ` Bjørn Mork
2014-07-22 17:25   ` Tony Luck
2014-07-22 17:40     ` Nick Krause
2014-07-22 17:46       ` Borislav Petkov
2014-07-22 18:03         ` Nick Krause
2014-07-22 18:08           ` Borislav Petkov
2014-07-22 18:14       ` Levente Kurusa
2014-07-22 18:25         ` Nick Krause
2014-07-22 19:17           ` Randy Dunlap
2014-07-22 19:33           ` Steven Rostedt
2014-07-22 20:56             ` Nick Krause
2014-07-23 14:37               ` Steven Rostedt
2014-07-23 15:35                 ` Nick Krause
2014-07-23 16:07                   ` Steven Rostedt
2014-07-23 16:19                     ` Nick Krause

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.