Commit-ID: 7729bebc619307a0233c86f8585a4bf3eadc7ce4
Gitweb: https://git.kernel.org/tip/7729bebc619307a0233c86f8585a4bf3eadc7ce4
Author: Valentin Ilie <valentin.i...@gmail.com>
AuthorDate: Fri, 5 Jan 2018 23:12:59 +
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate: Sat
Commit-ID: 7729bebc619307a0233c86f8585a4bf3eadc7ce4
Gitweb: https://git.kernel.org/tip/7729bebc619307a0233c86f8585a4bf3eadc7ce4
Author: Valentin Ilie
AuthorDate: Fri, 5 Jan 2018 23:12:59 +
Committer: Ingo Molnar
CommitDate: Sat, 6 Jan 2018 11:48:34 +0100
ia64, sched/cputime: Fix
Remove extra parentheses.
Signed-off-by: Valentin Ilie <valentin.i...@gmail.com>
---
arch/ia64/kernel/time.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/ia64/kernel/time.c b/arch/ia64/kernel/time.c
index c6ecb97..9025699 100644
--- a/arch/ia64/kernel/time.c
Remove extra parentheses.
Signed-off-by: Valentin Ilie
---
arch/ia64/kernel/time.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/ia64/kernel/time.c b/arch/ia64/kernel/time.c
index c6ecb97..9025699 100644
--- a/arch/ia64/kernel/time.c
+++ b/arch/ia64/kernel/time.c
On 16 January 2015 at 17:30, Paul Bolle wrote:
> Before you resend: what happens to the resources of a userspace program
> when its main() returns?
On top of that, I should have used free, not kfree. I will resend v2 ..
Sorry for the mess
--
Valentin
--
To unsubscribe from this list: send the
Call kfree on list2 after using it.
Signed-off-by: Valentin Ilie
---
tools/vm/page_owner_sort.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tools/vm/page_owner_sort.c b/tools/vm/page_owner_sort.c
index 77147b4..92292d5 100644
--- a/tools/vm/page_owner_sort.c
+++ b/tools/vm
Call kfree on list2 after using it.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
tools/vm/page_owner_sort.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tools/vm/page_owner_sort.c b/tools/vm/page_owner_sort.c
index 77147b4..92292d5 100644
--- a/tools/vm/page_owner_sort.c
On 16 January 2015 at 17:30, Paul Bolle pebo...@tiscali.nl wrote:
Before you resend: what happens to the resources of a userspace program
when its main() returns?
On top of that, I should have used free, not kfree. I will resend v2 ..
Sorry for the mess
--
Valentin
--
To unsubscribe from
For the logging process, we use either a file or stdout.
In the case of a file, we need to fix that by calling fclose at the right time.
In the case we end up calling fclose to stdout, we will not impact the program
because we do it at its very end.
Signed-off-by: Valentin Ilie
For the logging process, we use either a file or stdout.
In the case of a file, we need to fix that by calling fclose at the right time.
In the case we end up calling fclose to stdout, we will not impact the program
because we do it at its very end.
Signed-off-by: Valentin Ilie valentin.i
>
> Gabriel.
>
> Best Regards.
>
>
>
> On 05/06/2014 11:29 AM, Valentin Ilie wrote:
>>
>> On 22 April 2014 16:15, Valentin Ilie wrote:
>>>
>>> When it fails to allocate div, gate should be free'd before return
>>>
>>> Signed-of
?
Gabriel.
Best Regards.
On 05/06/2014 11:29 AM, Valentin Ilie wrote:
On 22 April 2014 16:15, Valentin Ilie valentin.i...@gmail.com wrote:
When it fails to allocate div, gate should be free'd before return
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/clk/st
On 22 April 2014 16:15, Valentin Ilie wrote:
> When it fails to allocate div, gate should be free'd before return
>
> Signed-off-by: Valentin Ilie
> ---
> drivers/clk/st/clkgen-pll.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drive
On 22 April 2014 16:15, Valentin Ilie valentin.i...@gmail.com wrote:
When it fails to allocate div, gate should be free'd before return
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/clk/st/clkgen-pll.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
When it fails to allocate div, gate should be free'd before return
Signed-off-by: Valentin Ilie
---
drivers/clk/st/clkgen-pll.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/st/clkgen-pll.c b/drivers/clk/st/clkgen-pll.c
index bca0a0b..a886702 100644
When it fails to allocate div, gate should be free'd before return
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/clk/st/clkgen-pll.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/st/clkgen-pll.c b/drivers/clk/st/clkgen-pll.c
index bca0a0b
When it fails to allocate a slot, edesc should be free'd before return;
Signed-off-by: Valentin Ilie
---
drivers/dma/edma.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c
index 098a8da..79ebac6 100644
--- a/drivers/dma/edma.c
+++ b/drivers/dma/edma.c
When the if condition is true, "fixed" and "gate" should be free'd before
return.
Signed-off-by: Valentin Ilie
---
drivers/clk/sunxi/clk-sunxi.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-s
When the if condition is true, fixed and gate should be free'd before
return.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/clk/sunxi/clk-sunxi.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk
When it fails to allocate a slot, edesc should be free'd before return;
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/dma/edma.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c
index 098a8da..79ebac6 100644
--- a/drivers/dma/edma.c
On 2 September 2013 18:23, Valentin Ilie wrote:
> Convert a compound if-else blob to a switch statement.
>
> Signed-off-by: Valentin Ilie
> ---
> arch/arm/mach-iop32x/iq31244.c | 29 +++--
> 1 file changed, 11 insertions(+), 18 deletions(-)
>
> di
On 2 September 2013 18:23, Valentin Ilie valentin.i...@gmail.com wrote:
Convert a compound if-else blob to a switch statement.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
arch/arm/mach-iop32x/iq31244.c | 29 +++--
1 file changed, 11 insertions(+), 18
On 21 September 2013 12:30, Valentin Ilie wrote:
> When trb_hw is NULL, trb should be free'd before return.
>
> Signed-off-by: Valentin Ilie
> ---
> drivers/usb/gadget/mv_u3d_core.c |2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/usb/gadget/mv_u3d
On 21 September 2013 12:30, Valentin Ilie valentin.i...@gmail.com wrote:
When trb_hw is NULL, trb should be free'd before return.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/usb/gadget/mv_u3d_core.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb
When trb_hw is NULL, trb should be free'd before return.
Signed-off-by: Valentin Ilie
---
drivers/usb/gadget/mv_u3d_core.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/gadget/mv_u3d_core.c b/drivers/usb/gadget/mv_u3d_core.c
index 561b30e..4d31177 100644
--- a/drivers/usb
When trb_hw is NULL, trb should be free'd before return.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/usb/gadget/mv_u3d_core.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/gadget/mv_u3d_core.c b/drivers/usb/gadget/mv_u3d_core.c
index 561b30e..4d31177
Hello,
I am trying to solve a NULL pointer deference in
drivers/power/pm2301_charger.c (and to submit a trivial patch) but I
am not sure how to do it.
Line 958 - 961
pm2 = (struct pm2xxx_charger*)i2c_get_clientdata(pm2xxx_i2c_client);
if (!pm2) {
dev_err(pm2->dev, "no
Hello,
I am trying to solve a NULL pointer deference in
drivers/power/pm2301_charger.c (and to submit a trivial patch) but I
am not sure how to do it.
Line 958 - 961
pm2 = (struct pm2xxx_charger*)i2c_get_clientdata(pm2xxx_i2c_client);
if (!pm2) {
dev_err(pm2-dev, no
SNDRV_PCM_FORMAT_S8 isn't supposed to fall through to SNDRV_PCM_FORMAT_S16_LE
Signed-off-by: Valentin Ilie
---
sound/soc/blackfin/bf6xx-i2s.c |1 +
1 file changed, 1 insertion(+)
diff --git a/sound/soc/blackfin/bf6xx-i2s.c b/sound/soc/blackfin/bf6xx-i2s.c
index c02405c..5810a06 100644
SNDRV_PCM_FORMAT_S8 isn't supposed to fall through to SNDRV_PCM_FORMAT_S16_LE
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
sound/soc/blackfin/bf6xx-i2s.c |1 +
1 file changed, 1 insertion(+)
diff --git a/sound/soc/blackfin/bf6xx-i2s.c b/sound/soc/blackfin/bf6xx-i2s.c
index
Convert a compound if-else blob to a switch statement.
Signed-off-by: Valentin Ilie
---
arch/arm/mach-iop32x/iq31244.c | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/arch/arm/mach-iop32x/iq31244.c b/arch/arm/mach-iop32x/iq31244.c
index
Convert a compound if-else blob to a switch statement.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
arch/arm/mach-iop32x/iq31244.c | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/arch/arm/mach-iop32x/iq31244.c b/arch/arm/mach-iop32x
Fix checkpatch warnings.
Replace __attribute__((__packed__)) with __packed.
Replace spaces with tabs.
Signed-off-by: Valentin Ilie
---
drivers/nfc/mei_phy.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/nfc/mei_phy.c b/drivers/nfc/mei_phy.c
index 1201bdb
Fix checkpatch warnings.
Replace __attribute__((__packed__)) with __packed.
Replace spaces with tabs.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/nfc/mei_phy.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/nfc/mei_phy.c b/drivers/nfc
Remove trailing whitespaces
Replace seq_puts with seq_printf when there are no extra parameters.
Fix indentation problems
Signed-off-by: Valentin Ilie
---
crypto/proc.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/crypto/proc.c b/crypto/proc.c
index
Remove trailing whitespaces
Replace seq_puts with seq_printf when there are no extra parameters.
Fix indentation problems
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
crypto/proc.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/crypto/proc.c b
Fix checkpatch warning about seq_printf.
Signed-off-by: Valentin Ilie
---
drivers/staging/fwserial/fwserial.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/fwserial/fwserial.c
b/drivers/staging/fwserial/fwserial.c
index 5c64e3a..e5818a1 100644
Fix checkpatch warning about seq_printf.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/staging/fwserial/fwserial.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/fwserial/fwserial.c
b/drivers/staging/fwserial/fwserial.c
index
Replace disallocated with deallocated
Signed-off-by: Valentin Ilie
---
drivers/staging/speakup/selection.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/selection.c
b/drivers/staging/speakup/selection.c
index 775af26..f0fb003 100644
Replace disallocated with deallocated
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/staging/speakup/selection.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/selection.c
b/drivers/staging/speakup/selection.c
index 775af26
Remove unused variable 'c'
Signed-off-by: Valentin Ilie
---
arch/x86/kernel/smpboot.c |1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c
index a6ceaed..7bc998a 100644
--- a/arch/x86/kernel/smpboot.c
+++ b/arch/x86/kernel/smpboot.c
Remove unused variable 'c'
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
arch/x86/kernel/smpboot.c |1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c
index a6ceaed..7bc998a 100644
--- a/arch/x86/kernel/smpboot.c
+++ b/arch/x86
Replace printk(KERN_ERR with pr_err
Add space before {
Removed OOM messages
Signed-off-by: Valentin Ilie
---
net/can/af_can.c | 21 -
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/net/can/af_can.c b/net/can/af_can.c
index c48e522..8bacf28 100644
--- a/net
Replace printk(KERN_ERR with pr_err
Add space before {
Signed-off-by: Valentin Ilie
---
net/can/af_can.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/net/can/af_can.c b/net/can/af_can.c
index c48e522..a14c0aa 100644
--- a/net/can/af_can.c
+++ b/net/can
Replace printk(KERN_ERR with pr_err
Add space before {
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
net/can/af_can.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/net/can/af_can.c b/net/can/af_can.c
index c48e522..a14c0aa 100644
--- a/net/can
Replace printk(KERN_ERR with pr_err
Add space before {
Removed OOM messages
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
net/can/af_can.c | 21 -
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/net/can/af_can.c b/net/can/af_can.c
index c48e522
Fixed the following checkpatch warnings:
- ERROR: space required after that ','.
- WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
Signed-off-by: Valentin Ilie
---
drivers/amba/bus.c | 21 +
1 file changed, 9 insertions(+), 12 deletions
Fixed the following checkpatch warnings:
- ERROR: space required after that ','.
- WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
drivers/amba/bus.c | 21 +
1 file changed, 9 insertions
Fixed checkpatch warnings.
Replaced printk(KERN_WARNING with pr_warn
Signed-off-by: Valentin Ilie
---
kernel/panic.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/kernel/panic.c b/kernel/panic.c
index e1b2822..8f1709f 100644
--- a/kernel/panic.c
+++ b
Fixed checkpatch warnings.
Replaced printk(KERN_WARNING with pr_warn
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
kernel/panic.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/kernel/panic.c b/kernel/panic.c
index e1b2822..8f1709f 100644
From: valentin
Fixed some checkpatch warnings.
Signed-off-by: Valentin Ilie
---
kernel/workqueue.c | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 692d976..c4d57c8 100644
--- a/kernel/workqueue.c
From: valentin valentin.i...@gmail.com
Fixed some checkpatch warnings.
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
kernel/workqueue.c | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index
Replaced printk(KERN_WARNING.. with pr_warn..
Signed-off-by: Valentin Ilie
---
kernel/panic.c | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/kernel/panic.c b/kernel/panic.c
index d2a5f4e..be76bc4 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
Fixed coding style issues. (checkpatch warnings)
Signed-off-by: Valentin Ilie
---
kernel/dma.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/kernel/dma.c b/kernel/dma.c
index 6c6262f..082cd74 100644
--- a/kernel/dma.c
+++ b/kernel/dma.c
@@ -38,6 +38,7
Fixed coding style issues. (checkpatch warnings)
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
kernel/dma.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/kernel/dma.c b/kernel/dma.c
index 6c6262f..082cd74 100644
--- a/kernel/dma.c
+++ b/kernel
Replaced printk(KERN_WARNING.. with pr_warn..
Signed-off-by: Valentin Ilie valentin.i...@gmail.com
---
kernel/panic.c | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/kernel/panic.c b/kernel/panic.c
index d2a5f4e..be76bc4 100644
--- a/kernel/panic.c
+++ b
56 matches
Mail list logo