From: Oleg Drokin
As per Rusty Russel, cpus_* functions are deprecated.
When mixing cpumask_copy with cpus_weight, they operate on different
sized masks if CPUMASK_OFFSTACK is enabled, causing an
immediate assertion failure.
Copying of cpumasks by assignment is also not allowed now.
Additionally
From: Oleg Drokin
Rusty Russel advises that cpus_* functions are deprecated to work
on cpumasks and cpumask_* functions should be called instead,
otherwise problems with CPUMASK_OFFSTACK arise.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/libcfs/linux/linux-cpu.c | 78 +++--
From: Oleg Drokin
A recent crash report with CONFIG_CPUMASK_OFFSTACK enabled
set off a chain of emails through which I learned that cpus_* functions
are deprecated and should not be really used to operate on cpumasks.
These two patches do just that, replacing cpus_* and cpu_* calls
with cpumask_
tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
char-misc-testing
head: b3de8e3719e582f3182bb504295e4a8e43c8c96f
commit: 96c1d0581d00f7abe033350edb021a9d947d8d81 [25/45] Drivers: hv: vmbus:
Add support for VMBus panic notifier handler
reproduce:
# apt-get install sp
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not
declared. Should it be static?
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not
declared. Should it be static?
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not
declared. Shoul
On Sun, Mar 1, 2015 at 5:18 PM, Greg Kroah-Hartman
wrote:
> On Fri, Feb 27, 2015 at 09:44:42PM -0800, Tolga Ceylan wrote:
>> Replaced C99 '//' comments with C89 '/**/'
>>
>> Signed-off-by: Tolga Ceylan
>> ---
>> drivers/staging/fbtft/fb_ssd1331.c | 30 +++---
>> 1 file ch
On Sat, Feb 07, 2015 at 08:19:01PM -0800, Shirish Gajera wrote:
> This patch fixes the checkpatch.pl warning:
>
> WARNING: line over 80 characters
> All line over 80 characters in driver/staging/speakup/* are fixed.
>
> Signed-off-by: Shirish Gajera
> ---
> drivers/staging/speakup/main.c
> -Original Message-
> From: Greg KH [mailto:gre...@linuxfoundation.org]
> Sent: Sunday, March 1, 2015 7:34 PM
> To: Jake Oshins
> Cc: rafael.j.wyso...@intel.com; KY Srinivasan; linux-ker...@vger.kernel.org;
> de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;
> vkuzn...@re
On Tue, Feb 17, 2015 at 11:41:49AM -0800, Jake Oshins wrote:
> This patch adds some wrapper functions in the pnp layer. The intent is
> to allow memory address space claims by devices which are descendants
> (a child or grandchild of) a device which is already part of the pnp
> layer. This allows
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Mon, Mar 02, 2015 at 11:18:09AM +0800, Matt Tyler wrote:
> hey greg, I've updated to the latest linux-next, applied the patch and
> compiled with no issues.
Please work against my staging-testing
hey greg, I've updated to the latest linux-next, applied the patch and compiled
with no issues.
I think it might be due to clashes with another patch that has been proposed by
Tal Shorer to fix style violations in linux-proc.c (which my patch merges into
module.c)
Tal - do you want to take my
> -Original Message-
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
> Sent: Friday, February 27, 2015 8:14 AM
> To: KY Srinivasan; de...@linuxdriverproject.org
> Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Radim Krčmář;
> Greg Kroah-Hartman; linux-...@vger.kernel.o
On Wed, Feb 18, 2015 at 05:37:55PM -0300, Mario J. Rugiero wrote:
> This fixes all appearances of the warning but one, as that one seems to be
> intentional:
>
> WARNING: Single statement macros should not use a do {} while (0) loop
> #1221: FILE: lustre/include/lustre_dlm.h:1221:
> +#define LDLM_
On Tue, Feb 10, 2015 at 08:36:19PM +0200, Tal Shorer wrote:
> fix the following coding style error in
> drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:
> initialization of lnet_table_header (static pointer) to NULL
>
> Signed-off-by: Tal Shorer
> ---
> drivers/staging/lustre/lustre/lib
On Tue, Feb 10, 2015 at 08:36:20PM +0200, Tal Shorer wrote:
> fix the following coding style error in
> drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:
> initialization of min_watchdog_ratelimit (static int) to 0
>
> for clarity's sake, the "= 0" is kept as part of a comment
>
> Signed-
On Tue, Feb 10, 2015 at 08:36:21PM +0200, Tal Shorer wrote:
> following coding style error in
> drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:
> missing spaces around '='
>
> Signed-off-by: Tal Shorer
> ---
> drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c | 2 +-
> 1 file cha
On Tue, Feb 10, 2015 at 07:24:40PM +0800, Matt Tyler wrote:
> module.c was previously the sole exporter of symbols from module.c
> This patch removes the global symbols by merging the two files.
>
> Signed-off-by: Matthew Tyler
> ---
> drivers/staging/lustre/lustre/libcfs/Makefile | 2 +-
On Sat, Feb 21, 2015 at 06:53:30PM -0800, Joe Perches wrote:
> The seq_printf return value, because it's frequently misused,
> will eventually be converted to void.
>
> See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
> seq_has_overflowed() and make public")
>
> Convert these us
On Fri, Feb 27, 2015 at 09:06:56PM -0800, Tolga Ceylan wrote:
> Removed space before ','
>
> Signed-off-by: Tolga Ceylan
> ---
> drivers/staging/fbtft/fbtft-core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/fbtft/fbtft-core.c
> b/drivers/staging/fb
On Fri, Feb 27, 2015 at 09:44:42PM -0800, Tolga Ceylan wrote:
> Replaced C99 '//' comments with C89 '/**/'
>
> Signed-off-by: Tolga Ceylan
> ---
> drivers/staging/fbtft/fb_ssd1331.c | 30 +++---
> 1 file changed, 15 insertions(+), 15 deletions(-)
Doesn't apply to my tree
On Fri, Feb 27, 2015 at 03:08:58AM +, Sheng Yong wrote:
> For value which has size_t type, use %zu to avoid compile warning of
> printk format.
>
> Signed-off-by: Sheng Yong
> ---
> drivers/staging/fbtft/fbtft-core.c | 2 +-
> drivers/staging/fbtft/fbtft-io.c | 4 ++--
> 2 files changed, 3
On Sat, Feb 28, 2015 at 03:51:47PM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following error:
> code indent should use tabs where possible
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/staging/fbtft/flexfb.c | 9 +
> 1 file changed, 5 insert
On Sat, Feb 28, 2015 at 06:37:30PM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following error:
> space required after that ';'
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/staging/fbtft/fbtft_device.c | 145
> +++
>
On Mon, Feb 23, 2015 at 07:20:30AM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following errors:
> ERROR: space required after that ','
> ERROR: trailing whitespace
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/staging/fbtft/fb_st7735r.c | 18
On Mon, Feb 23, 2015 at 05:24:51PM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following error:
> space prohibited before that ','
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/staging/fbtft/fb_ra8875.c | 150
> +++---
On Sat, Feb 21, 2015 at 09:36:49PM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following warnings:
> space prohibited before that close parenthesis ')'
> space prohibited after that open parenthesis '('
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/st
On Mon, Mar 02, 2015 at 12:38:35AM +0100, Salah Triki wrote:
> This patch removes unnecessary checks of unit magic, channel magic, board
> magic and board state.
>
> Signed-off-by: Salah Triki
> ---
> drivers/staging/dgnc/dgnc_sysfs.c | 14 --
> 1 file changed, 14 deletions(-)
>
> d
This patch removes unnecessary checks of unit magic, channel magic, board
magic and board state.
Signed-off-by: Salah Triki
---
drivers/staging/dgnc/dgnc_sysfs.c | 14 --
1 file changed, 14 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_sysfs.c
b/drivers/staging/dgnc/dgnc_sysf
This patch fixes the following checkpatch.pl warning:
braces {} are not necessary for single statement blocks
Signed-off-by: Salah Triki
---
drivers/staging/dgnc/dgnc_tty.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c
in
On 27 February 2015 at 23:14, Andrey Ryabinin wrote:
> binder_update_page_range() initializes only addr and size
> fields in 'struct vm_struct tmp_area;' and passes it to
> map_vm_area().
>
> Before 71394fe50146 ("mm: vmalloc: add flag preventing guard hole allocation")
> this was because map_vm_a
On Sun, Mar 01, 2015 at 06:19:57PM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch removes EXTRA_CFLAGS from the Makefile and defines DDG_PART
> in dgnc_driver.h
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/staging/dgnc/Makefile | 2 --
> drivers/staging/dgnc
From: Matteo Semenzato
This patch removes EXTRA_CFLAGS from the Makefile and defines DDG_PART
in dgnc_driver.h
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/Makefile | 2 --
drivers/staging/dgnc/dgnc_driver.h | 1 +
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/
On Sun, Mar 01, 2015 at 09:41:26AM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following warning:
> Use of EXTRA_CFLAGS is deprecated, please use `ccflags-y instead.
>
> Signed-off-by: Matteo Semenzato
> ---
> drivers/staging/dgnc/Makefile | 2 +-
> 1 file
On Sun, Mar 01, 2015 at 09:31:41AM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> This patch fixes the following warning:
> Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev,
> ... then pr_err(... to printk(KERN_ERR ...
>
> Signed-off-by: Matteo Semenzato
> --
From: Matteo Semenzato
This patch fixes the following warning:
Use of EXTRA_CFLAGS is deprecated, please use `ccflags-y instead.
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/Makefile b
From: Matteo Semenzato
This patch fixes the following warning:
Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev,
... then pr_err(... to printk(KERN_ERR ...
Signed-off-by: Matteo Semenzato
---
drivers/staging/dgnc/dgnc_sysfs.c | 4 ++--
1 file changed, 2 insertions(+), 2
36 matches
Mail list logo
