Looking at the driver the define you are fixing isn't even used.
So it could be removed.
While you are at it you may want to remove the peice of completey
redundant code.
.
.
if (ret)
goto error_ret;
error_ret:
.
.
Wow.
On Sat, Jun 28, 2014 at 05:10:46AM +0200, Guillaume
On Fri, Jun 27 2014 at 10:57:28 PM, "Chalamarla, Tirumalesh"
wrote:
> Marc,
>
> What is your opinion on ITS emulation . is it should be part
> of KVM or VFIO.
Making any sort of emulation part of VFIO sounds quite wrong. That's not
what VFIO is about, at all. Emulation belongs
On Fri, 2014-06-27 at 23:20 -0700, Austin Schuh wrote:
> For workqueues, as long as the helper doesn't block on a lock which
> requires the work queue to be freed up, it will eventually become
> unblocked and make progress. The helper _should_ only need the pool
> lock, which will wake the helper
On Fri, Jun 27, 2014 at 02:55:15PM +0200, Michael Grzeschik wrote:
> Hi,
>
> On Fri, Jun 27, 2014 at 04:53:53PM +0530, Punnaiah Choudary Kalluri wrote:
> > Zynq soc uses Chipidea/Synopsys usb IP core(CI13612). This patch adds
> > necessary glue to allow the chipidea driver to work on zynq soc.
> >
Am 27.06.2014 19:27, schrieb John Stultz:
Its been pointed out that the RTC hctosys functionality doesn't
work well with RTC modules, which may not be loaded until after
late_init().
While there have been other attempts to sovle this, this patchset
is a very quick 10 minute effort to show how I'
On Fri, Jun 27, 2014 at 11:28 PM, Andy Lutomirski wrote:
> On Fri, Jun 27, 2014 at 11:12 PM, Alexei Starovoitov
> wrote:
>> On Fri, Jun 27, 2014 at 5:19 PM, Andy Lutomirski wrote:
>>> On Fri, Jun 27, 2014 at 5:05 PM, Alexei Starovoitov
>>> wrote:
eBPF programs are safe run-to-completion
Am 28.06.2014 09:18, schrieb Alexander Holler:
Am 27.06.2014 19:27, schrieb John Stultz:
Its been pointed out that the RTC hctosys functionality doesn't
work well with RTC modules, which may not be loaded until after
late_init().
While there have been other attempts to sovle this, this patchset
Warns or generates patch for NULL check before the following functions:
kfree
usb_free_urb
debugfs_remove
debugfs_remove_recursive
Cc: Julia Lawall
Cc: Gilles Muller
Cc: Joe Perches
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
V3:
-Update print_main message.
-Add patch mode
On 23/06/14 10:08, Hartmut Knaack wrote:
Philippe Reynes schrieb:
This driver add partial support of the
maxim 1027/1029/1031. Differential mode is not
supported.
It was tested on armadeus apf27 board.
Signed-off-by: Philippe Reynes
Reviewed-by: Hartmut Knaack
---
Not sure if it's too lat
>> There seems to be a regression in the upcoming Linux 3.16-rc2 release
>> candidate that I bisected down to this first bad commit:
>> [dbb42748ac4929987c1449ecb296b39ef8956b62] drm/i915: Move the C3 LP
>> write bit setup to gen3_init_clock_gating() for KMS.
>
> Can you attach the dmesg with rc2 a
> V3:
>-Update print_main message.
Does the discussion topic need also an adjustment?
How do you think about my previous update suggestion "Deletion of
unnecessary checks before specific function calls"?
https://systeme.lip6.fr/pipermail/cocci/2014-March/000675.html
https://lkml.org/lkml/20
In virtualized environment there are mainly three problems
related to spinlocks that affects performance.
1. LHP (lock holder preemption)
2. Lock Waiter Preemption (LWP)
3. Starvation/fairness
Though Ticketlocks solve fairness problem it worsens LWP, LHP problems. Though
pv-ticketlocks tried to ad
Fix checkpatch warning:
"WARNING: debugfs_remove(NULL) is safe this check is probably not required"
Cc: Bjorn Helgaas
Cc: Ryan Desfosses
Cc: linux-...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/pci/hotplug/cpqphp_sysfs.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Hi Everybody,
Today I was trying to build 3.15.2 on AMD64 from kernel 3.14.8 and get this:
CC fs/mbcache.o
fs/mbcache.c: In function ‘__spin_lock_mb_cache_entry’:
fs/mbcache.c:134: error: SSE register return with SSE disabled
make[1]: *** [fs/mbcache.o] Error 1
google doesn't reveal much
On Sat, 28 Jun 2014 11:08:35 +0200
SF Markus Elfring wrote:
>
> > V3:
> >-Update print_main message.
>
> Does the discussion topic need also an adjustment?
>
> How do you think about my previous update suggestion "Deletion of
> unnecessary checks before specific function calls"?
> https:/
On Sat, Jun 28, 2014 at 10:52:24AM +0100, Nick Warne wrote:
> Hi Everybody,
>
> Today I was trying to build 3.15.2 on AMD64 from kernel 3.14.8 and get this:
>
> CC fs/mbcache.o
> fs/mbcache.c: In function ‘__spin_lock_mb_cache_entry’:
> fs/mbcache.c:134: error: SSE register return with SSE
--
I' ANNA MARIOUS DIAGNOSED OF CANCER. I DONATE TO YOU $8,600,000.SO YOU ARE TO
SEND YOUR TELE NUMBER AND ALTERNATIVE EMAIL TO MY FAMILY ATTORNEY, BRAIN JAMES
WITH DETAILS VIA THIS EMAIL; ( brainjames_2...@mynet.com )
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel"
Fix checkpatch warning:
WARNING: kfree(NULL) is safe this check is probably not required
Cc: "K. Y. Srinivasan"
Cc: Haiyang Zhang
Cc: de...@linuxdriverproject.org
Signed-off-by: Fabian Frederick
---
drivers/net/hyperv/netvsc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --gi
The "uptime" tracer added in:
commit 8aacf017b065a805d27467843490c976835eb4a5
tracing: Add "uptime" trace clock that uses jiffies
has wraparound problems when the system has been up more
than 1 hour 11 minutes and 34 seconds. It converts jiffies
to nanoseconds using:
(u64)jiffies_to
update uptime description.
Cc: Tony Luck
Cc: Steven Rostedt
Cc: Frederic Weisbecker
Signed-off-by: Xie XiuQi
---
Documentation/trace/ftrace.txt | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/Documentation/trace/ftrace.txt b/Documentation/trace/ftrace.txt
index 2479b2a..
The "uptime" tracer added in:
commit 8aacf017b065a805d27467843490c976835eb4a5
tracing: Add "uptime" trace clock that uses jiffies
has wraparound problems when the system has been up more
than 1 hour 11 minutes and 34 seconds. It converts jiffies
to nanoseconds using:
(u64)jiffies_to
On 28/06/14 11:26, Nick Warne wrote:
On 28/06/14 11:12, Borislav Petkov wrote:
On Sat, Jun 28, 2014 at 10:52:24AM +0100, Nick Warne wrote:
Hi Everybody,
Today I was trying to build 3.15.2 on AMD64 from kernel 3.14.8 and get this:
CC fs/mbcache.o
fs/mbcache.c: In function ‘__spin_lock_
Fix checkpatch warning:
WARNING: usb_free_urb(NULL) is safe this check is probably not required
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/media/usb/ttusb-budget/dvb-ttusb-budget.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Fix checkpatch warning:
WARNING: kfree(NULL) is safe this check is probably not required
Cc: Greg Kroah-Hartman
Cc: Sarah Sharp
Cc: de...@driverdev.osuosl.org
Signed-off-by: Fabian Frederick
---
drivers/staging/rtl8821ae/rtl8821ae/hal_btc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
>> How do you think about my previous update suggestion "Deletion of
>> unnecessary checks before specific function calls"?
>> https://systeme.lip6.fr/pipermail/cocci/2014-March/000675.html
>> https://lkml.org/lkml/2014/3/5/344
> I didn't see you made the same kind of script.
Will my approach
Ok, I rebuilt the host kernel with latest linus+tip/master and my queue.
The guest kernel is v3.15-8992-g08f7cc749389 with a is a bunch of RAS
patches. Before I start doing the coarse-grained bisection by testing
-rcs and major numbers, I wanted to catch a #DF and try to analyze at
least why it hap
>From 8e67ad03ab03839456816e922c57a7ab3bcf5474 Mon Sep 17 00:00:00 2001
From: Tejun Heo
Date: Sat, 28 Jun 2014 07:47:57 -0400
Currently, the blkio subsystem attributes all of writeback IOs to the
root. One of the issues is that there's no way to tell who originated
a writeback IO from block laye
Fix checkpatch warning:
WARNING: debugfs_remove(NULL) is safe this check is probably not required
Cc: Roland Dreier
Cc: Sean Hefty
Cc: linux-r...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/infiniband/ulp/ipoib/ipoib_fs.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions
On 2014/5/20 6:35, Luck, Tony wrote:
>> Another problem, maybe we should use get_jiffies_64() instead of jiffies
>> directly here OR we'll meet wraparound problems on 32bit system. But a
>> same problem is that the jiffies_lock is not safe in NMI context...
>
> To quote Winnie the Pooh - "Bother".
Hello, Li.
On Fri, Jun 27, 2014 at 05:13:12PM +0800, Li Zefan wrote:
> + for_each_subsys(ss, i) {
> + if (!(opts.subsys_mask & (1 << i)) ||
> + ss->root == &cgrp_dfl_root)
> + continue;
> +
> + if (!percpu_ref_tryget_live(&ss->root->c
If we don't check the current backing device status, balance_dirty_pages can
fall into infinite pausing routine.
This can be occurred when a lot of directories make a small number of dirty
dentry pages including files.
Reported-by: Brian Chadwick
Signed-off-by: Jaegeuk Kim
---
fs/f2fs/node.c |
Fix checkpatch warning:
WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not
required
Cc: Linus Walleij
Cc: Samuel Ortiz
Cc: linux-arm-ker...@lists.infradead.org
Signed-off-by: Fabian Frederick
---
drivers/mfd/ab8500-debugfs.c | 3 +--
1 file changed, 1 insertion(+), 2 d
On Sat, 28 Jun 2014 13:38:15 +0200
SF Markus Elfring wrote:
>
> >> How do you think about my previous update suggestion "Deletion of
> >> unnecessary checks before specific function calls"?
> >> https://systeme.lip6.fr/pipermail/cocci/2014-March/000675.html
> >> https://lkml.org/lkml/2014/3/5/34
On Tue, Jun 17, 2014 at 09:07:59PM -0400, Tejun Heo wrote:
> Hello,
>
> Currently, percpu_ref destruction - freeing the percpu counter -
> happens when the percpu_ref is released. This while a bit more
> convenient restricts how percpu_ref can be used. It can't be
> initialized with static percp
On Wed, Jun 18, 2014 at 09:19:26AM -0700, Jens Axboe wrote:
> > percpu/for-3.17 6fbc07bbe2b5 ("percpu: invoke __verify_pcpu_ptr() from the
> > generic part of accessors and operations")
> >+[1] [PATCHSET percpu/for-3.17] percpu: implement percpu_ref_reinit()
> >
> >and available in the following
Fix checkpatch warning:
WARNING: kfree(NULL) is safe this check is probably not required
Cc: Stanislav Yakovlev
Cc: "John W. Linville"
Cc: linux-wirel...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/net/wireless/ipw2x00/libipw_module.c | 3 +--
1 file changed, 1 insertion(+), 2
On Sat, Jun 28, 2014 at 11:55:07AM +0100, Nick Warne wrote:
> Whoops - that is WRONG (too many ssh terminals)
>
> gcc version 4.2.4
That's some old compiler. Anyway, I can't trigger it here with gcc 4.6
and 4.9.
Can you do
$ make clean
$ make V=1 fs/mbcache.i >>w.log 2>&1
$ make V=1 fs/mbcache.
Fix checkpatch warning:
WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not
required
Cc: Kevin Hilman
Cc: Nishanth Menon
Cc: linux...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/power/avs/smartreflex.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions
Cc: Johan Hovold
Cc: Greg Kroah-Hartman
Cc: linux-...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/usb/serial/mos7840.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/usb/serial/mos7840.c b/drivers/usb/serial/mos7840.c
index 393be56..3d88eef 10064
> -Original Message-
> From: Fabian Frederick [mailto:f...@skynet.be]
> Sent: Saturday, June 28, 2014 6:37 AM
> To: linux-kernel@vger.kernel.org
> Cc: Fabian Frederick; KY Srinivasan; Haiyang Zhang;
> de...@linuxdriverproject.org
> Subject: [PATCH 1/1] drivers/net/hyperv/netvsc.c: remove
On Sun, Jun 15, 2014 at 01:37:33PM -0700, Joe Perches wrote:
> Use the zeroing function instead of dma_alloc_coherent & memset(,0,)
Applied
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http:
Hello Masahiro,
On 27-06-14 08:38, Masahiro Yamada wrote:
KBuild (ab)uses the asm statement to write to a file and
llvm integrated as chokes about these invalid asm statements.
Workaround it by making it look like valid asm code.
Signed-off-by: Jeroen Hofstee
I think Linux has the same proble
Fix checkpatch warning:
WARNING: kfree(NULL) is safe this check is probably not required
Cc: Nagalakshmi Nandigama
Cc: Sreekanth Reddy
Cc: linux-s...@vger.kernel.org
Cc: dl-mptfusionli...@lsi.com
Signed-off-by: Fabian Frederick
---
drivers/message/fusion/mptfc.c | 3 +--
1 file changed, 1 inse
On Sat, Jun 28, 2014 at 11:41 AM, Ken Adams wrote:
>
> On 6/27/14 8:56 PM, "Ben Skeggs" wrote:
>
>>On Sat, Jun 28, 2014 at 4:51 AM, Ken Adams wrote:
>>> quick note re: tegra and gpu bars...
>>>
>>> to this point we've explicitly avoided providing user-mode mappings due
>>>to
>>> power management
Fixes the tab error found in base.h, reported by checkpatch.pl tool
Signed-off-by: Morgan Stuart
---
drivers/staging/rtl8192ee/base.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192ee/base.h b/drivers/staging/rtl8192ee/base.h
index c7929a7..1d6e5a7
This patch removes special case of last element and loops from idx to 0.
Cc: David Airlie
Cc: dri-de...@lists.freedesktop.org
Signed-off-by: Fabian Frederick
---
This is untested.
drivers/gpu/drm/drm_buffer.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/gpu
The macro "REGMAP_ALLOW_WRITE_DEBUGFS" can be used to enable write
support on the registers file in the debugfs. The mode of the file is
fixed to 0400 so it is not possible to write the file ever.
This patch fixes the mode by setting it to the correct value depending
on the macro.
Cc: Dimitris Pa
This patch does away with cast on void * and the if as it is unnecessary.
The following Coccinelle semantic patch was used for making the change:
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T *)x)->f
|
- (T *)
e
)
Signed-off-by: Himangi Saraogi
-
The following changes since commit a497c3ba1d97fc69c1e78e7b96435ba8c2cb42ee:
Linux 3.16-rc2 (2014-06-21 19:02:54 -1000)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git
tags/regulator-v3.16-rc2
for you to fetch changes up to 117674
This patch removes the cast on data of type void * as it is not needed.
The following Coccinelle semantic patch was used for making the change:
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T *)x)->f
|
- (T *)
e
)
Signed-off-by: Himangi Saraogi
Acke
This patch does away with cast on void * and the if as it is unnecessary.
The following Coccinelle semantic patch was used for making the change:
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T *)x)->f
|
- (T *)
e
)
Signed-off-by: Himangi Saraogi
A
The following changes since commit a497c3ba1d97fc69c1e78e7b96435ba8c2cb42ee:
Linux 3.16-rc2 (2014-06-21 19:02:54 -1000)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git
tags/spi-v3.16-rc2
for you to fetch changes up to 7216a4183947853f35
This patch does away with cast on void * and the if as it is unnecessary.
The following Coccinelle semantic patch was used for making the change:
@r@
expression x;
void* e;
type T;
identifier f;
@@
(
*((T *)e)
|
((T *)x)[...]
|
((T *)x)->f
|
- (T *)
e
)
Signed-off-by: Himangi Saraogi
A
Minor fix of $subject to be inline with existing logs needed.
PM / AVS: SmartReflex: ..
On 06/28/2014 07:33 AM, Fabian Frederick wrote:
Fix checkpatch warning:
WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not
required
Cc: Kevin Hilman
Cc: Nishanth Menon
Cc: linux...
Fix checkpatch warning:
WARNING: debugfs_remove(NULL) is safe this check is probably not required
Cc: Jiri Kosina
Cc: linux-in...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
drivers/hid/hid-picolcd_debugfs.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/dr
Fix checkpatch warning:
WARNING: debugfs_remove_recursive(NULL) is safe this check is probably not
required
Cc: Kevin Hilman
Cc: Nishanth Menon
Cc: linux...@vger.kernel.org
Signed-off-by: Fabian Frederick
---
V2:
-Subject update (suggested by Nishanth Menon)
-Blank line before
On 2014-06-28 06:13, Tejun Heo wrote:
On Wed, Jun 18, 2014 at 09:19:26AM -0700, Jens Axboe wrote:
percpu/for-3.17 6fbc07bbe2b5 ("percpu: invoke __verify_pcpu_ptr() from the generic
part of accessors and operations")
+[1] [PATCHSET percpu/for-3.17] percpu: implement percpu_ref_reinit()
and av
Dear Gregory CLEMENT,
On Fri, 27 Jun 2014 00:43:25 +0200, Gregory CLEMENT wrote:
> Quotin the ARM datasheet "When set, coherent linefill requests are
Quotin -> Quoting
> sent speculatively to the L2C-310 in parallel with the tag look-up. If
> the tag look-up misses, the confirmed linefill is sen
Dear Gregory CLEMENT,
On Fri, 27 Jun 2014 00:43:27 +0200, Gregory CLEMENT wrote:
> diff --git a/arch/arm/mach-mvebu/board-v7.c b/arch/arm/mach-mvebu/board-v7.c
> index 8bb742fdf5ca..a24ea105e709 100644
> --- a/arch/arm/mach-mvebu/board-v7.c
> +++ b/arch/arm/mach-mvebu/board-v7.c
> @@ -45,6 +45,7
On Wed, Jun 25, 2014 at 02:12:29PM +0530, Viresh Kumar wrote:
> cpufreq-cpu0 driver supports only platforms which have single clock line
> shared
> among all CPUs.
>
> We already have platforms where this limitation doesn't hold true.For example
> on
> Qualcomm's KRAIT all CPUs have separate clo
checkpath.pl was complaining about value_mask:
ERROR: Macros with complex values should be enclosed in parenthesis
I fixed this by simply removing it since it's not used (as well as another
macro).
Got rid of the un-necessary error_ret label as well. Both changes were
suggested
by Michael Welli
On Sat, Jun 28, 2014 at 12:26:14AM -0700, Alexei Starovoitov wrote:
> On Fri, Jun 27, 2014 at 11:28 PM, Andy Lutomirski wrote:
> > On Fri, Jun 27, 2014 at 11:12 PM, Alexei Starovoitov
> > wrote:
> > If you want to add GPL-only functions in the future, that would be one
> > thing. But if someone
On Sat, Jun 28, 2014 at 06:48:48AM +0200, Guillaume Morin wrote:
> From: Guillaume Morin
>
> Simple code style fixes
What exactly are those code style fixes? Please be explicit.
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo
On Sat, Jun 28, 2014 at 05:13:19PM +0200, Guillaume Morin wrote:
> checkpath.pl was complaining about value_mask:
> ERROR: Macros with complex values should be enclosed in parenthesis
>
> I fixed this by simply removing it since it's not used (as well as another
> macro).
> Got rid of the un-nece
> +the firmware license provided includes an implicit or explicit
> +patent grant to end users to ensure full functionality of device
> +operation with the firmware. If the licence is long and involved, it's
> permitted to include it in a separate file and refer to it from the
> WHENCE file.
> A
On Fri, 27 Jun 2014, Greg Kroah-Hartman wrote:
> On Fri, Jun 27, 2014 at 04:11:35PM -0400, Alan Stern wrote:
> > On Fri, 27 Jun 2014, Greg Kroah-Hartman wrote:
> >
> > > > One side point: The patch changes the string displayed for the
> > > > power/runtime_status attribute file when disable_dept
On Fri, Jun 27, 2014 at 11:43 PM, Alexei Starovoitov wrote:
> On Fri, Jun 27, 2014 at 11:25 PM, Andy Lutomirski wrote:
>> On Fri, Jun 27, 2014 at 10:55 PM, Alexei Starovoitov
>> wrote:
>>> On Fri, Jun 27, 2014 at 5:16 PM, Andy Lutomirski
>>> wrote:
On Fri, Jun 27, 2014 at 5:05 PM, Alexei
On Fri, 27 Jun 2014 12:25:20 -0500
> + rate = 16 * max(115200U, (unsigned int)baud);
> +
This assumes an arbitarily configurable clock, which is not I think the
usual case.
> + /*
> + * Request a different clock rate if necessary, and
> + * record
On Sat, Jun 28, 2014 at 8:21 AM, Greg KH wrote:
> On Sat, Jun 28, 2014 at 12:26:14AM -0700, Alexei Starovoitov wrote:
>> On Fri, Jun 27, 2014 at 11:28 PM, Andy Lutomirski
>> wrote:
>> > On Fri, Jun 27, 2014 at 11:12 PM, Alexei Starovoitov
>> > wrote:
>> > If you want to add GPL-only functions
checkpath.pl was complaining about value_mask:
ERROR: Macros with complex values should be enclosed in parenthesis
I fixed this by simply removing it since it's not used (as well as another
macro). Got rid of the un-necessary error_ret label as well.
Signed-off-by: Guillaume Morin
Reported-by:
From: Anil Belur
- this fixes few sparse warnings for missing static in functions:
drivers/staging/lustre/lnet/lnet/api-ni.c:63:6: warning: symbol
'lnet_get_routes' was not declared. Should it be static?
drivers/staging/lustre/lnet/lnet/api-ni.c:69:6: warning: symbol
'lnet_get_networks' was no
On Sat, Jun 28, 2014 at 09:19:34PM +0530, Kiran Kankipati wrote:
> Hi Something similar to that I got inconsistency issue in LZ4-HC in 64bit
> mode.
>
> I posted today in kernel bugzee:
> https://bugzilla.kernel.org/show_bug.cgi?id=79091
I don't do bugzilla :)
Let's drag this back to lkml as
On Fri, Jun 27, 2014 at 5:06 PM, Alexei Starovoitov wrote:
> Safety of eBPF programs is statically determined by the verifier, which
> detects:
This is a very high-level review. I haven't tried to read all the
code yet, and this is mostly questions rather than real comments.
> - loops
> - out
On Sat, Jun 28, 2014 at 05:54:39PM +0200, Guillaume Morin wrote:
> checkpath.pl was complaining about value_mask:
> ERROR: Macros with complex values should be enclosed in parenthesis
>
> I fixed this by simply removing it since it's not used (as well as another
> macro). Got rid of the un-necess
Do all of you guys even care about cleaning up bugs? If
you do it would be great if I get a reply to where I can get
a up to date list of kernel bugs.
Cheers Nick
On Fri, Jun 27, 2014 at 10:45 PM, Nick Krause wrote:
> Do any of you use the kernel Bugzilla? If you do I was wondering if we
> can cl
From: Guillaume Morin
Simple code style fixes:
- "if(" -> "if ("
- "switch(" -> "switch ("
- move one open brace to the line of the declaration instead of
its own line
- remove trailing whitespace
Signed-off-by: Guillaume Morin
---
Changes since v1:
- added explicit description of the f
On Sat, Jun 28, 2014 at 09:39:12PM +0530, Kiran Kankipati wrote:
> Wow I see. Thanks Greg. I seen the 3.15.2 change logs, found you are involved
> in the LZO issue.
>
> I read about this issue in some news too that they found 20yr old bug in it.
>
> So thought reporting to you via email.
>
> I d
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
On Fri, Jun 27, 2014 at 10:45:47PM -0400, Nick Krause wrote:
> Do any of you use the kernel Bugzilla? If you do I was wondering if we
> can clean it up.
> Otherwise I was wondering were I can get an accurate list of open
> bugs in the newest
> kernels.
Some subsystem maintainers use the kernel bu
These haven't had a single ack by the listed maintainer
in all git history and the email addresses don't work.
And entry for Michael Neuffer is already in CREDITS.
Signed-off-by: Joe Perches
---
MAINTAINERS | 14 --
1 file changed, 14 deletions(-)
diff --git a/MAINTAINERS b/MAINTAI
On Sat, Jun 28, 2014 at 12:06:15PM -0400, Nick Krause wrote:
> Do all of you guys even care about cleaning up bugs? If
> you do it would be great if I get a reply to where I can get
> a up to date list of kernel bugs.
We care about fixing bugs. Most kernel devs don't really care about
maintaining
Oops, I just click send before I type up the reply. Here we go again.
On Tue, Jun 17, 2014 at 2:11 AM, Andy Shevchenko
wrote:
> Very dumb patch to just skip --param allow-store-data-races=0 introduced in
> newer GCC versions.
>
> +static char **handle_param(char *arg, char **next)
> +{
> +
Commit 582e9d37a9a3 'include/linux: remove strict_strto* definitions' has
obsoleted the strict_strtol function. Use kstrtol instead.
This fixes the following compilation error:
drivers/staging/emxx_udc/emxx_udc.c: In function ‘nbu2ss_drv_set_ep_info’:
drivers/staging/emxx_udc/emxx_udc.c:3287:
On Thu 2014-06-26 13:47:32, Andy Lutomirski wrote:
> On Thu, Jun 26, 2014 at 1:12 PM, H. Peter Anvin wrote:
> > The real question is if we care that sysret and iter don't match. On 32
> > bits the situation is even more complex.
>
> At least for 64 bits, iret vs sysret is purely a kernel implem
Hi!
> The non-DT support has to be maintained for now to not break
> OMAP3 legacy boot, and the legacy-style code will be cleaned
> up once OMAP3 is also converted to DT-boot only.
> @@ -587,24 +606,157 @@ static int omap_mbox_unregister(struct
> omap_mbox_device *mdev)
> return 0;
> }
>
Hi!
> --- /dev/null
> +++ b/drivers/edac/altera_edac.c
> @@ -0,0 +1,449 @@
> +/*
> + * Copyright Altera Corporation (C) 2014. All rights reserved.
> + * Copyright 2011-2012 Calxeda, Inc.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms and
Hi!
> This gives the userspace (Replicant) a chance to fully handle the
> pm_wakeup_event, before autosleep suspends the system alltogether
> again.
>
> This fixes suspend/resume on the OpenPhoenux GTA04, in combination with
> the Replicant 4.2.2 userspace, which needs to execute this to stay
> a
Hi!
> From: Thor Thayer
>
> Add the Altera SDRAM controller bindings and device tree changes to the
> Altera SoC project.
>
> Signed-off-by: Thor Thayer
> diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-sdram.txt
> b/Documentation/devicetree/bindings/arm/altera/socfpga-sdr
Hi!
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
> @@ -0,0 +1,15 @@
> +Altera SOCFPGA SDRAM Error Detection & Correction [EDAC]
> +
> +Required properties:
> +- compatible : should contain "altr,sdram-edac";
> +- reg : should contain the ECC register
On Sat, 2014-06-28 at 19:00 +0200, Vincent Stehlé wrote:
> Commit 582e9d37a9a3 'include/linux: remove strict_strto* definitions' has
> obsoleted the strict_strtol function. Use kstrtol instead.
[]
> diff --git a/drivers/staging/emxx_udc/emxx_udc.c
> b/drivers/staging/emxx_udc/emxx_udc.c
[]
> @@ -3
devm_ioremap_resource returns an ERR_PTR value, not NULL, on failure.
A simplified version of the semantic match that finds this problem is as
follows:
//
@@
expression e,e1;
statement S;
@@
*e = devm_ioremap_resource(...);
if (!e1) S
//
Signed-off-by: Himangi Saraogi
Acked-by: Julia Lawall
devm_ioremap_resource returns an ERR_PTR value, not NULL, on failure.
A simplified version of the semantic match that finds this problem is as
follows:
//
@@
expression e,e1;
statement S;
@@
*e = devm_ioremap_resource(...);
if (!e1) S
//
Signed-off-by: Himangi Saraogi
Acked-by: Julia Lawall
On Sat, Jun 28, 2014 at 10:02:19PM +0530, Kiran Kankipati wrote:
> I don't understand, are you having problems on the compress side, or the
> decompress side? Where in the lz4 is a bug happening?
> >> Sorry sorry.
> I dont have a clue. Well the reason is, if compress is happening well and
> decomp
3.4-stable review patch. If anyone has any objections, please let me know.
--
From: Stephane Grosjean
commit 0b5a958cf4df3a5cd578b861471e62138f55c85e upstream.
As remarked by Christopher R. Baker in his post at
http://marc.info/?l=linux-can&m=139707295706465&w=2
there's a po
3.4-stable review patch. If anyone has any objections, please let me know.
--
From: Tony Luck
commit 74614de17db6fb472370c426d4f934d8d616edf2 upstream.
When Linux sees an "action optional" machine check (where h/w has reported
an error that is not in the current execution path
3.10-stable review patch. If anyone has any objections, please let me know.
--
From: Johan Hovold
commit acf47d4f9c39b1cba467aa9442fc2efe0b1da741 upstream.
Fix potential I/O while runtime suspended due to missing PM operations
in send_setup.
Fixes: 383cedc3bb43 ("USB: serial:
3.10-stable review patch. If anyone has any objections, please let me know.
--
From: Alan Stern
commit 32b36eeae6a859670d2939a7d6136cb5e9ed64f8 upstream.
In usbtest, tests 5 - 8 use the scatter-gather library in usbcore
without any sort of timeout. If there's a problem in the
3.10-stable review patch. If anyone has any objections, please let me know.
--
From: Paul Bolle
commit 77c2f02edbeda9409a7cf3fd66233015820c213a upstream.
Commit 193ab2a60700 ("usb: gadget: allow multiple gadgets to be built")
apparently required that checks for CONFIG_USB_GADG
3.10-stable review patch. If anyone has any objections, please let me know.
--
From: Liu Bo
commit 5dca6eea91653e9949ce6eb9e9acab6277e2f2c4 upstream.
According to commit 865ffef3797da2cac85b3354b5b6050dc9660978
(fs: fix fsync() error reporting),
it's not stable to just check e
1 - 100 of 532 matches
Mail list logo
