: Greg Kroah-Hartman gre...@linuxfoundation.org
Cc: Brian Swetland swetl...@google.com
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Snild Dolkow snild.dol...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/staging/android
On 16:13 Mon 15 Apr , Dan Carpenter wrote:
On Mon, Apr 15, 2013 at 03:38:08PM +0200, Dolkow, Snild wrote:
Where is lowmem_shrink called from? I only see shrink called from the
bcache sysfs handler __bch_cache_set(). The return value isn't checked
there.
Up to now this function
On 08:19 Tue 16 Apr , Dan Carpenter wrote:
On Mon, Apr 15, 2013 at 04:11:18PM -0700, David Rientjes wrote:
On Mon, 15 Apr 2013, Greg Kroah-Hartman wrote:
The positive numbers are used to return information on the remaining
cache size (again, see the comment I pasted above). We
On 08:44 Thu 04 Apr , Masami Hiramatsu wrote:
(2013/04/03 17:28), oskar.and...@sonymobile.com wrote:
+/* it can take some time ( 100ms ) to initialise the
+ * blacklist so we delay this until we actually need it
+ */
+static void init_kprobe_blacklist(void)
+{
+ int i;
+
directory
Oskar Andero (2):
kprobes: split blacklist into common and arch
kprobes: replace printk with pr_-functions
Toby Collett (1):
kprobes: delay blacklist symbol lookup until we actually need it
arch/arc/kernel/kprobes.c | 3 +
arch/arm/kernel/kprobes.c
On 08:17 Thu 04 Apr , Masami Hiramatsu wrote:
(2013/04/03 17:28), oskar.and...@sonymobile.com wrote:
Some blackpoints are only valid for specific architectures. To let each
architecture specify its own blackpoints the list has been split in two
lists: common and arch. The common list is
-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Toby Collett toby.coll...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
kernel/kprobes.c | 98 ++--
1 file changed, 60 insertions(+), 38 deletions
Instead of using printk use pr_info/pr_err/pr_warn. This was
detected by the checkpatch.pl script.
Cc: Masami Hiramatsu masami.hiramatsu...@hitachi.com
Cc: David S. Miller da...@davemloft.net
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
Acked-by: Masami Hiramatsu masami.hiramatsu
included linux-arch in Cc for this post.
-Oskar
Björn Davidsson (1):
kprobes: move x86-specific blacklist symbols to arch directory
Oskar Andero (2):
kprobes: split blacklist into common and arch
kprobes: replace printk with pr_-functions
Toby Collett (1):
kprobes: delay blacklist symbol
masami.hiramatsu...@hitachi.com
Cc: David S. Miller da...@davemloft.net
Cc: linux-a...@vger.kernel.org
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Björn Davidsson bjorn.davids...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
arch/x86/kernel/kprobes
masami.hiramatsu...@hitachi.com
Cc: David S. Miller da...@davemloft.net
Cc: linux-a...@vger.kernel.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
arch/arc/kernel/kprobes.c | 3 ++
arch/arm/kernel/kprobes.c | 2 +
arch/avr32/kernel/kprobes.c| 3 ++
arch/ia64/kernel/kprobes.c
...@gmail.com
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Aleksej Makarov aleksej.maka...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/input/input.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff
On 18:33 Thu 04 Apr , Dmitry Torokhov wrote:
Hi Oskar,
On Thu, Mar 07, 2013 at 03:01:22PM +0100, oskar.and...@sonymobile.com wrote:
From: Aleksej Makarov aleksej.maka...@sonymobile.com
When waking up the platform by pressing a specific key, sending a
release on that key makes it
Thanks for your input guys!
On 04:16 Fri 05 Apr , Masami Hiramatsu wrote:
(2013/04/05 9:56), Joonsoo Kim wrote:
Hello, Oskar.
On Thu, Apr 04, 2013 at 02:51:26PM +0200, Oskar Andero wrote:
From: Toby Collett toby.coll...@sonymobile.com
The symbol lookup can take a long time
Instead of using printk use pr_info/pr_err/pr_warn. This was
detected by the checkpatch.pl script.
Cc: Masami Hiramatsu masami.hiramatsu...@hitachi.com
Cc: David S. Miller da...@davemloft.net
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
Acked-by: Masami Hiramatsu masami.hiramatsu
arch_kprobes_blacksyms.* as __weak symbols.
-Oskar
Björn Davidsson (1):
kprobes: move x86-specific blacklist symbols to arch directory
Oskar Andero (2):
kprobes: split blacklist into common and arch
kprobes: replace printk with pr_-functions
Toby Collett (1):
kprobes: delay blacklist
masami.hiramatsu...@hitachi.com
Cc: David S. Miller da...@davemloft.net
Cc: linux-a...@vger.kernel.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
kernel/kprobes.c | 88 +---
1 file changed, 59 insertions(+), 29 deletions(-)
diff --git a/kernel
masami.hiramatsu...@hitachi.com
Cc: David S. Miller da...@davemloft.net
Cc: linux-a...@vger.kernel.org
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Björn Davidsson bjorn.davids...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
Acked-by: Masami Hiramatsu
-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Toby Collett toby.coll...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
kernel/kprobes.c | 100 ++-
1 file changed, 62 insertions(+), 38 deletions
pointer. This
patch adds a null pointer check.
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: syunsuke.x.itou syunsuke.x.i...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/scsi/sd.c |7 ++-
1 files changed, 6
Hi James,
On 18:39 Mon 08 Apr , James Bottomley wrote:
On Mon, 2013-04-08 at 13:42 +0200, Oskar Andero wrote:
On 14:36 Thu 07 Mar , oskar.and...@sonymobile.com wrote:
From: syunsuke.x.itou syunsuke.x.i...@sonymobile.com
By repeatadly connecting/disconnecting a USB masstorage
Hi,
On 14:47 Tue 09 Jul , Oskar Andero wrote:
From: Lars Svensson lars1.svens...@sonymobile.com
Some error bits in the status field of R1/R1b response are only set
by the device in response to the command following the failing
command. The status is only read and checked after a r/w
dmitry.torok...@gmail.com
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Aleksej Makarov aleksej.maka...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/input/input.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff
On 09:46 Fri 05 Jul , Oskar Andero wrote:
Hi Dmitry,
On 18:33 Thu 04 Apr , Dmitry Torokhov wrote:
Hi Oskar,
On Thu, Mar 07, 2013 at 03:01:22PM +0100, oskar.and...@sonymobile.com wrote:
From: Aleksej Makarov aleksej.maka...@sonymobile.com
When waking up the platform
Hi Ulf,
On 18:50 Tue 22 Oct , Ulf Hansson wrote:
And this is after the patch has been applied:
KB reclen write rewritereadreread
51200 4 251 990 3280 3244
51200 8 4601545 4460 4463
Hii Dmitry,
On 14:09 Thu 25 Jul , Oskar Andero wrote:
From: Aleksej Makarov aleksej.maka...@sonymobile.com
When waking up the platform by pressing a specific key, sending a
release on that key makes it impossible to react on the event in
user-space. This is fixed by moving
dmitry.torok...@gmail.com
Reviewed-by: Radovan Lekanovic radovan.lekano...@sonymobile.com
Signed-off-by: Aleksej Makarov aleksej.maka...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/input/input.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff
Hi Dmitry,
On 03:14 Tue 26 Nov , Dmitry Torokhov wrote:
Hi Oskar,
On Fri, Nov 22, 2013 at 02:27:04PM +0100, Oskar Andero wrote:
From: Aleksej Makarov aleksej.maka...@sonymobile.com
When waking up the platform by pressing a specific key, sending a
release on that key makes
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron ji...@cam.ac.uk
Cc: Lars-Peter Clausen l...@metafoo.de
Signed-off-by: Oskar Andero oskar.and
Hi,
On 16:07 Wed 01 May , Lars-Peter Clausen wrote:
On 05/01/2013 12:21 AM, Oskar Andero wrote:
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron ji...@cam.ac.uk
Cc: Lars-Peter Clausen l...@metafoo.de
Signed-off-by: Oskar Andero oskar.and
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron ji...@cam.ac.uk
Cc: Lars-Peter Clausen l...@metafoo.de
Signed-off-by: Oskar Andero oskar.and
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron ji...@cam.ac.uk
Cc: Lars-Peter Clausen l...@metafoo.de
Signed-off-by: Oskar Andero oskar.and
On 22:00 Tue 16 Apr , David Rientjes wrote:
On Tue, 16 Apr 2013, Oskar Andero wrote:
The comment in shrinker.h is misleading, not the source code.
do_shrinker_shrink() will fail for anything negative and 0.
The comment is correct. The only acceptable negative return is -1
On 01:05 Thu 16 May , Andrew Morton wrote:
On Mon, 13 May 2013 16:16:33 +0200 Oskar Andero oskar.and...@sonymobile.com
wrote:
In a previous discussion on lkml it was noted that the shrinkers use the
magic value -1 to signal that something went wrong.
This patch-set implements
On 16:49 Wed 15 May , Glauber Costa wrote:
On 05/15/2013 06:47 PM, Oskar Andero wrote:
On 16:18 Wed 15 May , Glauber Costa wrote:
On 05/15/2013 06:10 PM, Oskar Andero wrote:
On 17:03 Tue 14 May , Glauber Costa wrote:
On 05/13/2013 06:16 PM, Oskar Andero wrote:
Hi
Cc: Hugh Dickins hu...@google.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
include/linux/shrinker.h | 7 ---
mm/vmscan.c | 2 +-
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/include/linux
On 13:52 Thu 16 May , Dave Chinner wrote:
On Thu, May 16, 2013 at 10:42:16AM +0200, Oskar Andero wrote:
The shrinkers must return -1 to indicate that it is busy. Instead, treat
any negative value as busy.
Why? The API defines return condition for aborting a scan and gives
a specific
On 08:33 Fri 17 May , Dave Chinner wrote:
On Thu, May 16, 2013 at 02:27:52PM +0200, Oskar Andero wrote:
On 13:52 Thu 16 May , Dave Chinner wrote:
On Thu, May 16, 2013 at 10:42:16AM +0200, Oskar Andero wrote:
The shrinkers must return -1 to indicate that it is busy. Instead, treat
Cc: Hugh Dickins hu...@google.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
mm/vmscan.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/mm/vmscan.c b/mm/vmscan.c
index 6bac41e..fbe6742 100644
--- a/mm/vmscan.c
+++ b/mm/vmscan.c
: Hugh Dickins hu...@google.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
mm/vmscan.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/mm/vmscan.c b/mm/vmscan.c
index 6bac41e..63fec86 100644
--- a/mm/vmscan.c
+++ b/mm/vmscan.c
Hi Ulf,
On 17:01 Wed 16 Oct , Ulf Hansson wrote:
Hi Oskar / Lars,
Sorry for the delayed response!
On 10 October 2013 15:28, Oskar Andero oskar.and...@sonymobile.com wrote:
From: Lars Svensson lars1.svens...@sonymobile.com
Some error bits in the status field of R1/R1b response
or the following data
transfer. In some situations this causes errors passing undetected.
The solution is to read the status and check for these errors after
each r/w operation.
Signed-off-by: Lars Svensson lars1.svens...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
Cc
Shrinkers return hardcoded -1 on error. Use errno.h values instead
to add more meaning to the errors.
Cc: Hugh Dickins hu...@google.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/staging/android/ashmem.c | 2
say? Is this a good idea or does it make no sense at all?
Thanks!
-Oskar
Oskar Andero (2):
mm: vmscan: let any negative return value from shrinker mean error
Clean-up shrinker return values
drivers/staging/android/ashmem.c | 2 +-
drivers/staging/zcache/zcache-main.c | 2 +-
fs/gfs2
...@linuxfoundation.org
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
include/linux/shrinker.h | 5 +++--
mm/vmscan.c | 2 +-
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/include/linux/shrinker.h b/include/linux/shrinker.h
index ac6b8ee..31e9406 100644
--- a/include/linux
On 17:03 Tue 14 May , Glauber Costa wrote:
On 05/13/2013 06:16 PM, Oskar Andero wrote:
Hi,
In a previous discussion on lkml it was noted that the shrinkers use the
magic value -1 to signal that something went wrong.
This patch-set implements the suggestion of instead using
On 16:18 Wed 15 May , Glauber Costa wrote:
On 05/15/2013 06:10 PM, Oskar Andero wrote:
On 17:03 Tue 14 May , Glauber Costa wrote:
On 05/13/2013 06:16 PM, Oskar Andero wrote:
Hi,
In a previous discussion on lkml it was noted that the shrinkers use the
magic value -1 to signal
Hi Dmitry,
On 18:33 Thu 04 Apr , Dmitry Torokhov wrote:
Hi Oskar,
On Thu, Mar 07, 2013 at 03:01:22PM +0100, oskar.and...@sonymobile.com wrote:
From: Aleksej Makarov aleksej.maka...@sonymobile.com
When waking up the platform by pressing a specific key, sending a
release on that key
or the following data
transfer. In some situations this causes errors passing undetected.
The solution is to read the status and check for these errors after
each r/w operation.
Signed-off-by: Lars Svensson lars1.svens...@sonymobile.com
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
Cc
From: Ren Bigcren bigcren@sonymobile.com
The device report an error capacity when read_capacity_16().
Using read_capacity_10() can get the correct capacity.
Signed-off-by: Ren Bigcren bigcren@sonymobile.com
Cc: Matthew Dharm mdharm-...@one-eyed-alien.net
Signed-off-by: Oskar Andero
: Richard Purdie rpur...@rpsys.net
Signed-off-by: Oskar Andero oskar.and...@sonymobile.com
---
drivers/video/backlight/lp855x_bl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/lp855x_bl.c
b/drivers/video/backlight/lp855x_bl.c
index a0e1e02..c0b41f1
On 08:44 Thu 04 Apr , Masami Hiramatsu wrote:
> (2013/04/03 17:28), oskar.and...@sonymobile.com wrote:
> >
> > +/* it can take some time ( > 100ms ) to initialise the
> > + * blacklist so we delay this until we actually need it
> > + */
> > +static void init_kprobe_blacklist(void)
> > +{
> >
gt;
> > -Oskar
> >
> > Björn Davidsson (1):
> > kprobes: move x86-specific blacklist symbols to arch directory
> >
> > Oskar Andero (2):
> > kprobes: split blacklist into common and arch
> > kprobes: replace printk with pr_-functions
> &
On 08:17 Thu 04 Apr , Masami Hiramatsu wrote:
> (2013/04/03 17:28), oskar.and...@sonymobile.com wrote:
> > Some blackpoints are only valid for specific architectures. To let each
> > architecture specify its own blackpoints the list has been split in two
> > lists: common and arch. The common
From: Toby Collett
The symbol lookup can take a long time and kprobes is
initialised very early in boot, so delay symbol lookup
until the blacklist is first used.
Cc: Masami Hiramatsu
Cc: David S. Miller
Reviewed-by: Radovan Lekanovic
Signed-off-by: Toby Collett
Signed-off-by: Oskar Andero
Instead of using printk use pr_info/pr_err/pr_warn. This was
detected by the checkpatch.pl script.
Cc: Masami Hiramatsu
Cc: David S. Miller
Signed-off-by: Oskar Andero
Acked-by: Masami Hiramatsu
---
kernel/kprobes.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions
included linux-arch in Cc for this post.
-Oskar
Björn Davidsson (1):
kprobes: move x86-specific blacklist symbols to arch directory
Oskar Andero (2):
kprobes: split blacklist into common and arch
kprobes: replace printk with pr_-functions
Toby Collett (1):
kprobes: delay blacklist symbol
-a...@vger.kernel.org
Reviewed-by: Radovan Lekanovic
Signed-off-by: Björn Davidsson
Signed-off-by: Oskar Andero
---
arch/x86/kernel/kprobes/core.c | 6 +-
kernel/kprobes.c | 3 ---
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/arch/x86/kernel/kprobes/core.c b/arch/x86
. Miller
Cc: linux-a...@vger.kernel.org
Signed-off-by: Oskar Andero
---
arch/arc/kernel/kprobes.c | 3 ++
arch/arm/kernel/kprobes.c | 2 +
arch/avr32/kernel/kprobes.c| 3 ++
arch/ia64/kernel/kprobes.c | 3 ++
arch/mips/kernel/kprobes.c | 3 ++
arch/mn10300/kernel
ed-by: Radovan Lekanovic
> Signed-off-by: Aleksej Makarov
> Signed-off-by: Oskar Andero
> ---
> drivers/input/input.c |5 -
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/input/input.c b/drivers/input/input.c
> index c044699..61ce
On 18:33 Thu 04 Apr , Dmitry Torokhov wrote:
> Hi Oskar,
>
> On Thu, Mar 07, 2013 at 03:01:22PM +0100, oskar.and...@sonymobile.com wrote:
> > From: Aleksej Makarov
> >
> > When waking up the platform by pressing a specific key, sending a
> > release on that key makes it impossible to react
Thanks for your input guys!
On 04:16 Fri 05 Apr , Masami Hiramatsu wrote:
> (2013/04/05 9:56), Joonsoo Kim wrote:
> > Hello, Oskar.
> >
> > On Thu, Apr 04, 2013 at 02:51:26PM +0200, Oskar Andero wrote:
> >> From: Toby Collett
> >>
> >> Th
Instead of using printk use pr_info/pr_err/pr_warn. This was
detected by the checkpatch.pl script.
Cc: Masami Hiramatsu
Cc: David S. Miller
Signed-off-by: Oskar Andero
Acked-by: Masami Hiramatsu
---
kernel/kprobes.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions
arch_kprobes_blacksyms.* as __weak symbols.
-Oskar
Björn Davidsson (1):
kprobes: move x86-specific blacklist symbols to arch directory
Oskar Andero (2):
kprobes: split blacklist into common and arch
kprobes: replace printk with pr_-functions
Toby Collett (1):
kprobes: delay blacklist
. Miller
Cc: linux-a...@vger.kernel.org
Signed-off-by: Oskar Andero
---
kernel/kprobes.c | 88 +---
1 file changed, 59 insertions(+), 29 deletions(-)
diff --git a/kernel/kprobes.c b/kernel/kprobes.c
index c8c2281..2458ae1 100644
--- a/kernel
-a...@vger.kernel.org
Reviewed-by: Radovan Lekanovic
Signed-off-by: Björn Davidsson
Signed-off-by: Oskar Andero
Acked-by: Masami Hiramatsu
---
arch/x86/kernel/kprobes/core.c | 7 +++
kernel/kprobes.c | 3 ---
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/arch/x86/kernel
From: Toby Collett
The symbol lookup can take a long time and kprobes is
initialised very early in boot, so delay symbol lookup
until the blacklist is first used.
Cc: Masami Hiramatsu
Cc: David S. Miller
Reviewed-by: Radovan Lekanovic
Signed-off-by: Toby Collett
Signed-off-by: Oskar Andero
ointer. This
> patch adds a null pointer check.
>
> Reviewed-by: Radovan Lekanovic
> Signed-off-by: syunsuke.x.itou
> Signed-off-by: Oskar Andero
> ---
> drivers/scsi/sd.c |7 ++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
>
> diff --git a/drivers
Hi James,
On 18:39 Mon 08 Apr , James Bottomley wrote:
> On Mon, 2013-04-08 at 13:42 +0200, Oskar Andero wrote:
> > On 14:36 Thu 07 Mar , oskar.and...@sonymobile.com wrote:
> > > From: "syunsuke.x.itou"
> > >
> > > By repeatadly co
Hi,
On 14:47 Tue 09 Jul , Oskar Andero wrote:
> From: Lars Svensson
>
> Some error bits in the status field of R1/R1b response are only set
> by the device in response to the command following the failing
> command. The status is only read and checked after a r/w comman
Signed-off-by: Aleksej Makarov
Signed-off-by: Oskar Andero
---
drivers/input/input.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/input/input.c b/drivers/input/input.c
index c044699..ee3ff16 100644
--- a/drivers/input/input.c
+++ b/drivers/input/input.c
Hi Ulf,
On 18:50 Tue 22 Oct , Ulf Hansson wrote:
> > And this is after the patch has been applied:
> > KB reclen write rewritereadreread
> >51200 4 251 990 3280 3244
> >51200 8 4601545 4460 4463
> >
Signed-off-by: Aleksej Makarov
Signed-off-by: Oskar Andero
---
drivers/input/input.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/input/input.c b/drivers/input/input.c
index 846ccdd..511d490 100644
--- a/drivers/input/input.c
+++ b/drivers/input/input.c
Hi Dmitry,
On 03:14 Tue 26 Nov , Dmitry Torokhov wrote:
> Hi Oskar,
>
> On Fri, Nov 22, 2013 at 02:27:04PM +0100, Oskar Andero wrote:
> > From: Aleksej Makarov
> >
> > When waking up the platform by pressing a specific key, sending a
> > release on that
Shrinkers return hardcoded -1 on error. Use errno.h values instead
to add more meaning to the errors.
Cc: Hugh Dickins
Cc: Greg Kroah-Hartman
Signed-off-by: Oskar Andero
---
drivers/staging/android/ashmem.c | 2 +-
drivers/staging/zcache/zcache-main.c | 2 +-
fs/gfs2/glock.c
hat do you say? Is this a good idea or does it make no sense at all?
Thanks!
-Oskar
Oskar Andero (2):
mm: vmscan: let any negative return value from shrinker mean error
Clean-up shrinker return values
drivers/staging/android/ashmem.c | 2 +-
drivers/staging/zcache/zcache-main.c | 2 +
The shrinkers must return -1 to indicate that it is busy. Instead of
relaying on magical numbers, let any negative value indicate error. This
opens up for using the errno.h error codes in the shrinker
implementations.
Cc: Hugh Dickins
Cc: Greg Kroah-Hartman
Signed-off-by: Oskar Andero
On 17:03 Tue 14 May , Glauber Costa wrote:
> On 05/13/2013 06:16 PM, Oskar Andero wrote:
> > Hi,
> >
> > In a previous discussion on lkml it was noted that the shrinkers use the
> > magic value "-1" to signal that something went wrong.
> >
&
On 16:18 Wed 15 May , Glauber Costa wrote:
> On 05/15/2013 06:10 PM, Oskar Andero wrote:
> > On 17:03 Tue 14 May , Glauber Costa wrote:
> >> On 05/13/2013 06:16 PM, Oskar Andero wrote:
> >>> Hi,
> >>>
> >>> In a previous
On 01:05 Thu 16 May , Andrew Morton wrote:
> On Mon, 13 May 2013 16:16:33 +0200 Oskar Andero
> wrote:
>
> > In a previous discussion on lkml it was noted that the shrinkers use the
> > magic value "-1" to signal that something went wrong.
> >
> &g
On 16:49 Wed 15 May , Glauber Costa wrote:
> On 05/15/2013 06:47 PM, Oskar Andero wrote:
> > On 16:18 Wed 15 May , Glauber Costa wrote:
> >> On 05/15/2013 06:10 PM, Oskar Andero wrote:
> >>> On 17:03 Tue 14 May , Glauber Costa wrote:
> >>>&g
The shrinkers must return -1 to indicate that it is busy. Instead, treat
any negative value as busy.
This fixes a potential bug if scan_objects returns a negative other than -1.
Cc: Glauber Costa
Cc: Dave Chinner
Cc: Andrew Morton
Cc: Hugh Dickins
Cc: Greg Kroah-Hartman
Signed-off-by: Oskar
On 13:52 Thu 16 May , Dave Chinner wrote:
> On Thu, May 16, 2013 at 10:42:16AM +0200, Oskar Andero wrote:
> > The shrinkers must return -1 to indicate that it is busy. Instead, treat
> > any negative value as busy.
>
> Why? The API defines return condition for abor
On 08:33 Fri 17 May , Dave Chinner wrote:
> On Thu, May 16, 2013 at 02:27:52PM +0200, Oskar Andero wrote:
> > On 13:52 Thu 16 May , Dave Chinner wrote:
> > > On Thu, May 16, 2013 at 10:42:16AM +0200, Oskar Andero wrote:
> > > > The shrinkers must return
On 22:00 Tue 16 Apr , David Rientjes wrote:
> On Tue, 16 Apr 2013, Oskar Andero wrote:
>
> > > > The comment in shrinker.h is misleading, not the source code.
> > > > do_shrinker_shrink() will fail for anything negative and 0.
> > >
> > > The c
Hi,
On 16:07 Wed 01 May , Lars-Peter Clausen wrote:
> On 05/01/2013 12:21 AM, Oskar Andero wrote:
> > This adds support for Microchip's 12 bit AD converters MCP3204 and
> > MCP3208. These chips communicates over SPI and supports single-ended
> > and pseudo-differ
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron
Cc: Lars-Peter Clausen
Signed-off-by: Oskar Andero
---
drivers/iio/adc/Kconfig | 10
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron
Cc: Lars-Peter Clausen
Signed-off-by: Oskar Andero
---
drivers/iio/adc/Kconfig | 10
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron
Cc: Lars-Peter Clausen
Signed-off-by: Oskar Andero
---
drivers/iio/adc/Kconfig | 10
: Brian Swetland
Reviewed-by: Radovan Lekanovic
Signed-off-by: Snild Dolkow
Signed-off-by: Oskar Andero
---
drivers/staging/android/lowmemorykiller.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/staging/android/lowmemorykiller.c
b/drivers/staging/android/lowmemorykiller.c
On 16:13 Mon 15 Apr , Dan Carpenter wrote:
> On Mon, Apr 15, 2013 at 03:38:08PM +0200, Dolkow, Snild wrote:
> > >Where is lowmem_shrink called from? I only see shrink called from the
> > >bcache sysfs handler __bch_cache_set(). The return value isn't checked
> > >there.
> > >
> > >Up to now
On 08:19 Tue 16 Apr , Dan Carpenter wrote:
> On Mon, Apr 15, 2013 at 04:11:18PM -0700, David Rientjes wrote:
> > On Mon, 15 Apr 2013, Greg Kroah-Hartman wrote:
> >
> > > > The positive numbers are used to return information on the remaining
> > > > cache size (again, see the comment I pasted
transfer. In some situations this causes errors passing undetected.
The solution is to read the status and check for these errors after
each r/w operation.
Signed-off-by: Lars Svensson
Signed-off-by: Oskar Andero
Cc: linux-...@vger.kernel.org
---
drivers/mmc/card/block.c | 105
Hii Dmitry,
On 14:09 Thu 25 Jul , Oskar Andero wrote:
> From: Aleksej Makarov
>
> When waking up the platform by pressing a specific key, sending a
> release on that key makes it impossible to react on the event in
> user-space. This is fixed by moving the input_rese
Hi Ulf,
On 17:01 Wed 16 Oct , Ulf Hansson wrote:
> Hi Oskar / Lars,
>
> Sorry for the delayed response!
>
> On 10 October 2013 15:28, Oskar Andero wrote:
> > From: Lars Svensson
> >
> > Some error bits in the status field of R1/R1b response are only
Add a BUG_ON to catch any illegal value from the shrinkers. This fixes a
potential bug if scan_objects returns a negative other than -1, which
would lead to undefined behaviour.
Cc: Glauber Costa
Cc: Dave Chinner
Cc: Andrew Morton
Cc: Hugh Dickins
Cc: Greg Kroah-Hartman
Signed-off-by: Oskar
Add a VM_BUG_ON to catch any illegal value from the shrinkers. It's a
potential bug if scan_objects returns a negative other than -1 and
would lead to undefined behaviour.
Cc: Glauber Costa
Cc: Dave Chinner
Cc: Andrew Morton
Cc: Hugh Dickins
Cc: Greg Kroah-Hartman
Signed-off-by: Oskar Andero
This adds support for Microchip's 12 bit AD converters MCP3204 and
MCP3208. These chips communicates over SPI and supports single-ended
and pseudo-differential configurations.
Cc: Jonathan Cameron
Cc: Lars-Peter Clausen
Signed-off-by: Oskar Andero
---
drivers/iio/adc/Kconfig
From: Ren Bigcren
The device report an error capacity when read_capacity_16().
Using read_capacity_10() can get the correct capacity.
Signed-off-by: Ren Bigcren
Cc: Matthew Dharm
Signed-off-by: Oskar Andero
---
drivers/usb/storage/unusual_devs.h | 7 +++
1 file changed, 7 insertions
1 - 100 of 104 matches
Mail list logo