* [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
@ 2017-09-10 9:40 Shreeya Patel
2017-09-10 10:13 ` [Outreachy kernel] " Julia Lawall
2017-09-11 16:32 ` Greg KH
0 siblings, 2 replies; 13+ messages in thread
From: Shreeya Patel @ 2017-09-10 9:40 UTC (permalink / raw)
To: gregkh, outreachy-kernel; +Cc: Shreeya Patel
Logging messages that show some type of 'out of memory' error
are generally unnecessary as there is a generic message and
a stack dump done by the memory subsystem.
These messages generally increase kernel size without much
added value.
Problem found by checkpatch.
Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
---
Changes in v4:
-Include v2 and v3 in the patch.
Changes in v3:
-Make the commit message more appropriate.
changes in v2:
-Original patch was not sent to the mailing list.
Also, add a tab before the return statement.
drivers/staging/rtlwifi/rc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
index 65de0c7..779a5c6 100644
--- a/drivers/staging/rtlwifi/rc.c
+++ b/drivers/staging/rtlwifi/rc.c
@@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
struct rtl_rate_priv *rate_priv;
rate_priv = kzalloc(sizeof(*rate_priv), gfp);
- if (!rate_priv) {
- pr_err("Unable to allocate private rc structure\n");
+ if (!rate_priv)
return NULL;
- }
rtlpriv->rate_priv = rate_priv;
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-10 9:40 [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message Shreeya Patel
@ 2017-09-10 10:13 ` Julia Lawall
2017-09-11 16:32 ` Greg KH
1 sibling, 0 replies; 13+ messages in thread
From: Julia Lawall @ 2017-09-10 10:13 UTC (permalink / raw)
To: Shreeya Patel; +Cc: gregkh, outreachy-kernel
On Sun, 10 Sep 2017, Shreeya Patel wrote:
> Logging messages that show some type of 'out of memory' error
> are generally unnecessary as there is a generic message and
> a stack dump done by the memory subsystem.
>
> These messages generally increase kernel size without much
> added value.
>
> Problem found by checkpatch.
>
> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
>
> Changes in v4:
> -Include v2 and v3 in the patch.
>
> Changes in v3:
> -Make the commit message more appropriate.
>
> changes in v2:
> -Original patch was not sent to the mailing list.
> Also, add a tab before the return statement.
>
> drivers/staging/rtlwifi/rc.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
> index 65de0c7..779a5c6 100644
> --- a/drivers/staging/rtlwifi/rc.c
> +++ b/drivers/staging/rtlwifi/rc.c
> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
> struct rtl_rate_priv *rate_priv;
>
> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
> - if (!rate_priv) {
> - pr_err("Unable to allocate private rc structure\n");
> + if (!rate_priv)
> return NULL;
> - }
>
> rtlpriv->rate_priv = rate_priv;
>
> --
> 2.7.4
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1505036405-3702-1-git-send-email-shreeya.patel23498%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 16:32 ` Greg KH
@ 2017-09-11 16:18 ` Shreeya Patel
2017-09-11 16:57 ` Greg KH
2017-09-11 17:01 ` [Outreachy kernel] " Julia Lawall
0 siblings, 2 replies; 13+ messages in thread
From: Shreeya Patel @ 2017-09-11 16:18 UTC (permalink / raw)
To: Greg KH; +Cc: outreachy-kernel
[-- Attachment #1: Type: text/plain, Size: 1903 bytes --]
On Monday 11 September 2017 10:02 PM, Greg KH wrote:
> On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
>> Logging messages that show some type of 'out of memory' error
>> are generally unnecessary as there is a generic message and
>> a stack dump done by the memory subsystem.
>>
>> These messages generally increase kernel size without much
>> added value.
>>
>> Problem found by checkpatch.
>>
>> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
>> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
>> ---
>>
>> Changes in v4:
>> -Include v2 and v3 in the patch.
>>
>> Changes in v3:
>> -Make the commit message more appropriate.
>>
>> changes in v2:
>> -Original patch was not sent to the mailing list.
>> Also, add a tab before the return statement.
>>
>> drivers/staging/rtlwifi/rc.c | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
>> index 65de0c7..779a5c6 100644
>> --- a/drivers/staging/rtlwifi/rc.c
>> +++ b/drivers/staging/rtlwifi/rc.c
>> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
>> struct rtl_rate_priv *rate_priv;
>>
>> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
>> - if (!rate_priv) {
>> - pr_err("Unable to allocate private rc structure\n");
>> + if (!rate_priv)
>> return NULL;
>> - }
>>
>> rtlpriv->rate_priv = rate_priv;
>>
> This patch is sent corrupted (leading space is gone), and git doesn't
> like it. Can you please fix it up and resend?
>
> thanks,
>
> greg k-h
Hello sir,
If i am not removing the leading spaces then checkpatch is showing me
errors for trailing whitespace.
But there are no leading spaces present in the file which i have modified.
When i convert my changes into the patch, the whitespace errors are
shown up.
Can you please help me with this to make me understand why this is
happening?
[-- Attachment #2: Type: text/html, Size: 2529 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-10 9:40 [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message Shreeya Patel
2017-09-10 10:13 ` [Outreachy kernel] " Julia Lawall
@ 2017-09-11 16:32 ` Greg KH
2017-09-11 16:18 ` Shreeya Patel
1 sibling, 1 reply; 13+ messages in thread
From: Greg KH @ 2017-09-11 16:32 UTC (permalink / raw)
To: Shreeya Patel; +Cc: outreachy-kernel
On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
> Logging messages that show some type of 'out of memory' error
> are generally unnecessary as there is a generic message and
> a stack dump done by the memory subsystem.
>
> These messages generally increase kernel size without much
> added value.
>
> Problem found by checkpatch.
>
> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
>
> Changes in v4:
> -Include v2 and v3 in the patch.
>
> Changes in v3:
> -Make the commit message more appropriate.
>
> changes in v2:
> -Original patch was not sent to the mailing list.
> Also, add a tab before the return statement.
>
> drivers/staging/rtlwifi/rc.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
> index 65de0c7..779a5c6 100644
> --- a/drivers/staging/rtlwifi/rc.c
> +++ b/drivers/staging/rtlwifi/rc.c
> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
> struct rtl_rate_priv *rate_priv;
>
> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
> - if (!rate_priv) {
> - pr_err("Unable to allocate private rc structure\n");
> + if (!rate_priv)
> return NULL;
> - }
>
> rtlpriv->rate_priv = rate_priv;
>
This patch is sent corrupted (leading space is gone), and git doesn't
like it. Can you please fix it up and resend?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 16:18 ` Shreeya Patel
@ 2017-09-11 16:57 ` Greg KH
2017-09-11 17:00 ` Shreeya Patel
2017-09-11 17:01 ` [Outreachy kernel] " Julia Lawall
1 sibling, 1 reply; 13+ messages in thread
From: Greg KH @ 2017-09-11 16:57 UTC (permalink / raw)
To: Shreeya Patel; +Cc: outreachy-kernel
On Mon, Sep 11, 2017 at 09:48:15PM +0530, Shreeya Patel wrote:
>
Please turn html email off in your email client, the kernel mailing
lists will reject it.
> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
> This patch is sent corrupted (leading space is gone), and git doesn't
> like it. Can you please fix it up and resend?
>
> Hello sir,
>
> If i am not removing the leading spaces then checkpatch is showing me errors
> for trailing whitespace.
That is correct.
> But there are no leading spaces present in the file which i have modified.
> When i convert my changes into the patch, the whitespace errors are shown up.
> Can you please help me with this to make me understand why this is happening?
The problem isn't in the original file, but in the patch format itself
that you sent. What ever tool you used, stripped off the leading space,
making it impossible to apply. So please fix that.
Try emailing the patch to yourself and applying it to ensure you are
doing this correctly.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 16:57 ` Greg KH
@ 2017-09-11 17:00 ` Shreeya Patel
0 siblings, 0 replies; 13+ messages in thread
From: Shreeya Patel @ 2017-09-11 17:00 UTC (permalink / raw)
To: Greg KH; +Cc: outreachy-kernel
On Monday 11 September 2017 10:27 PM, Greg KH wrote:
> On Mon, Sep 11, 2017 at 09:48:15PM +0530, Shreeya Patel wrote:
> Please turn html email off in your email client, the kernel mailing
> lists will reject it.
>
>> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
>
>> This patch is sent corrupted (leading space is gone), and git doesn't
>> like it. Can you please fix it up and resend?
>>
>> Hello sir,
>>
>> If i am not removing the leading spaces then checkpatch is showing me errors
>> for trailing whitespace.
> That is correct.
>
>> But there are no leading spaces present in the file which i have modified.
>> When i convert my changes into the patch, the whitespace errors are shown up.
>> Can you please help me with this to make me understand why this is happening?
> The problem isn't in the original file, but in the patch format itself
> that you sent. What ever tool you used, stripped off the leading space,
> making it impossible to apply. So please fix that.
>
> Try emailing the patch to yourself and applying it to ensure you are
> doing this correctly.
>
> thanks,
>
> greg k-h
ok, i will do that.
Thank you.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 16:18 ` Shreeya Patel
2017-09-11 16:57 ` Greg KH
@ 2017-09-11 17:01 ` Julia Lawall
2017-09-11 17:04 ` Shreeya Patel
1 sibling, 1 reply; 13+ messages in thread
From: Julia Lawall @ 2017-09-11 17:01 UTC (permalink / raw)
To: Shreeya Patel; +Cc: Greg KH, outreachy-kernel
On Mon, 11 Sep 2017, Shreeya Patel wrote:
>
>
>
> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
>
> On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
>
> Logging messages that show some type of 'out of memory' error
> are generally unnecessary as there is a generic message and
> a stack dump done by the memory subsystem.
>
> These messages generally increase kernel size without much
> added value.
>
> Problem found by checkpatch.
>
> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
>
> Changes in v4:
> -Include v2 and v3 in the patch.
>
> Changes in v3:
> -Make the commit message more appropriate.
>
> changes in v2:
> -Original patch was not sent to the mailing list.
> Also, add a tab before the return statement.
>
> drivers/staging/rtlwifi/rc.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
> index 65de0c7..779a5c6 100644
> --- a/drivers/staging/rtlwifi/rc.c
> +++ b/drivers/staging/rtlwifi/rc.c
> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
> struct rtl_rate_priv *rate_priv;
>
> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
> - if (!rate_priv) {
> - pr_err("Unable to allocate private rc structure\n");
> + if (!rate_priv)
> return NULL;
> - }
>
> rtlpriv->rate_priv = rate_priv;
>
> This patch is sent corrupted (leading space is gone), and git doesn't
> like it. Can you please fix it up and resend?
>
> thanks,
>
> greg k-h
>
>
> Hello sir,
>
> If i am not removing the leading spaces then checkpatch is showing me errors
> for trailing whitespace.
Maybe you are not using checkpatch properly? What is your command line?
julia
> But there are no leading spaces present in the file which i have modified.
> When i convert my changes into the patch, the whitespace errors are shown
> up.
> Can you please help me with this to make me understand why this is
> happening?
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/6e7b1bf6-89b9-a40e-4a67-
> af79312b12a7%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 17:01 ` [Outreachy kernel] " Julia Lawall
@ 2017-09-11 17:04 ` Shreeya Patel
2017-09-11 17:11 ` Julia Lawall
0 siblings, 1 reply; 13+ messages in thread
From: Shreeya Patel @ 2017-09-11 17:04 UTC (permalink / raw)
To: Julia Lawall; +Cc: Greg KH, outreachy-kernel
[-- Attachment #1: Type: text/plain, Size: 2793 bytes --]
On Monday 11 September 2017 10:31 PM, Julia Lawall wrote:
>
> On Mon, 11 Sep 2017, Shreeya Patel wrote:
>
>>
>>
>> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
>>
>> On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
>>
>> Logging messages that show some type of 'out of memory' error
>> are generally unnecessary as there is a generic message and
>> a stack dump done by the memory subsystem.
>>
>> These messages generally increase kernel size without much
>> added value.
>>
>> Problem found by checkpatch.
>>
>> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
>> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
>> ---
>>
>> Changes in v4:
>> -Include v2 and v3 in the patch.
>>
>> Changes in v3:
>> -Make the commit message more appropriate.
>>
>> changes in v2:
>> -Original patch was not sent to the mailing list.
>> Also, add a tab before the return statement.
>>
>> drivers/staging/rtlwifi/rc.c | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
>> index 65de0c7..779a5c6 100644
>> --- a/drivers/staging/rtlwifi/rc.c
>> +++ b/drivers/staging/rtlwifi/rc.c
>> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
>> struct rtl_rate_priv *rate_priv;
>>
>> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
>> - if (!rate_priv) {
>> - pr_err("Unable to allocate private rc structure\n");
>> + if (!rate_priv)
>> return NULL;
>> - }
>>
>> rtlpriv->rate_priv = rate_priv;
>>
>> This patch is sent corrupted (leading space is gone), and git doesn't
>> like it. Can you please fix it up and resend?
>>
>> thanks,
>>
>> greg k-h
>>
>>
>> Hello sir,
>>
>> If i am not removing the leading spaces then checkpatch is showing me errors
>> for trailing whitespace.
> Maybe you are not using checkpatch properly? What is your command line?
>
> julia
perl scripts/checkpatch.pl -f
/tmp/0001-Staging-rtlwifi-Remove-unnecessary-out-of-memory-mes.patch
>
>> But there are no leading spaces present in the file which i have modified.
>> When i convert my changes into the patch, the whitespace errors are shown
>> up.
>> Can you please help me with this to make me understand why this is
>> happening?
>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "outreachy-kernel" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to outreachy-kernel+unsubscribe@googlegroups.com.
>> To post to this group, send email to outreachy-kernel@googlegroups.com.
>> To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/6e7b1bf6-89b9-a40e-4a67-
>> af79312b12a7%40gmail.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
>>
[-- Attachment #2: Type: text/html, Size: 3765 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 17:04 ` Shreeya Patel
@ 2017-09-11 17:11 ` Julia Lawall
2017-09-11 17:14 ` Shreeya Patel
2017-09-11 17:19 ` Shreeya Patel
0 siblings, 2 replies; 13+ messages in thread
From: Julia Lawall @ 2017-09-11 17:11 UTC (permalink / raw)
To: Shreeya Patel; +Cc: Greg KH, outreachy-kernel
On Mon, 11 Sep 2017, Shreeya Patel wrote:
>
>
>
> On Monday 11 September 2017 10:31 PM, Julia Lawall wrote:
>
>
> On Mon, 11 Sep 2017, Shreeya Patel wrote:
>
>
>
> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
>
> On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
>
> Logging messages that show some type of 'out of memory' error
> are generally unnecessary as there is a generic message and
> a stack dump done by the memory subsystem.
>
> These messages generally increase kernel size without much
> added value.
>
> Problem found by checkpatch.
>
> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
> ---
>
> Changes in v4:
> -Include v2 and v3 in the patch.
>
> Changes in v3:
> -Make the commit message more appropriate.
>
> changes in v2:
> -Original patch was not sent to the mailing list.
> Also, add a tab before the return statement.
>
> drivers/staging/rtlwifi/rc.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
> index 65de0c7..779a5c6 100644
> --- a/drivers/staging/rtlwifi/rc.c
> +++ b/drivers/staging/rtlwifi/rc.c
> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
> struct rtl_rate_priv *rate_priv;
>
> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
> - if (!rate_priv) {
> - pr_err("Unable to allocate private rc structure\n");
> + if (!rate_priv)
> return NULL;
> - }
>
> rtlpriv->rate_priv = rate_priv;
>
> This patch is sent corrupted (leading space is gone), and git doesn't
> like it. Can you please fix it up and resend?
>
> thanks,
>
> greg k-h
>
>
> Hello sir,
>
> If i am not removing the leading spaces then checkpatch is showing me errors
> for trailing whitespace.
>
> Maybe you are not using checkpatch properly? What is your command line?
>
> julia
>
> perl scripts/checkpatch.pl -f
> /tmp/0001-Staging-rtlwifi-Remove-unnecessary-out-of-memory-mes.patch
-f is for files, eg foo.c, not for patches. You should be able to just
give the patch file name directly.
Also, could you change your mailer to make something visible when you
quote other people? Otherwise, it's hard to sort out what is new and what
is quoted from others.
thanks,
julia
>
>
>
> But there are no leading spaces present in the file which i have modified.
> When i convert my changes into the patch, the whitespace errors are shown
> up.
> Can you please help me with this to make me understand why this is
> happening?
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visithttps://groups.google.com/d/msgid/ou
> treachy-kernel/6e7b1bf6-89b9-a40e-4a67-
> af79312b12a7%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
>
>
>
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 17:11 ` Julia Lawall
@ 2017-09-11 17:14 ` Shreeya Patel
2017-09-11 17:19 ` Shreeya Patel
1 sibling, 0 replies; 13+ messages in thread
From: Shreeya Patel @ 2017-09-11 17:14 UTC (permalink / raw)
To: Julia Lawall; +Cc: Greg KH, outreachy-kernel
On Monday 11 September 2017 10:41 PM, Julia Lawall wrote:
>
> On Mon, 11 Sep 2017, Shreeya Patel wrote:
>
>>
>>
>> On Monday 11 September 2017 10:31 PM, Julia Lawall wrote:
>>
>>
>> On Mon, 11 Sep 2017, Shreeya Patel wrote:
>>
>>
>>
>> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
>>
>> On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
>>
>> Logging messages that show some type of 'out of memory' error
>> are generally unnecessary as there is a generic message and
>> a stack dump done by the memory subsystem.
>>
>> These messages generally increase kernel size without much
>> added value.
>>
>> Problem found by checkpatch.
>>
>> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
>> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
>> ---
>>
>> Changes in v4:
>> -Include v2 and v3 in the patch.
>>
>> Changes in v3:
>> -Make the commit message more appropriate.
>>
>> changes in v2:
>> -Original patch was not sent to the mailing list.
>> Also, add a tab before the return statement.
>>
>> drivers/staging/rtlwifi/rc.c | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
>> index 65de0c7..779a5c6 100644
>> --- a/drivers/staging/rtlwifi/rc.c
>> +++ b/drivers/staging/rtlwifi/rc.c
>> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
>> struct rtl_rate_priv *rate_priv;
>>
>> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
>> - if (!rate_priv) {
>> - pr_err("Unable to allocate private rc structure\n");
>> + if (!rate_priv)
>> return NULL;
>> - }
>>
>> rtlpriv->rate_priv = rate_priv;
>>
>> This patch is sent corrupted (leading space is gone), and git doesn't
>> like it. Can you please fix it up and resend?
>>
>> thanks,
>>
>> greg k-h
>>
>>
>> Hello sir,
>>
>> If i am not removing the leading spaces then checkpatch is showing me errors
>> for trailing whitespace.
>>
>> Maybe you are not using checkpatch properly? What is your command line?
>>
>> julia
>>
>> perl scripts/checkpatch.pl -f
>> /tmp/0001-Staging-rtlwifi-Remove-unnecessary-out-of-memory-mes.patch
> -f is for files, eg foo.c, not for patches. You should be able to just
> give the patch file name directly.
>
> Also, could you change your mailer to make something visible when you
> quote other people? Otherwise, it's hard to sort out what is new and what
> is quoted from others.
Ok. I will do that soon.
Thanks.
>
> thanks,
> julia
>>
>>
>> But there are no leading spaces present in the file which i have modified.
>> When i convert my changes into the patch, the whitespace errors are shown
>> up.
>> Can you please help me with this to make me understand why this is
>> happening?
>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "outreachy-kernel" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to outreachy-kernel+unsubscribe@googlegroups.com.
>> To post to this group, send email to outreachy-kernel@googlegroups.com.
>> To view this discussion on the web visithttps://groups.google.com/d/msgid/ou
>> treachy-kernel/6e7b1bf6-89b9-a40e-4a67-
>> af79312b12a7%40gmail.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
>>
>>
>>
>>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 17:11 ` Julia Lawall
2017-09-11 17:14 ` Shreeya Patel
@ 2017-09-11 17:19 ` Shreeya Patel
2017-09-11 17:25 ` Julia Lawall
1 sibling, 1 reply; 13+ messages in thread
From: Shreeya Patel @ 2017-09-11 17:19 UTC (permalink / raw)
To: Julia Lawall; +Cc: Greg KH, outreachy-kernel
On Monday 11 September 2017 10:41 PM, Julia Lawall wrote:
>
> On Mon, 11 Sep 2017, Shreeya Patel wrote:
>
>>
>>
>> On Monday 11 September 2017 10:31 PM, Julia Lawall wrote:
>>
>>
>> On Mon, 11 Sep 2017, Shreeya Patel wrote:
>>
>>
>>
>> On Monday 11 September 2017 10:02 PM, Greg KH wrote:
>>
>> On Sun, Sep 10, 2017 at 03:10:05PM +0530, Shreeya Patel wrote:
>>
>> Logging messages that show some type of 'out of memory' error
>> are generally unnecessary as there is a generic message and
>> a stack dump done by the memory subsystem.
>>
>> These messages generally increase kernel size without much
>> added value.
>>
>> Problem found by checkpatch.
>>
>> Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
>> Acked-by: Julia Lawall <julia.lawall@lip6.fr>
>> ---
>>
>> Changes in v4:
>> -Include v2 and v3 in the patch.
>>
>> Changes in v3:
>> -Make the commit message more appropriate.
>>
>> changes in v2:
>> -Original patch was not sent to the mailing list.
>> Also, add a tab before the return statement.
>>
>> drivers/staging/rtlwifi/rc.c | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/drivers/staging/rtlwifi/rc.c b/drivers/staging/rtlwifi/rc.c
>> index 65de0c7..779a5c6 100644
>> --- a/drivers/staging/rtlwifi/rc.c
>> +++ b/drivers/staging/rtlwifi/rc.c
>> @@ -281,10 +281,8 @@ static void *rtl_rate_alloc_sta(void *ppriv,
>> struct rtl_rate_priv *rate_priv;
>>
>> rate_priv = kzalloc(sizeof(*rate_priv), gfp);
>> - if (!rate_priv) {
>> - pr_err("Unable to allocate private rc structure\n");
>> + if (!rate_priv)
>> return NULL;
>> - }
>>
>> rtlpriv->rate_priv = rate_priv;
>>
>> This patch is sent corrupted (leading space is gone), and git doesn't
>> like it. Can you please fix it up and resend?
>>
>> thanks,
>>
>> greg k-h
>>
>>
>> Hello sir,
>>
>> If i am not removing the leading spaces then checkpatch is showing me errors
>> for trailing whitespace.
>>
>> Maybe you are not using checkpatch properly? What is your command line?
>>
>> julia
>>
>> perl scripts/checkpatch.pl -f
>> /tmp/0001-Staging-rtlwifi-Remove-unnecessary-out-of-memory-mes.patch
> -f is for files, eg foo.c, not for patches. You should be able to just
> give the patch file name directly.
When I give the file name, it is showing me 'permission denied'.
>
> Also, could you change your mailer to make something visible when you
> quote other people? Otherwise, it's hard to sort out what is new and what
> is quoted from others.
>
> thanks,
> julia
>
>>
>>
>> But there are no leading spaces present in the file which i have modified.
>> When i convert my changes into the patch, the whitespace errors are shown
>> up.
>> Can you please help me with this to make me understand why this is
>> happening?
>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "outreachy-kernel" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to outreachy-kernel+unsubscribe@googlegroups.com.
>> To post to this group, send email to outreachy-kernel@googlegroups.com.
>> To view this discussion on the web visithttps://groups.google.com/d/msgid/ou
>> treachy-kernel/6e7b1bf6-89b9-a40e-4a67-
>> af79312b12a7%40gmail.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
>>
>>
>>
>>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 17:19 ` Shreeya Patel
@ 2017-09-11 17:25 ` Julia Lawall
2017-09-11 17:29 ` Shreeya Patel
0 siblings, 1 reply; 13+ messages in thread
From: Julia Lawall @ 2017-09-11 17:25 UTC (permalink / raw)
To: Shreeya Patel; +Cc: Greg KH, outreachy-kernel
> > > perl scripts/checkpatch.pl -f
> > > /tmp/0001-Staging-rtlwifi-Remove-unnecessary-out-of-memory-mes.patch
> > -f is for files, eg foo.c, not for patches. You should be able to just
> > give the patch file name directly.
>
> When I give the file name, it is showing me 'permission denied'.
What is the exact command line?
scripts/checkpatch.pl patchfile
should work.
julia
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Outreachy kernel] Re: [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message.
2017-09-11 17:25 ` Julia Lawall
@ 2017-09-11 17:29 ` Shreeya Patel
0 siblings, 0 replies; 13+ messages in thread
From: Shreeya Patel @ 2017-09-11 17:29 UTC (permalink / raw)
To: Julia Lawall; +Cc: Greg KH, outreachy-kernel
On Monday 11 September 2017 10:55 PM, Julia Lawall wrote:
>>>> perl scripts/checkpatch.pl -f
>>>> /tmp/0001-Staging-rtlwifi-Remove-unnecessary-out-of-memory-mes.patch
>>> -f is for files, eg foo.c, not for patches. You should be able to just
>>> give the patch file name directly.
>> When I give the file name, it is showing me 'permission denied'.
> What is the exact command line?
>
> scripts/checkpatch.pl patchfile
>
> should work.
>
> julia
Oh, yes.
Sorry, my mistake, it worked with 0 errors now :)
Thanks.
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2017-09-11 17:29 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-10 9:40 [PATCH v4] Staging: rtlwifi: Remove unnecessary 'out of memory' message Shreeya Patel
2017-09-10 10:13 ` [Outreachy kernel] " Julia Lawall
2017-09-11 16:32 ` Greg KH
2017-09-11 16:18 ` Shreeya Patel
2017-09-11 16:57 ` Greg KH
2017-09-11 17:00 ` Shreeya Patel
2017-09-11 17:01 ` [Outreachy kernel] " Julia Lawall
2017-09-11 17:04 ` Shreeya Patel
2017-09-11 17:11 ` Julia Lawall
2017-09-11 17:14 ` Shreeya Patel
2017-09-11 17:19 ` Shreeya Patel
2017-09-11 17:25 ` Julia Lawall
2017-09-11 17:29 ` Shreeya Patel
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.