Re: [PATCH v5 1/3] mtd: nand: gpmi: add gpmi_move_bits function

2014-11-30 Thread Brian Norris
On Wed, Nov 26, 2014 at 05:57:08PM +0100, Boris Brezillon wrote: On Wed, 26 Nov 2014 17:53:11 +0100 Boris Brezillon boris.brezil...@free-electrons.com wrote: +/** + * gpmi_move_bits - move bits from one memory region to another + * @dst: destination buffer + * @dst_bit_off: bit offset

Re: [PATCH 0/2] fix some problems for cpufreq

2014-11-30 Thread Wang Weidong
On 2014/11/30 6:30, Rafael J. Wysocki wrote: On Saturday, November 29, 2014 09:40:02 AM Wang Weidong wrote: On 2014/11/29 9:26, Rafael J. Wysocki wrote: On Friday, November 28, 2014 10:43:37 AM Wang Weidong wrote: Hi Rafael and Viresh Sorry to trouble you again. As for: acpi-cpufreq: get

[PATCH 0/3] btrfs: fix RCU string sparse noise

2014-11-30 Thread Omar Sandoval
Hi everyone, These patches clean up the big stack of sparse RCU errors I introduced into the integration tree as reported by the kbuild test robot: On Thu, Nov 27, 2014 at 06:45:20AM +0800, kbuild test robot wrote: tree: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git

[PATCH 3/3] btrfs: refactor btrfs_device-name updates

2014-11-30 Thread Omar Sandoval
The rcu_string API introduced some new sparse errors but also revealed existing ones. First of all, the name in struct btrfs_device should be annotated as __rcu to prevent unsafe reads. Additionally, updates should go through rcu_dereference_protected to make it clear what's going on. This

[PATCH 1/3] rcustring: clean up botched __rcu annotations

2014-11-30 Thread Omar Sandoval
The rcu_string returned by rcu_string_strdup isn't technically under RCU yet, and it makes more sense not to treat it as such. Additionally, an rcu_string passed to rcu_string_free should already be rcu_dereferenced and therefore not in the __rcu address space. Signed-off-by: Omar Sandoval

[PATCH 2/3] btrfs: fix suspicious RCU in BTRFS_IOC_DEV_INFO

2014-11-30 Thread Omar Sandoval
A naked read of the value of an RCU pointer isn't safe. Put the whole access in an RCU critical section, not just the pointer dereference. Signed-off-by: Omar Sandoval osan...@osandov.com --- fs/btrfs/ioctl.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH] Drivers:staging:clocking-wizard: Added a newline

2014-11-30 Thread Athira Sharikkal
From: Athira S athirasna...@gmail.com Fixed checkpatch warning: Missing a newline after declarations Signed-off-by: Athira S athirasna...@gmail.com --- drivers/staging/clocking-wizard/clk-xlnx-clock-wizard.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH] sound: use enum names instead of magic numbers

2014-11-30 Thread Takashi Iwai
At Sat, 29 Nov 2014 12:58:12 -0800, Joe Perches wrote: There's an enum defined for these magic numbers, might as well use it. Miscellanea: o Use ##__VA_ARGS__ Signed-off-by: Joe Perches j...@perches.com Any specific reason to hang to an irrelevant thread? Also... ---

[PATCH v3] Drivers:staging:clocking-wizard: Added a newline

2014-11-30 Thread Athira Sharikkal
Fixed checkpatch warning: Missing a newline after declarations Signed-off-by: Athira Sharikkal athirasna...@gmail.com --- drivers/staging/clocking-wizard/clk-xlnx-clock-wizard.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/clocking-wizard/clk-xlnx-clock-wizard.c

Re: [PATCH v5 0/3] mtd: nand: gpmi: add proper raw access support

2014-11-30 Thread Brian Norris
On Wed, Nov 26, 2014 at 05:53:10PM +0100, Boris Brezillon wrote: This series provides an implementation for raw accesses taking care of hidding the specific layout used by the GPMI controller. Best Regards, Boris Changes since v4: - fixed a few corner cases in gpmi_move_bits (tested

Re: [PATCH] sound: use enum names instead of magic numbers

2014-11-30 Thread Joe Perches
On Sun, 2014-11-30 at 09:41 +0100, Takashi Iwai wrote: At Sat, 29 Nov 2014 12:58:12 -0800, Joe Perches wrote: There's an enum defined for these magic numbers, might as well use it. Miscellanea: o Use ##__VA_ARGS__ Signed-off-by: Joe Perches j...@perches.com Any specific

Re: [PATCH v5 1/3] mtd: nand: gpmi: add gpmi_move_bits function

2014-11-30 Thread Boris Brezillon
Hi Brian, On Sun, 30 Nov 2014 00:11:10 -0800 Brian Norris computersforpe...@gmail.com wrote: On Wed, Nov 26, 2014 at 05:57:08PM +0100, Boris Brezillon wrote: On Wed, 26 Nov 2014 17:53:11 +0100 Boris Brezillon boris.brezil...@free-electrons.com wrote: +/** + * gpmi_move_bits - move

Re: [PATCH v4 2/2] mtd: hisilicon: add device tree binding documentation

2014-11-30 Thread Brian Norris
On Tue, Nov 04, 2014 at 08:47:01PM +0800, Zhou Wang wrote: Signed-off-by: Zhou Wang wangzhou@gmail.com --- .../devicetree/bindings/mtd/hisi504-nand.txt | 40 1 file changed, 40 insertions(+) create mode 100644

Re: kdbus: add documentation

2014-11-30 Thread Florian Weimer
* Greg Kroah-Hartman: +The focus of this document is an overview of the low-level, native kernel D-Bus +transport called kdbus. Kdbus exposes its functionality via files in a +filesystem called 'kdbusfs'. All communication between processes takes place +via ioctls on files exposed through

Re: [PATCH] i8k: Fix temperature bug handling in i8k_get_temp()

2014-11-30 Thread Guenter Roeck
On 11/18/2014 06:56 AM, Pali Rohár wrote: Static array prev[] was incorrectly initialized. It should be initialized to some invalid temperature value (above I8K_MAX_TEMP). Next, function should store invalid value to prev[] (above I8K_MAX_TEMP), not valid (= I8K_MAX_TEMP) because whole

Re: [PATCH v4 2/2] mtd: hisilicon: add device tree binding documentation

2014-11-30 Thread Brian Norris
One more thing: On Tue, Nov 04, 2014 at 08:47:01PM +0800, Zhou Wang wrote: diff --git a/Documentation/devicetree/bindings/mtd/hisi504-nand.txt b/Documentation/devicetree/bindings/mtd/hisi504-nand.txt new file mode 100644 index 000..c8b3988 --- /dev/null +++

Re: kdbus: add documentation

2014-11-30 Thread Florian Weimer
* Greg Kroah-Hartman: +7.4 Receiving messages +Also, if the connection allowed for file descriptor to be passed +(KDBUS_HELLO_ACCEPT_FD), and if the message contained any, they will be +installed into the receiving process after the KDBUS_CMD_MSG_RECV ioctl +returns. The receiving task is

Re: [PATCH] sound: use enum names instead of magic numbers

2014-11-30 Thread Takashi Iwai
At Sun, 30 Nov 2014 00:55:32 -0800, Joe Perches wrote: On Sun, 2014-11-30 at 09:41 +0100, Takashi Iwai wrote: At Sat, 29 Nov 2014 12:58:12 -0800, Joe Perches wrote: There's an enum defined for these magic numbers, might as well use it. Miscellanea: o Use ##__VA_ARGS__

Re: [RFC PATCH] proc, pidns: Add highpid

2014-11-30 Thread Florian Weimer
* Andy Lutomirski: The initial implementation is straightforward: highpid is simply a 64-bit counter. If a high-end system can fork every 3 ns (which would be amazing, given that just allocating a pid requires at atomic operation), it would take well over 1000 years for highpid to wrap. I'm

Re: kdbus: add documentation

2014-11-30 Thread Florian Weimer
* Andy Lutomirski: At the risk of opening a can of worms, wouldn't this be much more useful if you could share a pool between multiple connections? They would also be useful to reduce context switches when receiving data from all kinds of descriptors. At present, when polling, you receive

Re: [PATCH v4 0/2] mtd: hisilicon: add a new driver for NAND controller of hisilicon hip04 Soc

2014-11-30 Thread Brian Norris
On Tue, Nov 04, 2014 at 08:46:59PM +0800, Zhou Wang wrote: This patchset adds the support for NAND controller of hisilicon hip04 Soc. The NAND controller IP was developed by hisilicon and needs a new driver to support it. This patchset is based on v3.18-rc1. I have tested that NAND flash

Re: [PATCH v4 1/2] mtd: hisilicon: add a new NAND controller driver for hisilicon hip04 Soc

2014-11-30 Thread Brian Norris
On Tue, Nov 04, 2014 at 08:47:00PM +0800, Zhou Wang wrote: Signed-off-by: Zhou Wang wangzhou@gmail.com This driver mostly looks good. A few comments. --- drivers/mtd/nand/Kconfig|5 + drivers/mtd/nand/Makefile |1 + drivers/mtd/nand/hisi504_nand.c | 846

Re: [PATCH] i8k: Add support for temperature sensor labels

2014-11-30 Thread Pali Rohár
On Sunday 30 November 2014 01:07:15 Guenter Roeck wrote: On 11/29/2014 11:07 AM, Pali Rohár wrote: On Saturday 29 November 2014 19:58:28 Guenter Roeck wrote: On 11/29/2014 10:27 AM, Gabriele Mazzotta wrote: On Saturday 29 November 2014 18:18:18 Pali Rohár wrote: On Saturday 29 November

[PATCH] crypto: algif_skcipher - initialize upon init request

2014-11-30 Thread Stephan Mueller
When using the algif_skcipher, the following call sequence causess a re-initialization: 1. sendmsg with ALG_SET_OP and iov == NULL, iovlen == 0 (i.e initializing the cipher, but not sending data) 2. sendmsg with msg-msg-controllen == 0 and iov != NULL (using the initalized cipher handle by

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Christoffer Dall
On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: Use one looping instead of two, so can let code more simpler and get a little better performance. Signed-off-by: Chen Gang gang.chen.5...@gmail.com Performance? Please, this is vgic_create This does nothing else than introduce

[PATCHv1] staging: lustre: fixed sparse warnings related to static declarations

2014-11-30 Thread Janet Liu
drivers/staging/lustre/lustre/mdc/mdc_request.c:63:5: warning: symbol 'mdc_unpack_capa' was not declared. Should it be static? drivers/staging/lustre/lustre/mdc/mdc_request.c:150:5: warning: symbol 'mdc_getstatus' was not declared. Should it be static?

[PATCH 1/1] ACPI: Deletion of an unnecessary check before the function call pci_dev_put

2014-11-30 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Sun, 30 Nov 2014 10:59:10 +0100 The pci_dev_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software.

Re: [PATCH 5/7] KVM: arm64: guest debug, add support for single-step

2014-11-30 Thread Christoffer Dall
On Wed, Nov 26, 2014 at 07:27:06PM +, Peter Maydell wrote: On 25 November 2014 at 16:10, Alex Bennée alex.ben...@linaro.org wrote: This adds support for single-stepping the guest. As userspace can and will manipulate guest registers before restarting any tweaking of the registers has to

Re: [PATCH] i8k: Add support for temperature sensor labels

2014-11-30 Thread Pali Rohár
On Sunday 30 November 2014 02:25:09 Guenter Roeck wrote: On 11/29/2014 11:07 AM, Pali Rohár wrote: [ ... ] Original Dell DOS executable ignores all temperature sensors if type SMM function fails (if I decoded and understand that DOS assembler code correctly). So maybe we should do

[PATCH v2] acpi-cpufreq: make the freq_table store the same freq value

2014-11-30 Thread Wang Weidong
ACPI's P-states will report the acpi_processor_px *states to acpi-cpufreq. When the states likes these: [index:freq, 0:2400 1:2400 2:2000 3:1600...], we will initialize the freq_tables to those: [index:driver_data:freq, 0:0:2400, 1:2:2000 2:3,1600 ... CPUFREQ_TABLE_END] So when set the freqs

Re: [PATCH 5/7] KVM: arm64: guest debug, add support for single-step

2014-11-30 Thread Peter Maydell
On 30 November 2014 at 10:10, Christoffer Dall christoffer.d...@linaro.org wrote: In any case, I think it was related to how userspace observes the state of the CPU, because when you do the MMIO operation emulation in userspace, currently if you observe the PC though GET_ONE_REG, you'll see a

Re: [PATCH 5/7] KVM: arm64: guest debug, add support for single-step

2014-11-30 Thread Christoffer Dall
On Tue, Nov 25, 2014 at 04:10:03PM +, Alex Bennée wrote: This adds support for single-stepping the guest. As userspace can and will manipulate guest registers before restarting any tweaking of the registers has to occur just before control is passed back to the guest. Furthermore while

[PATCH 2/2 v3] adjtimex.2: add explanation on ADJ_TAI mode

2014-11-30 Thread Laurent Georget
This patch includes the adjtimex mode ADJ_TAI, intended to update the TAI offset. There are other FIXMEs in the man page and it's a bit outdated. I'll be on it once those first patches are accepted (to be sure to do the right thing). Thanks Laurent diff --git a/man2/adjtimex.2 b/man2/adjtimex.2

[PATCH 1/2 v3] adjtimex.2: add fields in struct timex description

2014-11-30 Thread Laurent Georget
Hello, this is a refactored series of patches for adjtimex(2) man page. Please ignore all previous patches I sent for adjtimex.2. The first patch updates the man page with the new fields added in struct timex since last edition of the man page. Thank you for your time. These are my first

Re: [PATCH 6/7] KVM: arm64: re-factor hyp.S debug register code

2014-11-30 Thread Christoffer Dall
On Tue, Nov 25, 2014 at 04:10:04PM +, Alex Bennée wrote: This is a pre-cursor to sharing the code with the guest debug support. This replaces the big macro that fishes data out of a fixed location with a more general helper macro to restore a set of debug registers. It uses macro

Re: [PATCH 7/7] KVM: arm64: guest debug, HW assisted debug support

2014-11-30 Thread Christoffer Dall
On Tue, Nov 25, 2014 at 04:10:05PM +, Alex Bennée wrote: This adds support for userspace to control the HW debug registers for guest debug. We'll only copy the $ARCH defined number across as that's all that hyp.S will use anyway. I've moved some helper functions into the hw_breakpoint.h

[PATCH] ioc3: fix incorrect use of htons/ntohs

2014-11-30 Thread Lino Sanfilippo
The protocol type in the ip header struct is a single byte variable. So there is no need to swap bytes depending on host endianness. Signed-off-by: Lino Sanfilippo linosanfili...@gmx.de --- Please note that I could not test this, since I dont have access to the concerning hardware.

[PATCH] pxa168: close race between napi and irq activation

2014-11-30 Thread Lino Sanfilippo
In pxa168_eth_open() the irqs are enabled before napi. This opens a tiny time window in which the irq handler is processed, disables irqs but then is not able to schedule the not yet activated napi, leaving irqs disabled forever (since irqs are reenabled in napi poll function). Fix this race by

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Chen Gang
On 11/30/2014 06:07 PM, Christoffer Dall wrote: On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: Use one looping instead of two, so can let code more simpler and get a little better performance. Signed-off-by: Chen Gang gang.chen.5...@gmail.com Performance? Please, this is

[PATCH] drivers:staging:media:omap24xx:v4l2-int-device.h: Added a '*' before ')'--(parenthesis closing)

2014-11-30 Thread Anjana Sasindran
This patch fix a checkpatch.pl error: ERROR: space prohibited before ')' parenthesis closing Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com --- drivers/staging/media/omap24xx/v4l2-int-device.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] mn10300: remove redundant hex()

2014-11-30 Thread Arjun Sreedharan
replace rendundant hex() with kernel's hex_to_bin() Signed-off-by: Arjun Sreedharan arjun...@gmail.com --- arch/mn10300/kernel/gdb-stub.c | 61 -- 1 file changed, 23 insertions(+), 38 deletions(-) diff --git a/arch/mn10300/kernel/gdb-stub.c

[PATCH] iscsi-target: fix error path in iscsi_target_init_module()

2014-11-30 Thread Lino Sanfilippo
In iscsi_target_init_module() unwind transport protocol registration in case that iscsit_load_discovery_tpg() failed. Signed-off-by: Lino Sanfilippo linosanfili...@gmx.de --- drivers/target/iscsi/iscsi_target.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH] kvm: arm: vgic: Let one looping code instead of two looping code

2014-11-30 Thread Christoffer Dall
On Mon, Dec 01, 2014 at 10:04:32PM +0800, Chen Gang wrote: On 11/30/2014 06:07 PM, Christoffer Dall wrote: On Sun, Nov 30, 2014 at 12:18:04PM +0800, Chen Gang wrote: Use one looping instead of two, so can let code more simpler and get a little better performance. Signed-off-by: Chen

Re: Crash when copying from broken external hdd

2014-11-30 Thread Richard Weinberger
On Sat, Nov 29, 2014 at 11:52 AM, Simon Danner danner.si...@gmail.com wrote: Hello, i get the following crash after i try to copy files from a broken external hdd to another external hdd. It happens after a few minutes, with latest git and 3.17.4 from Arch. Attached screenshot is from latest

[PATCH 6/6] UBI: Fastmap: Fix memory leak while attaching

2014-11-30 Thread Richard Weinberger
Currently we leak a few ubi_ainf_pebs while attaching. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/attach.c | 61 +-- drivers/mtd/ubi/fastmap.c | 13 -- 2 files changed, 33 insertions(+), 41 deletions(-) diff --git

[PATCH 1/6] UBI: Fastmap: Fix memory leaks while closing the WL sub-system

2014-11-30 Thread Richard Weinberger
Add a ubi_fastmap_close() to free all resources used by fastmap at WL shutdown. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/wl.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/drivers/mtd/ubi/wl.c b/drivers/mtd/ubi/wl.c index

[PATCH 5/6] UBI: Fastmap: Fix fastmap usage in ubi_volume_notify()

2014-11-30 Thread Richard Weinberger
There is no need to switch to ro mode if ubi_update_fastmap() fails. Also get rid of the ifdef. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/build.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/ubi/build.c

[PATCH 2/6] UBI: Fastmap: Don't allocate new ubi_wl_entry objects

2014-11-30 Thread Richard Weinberger
There is no need to allocate new ones every time, we can reuse the existing ones. This makes the code cleaner and more easy to follow. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 31 +-- drivers/mtd/ubi/wl.c | 11 +++

Fastmap update v2 (pile 2)

2014-11-30 Thread Richard Weinberger
Artem, this is pile 2 of 7. [PATCH 1/6] UBI: Fastmap: Fix memory leaks while closing the WL [PATCH 2/6] UBI: Fastmap: Don't allocate new ubi_wl_entry objects [PATCH 3/6] UBI: Fastmap: Notify user in case of an [PATCH 4/6] UBI: Fastmap: Wrap fastmap specific function in a ifdef [PATCH 5/6] UBI:

[PATCH 4/6] UBI: Fastmap: Wrap fastmap specific function in a ifdef

2014-11-30 Thread Richard Weinberger
...such that we can implement NOP variants of some functions. This will help to reduce fastmap specific ifdefs in other c files. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/ubi.h | 4 1 file changed, 4 insertions(+) diff --git a/drivers/mtd/ubi/ubi.h

[PATCH 3/6] UBI: Fastmap: Notify user in case of an ubi_update_fastmap() failure

2014-11-30 Thread Richard Weinberger
If ubi_update_fastmap() fails notify the user. This is not a hard error as ubi_update_fastmap() makes sure that upon failure the current on-flash fastmap will no be used upon next UBI attach. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/wl.c | 6 +- 1 file changed, 5

[PATCH 6/6] UBI: Fastmap: Make WL pool size 50% of user pool size

2014-11-30 Thread Richard Weinberger
Don't use a fixed size for the WL pool. Make it instead 50% of the user pool. We don't make it 100% as it is not as heavily used as the user pool. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/build.c | 2 +- drivers/mtd/ubi/ubi-media.h | 2 -- 2 files changed, 1

[PATCH 5/6] UBI: Fastmap: Switch to ro mode if invalidate_fastmap() fails

2014-11-30 Thread Richard Weinberger
We have to switch to ro mode to guarantee that upon next UBI attach all data is consistent. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/ubi/fastmap.c b/drivers/mtd/ubi/fastmap.c

[PATCH 1/6] UBI: Remove alloc_ai() slab name from parameter list

2014-11-30 Thread Richard Weinberger
There is always exactly one ubi_attach_info object allocated, therefore we don't have to care about the name. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/attach.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/ubi/attach.c

Fastmap update v2 (pile 3)

2014-11-30 Thread Richard Weinberger
Artem, this is pile 3 of 7. [PATCH 1/6] UBI: Remove alloc_ai() slab name from parameter list [PATCH 2/6] UBI: Fastmap: Fix race in ubi_eba_atomic_leb_change() [PATCH 3/6] UBI: Fastmap: Remove bogus ubi_assert() [PATCH 4/6] UBI: Fastmap: Remove eba_orphans logic [PATCH 5/6] UBI: Fastmap: Switch

[PATCH 4/6] UBI: Fastmap: Remove eba_orphans logic

2014-11-30 Thread Richard Weinberger
This logic is in vain as we treat protected PEBs also as used, so this case must not happen. If a PEB is found which is in the EBA table but not known as used has to be issued as fatal error. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 91

[PATCH 2/6] UBI: Fastmap: Fix race in ubi_eba_atomic_leb_change()

2014-11-30 Thread Richard Weinberger
This function a) requests a new PEB, b) writes data to it, c) returns the old PEB and d) registers the new PEB in the EBA table. For the non-fastmap case this works perfectly fine and is powercut safe. Is fastmap enabled this can lead to issues. If a new fastmap is written between a) and c) the

[PATCH 3/6] UBI: Fastmap: Remove bogus ubi_assert()

2014-11-30 Thread Richard Weinberger
It is legal to have PEBs left in the used list. This can happen if UBI copies a PEB and a powercut happens between writing a new fastmap and adding this PEB into the EBA table. In this case the old PEB will be used. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c |

[PATCH] skge: Unmask interrupts in case of spurious interrupts

2014-11-30 Thread Lino Sanfilippo
In case of a spurious interrupt dont forget to reenable the interrupts that have been masked by reading the interrupt source register. Signed-off-by: Lino Sanfilippo linosanfili...@gmx.de --- drivers/net/ethernet/marvell/skge.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: [PATCH 2/2] ARM: imx: src: support vf610 system reset controller

2014-11-30 Thread Arnd Bergmann
On Saturday 29 November 2014 01:15:57 Stefan Agner wrote: On 2014-11-28 23:22, Arnd Bergmann wrote: On Friday 28 November 2014 23:09:09 Stefan Agner wrote: On 2014-11-28 22:24, Arnd Bergmann wrote: On Friday 28 November 2014 22:02:01 Stefan Agner wrote: If the SRC is also capable of

[PATCH] sky2: avoid pci write posting after disabling irqs

2014-11-30 Thread Lino Sanfilippo
In sky2_change_mtu setting B0_IMSK to 0 may be delayed due to PCI write posting which could result in irqs being still active when synchronize_irq is called. Since we are not prepared to handle any further irqs after synchronize_irq (our resources are freed after that) force the write by a

[PATCH 6/6] UBI: Fastmap: Rework fastmap error paths

2014-11-30 Thread Richard Weinberger
If UBI is unable to write the fastmap to the device we have make sure that upon next attach UBI will fall back to scanning mode. In case we cannot ensure that they only thing we can do is falling back to read-only mode. The current error handling code is not powercut proof. It could happen that a

[PATCH 2/6] UBI: Fastmap: Fix race after ubi_wl_get_peb()

2014-11-30 Thread Richard Weinberger
ubi_wl_get_peb() returns a fresh PEB which can be used by user of UBI. Due to the pool logic fastmap will correctly map this PEB upon attach time because it will be scanned. If a new fastmap is written (due to heavy parallel io) while the before the fresh PEB is assigned to the EBA table it will

[PATCH 1/6] UBI: Fastmap: Fix leb_count unbalance

2014-11-30 Thread Richard Weinberger
If a LEB is unmapped we have to decrement leb_count as well. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mtd/ubi/fastmap.c b/drivers/mtd/ubi/fastmap.c index 3d913f5..5db41a5 100644 ---

[PATCH 4/6] UBI: Fastmap: Locking updates

2014-11-30 Thread Richard Weinberger
a) Rename ubi-fm_sem to ubi-fm_eba_sem as this semaphore protects EBA changes. b) Turn ubi-fm_mutex into a rw semaphore. It will still serialize fastmap writes but also ensures that ubi_wl_put_peb() is not interrupted by a fastmap write. We use a rw semaphore to allow ubi_wl_put_peb() still to be

[PATCH 5/6] UBI: Fastmap: Prepare for variable sized fastmaps

2014-11-30 Thread Richard Weinberger
The current code assumes that each fastmap has the same amount of PEBs. So far this is true but will change soon. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git

Fastmap update v2 (pile 4)

2014-11-30 Thread Richard Weinberger
Artem, this is pile 4 of 7. [PATCH 1/6] UBI: Fastmap: Fix leb_count unbalance [PATCH 2/6] UBI: Fastmap: Fix race after ubi_wl_get_peb() [PATCH 3/6] UBI: Fastmap: Set used_ebs only for static volumes [PATCH 4/6] UBI: Fastmap: Locking updates [PATCH 5/6] UBI: Fastmap: Prepare for variable sized

[PATCH 3/6] UBI: Fastmap: Set used_ebs only for static volumes

2014-11-30 Thread Richard Weinberger
If we set it for dynamic ones we might confuse various self checks. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/ubi/fastmap.c b/drivers/mtd/ubi/fastmap.c index 5db41a5..749e2e4

Re: [PATCH v3 3/5] pinctrl: exynos: add exynos5410 SoC specific data

2014-11-30 Thread Tomasz Figa
Hi Andreas, 2014-11-28 21:07 GMT+09:00 Andreas Färber afaer...@suse.de: Am 28.11.2014 um 12:59 schrieb Linus Walleij: On Sat, Nov 22, 2014 at 11:26 PM, Andreas Färber afaer...@suse.de wrote: From: Hakjoo Kim ruppi@hardkernel.com Add Samsung EXYNOS5410 SoC specific data to enable pinctrl

Re: [PATCH v3 2/8] KVM: arm64: Enable the KVM-VFIO device

2014-11-30 Thread Christoffer Dall
On Sun, Nov 23, 2014 at 07:35:54PM +0100, Eric Auger wrote: Used by KVM-enabled VFIO-based device passthrough support in QEMU. Signed-off-by: Joel Schopp joel.sch...@amd.com Signed-off-by: Eric Auger eric.au...@linaro.org --- Extracted from [RFC PATCH] arm64: KVM: add irqfd support

[PATCH 6/6] UBI: Fastmap: Add new module parameter fm_debug

2014-11-30 Thread Richard Weinberger
If fm_debug is set fastmap debugging is enabled by default. This is useful if one wants to debug fastmap on an UBI device with serves the rootfs. The the UBI attach mechanism runs long before debugfs can be mounted and chk_fastmap set. Signed-off-by: Richard Weinberger rich...@nod.at ---

[PATCH 2/6] UBI: Fastmap: Enhance fastmap checking

2014-11-30 Thread Richard Weinberger
Don't update the fastmap upon detach if fastmap checking is enabled. This is poor men's power cut testing feature. :-) Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/build.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/ubi/build.c

Re: [RFC PATCH 0/3] Generic TrustZone Driver in Linux Kernel

2014-11-30 Thread Javier González
Jason, On 29 Nov 2014, at 19:18, Jason Cooper ja...@lakedaemon.net wrote: Javier, On Fri, Nov 28, 2014 at 04:03:33PM +0100, Javier González wrote: Hi all, This set of patches is a first iteration to introduce a generic TrustZone driver to the Linux Kernel. Since there is no place to add

[PATCH 4/6] UBI: Fastmap: Add self check to detect absent PEBs

2014-11-30 Thread Richard Weinberger
This self check allows Fastmap to detect absent PEBs while writing a new fastmap to the MTD device. It will help to find implementation issues in Fastmap. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 86 +-- 1 file

[PATCH 5/6] UBI: Fastmap: Make self_check_eba() depend on fastmap self checking

2014-11-30 Thread Richard Weinberger
...instead of generic self checking. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/attach.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/ubi/attach.c b/drivers/mtd/ubi/attach.c index 40e6bf6..9bc70eb 100644 --- a/drivers/mtd/ubi/attach.c

Fastmap update v2 (pile 5)

2014-11-30 Thread Richard Weinberger
Artem, this is pile 5 of 7. [PATCH 1/6] UBI: Add initial support for fastmap self checks [PATCH 2/6] UBI: Fastmap: Enhance fastmap checking [PATCH 3/6] UBI: Fix stale pointers in ubi-lookuptbl [PATCH 4/6] UBI: Fastmap: Add self check to detect absent PEBs [PATCH 5/6] UBI: Fastmap: Make

[PATCH 3/6] UBI: Fix stale pointers in ubi-lookuptbl

2014-11-30 Thread Richard Weinberger
In some error paths the WL sub-system gives up on a PEB and frees it's ubi_wl_entry struct but does not set the entry in ubi-lookuptbl to NULL. Fastmap can stumble over such a stale pointer as it uses ubi-lookuptbl to find all PEBs. Fix this by introducing a new helper function which free()s a WL

[PATCH 1/6] UBI: Add initial support for fastmap self checks

2014-11-30 Thread Richard Weinberger
Using this debugfs knob fastmap self checks can be controlled. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/debug.c | 11 +++ drivers/mtd/ubi/debug.h | 5 + drivers/mtd/ubi/ubi.h | 4 3 files changed, 20 insertions(+) diff --git

Re: [PATCH v2 3/5] pinctrl: exynos: Fix GPIO setup failure because domain clock being gated

2014-11-30 Thread Tomasz Figa
Hi Krzysztof, 2014-11-28 23:08 GMT+09:00 Krzysztof Kozlowski k.kozlow...@samsung.com: On pią, 2014-11-28 at 15:04 +0100, Linus Walleij wrote: On Wed, Nov 26, 2014 at 3:24 PM, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: The audio subsystem on Exynos 5420 has separate clocks and GPIO.

[PATCH 5/6] UBI: Fastmap: Introduce may_reserve_for_fm()

2014-11-30 Thread Richard Weinberger
...and kill another #ifdef. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap-wl.c | 19 +++ drivers/mtd/ubi/wl.c | 7 +-- drivers/mtd/ubi/wl.h | 8 3 files changed, 28 insertions(+), 6 deletions(-) diff --git

[PATCH 6/6] UBI: Fastmap: Remove else after return.

2014-11-30 Thread Richard Weinberger
checkpatch.pl complains: WARNING: else is not generally useful after a break or return Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap-wl.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/mtd/ubi/fastmap-wl.c

[PATCH 1/6] UBI: Move fastmap specific functions out of wl.c

2014-11-30 Thread Richard Weinberger
Fastmap is tightly connected to the WL sub-system, many fastmap-specific functionslive in wl.c. To get rid of most #ifdefs in wl.c move this functions into a new file and include it into wl.c Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap-wl.c | 361

Fastmap update v2 (pile 6)

2014-11-30 Thread Richard Weinberger
Artem, this is pile 6 of 7. [PATCH 1/6] UBI: Move fastmap specific functions out of wl.c [PATCH 2/6] UBI: Add accessor functions for WL data structures [PATCH 3/6] UBI: Fastmap: Wire up WL accessor functions [PATCH 4/6] UBI: Fastmap: Introduce ubi_fastmap_init() [PATCH 5/6] UBI: Fastmap:

[PATCH 2/6] UBI: Add accessor functions for WL data structures

2014-11-30 Thread Richard Weinberger
Fastmap need access to various WL data structures as fastmap tightly depends on WL. To make the access less invasive add accessor functions. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/ubi.h | 36 1 file changed, 36 insertions(+)

[PATCH 4/6] UBI: Fastmap: Introduce ubi_fastmap_init()

2014-11-30 Thread Richard Weinberger
...and kill another #ifdef in wl.c. :-) Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/wl.c | 8 +--- drivers/mtd/ubi/wl.h | 7 +++ 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/mtd/ubi/wl.c b/drivers/mtd/ubi/wl.c index 2892039..d84546b

[PATCH 3/6] UBI: Fastmap: Wire up WL accessor functions

2014-11-30 Thread Richard Weinberger
Use the new WL accessor functions in fastmap. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap.c | 28 1 file changed, 12 insertions(+), 16 deletions(-) diff --git a/drivers/mtd/ubi/fastmap.c b/drivers/mtd/ubi/fastmap.c index

[PATCH 3/4] UBI: Fastmap: Remove is_fm_block()

2014-11-30 Thread Richard Weinberger
This function was added to fastmap in a very early stage to have paranoid assertions. With the current fastmap implementation this assert will never trigger as fastmap PEBs are not seen by the WL sub-system. Remove it to save us some CPU cycles. Signed-off-by: Richard Weinberger rich...@nod.at

Re: [PATCH v4] Thermal: int3406_thermal: solve the dependency build error

2014-11-30 Thread Zhang Rui
On Fri, 2014-11-07 at 15:11 -0400, Eduardo Valentin wrote: Hi, On Tue, Oct 28, 2014 at 02:11:59PM +0800, Aaron Lu wrote: Jim found that the current kernel may trigger a build error with some config: drivers/built-in.o: In function `int3406_thermal_probe':

[PATCH 4/4] UBI: Fastmap: Optimize anchor PEB search

2014-11-30 Thread Richard Weinberger
The UBI RB trees are sorted by the PEB number, hence we don't have to traverse the whole tree because ubi_rb_for_each_entry() walks the tree in sort order. Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap-wl.c | 7 ++- 1 file changed, 6 insertions(+), 1

[PATCH 2/4] Add drivers/mtd/ubi/fastmap-wl.c to MAINTAINERS file

2014-11-30 Thread Richard Weinberger
Signed-off-by: Richard Weinberger rich...@nod.at --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index ea4d005..87d4d36 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -9631,6 +9631,7 @@ M:Richard Weinberger rich...@nod.at L:

[PATCH 1/4] UBI: Fastmap: Add blank line after declarations

2014-11-30 Thread Richard Weinberger
Another checkpatch complaint: WARNING: Missing a blank line after declarations Signed-off-by: Richard Weinberger rich...@nod.at --- drivers/mtd/ubi/fastmap-wl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mtd/ubi/fastmap-wl.c b/drivers/mtd/ubi/fastmap-wl.c index 7a72af2..067aa90

Fastmap update v2 (pile 7)

2014-11-30 Thread Richard Weinberger
Artem, this is pile 7 of 7. [PATCH 1/4] UBI: Fastmap: Add blank line after declarations [PATCH 2/4] Add drivers/mtd/ubi/fastmap-wl.c to MAINTAINERS file [PATCH 3/4] UBI: Fastmap: Remove is_fm_block() [PATCH 4/4] UBI: Fastmap: Optimize anchor PEB search Thanks, //richard -- To unsubscribe from

Re: [PATCH 2/2] kernel: add support for live patching

2014-11-30 Thread Pavel Machek
On Thu 2014-11-06 16:51:02, Jiri Slaby wrote: On 11/06/2014, 03:39 PM, Seth Jennings wrote: This commit introduces code for the live patching core. It implements an ftrace-based mechanism and kernel interface for doing live patching of kernel and kernel module functions. Hi, nice! So

[PATCH] irqdomain: Correct early allocation of irq domains with IRQs off

2014-11-30 Thread Borislav Petkov
From: Borislav Petkov b...@suse.de I'm seeing the following lockdep splat on an x2APIC machine with interrupts remapping. The problem is that enable_IR_x2apic() disables interrupts before doing any further initialization. However, after having moved to irq domains, domain allocation cannot happen

Re: [PATCH v3 1/3] drm: add bus_formats and nbus_formats fields to drm_display_info

2014-11-30 Thread Boris Brezillon
Hi Laurent, On Sat, 29 Nov 2014 00:13:47 +0200 Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Boris, Thank you for the patch. I just have two small comments. On Tuesday 18 November 2014 14:46:18 Boris Brezillon wrote: Add bus_formats and nbus_formats fields and

Re: [PATCH v3 0/3] drm: describe display bus format

2014-11-30 Thread Boris Brezillon
Hi Laurent, On Sat, 29 Nov 2014 00:29:10 +0200 Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Boris, On Thursday 27 November 2014 14:37:50 Boris Brezillon wrote: On Tue, 18 Nov 2014 14:46:17 +0100 Boris Brezillon wrote: Hello, This series makes use of the

Re: [PATCH v3 3/8] VFIO: platform: forwarded state tested when selecting IRQ handler

2014-11-30 Thread Christoffer Dall
The subject reads strangely, perhaps just: VFIO: platform: test forward state when selecting IRQ handler On Sun, Nov 23, 2014 at 07:35:55PM +0100, Eric Auger wrote: In case the IRQ is forwarded, the VFIO platform IRQ handler does not need to disable the IRQ anymore. When setting the IRQ

Re: [PATCH v3 4/8] KVM: kvm-vfio: User API for IRQ forwarding

2014-11-30 Thread Christoffer Dall
On Sun, Nov 23, 2014 at 07:35:56PM +0100, Eric Auger wrote: This patch adds and document a new KVM_DEV_VFIO_DEVICE group documents and 2 device attributes: KVM_DEV_VFIO_DEVICE_FORWARD_IRQ, KVM_DEV_VFIO_DEVICE_UNFORWARD_IRQ. The purpose is to be able to set a VFIO device IRQ as forwarded or

Re: [PATCH v3 6/8] KVM: kvm-vfio: wrapper to VFIO external API device helpers

2014-11-30 Thread Christoffer Dall
On Sun, Nov 23, 2014 at 07:35:58PM +0100, Eric Auger wrote: Provide wrapper functions that allow KVM-VFIO device code to interact with a vfio device: - kvm_vfio_device_get_external_user gets a handle to a struct vfio_device from the vfio device file descriptor and increments its reference

Re: [PATCH] drivers:gpu:drm Remove unneeded struct of type psb_intel_i2c_chan in the header file psb_drv.h

2014-11-30 Thread Patrik Jakobsson
On Sun, Nov 30, 2014 at 3:24 AM, Nicholas Krause xerofo...@gmail.com wrote: Removes unneeeded struct *lvds_i2c_bus of type, psb_intel_i2c_chan as this struct is no needed due to never being used in the header file, psb_drv.h and therefore should be removed from this header file.

  1   2   3   4   >