v1 -> v2:
* replace all the spots by function get_vm_area_size().
Use wrapper function get_vm_area_size to calculate size of vm area.
Signed-off-by: Wanpeng Li
---
mm/vmalloc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index 93
v1 -> v2:
* add comments to describe alloc_usemap_and_memmap
After commit 9bdac91424075("sparsemem: Put mem map for one node together."),
vmemmap for one node will be allocated together, its logic is similar as
memory allocation for pageblock flags. This patch introduce
alloc_usemap_and_memmap
Prevent crash_kexec() from deadlocking on ioapic_lock. When crash_kexec()
is executed on a cpu, the cpu will get ioapic_lock in disable_IO_APIC().
So if the cpu gets NMI while locking ioapic_lock, a deadlock will happen.
In this patch, ioapic_lock is initialized before disable_IO_APIC().
To confir
v1 -> v2:
* remove failed.
preallocate_pmds will continue to preallocate pmds even if failure
occurrence, and then free all the preallocate pmds if there is
failure, this patch fix it by stop preallocate if failure occurrence
and go to free path.
Reviewed-by: Dave Hansen
Signed-off-by: Wanpeng
Hi Mike,
as we discussed with Kevin in my arm-soc pull request I should send
these two patches separately and directly to you.
I have rebased them on 3.11-rc2 which is the kernel version you have in
your clk-next branch. Thanks for pulling them.
Thanks,
Michal
The following changes since commit
Hi Prabhakar,
On Tue, Aug 20, 2013 at 11:44 AM, Prabhakar Lad
wrote:
> Hi Manish,
>
> Thanks for the patch.
>
> On Tue, May 21, 2013 at 2:24 PM, Vishwanathrao Badarkhe, Manish
> wrote:
>> Add device tree based support for TI's tps6507x touchscreen.
>>
>> Tested on da850-evm.
>>
>> Signed-off-by:
Dear developers,
At first thanks for all your work!
kernel version: 3.9.9-t23
kernel config: https://paste.debian.net/27351/
lspci -nvv output is attached.
I merged two kernel issues into one mail to find relations easier.
As both appeared only once I did not invest more time to try with
newer
It's not used globally and could be static.
Signed-off-by: Wanpeng Li
---
fs/fs-writeback.c | 2 +-
include/linux/writeback.h | 2 --
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c
index 87d7781..54b3c31 100644
--- a/fs/fs-writeback.c
Hi Stephen,
Thanks for your review.
On 8/20/2013 1:01 AM, Stephen Warren wrote:
On 08/16/2013 04:13 AM, George Cherian wrote:
Adding extcon driver for USB ID detection to dynamically
configure USB Host/Peripheral mode.
diff --git a/Documentation/devicetree/bindings/extcon/extcon-dra7xx.txt
b
On Mon, Aug 19, 2013 at 07:27:51PM -0500, Aravind Gopalakrishnan wrote:
> Aravind Gopalakrishnan (1):
> AMD64_EDAC: Fix incorrect wrap arounds due to left shift beyond 32
> bits.
>
> drivers/edac/amd64_edac.c |9 +
> 1 file changed, 5 insertions(+), 4 deletions(-)
Just a minor
From: Casey Schaufler
Date: Mon, 19 Aug 2013 14:42:55 -0700
> On 8/19/2013 12:50 PM, David Miller wrote:
>> It's so that you can pass a generic ipv4/ipv6 address blob into
>> things like printf formatting, and since there is an address family
>> member present, it knows what's in there and theref
On Tue, Aug 20, 2013 at 01:41:40PM +0800, Chen Gang wrote:
>
> need "if (ret < 0)" instead of. ;-)
sure, it's details
>
> > sure you'll have to change shmem_show_mpol statement to return int code.
> > Won't this be more short and convenient?
> >
> >
>
> Hmm... if return -ENOSPC, in common p
* Andi Kleen wrote:
> > > Introduced by commit 9a55fdbe941e ("x86, asmlinkage, paravirt: Add
> > > __visible/asmlinkage to xen paravirt ops"). The 2 definitions used to be
> > > identical ... maybe there should be only one.
> >
> > Andi, please send a fix for this build warning, against
> > t
To iterate over all policies we currently iterate over all CPUs and
then get the policy for each of them. Let's use the newly created
cpufreq_policy_list for this purpose.
Signed-off-by: Viresh Kumar
---
drivers/cpufreq/cpufreq.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
d
cpufreq_policy_cpu per-cpu variables are used for storing cpu that manages a cpu
or its policy->cpu. We are also storing policy for each cpu in cpufreq_cpu_data
and so this information is just redundant.
Better use cpufreq_cpu_data to retrieve policy and get policy->cpu from there.
And so this pat
We don't need to check if event is CPUFREQ_GOV_POLICY_INIT and put governor
module as we are sure event can only be START/STOP here.
And so remove this useless check.
Signed-off-by: Viresh Kumar
---
drivers/cpufreq/cpufreq.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/cpufreq/c
cpufreq_policy_list is a list of active policies. We do remove policies from
this list when all CPUs belonging to that policy are removed. But during suspend
we don't really free a policy struct as it will be used again during resume. And
so we didn't remove it from cpufreq_policy_list as well..
T
Following patch caused this:
commit 3de9bdeb28638e164d1f0eb38dd68e3f5d2ac95c
Author: Viresh Kumar
Date: Tue Aug 6 22:53:13 2013 +0530
cpufreq: improve error checking on return values of __cpufreq_governor()
Lets fix it.
Signed-off-by: Viresh Kumar
---
drivers/cpufreq/cpufreq.c | 2 +-
Hi Rafael,
You recently did this:
commit 878f6e074e9a7784a6e351512eace4ccb3542eef
Author: Rafael J. Wysocki
Date: Sun Aug 18 15:35:59 2013 +0200
Revert "cpufreq: Use cpufreq_policy_list for iterating over policies"
Revert commit eb60852 (cpufreq: Use cpufreq_policy_list for itera
On 19 August 2013 16:57, Viresh Kumar wrote:
> It wasn't my patch actually.. It only made it visible that's it :)
> The problem is:
> - On suspend all CPUs are removed and so governors are
> stopped.
> - On resume, handle_update() is called for the boot cpu and
> cpu_add_dev for all others.
>
> ha
On 20 August 2013 04:52, Rafael J. Wysocki wrote:
> From my experience, however, it is good to figure out what needs to be
> included
> into your test kernel and configure away everything else. Also, I'd recommend
> to build as much as possible into the kernel image and avoid compiling too
> ma
On 19 August 2013 17:15, Amit Kucheria wrote:
> Why do you create .deb packages to test development kernels? It _is_ slow.
>
> Instead I'd just add a new grub menu entry to /boot/grub/grub.cfg and
> point it to /boot/MyzImage. After a 'make install; make
> modules_install' just link /boot/MyzImage
This patch reverts commit 047b93a35961f7a6561e6f5dcb040738f822b892 which breaks
MAX7301 GPIO driver because that commit was dependant on a rejected patch that
was implementing selection of SPI speed from the Device Tree.
Signed-off-by: Christophe Leroy
--- linux-3.11-rc6/drivers/gpio/gpio-max730
On Tue, Aug 20, 2013 at 10:50:40AM +0530, Vineet Gupta wrote:
> Hi Greg,
>
> On 08/19/2013 08:27 PM, greg Kroah-Hartman wrote:
> > On Mon, Aug 19, 2013 at 09:08:49AM +, Vineet Gupta wrote:
> >> Hi Greg,
> >>
> >> I'd posted these patches for stable backport. Do I need to do anything
> >> more
2013/8/20 Marc Dionne :
> On Mon, Aug 19, 2013 at 7:49 PM, Linus Torvalds
> wrote:
>> On Mon, Aug 19, 2013 at 1:33 PM, Marc Dionne wrote:
>>>
>>> By my reading that commit (f0c3b5093add) also made proc_readdir always
>>> return 0, so with this patch the effect I see is that no pid entries
>>> are
Hi Emilio,
On Mon, Aug 19, 2013 at 05:14:57PM -0300, Emilio López wrote:
> El 05/08/13 17:43, Maxime Ripard escribió:
> >The A31 has a mostly different clock set compared to the other older
> >SoCs currently supported in the Allwinner clock driver.
> >
> >Add support for the basic useful clocks. T
From: Wei WANG
Set a bit to enable rts5227 and rts5249 to enter a deeper internal
power-saving mode in S3, and recover it after resuming.
Signed-off-by: Wei WANG
---
drivers/mfd/rtl8411.c|2 +-
drivers/mfd/rts5209.c|2 +-
drivers/mfd/rts5227.c|6 +-
driv
From: Wei WANG
v3:
Seperate copyright changes to a distinct patch
Modify some coding style and naming style
Fix a bug that sd30_drive_sel would be assigned a wrong value, in rts5227 and
rts5249
v2:
Use macro when initializing vendor settings
Wei WANG (6):
mfd:rtsx: Read vendor setting from c
From: Wei WANG
Clear hw_pfm_en to disable hardware PFM mode, to fix a bug that in some
situation registers in 0xFDxx domain can't be accessed.
Signed-off-by: Wei WANG
---
drivers/mfd/rtl8411.c|2 ++
include/linux/mfd/rtsx_pci.h |1 +
2 files changed, 3 insertions(+)
diff --git
From: Wei WANG
These actions are individual for each reader model, so should be put in
extra_init_hw instead of rtsx_pci_init_hw.
Signed-off-by: Wei WANG
---
drivers/mfd/rts5209.c |4
drivers/mfd/rts5227.c |2 ++
drivers/mfd/rts5229.c |4
drivers/mfd/rts5249.c |2
Am Sonntag, 18. August 2013, 20:05:52 schrieb Stephan Mueller:
Hi Ted, Sandy,
For FIPS 140-2, there is currently a draft of an Implementation Guidance
discussed covering the requirements of seed sources for deterministic
random number generators. The standard seed source when having no
hardwar
On Tuesday 20 August 2013 11:29 AM, ujhely...@gmail.com wrote:
> From: Matus Ujhelyi
>
> Some phy's can be configured to enable wake on lan (e.g. at803x or marvell
> 88E1318S).
> There is no way how to enable wol on CPSW with such connected phys. This patch
> adds this support. It is provided by
From: Wei WANG
Some actions to clear power state should be handled in .shutdown
callback in rtsx_pci_driver. This patch adopts the following measures to
catch this goal:
1. Add a function rtsx_pci_power_off to abstract the common ops in
.shutdown and .suspend
2. Add pcr->ops->force_power_down to
From: Wei WANG
Update copyright date, and remove author address.
Signed-off-by: Wei WANG
---
drivers/mfd/rtl8411.c |4 ++--
drivers/mfd/rts5209.c |3 +--
drivers/mfd/rts5227.c |5 +
drivers/mfd/rts5229.c |3 +--
drivers/mfd/rts5249.c
From: Wei WANG
Normally OEMs will set vendor setting to the config space of Realtek
card reader in BIOS stage. This patch reads the setting at the first,
and configure the internal registers according to it, to improve card
reader's compatibility condition.
Signed-off-by: Wei WANG
---
drivers/
Hello Mike,
On Mon, 19 Aug 2013 15:37:14 -0700, Mike Turquette
wrote:
> Quoting Nicolas Ferre (2013-08-19 09:23:46)
>> On 08/08/2013 17:23, boris brezillon :
>> > Hello Jean-Christophe,
>> >
>> > On 08/08/2013 16:07, Jean-Christophe PLAGNIOL-VILLARD wrote:
>> >> On 12:14 Wed 07 Aug , Boris BR
Hi Manish,
Thanks for the patch.
On Tue, May 21, 2013 at 2:24 PM, Vishwanathrao Badarkhe, Manish
wrote:
> Add device tree based support for TI's tps6507x touchscreen.
>
> Tested on da850-evm.
>
> Signed-off-by: Vishwanathrao Badarkhe, Manish
> ---
> Changes since V2:
> - Removed unnecessary co
Hello Nicolas,
On Mon, 19 Aug 2013 18:46:13 +0200, Nicolas Ferre
wrote:
> On 08/08/2013 06:09, boris brezillon :
>> Hello Arnd,
>>
>> On 07/08/2013 22:24, Arnd Bergmann wrote:
>>> On Thursday 01 August 2013, Boris BREZILLON wrote:
This patch moves peripheral id definitions from machine speci
From: Matus Ujhelyi
Some phy's can be configured to enable wake on lan (e.g. at803x or marvell
88E1318S).
There is no way how to enable wol on CPSW with such connected phys. This patch
adds this support. It is provided by calling the phy's related code.
Tested on board with at8030x connected ph
On Tuesday 20 August 2013 11:08 AM, ujhely...@gmail.com wrote:
> From: Matus Ujhelyi
>
> Some phy's can be configured to enable wake on lan (e.g. at803x or marvell
> 88E1318S).
> There is no way how to enable wol on CPSW with such connected phys. This patch
> adds this support. It is provided by
On Tue, Aug 20, 2013 at 10:55:40AM +0530, Arun KS wrote:
> >From 932a134abeac597f18c86c513704709ad154994b Mon Sep 17 00:00:00 2001
> From: Arun KS
> Date: Mon, 19 Aug 2013 12:06:33 +0530
> Subject: seq_file: Fix overflow condition in seq_commit
>
> seq_path()/seq_commit() is treating a d_path() f
On 08/20/2013 01:30 PM, Cyrill Gorcunov wrote:
> On Tue, Aug 20, 2013 at 11:56:15AM +0800, Chen Gang wrote:
>> For the implementation (patch 1/3), need fill buffer as full as
>> possible when buffer space is not enough.
>>
>> For the caller (patch 2/3, 3/3), need check the return value of
>> mpol_t
From: Matus Ujhelyi
Some phy's can be configured to enable wake on lan (e.g. at803x or marvell
88E1318S).
There is no way how to enable wol on CPSW with such connected phys. This patch
adds this support. It is provided by calling the phy's related code.
Tested on board with at8030x connected ph
On Tue, Aug 20, 2013 at 11:56:15AM +0800, Chen Gang wrote:
> For the implementation (patch 1/3), need fill buffer as full as
> possible when buffer space is not enough.
>
> For the caller (patch 2/3, 3/3), need check the return value of
> mpol_to_str().
>
> Signed-off-by: Chen Gang
Won't simple
- Original Nachricht
Von: Greg Kroah-Hartman
An: Stefan Achatz
Datum: 15.08.2013 18:40
Betreff: Re: [PATCH 35/36] hid: roccat: convert class code to use bin_attrs
in groups
> On Thu, Aug 15, 2013 at 06:03:46PM +0200, Stefan Achatz wrote:
> > Hello Greg,
> > looking through t
Hi Peter,
Thank you for the clarification.
On 08/19/2013 04:00 PM, Peter Zijlstra wrote:
> On Mon, Aug 19, 2013 at 09:47:47AM +0530, Preeti U Murthy wrote:
>> Hi Peter,
>>
>> On 08/16/2013 03:42 PM, Peter Zijlstra wrote:
>>
>> I have a few comments and clarification to seek.
>>
>> 1. How are you e
[ 24.990076] [ INFO: possible circular locking dependency detected ]
[ 24.990086] 3.11.0-rc6+ #154 Not tainted
[ 24.990094] ---
[ 24.990103] crda/1159 is trying to acquire lock:
[ 24.990111] (genl_mutex){+.+.+.}, at: []
genl_lock+0x12/0
>From 932a134abeac597f18c86c513704709ad154994b Mon Sep 17 00:00:00 2001
From: Arun KS
Date: Mon, 19 Aug 2013 12:06:33 +0530
Subject: seq_file: Fix overflow condition in seq_commit
seq_path()/seq_commit() is treating a d_path() failure as an overflow
condition, but it isn't.
seq_read handles over
There is a corener case when no peripheral irqs route to secondary
cores.
Let's take dual core system for example, the sequence is as following:
Core 0 Core1
1. set waiting bit and enter waiting loop
2. set waiting bit and po
Hi Andrew,
On Tue, Aug 20, 2013 at 4:35 AM, Andrew Morton
wrote:
> On Mon, 19 Aug 2013 17:02:37 +0530 Arun KS wrote:
>
>> seq_path()/seq_commit() is treating a d_path() failure as an overflow
>> condition, but it isn't.
>
> This is too brief. When fixing a bug please always describe the
> user-
On Friday 16 August 2013 10:37 PM, Lad, Prabhakar wrote:
> From: "Lad, Prabhakar"
>
> Add ethernet device tree node information and pinmux for mii to da850 by
> providing interrupt details and local mac address.
>
> Signed-off-by: Lad, Prabhakar
Queuing this for v3.12
Thanks,
Sekhar
--
To uns
Hi Greg,
On 08/19/2013 08:27 PM, greg Kroah-Hartman wrote:
> On Mon, Aug 19, 2013 at 09:08:49AM +, Vineet Gupta wrote:
>> Hi Greg,
>>
>> I'd posted these patches for stable backport. Do I need to do anything more
>> to get them included.
>>
>> https://patchwork.kernel.org/patch/2841153/ [1/2
On 08/20/2013 01:09 PM, Joe Perches wrote:
> On Tue, 2013-08-20 at 11:38 +0800, Chen Gang wrote:
>> Need check the return value of proc_put_char(), just like another have
>> done in __do_proc_doulongvec_minmax().
>>
>> Signed-off-by: Chen Gang
>> ---
>> kernel/sysctl.c |5 -
>> 1 files ch
On Tue, 2013-08-20 at 11:38 +0800, Chen Gang wrote:
> Need check the return value of proc_put_char(), just like another have
> done in __do_proc_doulongvec_minmax().
>
> Signed-off-by: Chen Gang
> ---
> kernel/sysctl.c |5 -
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --
On Mon, Aug 19, 2013 at 09:28:32PM -0700, Paul E. McKenney wrote:
> On Mon, Aug 19, 2013 at 08:24:11PM -0700, Josh Triplett wrote:
> > On Mon, Aug 19, 2013 at 07:37:17PM -0700, Paul E. McKenney wrote:
> > > Hello!
> > >
> > > This series provides a few documentation updates:
> > >
> > > 1.
On 2013-08-20 2:28 AM, Greg Kroah-Hartman wrote:
> On Tue, Aug 20, 2013 at 08:26:11AM +0800, Tom Gundersen wrote:
>> On Tue, Aug 20, 2013 at 8:03 AM, Greg Kroah-Hartman
>> wrote:
>> > On Tue, Aug 20, 2013 at 07:59:47AM +0800, Tom Gundersen wrote:
>> >> Hi guys,
>> >>
>> >> Starting with 3.10.6 (an
On 08/20/2013 12:43 PM, Chen Gang wrote:
> On 08/20/2013 12:18 PM, Paul E. McKenney wrote:
>> On Tue, Aug 20, 2013 at 11:51:23AM +0800, Chen Gang wrote:
>>>
>>>
>>> If 'hc' is false, 'al' will never be false, either (only need check
>>> "irdp->qlen != rdp->qlen_lazy' when 'rdp->nxtlist' existance).
On 08/20/2013 12:18 PM, Paul E. McKenney wrote:
> On Tue, Aug 20, 2013 at 11:51:23AM +0800, Chen Gang wrote:
>>
>>
>> If 'hc' is false, 'al' will never be false, either (only need check
>> "irdp->qlen != rdp->qlen_lazy' when 'rdp->nxtlist' existance).
>>
>> Recommend to improve the related code, li
On Tue, Aug 20, 2013 at 7:18 AM, Dave Jones wrote:
>
> btw, anyone have thoughts on a patch something like below ?
And another(sorry if message is reformatted by the mail agent,
and it took my an hour to get the agent back to the correct format but failed,
and thanks a lot for any howto send plai
On 08/20/2013 12:10 PM, Paul E. McKenney wrote:
> On Tue, Aug 20, 2013 at 11:50:02AM +0800, Chen Gang wrote:
>> According to the comment above rcu_cpu_has_callbacks(): "If there are
>> no callbacks, all of them are deemed to be lazy".
>>
>> So when both 'hc' and 'al' are false, '*all_lazy' should b
On Mon, Aug 19, 2013 at 08:24:11PM -0700, Josh Triplett wrote:
> On Mon, Aug 19, 2013 at 07:37:17PM -0700, Paul E. McKenney wrote:
> > Hello!
> >
> > This series provides a few documentation updates:
> >
> > 1. Update rcu_barrier() documentation to note that it no longer is
> > guaranteed to
On Mon, Aug 12, 2013 at 12:22:26PM +0200, Sascha Hauer wrote:
> of_property_read_bool properly compiles away, no need to ifdef this
> for non DT builds.
>
> Signed-off-by: Sascha Hauer
Pushed only this one to l2-mtd.git, since it's independent.
Thanks,
Brian
--
To unsubscribe from this list: se
Hello Seth,
On Fri, Aug 16, 2013 at 05:00:34PM -0500, Seth Jennings wrote:
> On Wed, Aug 14, 2013 at 02:55:34PM +0900, Minchan Kim wrote:
> > This patch moves zsmalloc under zram directory because there
> > isn't any other user any more.
> >
> > Before that, description will explain why we have n
On Mon, Aug 19, 2013 at 10:42:26AM +0200, Sascha Hauer wrote:
> On Sat, Aug 17, 2013 at 01:17:02PM -0700, Brian Norris wrote:
> > On Mon, Aug 12, 2013 at 12:22:24PM +0200, Sascha Hauer wrote:
> > > This patch adds a flag to struct flash_info indicating that
> > > fast_read is not supported. This no
On Tue, Aug 20, 2013 at 11:51:23AM +0800, Chen Gang wrote:
>
>
> If 'hc' is false, 'al' will never be false, either (only need check
> "irdp->qlen != rdp->qlen_lazy' when 'rdp->nxtlist' existance).
>
> Recommend to improve the related code, like the diff below.
Are you sure that this represents
On Mon, Aug 19, 2013 at 9:08 PM, Dave Chinner wrote:
> On Mon, Aug 19, 2013 at 08:28:20PM -0700, Andy Lutomirski wrote:
>> On Mon, Aug 19, 2013 at 7:36 PM, Dave Chinner wrote:
>> > On Fri, Aug 16, 2013 at 04:22:10PM -0700, Andy Lutomirski wrote:
>> >> Filesystems that defer cmtime updates should
On Tue, Aug 20, 2013 at 11:50:02AM +0800, Chen Gang wrote:
> According to the comment above rcu_cpu_has_callbacks(): "If there are
> no callbacks, all of them are deemed to be lazy".
>
> So when both 'hc' and 'al' are false, '*all_lazy' should be true, not
> false.
If there are no callbacks, what
On Mon, Aug 19, 2013 at 08:30:02PM -0700, Andy Lutomirski wrote:
> On Mon, Aug 19, 2013 at 7:38 PM, Dave Chinner wrote:
> > On Fri, Aug 16, 2013 at 04:22:12PM -0700, Andy Lutomirski wrote:
> >> A fancier implementation could probably avoid an extra journal
> >> transaction by adding a mapping_test
On Mon, Aug 19, 2013 at 08:28:20PM -0700, Andy Lutomirski wrote:
> On Mon, Aug 19, 2013 at 7:36 PM, Dave Chinner wrote:
> > On Fri, Aug 16, 2013 at 04:22:10PM -0700, Andy Lutomirski wrote:
> >> Filesystems that defer cmtime updates should update cmtime when any
> >> of these events happen after a
On Mon, Aug 19, 2013 at 8:33 PM, Dave Chinner wrote:
> On Mon, Aug 19, 2013 at 08:20:12PM -0700, Andy Lutomirski wrote:
>> On Mon, Aug 19, 2013 at 7:28 PM, Dave Chinner wrote:
>> > On Fri, Aug 16, 2013 at 04:22:09PM -0700, Andy Lutomirski wrote:
>> >> This is like file_update_time, except that it
On Mon, Aug 19, 2013 at 02:16:36PM -0700, Linus Torvalds wrote:
> On Mon, Aug 19, 2013 at 1:29 PM, Christoph Lameter wrote:
> > On Mon, 19 Aug 2013, Simon Kirby wrote:
> >
> >>[... ] The
> >> alloc/free traces are always the same -- always alloc_pipe_info and
> >> free_pipe_info. This is seen
Hi All,
Anybody help me review this?
Thanks,
LiWang.
On 08/14/2013 10:20 AM, Li Wang wrote:
When cpuidle_idle_call() return 0, it shows that linux system is using
idle framwork driver. Now, local irq has already been enabled in
cpuidle_idle_call(). So, it need not enable local irq again, when
Hi Thierry,
On 8/20/2013 05:20, Thierry Reding wrote:
On Mon, Aug 19, 2013 at 11:11:06AM +0800, Bo Shen wrote:
add atmel pwm controller driver based on PWM framework
this is basic function implementation of pwm controller
it can work with pwm based led and backlight
Please use the proper spe
mpol_to_str() may fail, and not fill the buffer (e.g. -EINVAL), so need
check about it, or buffer may not be zero based, and next seq_printf()
will cause issue.
Also print related warning when the buffer space is not enough.
Signed-off-by: Chen Gang
---
mm/shmem.c | 15 ++-
1 fil
mpol_to_str() may fail, and not fill the buffer (e.g. -EINVAL), so need
check about it, or buffer may not be zero based, and next seq_printf()
will cause issue.
Also print related warning when the buffer space is not enough.
Signed-off-by: Chen Gang
---
fs/proc/task_mmu.c | 16 ++
Need still try to fill buffer as full as possible if the buffer space
is not enough, commonly, the caller can bear of it (e.g. print warning
and still continue).
Signed-off-by: Chen Gang
---
mm/mempolicy.c | 14 ++
1 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/mm/
For the implementation (patch 1/3), need fill buffer as full as
possible when buffer space is not enough.
For the caller (patch 2/3, 3/3), need check the return value of
mpol_to_str().
Signed-off-by: Chen Gang
---
fs/proc/task_mmu.c | 16 ++--
mm/mempolicy.c | 14 ++-
On Tue, 2013-08-20 at 04:18 +0100, Al Viro wrote:
> On Wed, Aug 14, 2013 at 04:42:14PM -0600, Bjorn Helgaas wrote:
> > [+cc Al, linux-fsdevel for fdget/fdput usage]
>
> fdget/fdput use looks sane, the only thing is that I would rather
> have an explicit include of linux/file.h instead of relying u
If 'hc' is false, 'al' will never be false, either (only need check
"irdp->qlen != rdp->qlen_lazy' when 'rdp->nxtlist' existance).
Recommend to improve the related code, like the diff below.
--diff begin
diff --git a/kernel/rc
According to the comment above rcu_cpu_has_callbacks(): "If there are
no callbacks, all of them are deemed to be lazy".
So when both 'hc' and 'al' are false, '*all_lazy' should be true, not
false.
Signed-off-by: Chen Gang
---
kernel/rcutree.c |2 +-
1 files changed, 1 insertions(+), 1 dele
If this patch is correct, better to let CPU_ONLINE and CPU_DOWN_FAILED
share the same code.
And do we need a comment "/* fall through */" between CPU_UP_CANCELED
and CPU_DOWN_FAILED (or it is another bug, need a 'break' statement) ?
At last, also better to let CPU_DOWN_PREPARE and CPU_UP_CANCELE
When failure occures, __padata_add_cpu() and __padata_remove_cpu() will
return -ENOMEM, which need be noticed in any cases (even in cleaning up
cases).
Signed-off-by: Chen Gang
---
kernel/padata.c |8 ++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/kernel/padata.c b/k
As the up/down_read_non_owner() API has been brought back from the
dead, and has a single user (bcache), this module now breaks -rt, as
-rt has no API for non owner rwsems.
Now I looked at the code for bcache, and I don't see a way to fix it.
Perhaps there is, but I don't see it. Thus, I decided
In related functions, they have already had tag 'free', so use 'goto'
instead of 'break' to keep one (not multiple) style in each function.
Also remove useless checking 'err' code or move it to related 'if' code
block.
Signed-off-by: Chen Gang
---
kernel/sysctl.c | 25 +---
Need check the return value of proc_put_char(), just like another have
done in __do_proc_doulongvec_minmax().
Signed-off-by: Chen Gang
---
kernel/sysctl.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index 7822cd8..ee00986 100644
Patch 1 for bug fix: check return value after call proc_put_char() in
__do_proc_doulongvec_minmax()
Patch 2 for simplify code : Simplify code for C code readers.
Signed-off-by: Chen Gang
---
kernel/sysctl.c | 30 +-
1 files changed, 17 insertions(+), 13 deletions(
On Tue Aug 20 13, Rafael J. Wysocki wrote:
> On Monday, August 19, 2013 04:35:29 PM Jerry Snitselaar wrote:
> > On Tue Aug 20 13, Rafael J. Wysocki wrote:
> > > On Monday, August 19, 2013 09:16:14 AM Mika Westerberg wrote:
> > > > On Fri, Aug 16, 2013 at 06:26:35PM -0700, Jerry Snitselaar wrote:
>
On Mon, Aug 19, 2013 at 08:20:12PM -0700, Andy Lutomirski wrote:
> On Mon, Aug 19, 2013 at 7:28 PM, Dave Chinner wrote:
> > On Fri, Aug 16, 2013 at 04:22:09PM -0700, Andy Lutomirski wrote:
> >> This is like file_update_time, except that it acts on a struct inode *
> >> instead of a struct file *.
On Mon, Aug 19, 2013 at 7:38 PM, Dave Chinner wrote:
> On Fri, Aug 16, 2013 at 04:22:12PM -0700, Andy Lutomirski wrote:
>> A fancier implementation could probably avoid an extra journal
>> transaction by adding a mapping_test_clear_cmtime call in
>> ext4_writepages, but this should already be a co
On Mon, Aug 19, 2013 at 7:36 PM, Dave Chinner wrote:
> On Fri, Aug 16, 2013 at 04:22:10PM -0700, Andy Lutomirski wrote:
>> Filesystems that defer cmtime updates should update cmtime when any
>> of these events happen after a write via a mapping:
>>
>> - The mapping is written back to disk. This
xics_init() assumes kvm->nrcpus is already setup. kvm->nrcpus is setup
in kvm_cpu_init()
Unfortunately xics_init() and kvm_cpu_init() both use base_init(). So
depending on the order randomly determined by the compiler, xics_init()
may initialised see kvm->nrcpus as 0 and not setup any of the icp
On Mon, Aug 19, 2013 at 07:51:10PM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney"
>
> This commit adds a object_debug option to rcutorture to allow the
> debug-object-based checks for duplicate call_rcu() invocations to
> be deterministically tested.
>
> Signed-off-by: Paul E. McKenne
On Mon, Aug 19, 2013 at 07:37:17PM -0700, Paul E. McKenney wrote:
> Hello!
>
> This series provides a few documentation updates:
>
> 1.Update rcu_barrier() documentation to note that it no longer is
> guaranteed to wait for a full grace period. This guarantee was
> a victim of en
On Mon, Aug 19, 2013 at 07:37:46PM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney"
>
> Note that this commit also updates the formatting of serveral of the
> bibtex entries to conform to that of my .bib files. I started
> accumulating entries back in the 1980s, back when bibtex insiste
On Mon, Aug 19, 2013 at 07:05:58PM -0700, Paul E. McKenney wrote:
> On Mon, Aug 19, 2013 at 10:16:52AM -0700, Josh Triplett wrote:
> > On Mon, Aug 19, 2013 at 09:09:45AM -0700, Paul E. McKenney wrote:
> > > On Sun, Aug 18, 2013 at 09:19:25PM -0700, Josh Triplett wrote:
> > > > On Sun, Aug 18, 2013
On Mon, Aug 19, 2013 at 7:28 PM, Dave Chinner wrote:
> On Fri, Aug 16, 2013 at 04:22:09PM -0700, Andy Lutomirski wrote:
>> This is like file_update_time, except that it acts on a struct inode *
>> instead of a struct file *.
>>
>> Signed-off-by: Andy Lutomirski
>> ---
>> fs/inode.c | 72
On Wed, Aug 14, 2013 at 04:42:14PM -0600, Bjorn Helgaas wrote:
> [+cc Al, linux-fsdevel for fdget/fdput usage]
fdget/fdput use looks sane, the only thing is that I would rather
have an explicit include of linux/file.h instead of relying upon
linux/eventfd.h pulling it. Incidentally, there are onl
For some strings (e.g. version string), they are permitted to be larger
than PAGE_SIZE (although meaningless), so recommend to use scnprintf()
instead of sprintf().
Signed-off-by: Chen Gang
---
kernel/params.c |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/kernel
(2013/08/19 22:46), Petr Tesarik wrote:
On Sun, 18 Aug 2013 19:59:53 -0700
"Eric W. Biederman" wrote:
Sorry Eric, I'm not clear to what you mean by ``short one core''...
Which are you suggesting? Disabling BSP if crash happens on AP is
reasonable?
Or restricting cpus to a single one only
For some strings, they are permitted to be larger than PAGE_SIZE, so
need use scnprintf() instead of sprintf(), or it will cause issue.
One case is:
if a module version is crazy defined (length more than PAGE_SIZE),
'modinfo' command is still OK (print full contents),
but for "cat /sys/modu
1 - 100 of 705 matches
Mail list logo