From: Arnaldo Carvalho de Melo
Hi Ingo,
Please consider pulling,
- Arnaldo
The following changes since commit a7636d9ecfa3ab7800a7c04c1f89378229eff609:
kprobes: Optimize hot path by using percpu counter to collect 'nhit'
statistics (2016-02-09 11:08:58 +0100)
are available in the
From: Arnaldo Carvalho de Melo
To follow convention used in other tools/perf/ areas. Also remove the
need to check if it is NULL before calling the destructor, again, to
follow convention that goes back to free().
Cc: Alexei Starovoitov
Cc: He Kuang
cc: Jiri Olsa
Cc: Masami Hiramatsu
Cc: Nam
From: Arnaldo Carvalho de Melo
Cc: Adrian Hunter
Cc: Jiri Olsa
Cc: Namhyung Kim
Cc: Wang Nan
Link: http://lkml.kernel.org/n/tip-4j67nvlfwbnkg85b969ew...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
tools/perf/util/sort.c | 10 +-
1 file changed, 9 insertions(+), 1 dele
From: Wang Nan
A new patch in libbabeltrace [1] reveals a object leak problem in
'perf data' CTF support: perf code never releases the event_class
which is allocated in add_event() and stored in evsel's private field.
If libbabeltrace has the above patch applied, leaking event_class
prevents the
The patch
regulator: vexpress: rename vexpress regulator implementation
has been applied to the regulator tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the nex
On Mon, Feb 15, 2016 at 06:12:01PM +0100, Uwe Kleine-König wrote:
> Hello Greg,
>
> On Mon, Feb 15, 2016 at 08:49:37AM -0800, Greg Kroah-Hartman wrote:
> > On Mon, Feb 15, 2016 at 05:27:53PM +0100, Uwe Kleine-König wrote:
> > > Greg, can you drop this patch, or do you need a proper changelog for a
From: Arnaldo Carvalho de Melo
fixing the following problems, for instance, on RHEL6.7:
CC /tmp/build/perf/tests/bp_signal.o
cc1: warnings being treated as errors
tests/bp_signal.c: In function ‘__event’:
tests/bp_signal.c:106: error: declaration of ‘signal’ shadows a global
dec
From: Wang Nan
Fixing a leak, since code calling parse_events__free_terms() expect it
to free the list_head too.
Signed-off-by: Wang Nan
Acked-by: Jiri Olsa
Cc: Alexei Starovoitov
Cc: He Kuang
Cc: Masami Hiramatsu
Cc: Namhyung Kim
Cc: Zefan Li
Cc: pi3or...@163.com
[ Spun off from another
From: Wang Nan
Before this patch, if a sample is triggered inside a module not in
/lib/modules/`uname -r`/, even if the module is in buildid-cache, 'perf
report' will still be unable to find the correct symbol. For example:
# rm -rf ~/.debug/
# perf buildid-cache -a ./mymodule.ko
# perf p
From: Taeung Song
The '--system' option means $(sysconfdir)/perfconfig and '--user' means
$HOME/.perfconfig. If none is used, both system and user config file are
read. E.g.:
# perf config [] [options]
With an specific config file:
# perf config --user | --system
or both user
From: Wang Nan
We were just freeing them, better unlink and init its nodes to catch
bugs faster if we keep dangling references to them.
Signed-off-by: Wang Nan
Acked-by: Jiri Olsa
Cc: Alexei Starovoitov
Cc: He Kuang
Cc: Masami Hiramatsu
Cc: Namhyung Kim
Cc: Zefan Li
Cc: pi3or...@163.com
[
From: Arnaldo Carvalho de Melo
We were doing column alignment in the format function for each cell,
returning a string padded with spaces so that when the next column is
printed the cursor is at its column alignment.
This ends up needlessly printing trailing spaces, do it at the format
iterator,
Hello, Michal.
On Mon, Feb 15, 2016 at 09:54:38PM +0100, Michal Hocko wrote:
> I am not sure I understand. Does that mean that a node with no memory
> would have its WQ specific pool? I might be missing something but I
Yeah, I think it should.
> thought that cpu_to_node will return NUMA_NO_NODE
From: Taeung Song
To print syscall names, the audit-libs-python package is required.. If
not installed, it prints this error string:
# perf script syscall-counts
Install the audit-libs-python package to get syscall names.
But the package name is different in Ubuntu, mention that in the
From: Zubair Lutfullah Kakakhel
To compile for little-endian systems, you need to pass -EL to CC and LD.
EXTRA_CFLAGS works to pass -EL to CC.
Add EXTRA_LDFLAGS to pass -EL to LD.
Signed-off-by: Zubair Lutfullah Kakakhel
Cc: Peter Zijlstra
Link:
http://lkml.kernel.org/r/1455024818-15842-1-gi
2016-02-12 16:42+0100, Vitaly Kuznetsov:
> We have 3 functions dealing with messages and they all implement
> the same logic to finalize reads, move it to vmbus_signal_eom().
>
> Suggested-by: Radim Krcmar
Reviewed-by: Radim Krčmář
On Mon, Feb 15, 2016 at 06:31:29PM +, Sudeep Holla wrote:
> Understood. I knew lkml.org is broken if we tried very old links but
> expected to work at-least for couple of month old patches. Anyways
> hopefully [1] still works.
It's not old links in particular AFAICT, just general instability.
2016-02-12 16:42+0100, Vitaly Kuznetsov:
> wait_for_completion() may sleep, it enables interrupts and this
> is something we really want to avoid on crashes because interrupt
> handlers can cause other crashes. Switch to the recently introduced
> vmbus_wait_for_unload() doing busy wait instead.
(T
Declare the number of DMA requestor lines per platform :
- for pxa25x: 40 requestor lines
- for pxa27x: 75 requestor lines
- for pxa3xx: 100 requestor lines
This information will be used to activate the DMA flow control or not.
Signed-off-by: Robert Jarzmik
---
Since v1: rename of property to
The current number of requestor lines is limited to 31. This was an
error of a previous commit, as this number is platform dependent, and is
actually :
- for pxa25x: 40 requestor lines
- for pxa27x: 75 requestor lines
- for pxa3xx: 100 requestor lines
The previous testing did not reveal the fau
For pxa based platforms, the number of requestor lines should be
specified, so that the driver can check if the flow control should be
activated (when a requestor line is asked for) or not.
Signed-off-by: Robert Jarzmik
---
Since v1: rename of property to #dma-requests
modify default va
The DMA chip has a fixed number of requestor lines used for flow
control. This number is platform dependent. The pxa_dma dma driver will
use this value to activate or not the flow control.
There won't be any impact on mmp_pdma driver.
Signed-off-by: Robert Jarzmik
---
include/linux/platform_dat
On Mon, Feb 15, 2016 at 06:31:23PM +0800, Chen-Yu Tsai wrote:
> The X-Powers AXP809 PMIC has a similar set of regulators as the AXP221,
> though a few LDOs were removed, and a new switch output added. Like the
> AXP221, AXP809 also has DC1SW and DC5LDO, which are internally chained
> to DCDC1 and D
On Mon 15-02-16 13:21:25, Tejun Heo wrote:
> Hello, Michal.
>
> On Mon, Feb 15, 2016 at 06:33:46PM +0100, Michal Hocko wrote:
> > On Wed 10-02-16 10:55:03, Tejun Heo wrote:
> > [...]
> > > --- a/kernel/workqueue.c
> > > +++ b/kernel/workqueue.c
> > > @@ -570,6 +570,16 @@ static struct pool_workque
On Mon, Feb 15, 2016 at 6:01 PM, Clemens Gruber
wrote:
> +static int marvell_config_init(struct phy_device *phydev)
> +{
> + int err;
> +
> + /* Set registers from marvell,reg-init DT property */
> + err = marvell_of_reg_init(phydev);
> + if (err < 0)
> + ret
2016-02-12 16:42+0100, Vitaly Kuznetsov:
> We must handle HVMSG_TIMER_EXPIRED messages in the interrupt context
> and we offload all the rest to vmbus_on_msg_dpc() tasklet. This functions
> loops to see if there are new messages pending. In case we'll ever see
> HVMSG_TIMER_EXPIRED message there we
On Sat 06-02-16 22:22:20, Tetsuo Handa wrote:
> Michal Hocko wrote:
> > There is one notable exception to this, though, if the OOM victim was
> > in the process of coredumping the result would be incomplete. This is
> > considered a reasonable constrain because the overall system health is
> > more
On Sat 06-02-16 20:23:43, Tetsuo Handa wrote:
> Michal Hocko wrote:
[...]
> By always waking the OOM reaper up, we can delegate the duty of unlocking
> the OOM killer (by clearing TIF_MEMDIE or some other means) to the OOM
> reaper because the OOM reaper is tracking all TIF_MEMDIE tasks.
And again
On Mon, Feb 15, 2016 at 8:23 PM, Russell King - ARM Linux
wrote:
> On Mon, Feb 15, 2016 at 07:03:33PM +, Marc Zyngier wrote:
>> On 15/02/16 18:54, Rafael J. Wysocki wrote:
>> > That would explain it, thanks.
>> >
>> > So it looks like we should always use irq_work_queue() on UP even if
>> > CO
Andrew,
this can be either folded into 3/5 patch or go as a standalone one. I
would be inclined to have it standalone for the record (the description
should be pretty clear about the intention) and because the issue is
highly unlikely. OOM during the PM freezer doesn't happen in 99% cases.
On Sat
On Mon, Feb 15, 2016 at 9:09 PM, Guenter Roeck wrote:
> On 02/15/2016 11:58 AM, Tony Lindgren wrote:
>>
>> * Guenter Roeck [160215 11:41]:
>>>
>>> On 02/15/2016 11:01 AM, Tony Lindgren wrote:
>>>>
>>>>
>>>> https://kernelci.or
On Monday 15 February 2016 21:56:42 Viresh Kumar wrote:
> We are currently required to do two checks for regulator pointer:
> IS_ERR() and IS_NULL().
>
> And multiple instances are reported, about both of these not being used
> consistently and so resulting in crashes.
>
> Fix that by initializin
On Mon, Feb 15, 2016 at 8:58 PM, Tony Lindgren wrote:
> * Guenter Roeck [160215 11:41]:
>> On 02/15/2016 11:01 AM, Tony Lindgren wrote:
>> >
>> >https://kernelci.org/boot/all/job/next/kernel/next-20160215/
>> >
>> >The SMP ones seem to fail with some
Good morning lkml
http://www.drewlin.me/pitch.php?cause=1va2u3y80rqgv
Badari Pulavarty
The patch
regulator: hi655x: Document for hi655x regulator
has been applied to the regulator tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) a
The patch
regulator: hi655x: enable regulator for hi655x PMIC
has been applied to the regulator tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours
For pxa based platforms, the number of requestor lines should be
specified, so that the driver can check if the flow control should be
activated (when a requestor line is asked for) or not.
Signed-off-by: Robert Jarzmik
---
Since v1: rename of property to #dma-requests
modify default va
diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
index 4571ef1..b8ff6a3 100644
--- a/include/linux/blkdev.h
+++ b/include/linux/blkdev.h
@@ -1388,7 +1388,7 @@ static inline bool bvec_gap_to_prev(struct
request_queue *q,
static inline bool bio_will_gap(struct request_queue *q, stru
On 2016-02-14 Baoquan He wrote:
> On 02/13/16 at 08:57pm, Lasse Collin wrote:
> > The long comment in arch/x86/boot/compressed/misc.c explains the
> > need for the offset for gzip/Deflate. A similar comment in
> > lib/decompress_unxz.c explains it for XZ/LZMA2.
>
> Thank you so much, Lasse. You
The patch
regmap: irq: add devm apis for regmap_{add,del}_irq_chip
has been applied to the regmap tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regmap.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours)
On Mon, 15 Feb 2016 21:15:39 +0100
Arnd Bergmann wrote:
> Do you also want to add a 'Cc: sta...@vger.kernel.org' tag?
>
I currently have this in the Tags:
Acked-by: Nicolas Pitre
Fixes: ab3c9c686e22 ("branch tracer, intel-iommu: fix build with
CONFIG_BRANCH_TRACER=y")
Cc: sta...@
The patch
regulator: axp20x: Use of_match name as default regulator name
has been applied to the regulator tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the ne
On Monday 15 February 2016 17:01:09 Lars-Peter Clausen wrote:
> On 02/15/2016 04:46 PM, Arnd Bergmann wrote:
> > +static inline __deprecated int irq_to_gpio(unsigned int irq)
> > +{
> > + /* this has clearly not worked for a long time */
> > + return -EINVAL;
> > +}
> > +
> > #define IRQ_T
On Monday 15 February 2016 12:47:46 Steven Rostedt wrote:
> On Fri, 12 Feb 2016 22:26:42 +0100
> Arnd Bergmann wrote:
>
> > In my randconfig tests, I came across a bug that involves several
> > components:
> >
> >
>
> [..]
>
> >
> > This patch avoids the problem by changing __trace_if() to ch
On Sun 07-02-16 00:33:38, Tetsuo Handa wrote:
> Michal Hocko wrote:
> > On Sat 06-02-16 14:54:24, Tetsuo Handa wrote:
> > > Michal Hocko wrote:
> > > > > But if we consider non system-wide OOM events, it is not very
> > > > > unlikely to hit
> > > > > this race. This queue is useful for situations
On 02/15/2016 11:58 AM, Tony Lindgren wrote:
* Guenter Roeck [160215 11:41]:
On 02/15/2016 11:01 AM, Tony Lindgren wrote:
https://kernelci.org/boot/all/job/next/kernel/next-20160215/
The SMP ones seem to fail with some regulator issues?
There is another problem, introduced with
From: Буди Романто, AreMa Inc
Signed-off-by: Буди Романто, AreMa Inc
---
drivers/media/pci/Kconfig | 2 +-
drivers/media/pci/Makefile | 2 +-
drivers/media/pci/ptx/Kconfig | 21 ++
drivers/media/pci/ptx/Makefile | 8 +
drivers/media/pci/ptx/pt3_pci.c| 509 ++
Cc Kent and Keith.
Follows another version which should be more efficient.
Kent and Keith, I appreciate much if you may give a review on it.
diff --git a/include/linux/bio.h b/include/linux/bio.h
index 56d2db8..ef45fec 100644
--- a/include/linux/bio.h
+++ b/include/linux/bio.h
@@ -278,11 +278,
On Sun 07-02-16 13:09:33, Tetsuo Handa wrote:
[...]
> FYI, I again hit unexpected OOM-killer during genxref on linux-4.5-rc2 source.
> I think current patchset is too fragile to merge.
>
> [ 3101.626995] smbd invoked oom-killer:
> gfp_mask=0x27000c0(GFP_KER
On Mon, Feb 15, 2016 at 07:40:37PM +, Will Deacon wrote:
> On Wed, Feb 03, 2016 at 06:11:56PM +0100, Jan Glauber wrote:
> > The implemented Cortex A57 events are not A57 specific.
> > They are recommended by ARM and can be found on other
> > ARMv8 SOCs like Cavium ThunderX too. Therefore move
>
On Wed, Feb 03, 2016 at 06:12:00PM +0100, Jan Glauber wrote:
> ARMv8.1 increases the PMU event number space. Detect the
> presence of this PMUv3 type and extend the event mask.
>
> The event mask is moved to struct arm_pmu so different event masks
> can exist, depending on the PMU type.
>
> Signe
On Mon, Feb 15, 2016 at 11:36:13AM -0800, Greg Kroah-Hartman wrote:
> On Mon, Feb 15, 2016 at 12:50:41PM -0600, Michael Welling wrote:
> > On Mon, Feb 15, 2016 at 10:15:08AM -0800, Greg Kroah-Hartman wrote:
> > > On Sun, Feb 14, 2016 at 09:08:50PM -0600, Michael Welling wrote:
> > > > On Sun, Feb 1
> > There are two ways to fix the warnings:
> >
> > 1. get rid of the thunks and call the C functions directly; or
>
> No. Not until gcc learns about per-function callibg conventions (so that it
> can
> be marked as not clobbering registers).
It does already for static functions in 5.x (with -fi
Fix the following sparse warnings:
ti-vpe/cal.c:387:26: warning: incorrect type in return expression (different
address spaces)
ti-vpe/cal.c:459:26: warning: incorrect type in return expression (different
address spaces)
ti-vpe/cal.c:503:27: warning: incorrect type in argument 6 (different addre
On Mon, Feb 15, 2016 at 08:56:21AM -0800, Linus Torvalds wrote:
> On Feb 15, 2016 8:31 AM, "Josh Poimboeuf" wrote:
> >
> > So is the goal to optimize for size? If I replace the calls to
> > __preempt_schedule[_notrace]() with real C calls and remove the thunks,
> > it only adds about 2k to vmlinu
For the Marvell 88E1510, marvell_of_reg_init was called too late, in the
config_aneg function.
Since commit 113c74d83eef ("net: phy: turn carrier off on phy attach"),
this lead to the link not coming up at boot anymore, due to the phy
state machine being stuck at waiting for interrupts (off by defa
On Tue, Feb 02, 2016 at 09:40:04PM +0800, Zhouyi Zhou wrote:
> diff --git a/net/netfilter/nf_conntrack_h323_main.c
> b/net/netfilter/nf_conntrack_h323_main.c
> index 9511af0..8d24c4b 100644
> --- a/net/netfilter/nf_conntrack_h323_main.c
> +++ b/net/netfilter/nf_conntrack_h323_main.c
> @@ -110,6 +1
Hi,
On Mon, Feb 15, 2016 at 08:50:48PM +0100, Heinrich Schuchardt wrote:
> On 02/15/2016 08:40 PM, Aaro Koskinen wrote:
> > On Mon, Feb 15, 2016 at 08:21:33PM +0100, Heinrich Schuchardt wrote:
> >> flash-kernel has a database /usr/share/flash-kernel/db/all.db with
> >> entries like:
> >>
> >> Mach
* Guenter Roeck [160215 11:41]:
> On 02/15/2016 11:01 AM, Tony Lindgren wrote:
> >
> >https://kernelci.org/boot/all/job/next/kernel/next-20160215/
> >
> >The SMP ones seem to fail with some regulator issues?
> >
>
> There is another problem, introd
* Guenter Roeck [160215 11:47]:
> On Mon, Feb 15, 2016 at 11:42:27AM -0800, Tony Lindgren wrote:
> > * Rafael J. Wysocki [160215 11:28]:
> > >
> > > Guenter, Tony,
> > >
> > > Below is a patch to try, on top of linux-next.
> >
> > Fixes the issue on UP for me:
> >
> > Tested-by: Tony Lindgren
Dear,
Good day.
May I have your attention?We are a professional manufacturer&exporter of
magnet and magnetic products located in Shenzhen China,with two production
line the annual capacity exceeds 1,000 tons.
Thanks to skilled workers and engineers,some of them having more than 15 year
On Wed, Feb 03, 2016 at 06:11:59PM +0100, Jan Glauber wrote:
> With the long cycle counter bit (LC) disabled the cycle counter is not
> working on ThunderX SOC (ThunderX only implements Aarch64).
> Also, according to documentation LC == 0 is deprecated.
>
> To keep the code simple the patch does n
Hi Ganapathi,
[auto build test WARNING on bluetooth-next/master]
[also build test WARNING on v4.5-rc4 next-20160215]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Amitkumar-Karwar/Bluetooth
On 02/15/2016 08:40 PM, Aaro Koskinen wrote:
> Hi,
>
> On Mon, Feb 15, 2016 at 08:21:33PM +0100, Heinrich Schuchardt wrote:
>> flash-kernel has a database /usr/share/flash-kernel/db/all.db with
>> entries like:
>>
>> Machine: LeMaker Banana Pi
>> Kernel-Flavors: armmp armmp-lpae
>> Boot-Script-Pat
On Mon, Feb 15, 2016 at 11:42:27AM -0800, Tony Lindgren wrote:
> * Rafael J. Wysocki [160215 11:28]:
> >
> > Guenter, Tony,
> >
> > Below is a patch to try, on top of linux-next.
>
> Fixes the issue on UP for me:
>
> Tested-by: Tony Lindgren
>
> > Please let me know if the problem is still a
issue with SMP systems, see below.
Regards,
Tony
8< --
Unable to handle kernel NULL pointer dereference at virtual address 0030
pgd = c0204000
[0030] *pgd=
Internal error: Oops: 5 [#1] SMP ARM
Modules linked in:
CPU: 0 PID: 1 Comm: swapper/0 Not tainted
4.5.0-rc4-n
On Mon, Feb 8, 2016 at 3:29 PM, Arnd Bergmann wrote:
> Clang correctly points out that the section attribute for u300_gpio_confdata
> is in the wrong place:
>
> drivers/pinctrl/pinctrl-coh901.c:130:37: error: '__section__' attribute only
> applies to functions and global variables
>
> This moves
On Wed, Feb 03, 2016 at 06:11:56PM +0100, Jan Glauber wrote:
> The implemented Cortex A57 events are not A57 specific.
> They are recommended by ARM and can be found on other
> ARMv8 SOCs like Cavium ThunderX too. Therefore move
> these events to the common PMUv3 table.
I can't find anything in th
Hi,
On Mon, Feb 15, 2016 at 08:21:33PM +0100, Heinrich Schuchardt wrote:
> flash-kernel has a database /usr/share/flash-kernel/db/all.db with
> entries like:
>
> Machine: LeMaker Banana Pi
> Kernel-Flavors: armmp armmp-lpae
> Boot-Script-Path: /boot/boot.scr
> DTB-Id: sun7i-a20-bananapi.dtb
> U-B
backs' with next-20160215. An example
crash log and bisect results are attached below.
Please let me know if there is anything I can do to help tracking down
the problem.
It looks like we've uncovered some nastiness in the arch ARM code (see below).
[cut]
[1.34] Unable to han
On Mon, Feb 15, 2016 at 12:50:41PM -0600, Michael Welling wrote:
> On Mon, Feb 15, 2016 at 10:15:08AM -0800, Greg Kroah-Hartman wrote:
> > On Sun, Feb 14, 2016 at 09:08:50PM -0600, Michael Welling wrote:
> > > On Sun, Feb 14, 2016 at 03:17:36PM -0800, Greg Kroah-Hartman wrote:
> > > > On Sun, Feb 1
omap3 overo boots crash with
Unable to handle kernel NULL pointer dereference at virtual address 0030
pgd = c0204000
[0030] *pgd=
Internal error: Oops: 17 [#1] SMP ARM
...
[] (regulator_set_voltage) from [] (_set_opp_voltage+0x34/0x90)
[] (_set_opp_voltage) from [] (dev_pm_opp_set_
On Thu, 2016-02-11 at 15:05 +0530, Archit Taneja wrote:
> component_master_add_with_match can fail if the master's bind op doesn't
> go through successfully. In such a scenario, all the components in the
> master's match array have their 'master' pointer set to the given master.
> These pointers ne
On Tue, Feb 02, 2016 at 01:36:45PM -0500, phil.turnb...@oracle.com wrote:
> From: Phil Turnbull
>
> If nlh->nlmsg_len is zero then an infinite loop is triggered because
> 'skb_pull(skb, msglen);' pulls zero bytes.
>
> The calculation in nlmsg_len() underflows if 'nlh->nlmsg_len <
> NLMSG_HDRLEN'
ocki wrote:
> >>>> On Mon, Feb 15, 2016 at 6:05 PM, Guenter Roeck
> >>>> wrote:
> >>>>> Rafael,
> >>>>
> >>>> Hi,
> >>>>
> >>>> Thanks for the report!
> >>>>
> >&g
On Mon, Feb 15, 2016 at 07:03:33PM +, Marc Zyngier wrote:
> On 15/02/16 18:54, Rafael J. Wysocki wrote:
> > That would explain it, thanks.
> >
> > So it looks like we should always use irq_work_queue() on UP even if
> > CONFIG_SMP is set, shouldn't we?
>
> Something like that, yes. CONFIG_SMP
2016-02-12 16:38+0100, Paolo Bonzini:
> On 12/02/2016 14:59, Suravee Suthikulpanit wrote:
>> +case AVIC_INCMP_IPI_ERR_TARGET_NOT_RUN:
>> +kvm_lapic_reg_write(apic, APIC_ICR2, icrh);
>> +kvm_lapic_reg_write(apic, APIC_ICR, icrl);
>
> Wouldn't this cause a double injectio
On Sun, Feb 14, 2016 at 09:50:04PM +0800, Axel Lin wrote:
> ts4800_ic_ops is only referenced in this driver, so make it static.
> In additional, it's never get modified thus also make it const.
>
> Signed-off-by: Axel Lin
Reviewed-by: Damien Riegel
Thanks for catching this.
On 02/15/2016 07:38 PM, Aaro Koskinen wrote:
> Hi,
>
> On Mon, Feb 15, 2016 at 06:52:20PM +0100, Heinrich Schuchardt wrote:
>> On 02/15/2016 12:16 PM, Sergei Shtylyov wrote:
>>> On 2/15/2016 8:26 AM, Heinrich Schuchardt wrote:
Downstream packages like Debian flash-kernel rely on
/proc/de
On Fri, Feb 12, 2016 at 06:00:10PM +0100, Borislav Petkov wrote:
> Something for me to try when I get a chance.
Ok, so I wanted to know what happens in detail. Here's some ftracing
(debug patch at the end).
Now pay attention to this udevadm thing
[3.816977] rcu_pree-7 0d..2 4058241us :
te:
>>>>> Rafael,
>>>>
>>>> Hi,
>>>>
>>>> Thanks for the report!
>>>>
>>>>> I see crashes in various arm qemu tests due to 'cpufreq: governor: Replace
>>>>> timers with utilization update callback
On Thu, Dec 17, 2015 at 5:48 PM, Ricardo Ribalda Delgado
wrote:
> On named GPIOs there is currently no find out their numerical id.
>
> Because of this, a GPIO pin named like:
> /sys/class/gpio/PROG_B
>
> cannot be unexported by an application different than the one that
> exported it.
>
> This p
On Mon, Feb 15, 2016 at 05:46:56PM +, Will Deacon wrote:
> On Thu, Feb 11, 2016 at 06:40:02PM +, Marc Zyngier wrote:
> > With VHE, we place kernel {watch,break}-points at EL2 to get things
> > like kgdb and "perf -e mem:..." working.
> >
> > This requires a bit of repainting in the low-lev
On Mon, Feb 15, 2016 at 07:54:26PM +0100, Rafael J. Wysocki wrote:
> On Mon, Feb 15, 2016 at 7:49 PM, Marc Zyngier wrote:
> > Given that OMAP3 is a UP system, there is zero chance that it has
> > registered the magic hook that delivers IPIs (its interrupt controller
> > is not even capable of doin
On Thu, Feb 11, 2016 at 05:16:00PM -0800, Andy Lutomirski wrote:
> Are you running 32-bit userspace by any chance?
Sure, that's a 32-bit kernel testing partition. :)
> I'm guessing you're hitting this in __fpu_restore_sig:
Yeah, I was looking at that too.
> fpu__drop(fpu);
> if
;>> Thanks for the report!
>>>
>>>> I see crashes in various arm qemu tests due to 'cpufreq: governor: Replace
>>>> timers with utilization update callbacks' with next-20160215. An example
>>>> crash log and bisect results are attached below.
On Mon, Feb 15, 2016 at 07:41:21PM +0100, Rafael J. Wysocki wrote:
> Since this is ARM, arch_send_call_function_single_ipi() looks like this:
>
> void arch_send_call_function_single_ipi(int cpu)
> {
> smp_cross_call(cpumask_of(cpu), IPI_CALL_FUNC_SINGLE);
> }
>
> so I'm not sure how the
* Rafael J. Wysocki [160215 10:44]:
> On Mon, Feb 15, 2016 at 6:05 PM, Guenter Roeck wrote:
> > Rafael,
>
> Hi,
>
> Thanks for the report!
>
> > I see crashes in various arm qemu tests due to 'cpufreq: governor: Replace
> > timers with utilization
This patch fixes the alignment issue reported by checkpatch.pl:
CHECK: Alignment should match open parenthesis
Add a unsigned char *sgbuffer in rtsx_stor_access_xfer_buffer to make the
following memcpy logic easier to read.
Add a struct scatterlist *sg in the use_sg branch of
rtsx_transfer_data_
This patch fixes the following issues in rtsx_transport.c as reported by
checkpatch.pl:
CHECK: Blank lines aren't necessary after an open brace '{'
CHECK: Please don't use multiple blank lines
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 3 ---
1 file changed, 3 delet
This patch fixes the following styling issue in rtsx_transport.c
as reported by checkpatch.pl:
CHECK: spaces preferred around that '-' (ctx:VxV)
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/
Hello,
On Sun, Feb 14, 2016 at 09:07:04AM +0800, kernel test robot wrote:
> FYI, we noticed the below changes on
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> commit 82607adcf9cdf40fb7b5331269780c8f70ec6e35 ("workqueue: implement lockup
> detector")
>
>
> [
This patch removes all unnecessary parentheses found by checkpatch.pl.
Signed-off-by: Shaun Ren
---
Changes since v3
* Fixed patch conflicts due to commit 9a66d05d82db
("Staging: rts5208: fix check for dma mapping error"), and
commit b3232842dbef ("Staging: rts5208: remove unnecessary p
This patch changes all comparsions to NULL with !..., as reported by
checkpatch.pl.
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx_transport.c
b/drivers/staging/rts5208/
This patch fixes the following naming convention issue in rtsx_transport.c,
as reported by checkpatch.pl:
CHECK: Avoid CamelCase:
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rts5
This patch fixes all multiline comments to conform to the coding style,
which states that multiline comments should start with "/*" and end
with "*/" on a separate line.
Also cleans up some comments to make them more clear and/or reflect what
the code is doing.
Signed-off-by: Shaun Ren
---
Chang
This set of patches fixes the coding style issues in rtsx_transport.c.
Changes since v4
* Moved a declaration to the beginning of an if branch
Changes since v3
* Removed the extraneous SCSI comment in the first patch
* Removed the last patch (Staging: rts5208: Add missing dma_mapping_error)
This patch removes all spaces after casts in rtsx_transport.c, as reported
by checkpatch.pl:
CHECK: No space is necessary after a cast
Signed-off-by: Shaun Ren
---
drivers/staging/rts5208/rtsx_transport.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rts520
On Sat, Feb 13, 2016 at 08:26:22PM +, Colin King wrote:
> From: Colin Ian King
>
> busyp->bno is printed with a %llu format specifier when the
> intention is to print a hexadecimal value. Trivial fix to
> use %llx instead. Found with smatch static analysis:
>
> fs/xfs/xfs_discard.c:229 xfs_
The state in devfreq_cooling_state2power is unsigned so removed the < 0
check.
Signed-off-by: Alan Cox
---
drivers/thermal/devfreq_cooling.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/thermal/devfreq_cooling.c
b/drivers/thermal/devfreq_cooling.c
index 01f0015
301 - 400 of 925 matches
Mail list logo
