On Thu, May 4, 2017 at 12:31 PM, Christian König
wrote:
> From: Christian König
>
> We use this mask multiple times in the bus setup.
Couple of nitpicks below.
Otherwise, FWIW:
Reviewed-by: Andy Shevchenko
> +#define PCI_RES_TYPE_MASK (IORESOURCE_IO | IORESOURCE_MEM | \
> +
On Sun, Apr 30, 2017 at 6:26 PM, Karim Eshapa wrote:
> On Fri, 28 Apr 2017 12:02:58 +0200, Greg KH wrote:
>> On Wed, Apr 26, 2017 at 06:15:40PM +0200, Karim Eshapa wrote:
>>
>>> Fixing sparse warnings incorrect type assignment.
>>>
>>> Signed-off-by: Karim Eshapa
>>> ---
>>> drivers/staging/wlan
On Sun, May 7, 2017 at 9:20 AM, Geert Uytterhoeven wrote:
> Hi Dan,
>
> On Sat, May 6, 2017 at 4:21 AM, Linux Kernel Mailing List
> wrote:
>> Web:
>> https://git.kernel.org/torvalds/c/b0686260fecaa924d8eff2ace94bee70506bc308
>> Commit: b0686260fecaa924d8eff2ace94bee70506bc308
>> Paren
On 2017-05-08 00:02, Richard Weinberger wrote:
Ahh, *very* recent is the keyword then. ;)
I was a bit confused since in your paper the overhead is less than 1%.
Yes, only for very recent platforms (Skylake). While working on the
paper we were surprised that we found overheads that small.
Wh
A kernel cloned from
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git and
built with the latest RTL8723BS driver included (CONFIG_RTL8723BS=m)
fails when booting on an Intel Atom device with the RTL8723BS wifi
chipset due to an error in
drivers/staging/rtl8723bs/os_dep/ioctl_cf
On 04/05/17 21:38, Enric Balletbo i Serra wrote:
> Do not include the paragraph about writing to the Free Software
> Foundation's mailing address from the sample GPL notice. The FSF
> has changed addresses in the past, and may do so again. Linux
> already includes a copy of the GPL.
>
> Signed-off
On 02/05/17 07:46, Corentin Labbe wrote:
> Hello
Beyond picking up in a quick code inspection I'm afraid. Quentin
can you take a look at this? I guess its triggering as a result
of the call to read the temperature failing, but beyond that - no
idea!
Jonathan
>
> When inserting sun4i_gpadc_iio
On 04/05/17 21:38, Enric Balletbo i Serra wrote:
> Fix the output from checkpatch.pl like:
> Symbolic permissions 'S_IRUSR' are not preferred.
> Consider using octal permissions '0400'.
>
> Also fix the alignment DEVICE_ATTR variables as should match open
> parenthesis.
>
> Signed-off-by: Enr
On Sat, Apr 08, 2017 at 08:32:36PM +0200, Guillaume Brogi wrote:
> On Sat, Apr 08, 2017 at 12:31:25PM +0200, Greg Kroah-Hartman wrote:
> > On Sun, Mar 26, 2017 at 12:24:14AM +0100, Guillaume Brogi wrote:
> > >
> > > This patch fixes the following sparse warnings:
> > > drivers/staging/rtl8192u/r81
* Dan Williams wrote:
> On Sat, May 6, 2017 at 2:46 AM, Ingo Molnar wrote:
> >
> > * Dan Williams wrote:
> >
> >> On Fri, May 5, 2017 at 3:44 PM, Kani, Toshimitsu
> >> wrote:
> >> > On Fri, 2017-05-05 at 15:25 -0700, Dan Williams wrote:
> >> >> On Fri, May 5, 2017 at 1:39 PM, Kani, Toshimits
On Fri, May 05, 2017 at 12:43:40PM -0700, Andi Kleen wrote:
> On Fri, May 05, 2017 at 09:57:54AM +0200, Jiri Olsa wrote:
> > On Thu, May 04, 2017 at 03:26:20PM -0700, Andi Kleen wrote:
> > > On Tue, May 02, 2017 at 08:41:47AM +0200, Jiri Olsa wrote:
> > > > On Mon, May 01, 2017 at 12:47:46PM -0700,
Hi,
Le vendredi 05 mai 2017 à 10:04 +0200, Pali Rohár a écrit :
> On Sunday 30 April 2017 20:27:26 Paul Kocialkowski wrote:
> > This introduces a dedicated status change work to look for power
> > status change. It is triggered by external power change notifications
> > and periodically retries de
From: Ville Syrjälä
Add a new Kconfig option to enable/disable the extra warnings
from the vblank evade code. For now we'll keep the warning
about an actually missed vblank always enabled as that can have
an actual user visible impact. But if we miss the deadline
othrwise there's no real need to
On 04/05/17 21:10, Julia Lawall wrote:
> Drop static on a local variable, when the variable is initialized before
> use, on every possible execution path through the function. The static has
> no benefit, and dropping it reduces the code size.
>
> The semantic patch that fixes this problem is as
-to-iommu_get-put_resv_regions/20170507-153552
base: https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next
config: x86_64-kexec (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH
From: Markus Elfring
Date: Sun, 7 May 2017 21:24:51 +0200
A string which did not contain a data format specification should be put
into a sequence. Thus use the corresponding function "seq_puts".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
arch/
Hi Dan,
On Sat, May 6, 2017 at 4:21 AM, Linux Kernel Mailing List
wrote:
> Web:
> https://git.kernel.org/torvalds/c/b0686260fecaa924d8eff2ace94bee70506bc308
> Commit: b0686260fecaa924d8eff2ace94bee70506bc308
> Parent: d8f07aee3f2fd959878bf614d4e984900018eb9e
> Refname:refs/hea
Hi Oza,
[auto build test ERROR on v4.9-rc8]
[cannot apply to glikely/devicetree/next iommu/next next-20170505]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Oza-Pawandeep/OF-PCI-address-PCI-inb
Le lundi 01 mai 2017 à 12:55 +0200, Sebastian Reichel a écrit :
> Hi,
>
> On Sun, Apr 30, 2017 at 08:27:24PM +0200, Paul Kocialkowski wrote:
> > This uses the managed devices resources version of the
> > power_supply_register_no_ws function to register the power supply.
> >
> > Signed-off-by: Pau
From: Markus Elfring
Date: Sun, 7 May 2017 20:15:41 +0200
Strings which did not contain data format specifications should be put
into a sequence. Thus use the corresponding function "seq_puts" more.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
ar
On Sun, May 7, 2017 at 5:14 AM, Greg KH wrote:
> On Thu, May 04, 2017 at 10:06:51PM +0200, Riccardo wrote:
>> Fixed a brace coding style issue, found via checkpatch.
>>
>> Signed-off-by: Riccardo Marotti
>
> This name doesn't match the name on the From: line of your email :(
>
Sorry about that!
Create an ops variable to store tcon->ses->server->ops and cache
indirections and reduce code size a trivial bit.
$ size fs/cifs/cifsacl.o*
textdata bss dec hex filename
5338 136 85482156a fs/cifs/cifsacl.o.new
5371 136 85515158b fs/cifs
On Sun 2017-05-07 10:01:21, sanshan zhang wrote:
> From: zhang sanshan
>
> system can't enter suspend when enable led heartbeat.
>
> pm will call heartbeat_pm_notifier when suspend.
> system will prepare led states, and led_trigger_unregister
> will use led_trigger_set to set trigger.
> kobject_
On Thu, May 4, 2017 at 12:31 PM, Christian König
wrote:
> From: Christian König
>
> Just the defines and helper functions to read the possible sizes of a BAR and
> update it's size.
>
> See
> https://pcisig.com/sites/default/files/specification_documents/ECN_Resizable-BAR_24Apr2008.pdf
> and PCI
On Sat, May 6, 2017 at 5:06 AM, Borislav Petkov wrote:
> On Fri, May 05, 2017 at 09:42:14PM +0100, Matt Fleming wrote:
>> (Including the folks from SGI since this was hit on a UV system)
>
> Wasn't there a BIOS fix supplied at some point which obviated the need
> to boot with efi=old_map on SGI bo
From: Wanpeng Li
This can be triggered by hot-unplug one cpu.
==
[ INFO: possible circular locking dependency detected ]
4.11.0+ #17 Not tainted
---
step_after_susp/2640 is trying to acqui
I work for Arm on maintaining the TrustZone CryptoCell driver.
Signed-off-by: Gilad Ben-Yossef
---
The original patch went out with the patch set introducing the ccree
driver but somehow failed to end up in Linus's tree, maybe because
I omitted the "staging:" prefix.
---
MAINTAINERS | 8 ++
From: Anton Blanchard
The ir-spi driver has 2 issues which prevents it from working with
lirc:
1. The ir-spi driver uses 16 bits of SPI data to create one cycle of
the waveform. As such our SPI clock needs to be 16x faster than the
carrier frequency.
The driver is inconsistent in how it current
On 05/05/17 01:38, Brian Masney wrote:
> This patch renames the tsl2x7x_core.c file to tsl2x7x.c so that the
> naming convention is consistent with other IIO light drivers outside
> of staging.
>
> Signed-off-by: Brian Masney
Applied to the togreg branch of iio.git and pushed out as testing
for t
The only adc button connected to adc input is recovery button.
Signed-off-by: Randy Li
---
arch/arm/boot/dts/rk3288-firefly-reload.dts | 17 +
arch/arm/boot/dts/rk3288-firefly.dtsi | 13 +
2 files changed, 30 insertions(+)
diff --git a/arch/arm/boot/dts/rk3288-
vThe following changes since commit 39da7c509acff13fc8cb12ec1bb20337c988ed36:
Linux 4.11-rc6 (2017-04-09 09:49:44 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4.git
tags/ext4_for_linus
for you to fetch changes up to 17f423b5160767a8e
The following changes since commit 39da7c509acff13fc8cb12ec1bb20337c988ed36:
Linux 4.11-rc6 (2017-04-09 09:49:44 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/tytso/fscrypt.git
tags/fscrypt_for_linus
for you to fetch changes up to 6f9d696f016f
2017-05-04 10:14 GMT+09:00 Masahiro Yamada :
> Since commit 5399eb9b3908 ("dtbsinstall: don't move target directory
> out of the way"), the target __dtbs_install_prep is invoked just for
> creating the install directory, but all the necessary directories
> are automatically created by:
>cmd_dtb
On Fri, Apr 28, 2017 at 11:56 AM, Varadarajan Narayanan
wrote:
> Add initial pinctrl driver to support pin configuration with
> pinctrl framework for ipq8074.
>
> Signed-off-by: Manoharan Vijaya Raghavan
> Signed-off-by: Varadarajan Narayanan
Bjorn Andersson is the maintainer of the qcom pinct
Fixed a brace coding style issue, found via checkpatch.
Signed-off-by: Riccardo Marotti
---
drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
b/drivers/staging/rtl8192
On Sun, May 7, 2017 at 5:38 AM, Andy Lutomirski wrote:
>
> This series goes a long way toward cleaning up the mess. With all the
> patches applied, there is a single function that contains the meat of
> the code to flush the TLB on a given CPU, and all the tlb flushing
> APIs call it for both loc
On Sat, May 6, 2017 at 10:02 PM, Linus Torvalds
wrote:
> On Tue, May 2, 2017 at 8:44 PM, Dave Airlie wrote:
>> i915:
>> vblank evasion improvements
>
> These may be "improvements", but they end up being very noisy.
>
> I geta fair amount of messages like
>
> [drm] Atomic update on pipe (A) took
Fixes a sparse warning:
drivers/staging/media/atomisp/platform/intel-mid/intel_mid_pcihelpers.c:35:5:
warning: symbol 'qos' was not declared. Should it be static?
Signed-off-by: Valentin Vidic
---
drivers/staging/media/atomisp/platform/intel-mid/intel_mid_pcihelpers.c | 1 -
1 file changed, 1
On Sun, May 7, 2017 at 7:46 PM, Jens Axboe wrote:
> On 05/07/2017 11:12 AM, ville.syrj...@linux.intel.com wrote:
>> From: Ville Syrjälä
>>
>> Add a new Kconfig option to enable/disable the extra warnings
>> from the vblank evade code. For now we'll keep the warning
>> about an actually missed vbl
Hi Tomohiro,
On Sun, May 07, 2017 at 03:43:36AM +0900, Tomohiro Yoshidomi wrote:
> Mr.Torokhov
> Mr.Herrmann
>
> Sorry,
> I sent Playstation 1/2 joypads patch again.
> Mr.Torokhov's Reviewed-by added.
Please do not add "Reviewed-by"s unless explicitly given.
I took the liberty to adjust a coupl
On Fri, May 05, 2017 at 08:11:38AM +0200, Ingo Molnar wrote:
> * alexander.le...@verizon.com wrote:
> Could you please include all the patches as part of the pull request, so that
> individual patches can be discussed?
Sure, I'll resend.
> In particular I'm wondering about:
>
> > tools/inclu
On 05/07/2017 08:47 AM, Riccardo Marotti wrote:
> From: Riccardo
>
> Fixed a brace coding style issue, found via checkpatch.
>
> Signed-off-by: Riccardo Marotti
> ---
> drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --g
On Wed, May 3, 2017 at 7:11 PM, Darren Hart wrote:
> On Wed, May 03, 2017 at 05:42:02PM +0300, Andy Shevchenko wrote:
>> On Wed, May 3, 2017 at 5:36 PM, Ritesh Raj Sarraf wrote:
>> > Resending again, as Google servers are behaving weird lately.
>> >
>> > On Sun, 2017-04-30 at 15:54 +0300, Andy Sh
From: Markus Elfring
Date: Sun, 7 May 2017 19:00:09 +0200
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (4):
Combine two function calls into one in show_cacheinfo()
Use seq_putc() in show_cpu_summary()
Replace six seq_printf() calls by se
On Mon, 2017-05-08 at 00:45 +0530, Jaya Durga wrote:
> Fixed coding style issue
Please list the various renames you are doing.
BeaconPeriod-> beacon_period
DSConfig-> ds_config
ATIMWindow -> atim_window
etc...
And it's generally better to do these while avoiding
changing line b
Fix sparse warning "warning: Variable length array is used." by using
kmalloc_array to allocate the required amount of memory instead and
kfree to deallocate memory after use.
Signed-off-by: Guru Das Srinagesh
---
v2:
- Added missing check for NULL return value of kmalloc_array()
drivers/st
Hi Nicolas
2017-03-27 21:20 GMT+09:00 Nicolas Dichtel :
> Regularly, when a new header is created in include/uapi/, the developer
> forgets to add it in the corresponding Kbuild file. This error is usually
> detected after the release is out.
>
> In fact, all headers under uapi directories should
On Thu, May 4, 2017 at 12:31 PM, Christian König
wrote:
> From: Christian König
>
> This allows device drivers to request resizing their BARs.
>
> The function only tries to reprogram the windows of the bridge directly above
> the requesting device and only the BAR of the same type (usually mem,
On Fri, Apr 28, 2017 at 4:53 PM, Andy Shevchenko
wrote:
> Linus, for me it looks like better to revert that change, until we
> will have clear picture why existing configuration parameters can't
> work.
Yeah I'll revert the binding for fixes.
Yours,
Linus Walleij
Le 05/05/17 à 22:29, Christophe JAILLET a écrit :
> If 'devm_kzalloc' fails, a NULL pointer will be dereferenced.
> Return -ENOMEM instead, as done for some other memory allocation just a
> few lines above.
>
> Fixes: 98cd1552ea27 ("net: dsa: Mock-up driver")
>
> Signed-off-by: Christophe JAILLET
On Sun, 2017-05-07 at 02:23 -0700, Guru Das Srinagesh wrote:
> Fix sparse warning "warning: Variable length array is used." by using
> kmalloc_array to allocate the required amount of memory instead and
> kfree to deallocate memory after use.
[]
> diff --git a/drivers/staging/lustre/lustre/llite/xa
From: Markus Elfring
Date: Sun, 7 May 2017 10:50:35 +0200
A bit of data was put into a sequence by two separate function calls
at two places. Print the same data by single function calls instead.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drive
Hi Oleg,
On Sat, May 6, 2017 at 9:42 PM, Oleg Nesterov wrote:
> On 05/05, Geert Uytterhoeven wrote:
>> I'm using signal_wake_up() to abort a task blocked on
>> wait_for_completion_interruptible(), cfr. sh_msiof_slave_abort() in
>> "spi: sh-msiof: Add slave mode support"
>> (http://www.spinics.net
From: Markus Elfring
Date: Sun, 7 May 2017 10:57:40 +0200
A single character (line break) should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/thermal/tegra/soctherm.c
Fix all PARENTHESIS_ALIGNMENT type errors reported by checkpatch
in rtxt.c
Signed-off-by: Salvatore Benedetto
---
drivers/staging/vt6656/rxtx.c | 70 +++
1 file changed, 37 insertions(+), 33 deletions(-)
diff --git a/drivers/staging/vt6656/rxtx.c b/driver
From: Markus Elfring
Date: Sun, 7 May 2017 11:08:42 +0200
A bit of data was put into a sequence by two separate function calls.
Print the same data by a single function call instead.
Signed-off-by: Markus Elfring
---
drivers/thermal/tegra/soctherm.c | 4 +---
1 file changed, 1 insertion(+), 3
From: Markus Elfring
Date: Sun, 7 May 2017 11:40:04 +0200
Three update suggestions were taken into account
from static source code analysis.
Markus Elfring (3):
Combine two function calls into one
Use seq_putc() in regs_show()
Combine two seq_printf() calls into one
drivers/thermal/tegra
Le 05/06/17 à 04:54, Anup Patel a écrit :
> The Broadcom Stingray SoC is a new member in Broadcom iProc
> SoC family.
>
> This patch adds initial DTS files for Broadcom Stingray SoC
> and two of its reference boards (bcm958742k and bcm958742t).
>
> We have lot of reference boards and large number
One return case of `__collapse_huge_page_swapin()` does not invoke
tracepoint while every other return case does. This commit adds a
tracepoint invocation for the case.
Signed-off-by: SeongJae Park
---
mm/khugepaged.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/mm/khu
From: Markus Elfring
Date: Sun, 7 May 2017 14:58:55 +0200
A bit of data was put into a sequence by two separate function calls.
Print the same data by a single function call instead.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
security/integrity
Closing braces should match the first characters of the opening.
Signed-off-by: Geert Uytterhoeven
---
drivers/staging/ccree/ssi_hash.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/ccree/ssi_hash.c b/drivers/staging/ccree/ssi_
With gcc 4.1.2:
drivers/staging/ccree/ssi_hash.c:1990: error: unknown field
‘template_ahash’ specified in initializer
drivers/staging/ccree/ssi_hash.c:1991: error: unknown field ‘init’
specified in initializer
drivers/staging/ccree/ssi_hash.c:1991: warning: missing braces around
ini
With gcc 4.1.2:
drivers/nvme/host/lightnvm.c: In function ‘nvme_nvm_submit_io’:
drivers/nvme/host/lightnvm.c:498: warning: ‘rq’ is used uninitialized in
this function
Indeed, since commit 2e13f33a2464fc3a ("lightnvm: create cmd before
allocating request"), the request is passed to nvme_n
From: Markus Elfring
Date: Sun, 7 May 2017 15:35:15 +0200
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (3):
Use seq_putc() in ima_ascii_measurements_show()
Combine two function calls into one in ima_policy_show()
Replace nine seq_puts()
From: Markus Elfring
Date: Sun, 7 May 2017 15:23:44 +0200
Nine single characters should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
security/integrity/ima/ima_policy.c | 18
From: Markus Elfring
Date: Sun, 7 May 2017 14:45:01 +0200
Two single characters should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
security/integrity/ima/ima_fs.c | 4 ++--
* Andy Lutomirski wrote:
> As I've been working on polishing my PCID code, a major problem I've
> encountered is that there are too many x86 TLB flushing code paths and
> that they have too many inconsequential differences. The result was
> that earlier versions of the PCID code were a colossal
On Thu, May 4, 2017 at 12:31 PM, Christian König
wrote:
> From: Christian König
>
> Most BIOS don't enable this because of compatibility reasons.
>
> Manually enable a 64bit BAR of 64GB size so that we have
> enough room for PCI devices.
>
> v2: style cleanups, increase size, add resource name, s
On Sun, 07 May 2017, Pavel Machek wrote:
> On Thu 2017-01-19 12:21:32, Adam Goode wrote:
> > This allows the control of the red status LED, which is the dot of the "i"
> > in the word "ThinkPad" on the outside cover of newer models.
> >
> > In the manual, both this LED and the power LED are referr
On Thu, May 4, 2017 at 12:31 PM, Christian König
wrote:
> Hi everyone,
>
> this is the fifth incarnation of this set of patches. It enables device
> drivers to resize and most likely also relocate the PCI BAR of devices
> they manage to allow the CPU to access all of the device local memory at onc
Fix sparse warning "warning: Variable length array is used." by using
kmalloc_array to allocate the required amount of memory instead and
kfree to deallocate memory after use.
Signed-off-by: Guru Das Srinagesh
---
v3:
- Fixed checkpatch warning: Comparison to NULL could be written "!fullname"
On Sun, May 07, 2017 at 04:24:51PM -0700, Joe Perches wrote:
> On Sun, 2017-05-07 at 02:23 -0700, Guru Das Srinagesh wrote:
> > Fix sparse warning "warning: Variable length array is used." by using
> > kmalloc_array to allocate the required amount of memory instead and
> > kfree to deallocate memor
Each time a slab is deactivated, the page and freelist pointer should be
reset.
This patch just merges these two options into deactivate_slab().
Signed-off-by: Wei Yang
---
mm/slub.c | 21 -
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/mm/slub.c b/mm/slub.c
From: Riccardo
Fixed a brace coding style issue, found via checkpatch.
Signed-off-by: Riccardo Marotti
---
drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
b/driver
On 05/07/2017 05:02 AM, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 May 2017 13:43:50 +0200
>
> A bit of data was put into a sequence by two separate function calls.
> Print the same data by a single function call instead.
>
> Signed-off-by: Markus Elfring
Acked-by: John Joh
From: Colin Ian King
The null check on client->adapter->name is redundant as name is an
array of I2C_NAME_SIZE chars and hence can never be null. We may as
well remove this redundant check.
Detected by CoverityScan, CID#1375918 ("Array compared against 0")
Signed-off-by: Colin Ian King
---
dr
From: Colin Ian King
iscsi_lookup_endpoint can potentially return null and in 9 out of
the 10 calls to this function a null return is checked, so I think
it is pertinent to perform a null check here too and return -EINVAL
as in the other null cases.
Detected by CoverityScan, CID#147282 ("Derefer
From: Colin Ian King
The 2nd check of cnt > 8 is redundant as cnt is already checked
and thresholded to a maximum of 8 a few statements earlier.
Remove this redundant 2nd check.
Detected by CoverityScan, CID#114281 ("Logically dead code")
Signed-off-by: Colin Ian King
---
drivers/media/usb/pv
On 05/07/2017 05:03 AM, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 May 2017 13:50:28 +0200
>
> Two single characters (line breaks) should be put into a sequence.
> Thus use the corresponding function "seq_putc".
>
> This issue was detected by using the Coccinelle software.
>
On 03/05/17 06:24 PM, Gerd Hoffmann wrote:
> Hi,
>
>>> R600+ supports bigendian framebuffer formats, so no byteswapping on
>>> access is needed. Not sure whenever that includes 16bpp formats or
>>> whenever this is limited to the 8 bit-per-color formats [...]
>>
>> It includes 16bpp. Looking at
From: Markus Elfring
Date: Sun, 7 May 2017 16:32:04 +0200
Two single characters (line breaks) should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
arch/powerpc/mm/dump_hashpag
On 4/24/2017 8:47 AM, Jin, Yao wrote:
On 4/23/2017 9:55 PM, Jiri Olsa wrote:
On Thu, Apr 20, 2017 at 08:07:50PM +0800, Jin Yao wrote:
SNIP
+#define X86_BR_TYPE_MAP_MAX16
+
+static int
+common_branch_type(int type)
+{
+int i, mask;
+const int branch_map[X86_BR_TYPE_MAP_MAX] =
> >> Reviewed-by: Andy Shevchenko
> >> Reviewed-by: Koul, Vinod
>
> > A previous version of this has been applied and I don't see any differences
> > in the code changes.
>
> AFAIR only tags (above) had been extended in v3.
Thanks.
Yes, for v3 no change in code, and only tags "Reviewed-by" add
FYI, we noticed the following commit:
commit: 5ecda13711b3bd4a750b5740897bf13d1720de7c ("generic_file_read_iter():
make use of iov_iter_revert()")
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git master
in testcase: ocfs2test
with following parameters:
disk: 1HDD
From: Markus Elfring
Date: Sun, 7 May 2017 13:58:08 +0200
Two update suggestions were taken into account
from static source code analysis.
Markus Elfring (2):
Combine two function calls into one in aa_fs_seq_raw_abi_show()
Use seq_putc() in two functions
security/apparmor/apparmorfs.c | 11
From: Markus Elfring
Date: Sun, 7 May 2017 13:50:28 +0200
Two single characters (line breaks) should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
security/apparmor/apparmorfs
From: Markus Elfring
Date: Sun, 7 May 2017 13:43:50 +0200
A bit of data was put into a sequence by two separate function calls.
Print the same data by a single function call instead.
Signed-off-by: Markus Elfring
---
security/apparmor/apparmorfs.c | 7 +++
1 file changed, 3 insertions(+),
Thanks for explaining, Bhupesh.
BIOS issue of SGI uv1 is still not fixed. There's a quirk for uv1 to
use efi old map:
void __init efi_apply_memmap_quirks(void)
{
...
...
/* UV2+ BIOS has a fix for this issue. UV1 still needs the quirk. */
if (dmi_check_system(sgi
On 05/07/2017 01:14 AM, Wanpeng Li wrote:
> From: Wanpeng Li
>
> This can be triggered by hot-unplug one cpu.
>
> ==
> [ INFO: possible circular locking dependency detected ]
> 4.11.0+ #17 Not tainted
> --
On 05/07/2017 11:56 AM, Daniel Vetter wrote:
> On Sun, May 7, 2017 at 7:46 PM, Jens Axboe wrote:
>> On 05/07/2017 11:12 AM, ville.syrj...@linux.intel.com wrote:
>>> From: Ville Syrjälä
>>>
>>> Add a new Kconfig option to enable/disable the extra warnings
>>> from the vblank evade code. For now we
On 05/07/2017 08:14 AM, Geert Uytterhoeven wrote:
> With gcc 4.1.2:
Wow, that's ancient.
> drivers/nvme/host/lightnvm.c: In function ‘nvme_nvm_submit_io’:
> drivers/nvme/host/lightnvm.c:498: warning: ‘rq’ is used uninitialized in
> this function
>
> Indeed, since commit 2e13f33a2464fc3a
On Sat, May 6, 2017 at 1:28 AM, Luis Henriques wrote:
> Currently the ceph client doesn't respect the rlimit in fallocate. This
> means that a user can allocate a file with size > RLIMIT_FSIZE. This
> patch adds the call to inode_newsize_ok() to verify filesystem limits and
> ulimits. This shou
Hi,
Could not find the proper person hence putting it in general mailing
list, A file named "REPORTING-BUGS" is missing in the current release
of the linux kernel i.e. v4.11.
I found this is when using debian tools to compile a latest release
using command,
"fakeroot make-kpkg --initrd --revisio
On 05/04/17 at 09:25am, Thomas Garnier wrote:
> > I think this needs a "Fixes:" tag and Cc: .
Sorry for late response, should I resend with them?
>
> Agreed.
>
> >
> > Other than that:
> >
> > Reviewed-by: Dan Williams
>
> Thanks again!
>
> Reviewed-by: Thomas Garnier
> --
> Thomas
perf record -b -g
perf report --branch-history
This merges the LBRs with the callgraphs.
However it would be nice if it also works without callgraphs (-g)
set in perf record, so that only the LBRs are displayed.
But currently perf report errors in this case. For example,
perf record -b
perf re
On 05/07/17 19:17, Anil Nair wrote:
> Hi,
>
> Could not find the proper person hence putting it in general mailing
> list, A file named "REPORTING-BUGS" is missing in the current release
> of the linux kernel i.e. v4.11.
>
> I found this is when using debian tools to compile a latest release
> us
On Tue, May 02, 2017 at 08:55:07PM +0200, Michal Hocko wrote:
> On Tue 02-05-17 16:59:30, Laurent Dufour wrote:
> > On 28/04/2017 15:48, Michal Hocko wrote:
> [...]
> > > This is getting quite hairy. What is the expected page count of the
> > > hwpoison page?
>
> OK, so from the quick check of the
> * Dan Williams wrote:
>
> > On Sat, May 6, 2017 at 2:46 AM, Ingo Molnar wrote:
> > >
> > > * Dan Williams wrote:
> > >
> > >> On Fri, May 5, 2017 at 3:44 PM, Kani, Toshimitsu
> wrote:
> > >> > On Fri, 2017-05-05 at 15:25 -0700, Dan Williams wrote:
> > >> >> On Fri, May 5, 2017 at 1:39 PM, Ka
On Sat, May 06, 2017 at 01:36:20AM +0200, Borislav Petkov wrote:
> On Fri, May 05, 2017 at 09:42:14PM +0100, Matt Fleming wrote:
> > (Including the folks from SGI since this was hit on a UV system)
>
> Wasn't there a BIOS fix supplied at some point which obviated the need
> to boot with efi=old_ma
On Fri, May 05, 2017 at 08:05:09PM -0700, Nick Desaulniers wrote:
> Clang does not support this machine dependent option.
>
> Older versions of GCC (pre 3.0) may not support this option, added in
> 2000, but it's unlikely they can still compile the kernel.
>
> Signed-off-by: Nick Desaulniers
> -
1 - 100 of 254 matches
Mail list logo
