On Sat, Jan 02, 2016 at 06:59:16PM -0600, Serge E. Hallyn wrote:
> On Sat, Jan 02, 2016 at 01:50:49PM -0800, Jeremiah Mahler wrote:
> > Serge,
> >
> > On Sat, Jan 02, 2016 at 12:24:16PM -0600, Serge E. Hallyn wrote:
> > [...]
> > >
> > > Tried to reproduce with setting
Added annotation '__must_hold' to function ksocknal_send_keepalive_locked
which unlocks the lock ksocknal_data.ksnd_global_lock. As this lock is
not acquired in the current function, sparse warns about context imbalance
Signed-off-by: Niranjan Dighe
---
From: Bjørn Mork
Date: Sat, 02 Jan 2016 22:30:48 +0100
> SF Markus Elfring writes:
>
>> From: Markus Elfring
>> Date: Fri, 1 Jan 2016 17:35:03 +0100
>>
>> Omit explicit initialisation at the beginning for one local variable
>> that is redefined before its first use.
>
>
> This patch is
On Sat, 2016-01-02 at 18:50 +0100, SF Markus Elfring wrote:
> A few update suggestions were taken into account
> from static source code analysis.
While static analysis can be useful, I don't think these
specific conversions are generally useful.
Perhaps it would be more useful to convert the
On Sat, Jan 02, 2016 at 09:18:30PM +0100, Milan Broz wrote:
>
> But I cannot change thousands of cryptsetup installations that are actively
> using that code.
> This is clear userspace breakage which should not happen this way.
I'll try to add some compatibility code for your case, assuming
your
Well, OK with me, I will use yours instead because I don't know if they
will backport the thing, thanks so much for doing this.
Samuel Thibault wrote:
> cov...@ccs.covici.com, on Sat 02 Jan 2016 19:10:36 -0500, wrote:
> > I had a patch which also worked, but yours may be better -- I enclose it
After reading a page, we need to check whether there is any error.
Signed-off-by: Jaegeuk Kim
---
fs/f2fs/data.c | 8
1 file changed, 8 insertions(+)
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 89a978c..11b2111 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -448,6 +448,14
Original issue is fixed by:
f2fs: cover more area with nat_tree_lock
This reverts commit 24928634f81b1592e83b37dcd89ed45c28f12feb.
---
fs/f2fs/node.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c
index 4dab09f..6d5f548 100644
--- a/fs/f2fs/node.c
There was a subtle bug on nat cache management which incurs wrong nid allocation
or wrong block addresses when try_to_free_nats is triggered heavily.
This patch enlarges the previous coverage of nat_tree_lock to avoid data race.
Signed-off-by: Jaegeuk Kim
---
fs/f2fs/node.c | 29
From: Rafael J. Wysocki
Commit f06147f9fbf1 (ACPICA: Hardware: Enable firmware waking vector
for both 32-bit and 64-bit FACS) added three functions that aren't
present in upstream ACPICA, acpi_hw_set_firmware_waking_vectors(),
acpi_set_firmware_waking_vectors() and
cov...@ccs.covici.com, on Sat 02 Jan 2016 19:10:36 -0500, wrote:
> I had a patch which also worked, but yours may be better -- I enclose it
> here for your information.
Well, it's not up to serialio.h to include things for serialio.c. That
however makes me realize that the culprit is actually
On Sat, Jan 02, 2016 at 01:50:49PM -0800, Jeremiah Mahler wrote:
> Serge,
>
> On Sat, Jan 02, 2016 at 12:24:16PM -0600, Serge E. Hallyn wrote:
> [...]
> >
> > Tried to reproduce with setting CONFIG_CFQ_GROUP_IOSCHED=y, but did not
> > succeed. Could you send me the .config? Also, if someone
I had a patch which also worked, but yours may be better -- I enclose it
here for your information.
--- drivers/staging/speakup/serialio.h.old 2015-08-30 14:34:09.0
-0400
+++ drivers/staging/speakup/serialio.h 2015-10-07 06:27:04.880829874 -0400
@@ -1,22 +1,24 @@
#ifndef
On Tuesday, December 22, 2015 11:05:43 AM Viresh Kumar wrote:
> On 21-12-15, 22:32, SF Markus Elfring wrote:
> > From: Markus Elfring
> > Date: Mon, 21 Dec 2015 22:12:26 +0100
> >
> > The cpu_set_cclk() function was only used in a single source file so far.
> > Indicate this setting also by the
On Friday, December 18, 2015 07:21:35 AM Viresh Kumar wrote:
> On 17-12-15, 19:04, Bartlomiej Zolnierkiewicz wrote:
> > Commit 01fb4d3c39d3 ("PM / OPP: Parse 'opp--'
> > bindings") broke support for parsing standard opp-microvolt and
> > opp-microamp properties. Fix it by setting 'name' string to
On Wednesday, December 23, 2015 10:59:55 PM Julia Lawall wrote:
> The powercap_zone_ops and powercap_zone_constraint_ops structures are never
> modified, so declare them as const.
>
> Most of the actual changes adjust indentation to accomodate the const
> keyword.
>
> Done with the help of
On Tuesday, December 29, 2015 05:43:33 PM Pavel Machek wrote:
> On Tue 2015-12-29 11:03:21, Manuel Pégourié-Gonnard wrote:
> > The documentation for detach() said attach.
> >
> > Signed-off-by: Manuel Pégourié-Gonnard
> > Cc: triv...@kernel.org
>
> Acked-by: Pavel Machek
Applied, thanks!
--
On Wednesday, December 16, 2015 12:20:29 PM Chen Yu wrote:
> It is reported that, with CONFIG_HZ_PERIODIC=y cpu stays at the
> lowest frequency even if the usage goes to 100%, neither ondemand
> nor conservative governor works, however performance and
> userspace work as expected. If set with
On Tuesday, December 29, 2015 01:52:19 PM Lv Zheng wrote:
> The 20151218 ACPICA kernel-resident subsystem updates are linuxized based
> on the linux-pm/linux-next branch.
>
> The patchset has passed the following build/boot tests.
> Build tests are performed as follows:
> 1. i386 + allyes
> 2.
On Wednesday, December 23, 2015 05:19:39 PM Aleksey Makarov wrote:
> As discussed when Shannon Zhao sent a patch to add platform_device support
> to pl061 driver. Russel and other maintainers prefered that ACPI learned
> how to create AMBA devices rather than converting/adding platform_device
>
Samuel Thibault, on Sun 03 Jan 2016 00:25:29 +0100, wrote:
> 5e6dc54 broke the port information in the speakup driver:
> SERIAL_PORT_DFNS only gets defined if asm/serial.h is included.
>
> Along the way, make sure that we do have information for the requested
> serial port number (index)
(It'd
Hi Markus,
On Sun, Jan 3, 2016 at 7:54 AM, SF Markus Elfring
wrote:
> From: Markus Elfring
> Date: Sat, 2 Jan 2016 21:21:24 +0100
>
> This issue was detected by using the Coccinelle software.
>
> Adjust jump targets according to the current Linux coding style convention.
>
> Signed-off-by:
Hi,
My name is Jeffrey Skoll, a philanthropist and the founder of one of the
largest private foundations in the world. I believe strongly in ‘giving while
living.’ I had one idea that never changed in my mind — that you should use
your wealth to help people and I have decided to secretly give
5e6dc54 broke the port information in the speakup driver:
SERIAL_PORT_DFNS only gets defined if asm/serial.h is included.
Along the way, make sure that we do have information for the requested
serial port number (index)
Signed-off-by: Samuel Thibault
--- a/drivers/staging/speakup/serialio.c
On Sunday 03 January 2016 00:03:54 Pali Rohár wrote:
> On Saturday 02 January 2016 23:57:47 Arnd Bergmann wrote:
> > On Saturday 02 January 2016 16:22:03 Pali Rohár wrote:
> > > On Monday 28 December 2015 15:55:28 Arnd Bergmann wrote:
> > > > On Monday 28 December 2015 15:54:35 Pali Rohár wrote:
>
On Saturday 02 January 2016 23:57:47 Arnd Bergmann wrote:
> On Saturday 02 January 2016 16:22:03 Pali Rohár wrote:
> > On Monday 28 December 2015 15:55:28 Arnd Bergmann wrote:
> > > On Monday 28 December 2015 15:54:35 Pali Rohár wrote:
> > > > > I mean you can add the platform data to the
> > > >
On Saturday 02 January 2016 16:22:03 Pali Rohár wrote:
> On Monday 28 December 2015 15:55:28 Arnd Bergmann wrote:
> > On Monday 28 December 2015 15:54:35 Pali Rohár wrote:
> > > >
> > > > I mean you can add the platform data to the omap_auxdata_lookup[]
> > > > table for this board.
> > >
> > >
On Mon, Dec 21, 2015 at 6:05 PM, Marcelo Tosatti wrote:
> OK cool hopefully that makes it clear to Fenghua Yu what must
> be changed in the patchset.
>
>> - http://lkml.iu.edu/hypermail/linux/kernel/1511.0/02375.html
...beating a dead horse but this patch is in -next and breaks the
ARCH=um
On 12/17/2015 08:37 AM, Baolin Wang wrote:
> Hi Milan,
>
> On 16 December 2015 at 16:08, Milan Broz wrote:
>> On 12/16/2015 04:18 AM, Baolin Wang wrote:
>>> From the dm-crypt performance report, we found it shows low efficiency
>>> with crypto engine for some mode (like ecb or xts mode). Because
On Saturday 02 January 2016 11:59:29 Sudip Mukherjee wrote:
> >
> > Just to be sure we are talking about the same thing: you mean running a
> > 64-bit
> > kernel in a kvm guest with a 32-bit file system, right? Running a 32-bit
> > kvm guest on a 64-bit host would not be interesting of course.
>
Prefix public functions with "orangefs_" do don't
pollute the global namespace.
This fixes a build issue on UML which also has block_signals().
Signed-off-by: Richard Weinberger
---
fs/orangefs/orangefs-kernel.h | 4 ++--
fs/orangefs/orangefs-utils.c | 4 ++--
fs/orangefs/waitqueue.c |
Not every arch has io memory.
So, unbreak the build by fixing the dependencies.
Signed-off-by: Richard Weinberger
---
drivers/clocksource/Kconfig | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
index
...instead of open coding it.
Signed-off-by: Richard Weinberger
---
fs/ubifs/xattr.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/ubifs/xattr.c b/fs/ubifs/xattr.c
index e8b01b7..e53292d 100644
--- a/fs/ubifs/xattr.c
+++ b/fs/ubifs/xattr.c
@@ -267,7 +267,7 @@ static
This function does never return NULL, IS_ERR() is sufficent.
Signed-off-by: Richard Weinberger
---
fs/ext4/crypto_key.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/ext4/crypto_key.c b/fs/ext4/crypto_key.c
index c5882b3..436ea2a 100644
--- a/fs/ext4/crypto_key.c
We cannot use positive error codes in ERR_PTR().
IS_ERR() won't catch them.
Cc: sta...@vger.kernel.org
Signed-off-by: Richard Weinberger
---
fs/ubifs/super.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/fs/ubifs/super.c b/fs/ubifs/super.c
index
... instead of open coding it.
Signed-off-by: Richard Weinberger
---
fs/xattr.c | 38 ++
1 file changed, 14 insertions(+), 24 deletions(-)
diff --git a/fs/xattr.c b/fs/xattr.c
index 9b932b9..9e8a652 100644
--- a/fs/xattr.c
+++ b/fs/xattr.c
@@ -324,7 +324,6
Am 02.01.2016 um 23:04 schrieb Richard Weinberger:
> Prefix public functions with "orangefs_" do don't
> pollute the global namespace.
>
> This fixes a build issue on UML which also has block_signals().
>
> Signed-off-by: Richard Weinberger
> ---
> fs/orangefs/orangefs-kernel.h | 4 ++--
>
Hi,
My name is Jeffrey Skoll, a philanthropist and the founder of one of the
largest private foundations in the world. I believe strongly in ‘giving while
living.’ I had one idea that never changed in my mind — that you should use
your wealth to help people and I have decided to secretly give
On 01/02/2016 03:26 PM, Pali Rohár wrote:
Hello,
due to this Daniel Lezcano commit (ARM: OMAP3: cpuidle - remove rx51
cpuidle parameters table)
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=231900afba52d6faddfb480cde4132d4edc089bc
we need patch cpuidle34xx.c code
Serge,
On Sat, Jan 02, 2016 at 12:24:16PM -0600, Serge E. Hallyn wrote:
[...]
>
> Tried to reproduce with setting CONFIG_CFQ_GROUP_IOSCHED=y, but did not
> succeed. Could you send me the .config? Also, if someone could send
> the objdump -d output that might help. Though really, it seems
Quoting Heiko Stübner (2016-01-01 14:05:10)
> Am Freitag, 1. Januar 2016, 22:50:43 schrieb Arnd Bergmann:
> > On Friday 01 January 2016 18:06:30 Heiko Stübner wrote:
> > > "[PATCH] clk: rockchip: fix section mismatches with new child-clocks" [0]
> > >
> > > should be in Mike's + Stephen's inbox
On Saturday 02 January 2016 18:14:31 Tony Lindgren wrote:
> * Pali Rohár [160102 06:14]:
> > Hello,
> >
> > now I'm looking at differences between legacy board code and DTS
> > file for Nokia N900 and I see some inconsistency for twl4030-power
> > driver.
> >
> > In board code are defined more
SF Markus Elfring writes:
> From: Markus Elfring
> Date: Fri, 1 Jan 2016 17:32:07 +0100
>
> Reduce the scope for the local variable "desc" to one branch
> of an if statement.
This patch is harmless. But is also pointless.
You could at least try to explain why this must be changed. I'm not
On Sat, Jan 02, 2016 at 06:43:16AM -0500, Tejun Heo wrote:
> Hello,
>
> On Fri, Jan 01, 2016 at 12:18:17PM +0200, Michael S. Tsirkin wrote:
> > > My initial idea was to use a dedicated workqueue. Michael S. Tsirkin
> > > suggested using a system one. Tejun Heo confirmed that the system
> > >
This removes a few Sparse warnings.
Signed-off-by: Hugo Camboulive
---
drivers/staging/most/hdm-dim2/dim2_hal.c | 4 ++--
drivers/staging/most/hdm-dim2/dim2_hal.h | 7 ---
drivers/staging/most/hdm-dim2/dim2_hdm.c | 6 +++---
drivers/staging/most/hdm-dim2/dim2_hdm.h | 2 +-
4 files changed,
SF Markus Elfring writes:
> From: Markus Elfring
> Date: Fri, 1 Jan 2016 17:35:03 +0100
>
> Omit explicit initialisation at the beginning for one local variable
> that is redefined before its first use.
This patch is unnecessary. The variable initialisation is redundant.
See the difference?
On Sun, Dec 27, 2015 at 04:57:48PM +0200, Andy Shevchenko wrote:
> On Wed, Dec 23, 2015 at 9:18 PM, Akshay Bhat wrote:
> > "at" is not a valid vendor prefix, correcting the same to "atmel"
> >
>
> I'm afraid you can't just do this change alone as it's used in some
> DTS. Though you may
On Sun, Dec 27, 2015 at 05:02:48PM +0200, Andy Shevchenko wrote:
> On Wed, Dec 23, 2015 at 8:38 PM, Akshay Bhat wrote:
> > This adds devicetree documentation for the bindings of rtc-rx8010
> > driver.
> >
> > Signed-off-by: Akshay Bhat
> > ---
> >
On Saturday 02 Jan 2016 20:33:23 (+), Al Viro wrote:
> On Sat, Jan 02, 2016 at 08:30:21PM +, Hugo Camboulive wrote:
> > This removes a few Sparse warnings.
>
> > + g.dim2 = (struct dim2_regs __iomem *)dim_base_address;
>
> > -u8 dim_startup(void *dim_base_address, u32 mlb_clock);
> >
From: Markus Elfring
Date: Sat, 2 Jan 2016 21:40:10 +0100
Omit explicit initialisation at the beginning for one local variable
that is redefined before its first use.
Signed-off-by: Markus Elfring
---
drivers/nfc/mei_phy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
From: Markus Elfring
Date: Sat, 2 Jan 2016 21:33:04 +0100
Rename a jump label according to the current Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/nfc/mei_phy.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/nfc/mei_phy.c
From: Markus Elfring
Date: Sat, 2 Jan 2016 21:21:24 +0100
This issue was detected by using the Coccinelle software.
Adjust jump targets according to the current Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/nfc/mei_phy.c | 13 ++---
1 file changed, 6
From: Markus Elfring
Date: Sat, 2 Jan 2016 21:47:30 +0100
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (3):
Refactoring for mei_nfc_connect()
Refactoring for mei_nfc_if_version()
Delete an unnecessary variable initialisation in
On Fri, Dec 11, 2015 at 02:55:10PM +0100, Bartosz Golaszewski wrote:
> 2015-12-11 13:08 GMT+01:00 Wolfram Sang :
> > On Wed, Dec 02, 2015 at 11:25:17AM +0100, Bartosz Golaszewski wrote:
> >> Chips from the at24cs EEPROM series have an additional read-only memory
> >> area
> >> containing a
On 01/02/2016 11:16 AM, Tony Lindgren wrote:
> * Pali Rohár [160102 06:31]:
>> Hello,
>>
>> MPU OPP table table (omap36xx_vddcore_volt_data) defined in
>> opp3xxx_data.c does not match Nokia N900 phone. For a long time we have
>> dirty patch in linux-n900 tree for it, see:
>>
>>
On Sat, Jan 02, 2016 at 08:30:21PM +, Hugo Camboulive wrote:
> This removes a few Sparse warnings.
> + g.dim2 = (struct dim2_regs __iomem *)dim_base_address;
> -u8 dim_startup(void *dim_base_address, u32 mlb_clock);
> +u8 dim_startup(void __iomem *dim_base_address, u32 mlb_clock);
On 01/02/2016 09:03 PM, Stephan Mueller wrote:
> Am Samstag, 2. Januar 2016, 15:41:34 schrieb Milan Broz:
>
> Hi Milan,
>
...
>>> Hi Herbert,
>>>
>>> this patch breaks userspace in cryptsetup...
>>>
>>> We use algif_skcipher in cryptsetup (for years, even before
>>> there was Stephan's library)
On Sat, 2016-01-02 at 19:35 +0100, Mike Looijmans wrote:
> On 2-1-2016 11:39, Russell King - ARM Linux wrote:
> > On Thu, Dec 31, 2015 at 04:50:54PM +0900, Masahiro Yamada wrote:
> > > Hi.
> > >
> > > I am new to the Linux DMA APIs.
> > >
> > > First, I started by reading
Am Samstag, 2. Januar 2016, 15:41:34 schrieb Milan Broz:
Hi Milan,
> On 01/02/2016 12:52 PM, Milan Broz wrote:
> > On 12/25/2015 08:40 AM, Herbert Xu wrote:
> >> Dmitry Vyukov wrote:
> >>> I am testing with your two patches:
> >>> crypto: algif_skcipher - Use new skcipher interface
> >>>
Add a defconfig covering all known pxa board, ie. all selectable machine
files in arch/arm/mach-pxa/*.c.
This defconfig was built by doing :
- aggregation of all known defconfigs by cat
am200epdkit_defconfig
cm_x2xx_defconfig
cm_x300_defconfig
gcc correctly warns that the ctx variable in ext4_set_encrypted_filename
has gone out of scope in ext4_set_encrypted_filename if we enter the
retry path and a 'goto' into the previous code block can not guarantee
to get the contents back:
fs/ext4/namei.c: In function
This removes a few Sparse warnings.
Signed-off-by: Hugo Camboulive
---
drivers/staging/most/hdm-dim2/dim2_hal.c | 6 +++---
drivers/staging/most/hdm-dim2/dim2_hal.h | 6 +++---
drivers/staging/most/hdm-dim2/dim2_hdm.c | 6 +++---
drivers/staging/most/hdm-dim2/dim2_hdm.h | 2 +-
4 files changed,
Hi,
My name is Jeffrey Skoll, a philanthropist and the founder of one of the
largest private foundations in the world. I believe strongly in ‘giving while
living.’ I had one idea that never changed in my mind — that you should use
your wealth to help people and I have decided to secretly give
On 2-1-2016 11:39, Russell King - ARM Linux wrote:
On Thu, Dec 31, 2015 at 04:50:54PM +0900, Masahiro Yamada wrote:
Hi.
I am new to the Linux DMA APIs.
First, I started by reading Documentation/DMA-API.txt,
but I am confused with the term "consistent memory".
Just read "coherent memory"
On 26/12/15 13:04, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sat, 26 Dec 2015 13:53:15 +0100
>
> This issue was detected by using the Coccinelle software.
>
> Move the jump label directly before the desired log statement
> so that the variable "ret" does not need to be checked
From: Markus Elfring
Date: Sat, 2 Jan 2016 19:22:36 +0100
Omit explicit initialisation at the beginning for four local variables
which are redefined before their first use.
Signed-off-by: Markus Elfring
---
drivers/net/wireless/rsi/rsi_91x_pkt.c | 8
1 file changed, 4 insertions(+),
On Sat, Jan 02, 2016 at 06:54:37AM -0500, Tejun Heo wrote:
> On Fri, Jan 01, 2016 at 03:40:28PM -0800, Jeremiah Mahler wrote:
> > all,
> >
> > When running the latest linux-next (20151231) two of my machines
> > hang early in the boot sequence. The initial message is for a
> > NULL pointer
On Sat, Jan 2, 2016 at 2:02 AM, SF Markus Elfring
wrote:
> From: Markus Elfring
> Date: Fri, 1 Jan 2016 21:16:01 +0100
>
> Rename a jump label according to the current Linux coding style convention.
>
> Signed-off-by: Markus Elfring
> ---
> drivers/net/wireless/intel/iwlegacy/common.c | 4 ++--
Michael Turquette writes:
> Hi Arnd,
>
> Quoting Arnd Bergmann (2015-12-30 01:29:02)
>> It's also ok to merge the header file and binding with either the dts file
>> changes or the driver and then do the other part the following release.
>>
>> In the past, we've worked around the issue by
On Sat, Jan 02, 2016 at 08:17:51AM -0800, James Bottomley wrote:
> On Sat, 2016-01-02 at 10:39 +, Russell King - ARM Linux wrote:
> > On Thu, Dec 31, 2015 at 04:50:54PM +0900, Masahiro Yamada wrote:
> > > Hi.
> > >
> > > I am new to the Linux DMA APIs.
> > >
> > > First, I started by reading
From: Markus Elfring
Date: Sat, 2 Jan 2016 18:28:26 +0100
Omit explicit initialisation at the beginning for one local variable
that is redefined before its first use.
Signed-off-by: Markus Elfring
---
drivers/net/xen-netback/xenbus.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
From: Markus Elfring
Date: Sat, 2 Jan 2016 18:23:16 +0100
Replace an explicit initialisation for one local variable at the beginning
by an assignment.
Signed-off-by: Markus Elfring
---
drivers/net/xen-netback/xenbus.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
From: Markus Elfring
Date: Sat, 2 Jan 2016 18:01:57 +0100
Replace an explicit initialisation for one local variable at the beginning
by an assignment.
Signed-off-by: Markus Elfring
---
drivers/net/xen-netback/xenbus.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git
From: Markus Elfring
Date: Sat, 2 Jan 2016 17:50:21 +0100
One goto statement referred to a source code position directly behind it.
Thus omit such an unnecessary jump.
Signed-off-by: Markus Elfring
---
drivers/net/xen-netback/xenbus.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
From: Markus Elfring
Date: Sat, 2 Jan 2016 17:32:40 +0100
Remove the assignment for a local variable because its value is not
changed compared to the one from a previous function call.
Signed-off-by: Markus Elfring
---
drivers/net/xen-netback/xenbus.c | 1 -
1 file changed, 1 deletion(-)
From: Markus Elfring
Date: Sat, 2 Jan 2016 18:46:45 +0100
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (5):
Delete an unnecessary assignment in connect_rings()
Delete an unnecessary goto statement in connect_rings()
Replace a variable
* Pali Rohár [160102 06:31]:
> Hello,
>
> MPU OPP table table (omap36xx_vddcore_volt_data) defined in
> opp3xxx_data.c does not match Nokia N900 phone. For a long time we have
> dirty patch in linux-n900 tree for it, see:
>
>
* Pali Rohár [160102 06:14]:
> Hello,
>
> now I'm looking at differences between legacy board code and DTS file
> for Nokia N900 and I see some inconsistency for twl4030-power driver.
>
> In board code are defined more twl4030 power scripts which override
> defaults defined in twl4030-power
On Mon, Dec 28, 2015 at 12:05:51PM +0200, Kirill A. Shutemov wrote:
> On Thu, Dec 24, 2015 at 11:20:30AM -0500, Matthew Wilcox wrote:
> > diff --git a/include/linux/mm.h b/include/linux/mm.h
> > index 4bf3811..e14634f 100644
> > --- a/include/linux/mm.h
> > +++ b/include/linux/mm.h
> > @@ -1958,6
Hi,
* Pali Rohár [160102 06:46]:
> --- a/drivers/media/rc/ir-rx51.c
> +++ b/drivers/media/rc/ir-rx51.c
> @@ -25,9 +25,9 @@
> #include
> #include
> #include
> +#include
>
> -#include
> -#include
> +#include "../../../arch/arm/plat-omap/include/plat/dmtimer.h"
Well we don't want to
On Thu, Dec 31, 2015 at 10:30:27AM +1100, Dave Chinner wrote:
> > @@ -1637,6 +1669,7 @@ xfs_filemap_pfn_mkwrite(
> > static const struct vm_operations_struct xfs_file_vm_ops = {
> > .fault = xfs_filemap_fault,
> > .pmd_fault = xfs_filemap_pmd_fault,
> > + .pud_fault =
-rsi-Fine-tuning-for-two-function-implementations/20160102-224740
base:
https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git
master
config: x86_64-randconfig-s2-01030012 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make
On Sat, 2016-01-02 at 10:39 +, Russell King - ARM Linux wrote:
> On Thu, Dec 31, 2015 at 04:50:54PM +0900, Masahiro Yamada wrote:
> > Hi.
> >
> > I am new to the Linux DMA APIs.
> >
> > First, I started by reading Documentation/DMA-API.txt,
> > but I am confused with the term "consistent
Agreed.
This probably comes from the downstream code, I think.. But I agree
it's actually redundant.
Thank you for pointing this out.
On Sat, Jan 2, 2016 at 11:27 AM, Russell King - ARM Linux
wrote:
> On Sat, Dec 26, 2015 at 01:47:22PM -0800, Eric Anholt wrote:
>> +int __init
Hi Dmitry,
On 01/02/2016 01:19 PM, Dmitry Vyukov wrote:
On Sat, Jan 2, 2016 at 12:33 PM, Manfred Spraul
wrote:
Hi Dmitry,
shm locking differs too much from msg/sem locking, I never looked at it in
depth, so I'm not able to perform a proper review.
Except for the obvious: Races that can be
On Friday 01 January 2016 12:33:03 Ivaylo Dimitrov wrote:
> Hi Pali,
>
> On 1.01.2016 13:26, Pali Rohár wrote:
> > On Friday 01 January 2016 12:03:29 Ivaylo Dimitrov wrote:
> >> All calls to isp1704_write() are using parameter sequence of
> >> isp1704_write(isp, reg, val) but the function is
Tetsuo Handa wrote:
> Michal Hocko wrote:
> > On Mon 28-12-15 21:08:56, Tetsuo Handa wrote:
> > > Tetsuo Handa wrote:
> > > > I got OOM killers while running heavy disk I/O (extracting kernel
> > > > source,
> > > > running lxr's genxref command). (Environ: 4 CPUs / 2048MB RAM / no swap
> > > >
On Monday 28 December 2015 15:55:28 Arnd Bergmann wrote:
> On Monday 28 December 2015 15:54:35 Pali Rohár wrote:
> > On Monday 28 December 2015 15:41:01 Arnd Bergmann wrote:
> > > On Monday 28 December 2015 15:28:48 Pali Rohár wrote:
> > > > On Monday 28 December 2015 15:14:50 Arnd Bergmann wrote:
Hello,
I have taken another look at the implementation of the function
"rsi_send_mgmt_pkt".
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/net/wireless/rsi/rsi_91x_pkt.c?id=e8c58e7a5a106c3d557fccd01cd4d1128f9bab38#n114
I find the following statement combination
SF Markus Elfring :
> From: Markus Elfring
> Date: Sat, 2 Jan 2016 15:25:34 +0100
>
> Replace explicit initialisation for two local variables at the beginning
> by assignments.
It makes no sense for the 'adapter' variable.
--
Ueimor
--
To unsubscribe from this list: send the line
Hi,
My name is Jeffrey Skoll, a philanthropist and the founder of one of the
largest private foundations in the world. I believe strongly in ‘giving while
living.’ I had one idea that never changed in my mind — that you should use
your wealth to help people and I have decided to secretly give
Hi,
My name is Jeffrey Skoll, a philanthropist and the founder of one of the
largest private foundations in the world. I believe strongly in ‘giving while
living.’ I had one idea that never changed in my mind — that you should use
your wealth to help people and I have decided to secretly give
Hello,
due to this commit (ARM: OMAP2+: Disable code that currently does not
work with multiplaform)
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/media/rc/Kconfig?id=a62a6e98c370ccca37d353a5f763b532411a4c14
lirc driver for Nokia N900 (ir-rx51) cannot be enabled
From: Markus Elfring
Date: Sat, 2 Jan 2016 15:25:34 +0100
Replace explicit initialisation for two local variables at the beginning
by assignments.
Signed-off-by: Markus Elfring
---
drivers/net/wireless/rsi/rsi_91x_pkt.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff
From: Markus Elfring
Date: Sat, 2 Jan 2016 15:15:12 +0100
Omit explicit initialisation at the beginning for four local variables
which are redefined before their first use.
Signed-off-by: Markus Elfring
---
drivers/net/wireless/rsi/rsi_91x_pkt.c | 8
1 file changed, 4 insertions(+),
From: Markus Elfring
Date: Sat, 2 Jan 2016 14:54:30 +0100
Omit explicit initialisation at the beginning for five local variables
which are redefined before their first use.
Signed-off-by: Markus Elfring
---
drivers/net/wireless/rsi/rsi_91x_pkt.c | 10 +-
1 file changed, 5
On 01/02/2016 12:52 PM, Milan Broz wrote:
> On 12/25/2015 08:40 AM, Herbert Xu wrote:
>> Dmitry Vyukov wrote:
>>>
>>> I am testing with your two patches:
>>> crypto: algif_skcipher - Use new skcipher interface
>>> crypto: algif_skcipher - Require setkey before accept(2)
>>> on top of
Hi,
My name is Jeffrey Skoll, a philanthropist and the founder of one of the
largest private foundations in the world. I believe strongly in ‘giving while
living.’ I had one idea that never changed in my mind — that you should use
your wealth to help people and I have decided to secretly give
From: Markus Elfring
Date: Sat, 2 Jan 2016 15:36:25 +0100
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (3):
Delete unnecessary variable initialisations in rsi_send_mgmt_pkt()
Delete unnecessary variable initialisations in
Hello,
MPU OPP table table (omap36xx_vddcore_volt_data) defined in
opp3xxx_data.c does not match Nokia N900 phone. For a long time we have
dirty patch in linux-n900 tree for it, see:
https://github.com/pali/linux-n900/commit/4644c5801d7469e2be01d847c61df3d934dadd8c
Now when doing transition to
101 - 200 of 474 matches
Mail list logo