Re: bit fields && data tearing

2014-09-07 Thread James Bottomley
On Sun, 2014-09-07 at 16:39 -0700, H. Peter Anvin wrote: > How many PARISC systems do we have that actually do real work on Linux? I'd be very surprised if this problem didn't exist on all alignment requiring architectures, like PPC and Sparc as well. I know it would be very convenient if all

Re: bit fields && data tearing

2014-09-07 Thread James Bottomley
On Sun, 2014-09-07 at 16:41 -0400, Peter Hurley wrote: > On 09/07/2014 03:04 PM, James Bottomley wrote: > > On Sun, 2014-09-07 at 09:21 -0700, Paul E. McKenney wrote: > >> On Sat, Sep 06, 2014 at 10:07:22PM -0700, James Bottomley wrote: > >>> On Thu, 2014-09-04 at 21:06 -0700, Paul E. McKenney

Re: [PATCH v10 net-next 2/2] net: filter: split filter.h and expose eBPF to user space

2014-09-07 Thread David Miller
From: Ingo Molnar Date: Mon, 8 Sep 2014 07:23:29 +0200 > > * Alexei Starovoitov wrote: > >> > I don't think the speed up of the llvm submission is a good >> > argument, this sounds to me similar to the "please apply this >> > patch that reserves this new netlink family in >> >

Re: [PATCH v10 net-next 2/2] net: filter: split filter.h and expose eBPF to user space

2014-09-07 Thread Ingo Molnar
* Alexei Starovoitov wrote: > > I don't think the speed up of the llvm submission is a good > > argument, this sounds to me similar to the "please apply this > > patch that reserves this new netlink family in > > include/linux/netlink.h, I promise this new subsystem will be > > submitted

Re: [PATCH] drivers/usb/host/ehci-xilinx-of.c: Include "linux/of_irq.h" to avoid compiling error

2014-09-07 Thread Michal Simek
On 09/03/2014 05:50 PM, Chen Gang wrote: > Need include it for irq_of_parse_and_map(), the related error with > allmodconfig under microblaze: > > drivers/usb/host/ehci-xilinx-of.c: In function ‘ehci_hcd_xilinx_of_probe’: > drivers/usb/host/ehci-xilinx-of.c:156:2: error: implicit declaration

linux-next: manual merge of the integrity tree with the nfsd tree

2014-09-07 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the integrity tree got a conflict in fs/nfsd/vfs.c between commit 9b638b816081 ("NFSD: Put file after ima_file_check fail in nfsd_open()") from the nfsd tree and commit e8442e203c25 ("ima: pass 'opened' flag to identify newly created files") from the integrity

Re: [GIT PULL] cgroup fixes for v3.17-rc4

2014-09-07 Thread Tejun Heo
Hello, Linus. On Sun, Sep 07, 2014 at 08:17:11PM -0700, Linus Torvalds wrote: > In other words, I'll happily pull this, but your excuses for it are > wrong-headed. There is no "crazyness justifies this". That's crap. But > the argument of "nobody does this, so let's fix it before anybody >

Re: [PATCH v2 14/15] cpufreq: Add module to register cpufreq on Krait CPUs

2014-09-07 Thread Viresh Kumar
On 6 September 2014 04:17, Stephen Boyd wrote: > diff --git a/drivers/cpufreq/qcom-cpufreq.c b/drivers/cpufreq/qcom-cpufreq.c > new file mode 100644 > index ..aa8eb97144b6 > --- /dev/null > +++ b/drivers/cpufreq/qcom-cpufreq.c > @@ -0,0 +1,199 @@ > +/* Copyright (c) 2014, The Linux

linux-next: manual merge of the drm-intel tree with Linus' tree

2014-09-07 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in drivers/gpu/drm/i915/intel_display.c between commit a4bf214ffc72 ("drm/i915: Move intel_ddi_set_vc_payload_alloc(false) to haswell_crtc_disable()") from Linus' tree and commit 575f7ab754c4 ("drm/i915: Pass intel_crtc to

linux-next: build failure after merge of the drm tree

2014-09-07 Thread Stephen Rothwell
Hi Dave, After merging the drm tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/gpu/drm/i915/intel_display.c: In function 'intel_crtc_cursor_set_obj': drivers/gpu/drm/i915/intel_display.c:8302:24: error: 'dev_priv' undeclared (first use in this function)

Re: [PATCH v6 4/4] phy: exynos5-usbdrd: Calibrate LOS levels for exynos5420/5800

2014-09-07 Thread Vivek Gautam
On Fri, Sep 5, 2014 at 11:26 PM, Felipe Balbi wrote: > On Thu, Sep 04, 2014 at 12:01:19PM +0530, Vivek Gautam wrote: >> > Don't we have phy_power_on() >> > for that ? It looks like you could just as well do this from >> > phy_power_on() ? >> >> No, unfortunately keeping these calibration settings

Re: f2fs get_dnode_of_data oops

2014-09-07 Thread Jaegeuk Kim
Hi, Thank you for the report. Could you share a little bit more information about the file accessing f2fs_llseek? E.g., file size, file offset, file allocation information, or dump of that file. Thanks, On Sun, Sep 07, 2014 at 10:20:44PM +0300, Tommi Rantala wrote: > 2014-09-07 22:14 GMT+03:00

Re: [PATCH v2] i2c: rk3x: fix divisor calculation for SCL frequency

2014-09-07 Thread Doug Anderson
Addy, On Sun, Sep 7, 2014 at 8:38 PM, Addy Ke wrote: > I2C_CLKDIV register descripted in the previous version of > RK3x chip manual is incorrect. Plus 1 is required. > > The correct formula: > - T(SCL_HIGH) = T(PCLK) * (CLKDIVH + 1) * 8 > - T(SCL_LOW) = T(PCLK) * (CLKDIVL + 1) * 8 > - (SCL

[PATCH net-next] ethernet: ti: remove unwanted THIS_MODULE macro

2014-09-07 Thread Varka Bhadram
It removes the owner field updation of driver structure. It will be automatically updated by module_platform_driver() Signed-off-by: Varka Bhadram --- drivers/net/ethernet/ti/cpmac.c|1 - drivers/net/ethernet/ti/cpsw-phy-sel.c |1 - drivers/net/ethernet/ti/cpsw.c |1

[PATCH v2] i2c: rk3x: fix divisor calculation for SCL frequency

2014-09-07 Thread Addy Ke
I2C_CLKDIV register descripted in the previous version of RK3x chip manual is incorrect. Plus 1 is required. The correct formula: - T(SCL_HIGH) = T(PCLK) * (CLKDIVH + 1) * 8 - T(SCL_LOW) = T(PCLK) * (CLKDIVL + 1) * 8 - (SCL Divsor) = 8 * ((CLKDIVL + 1) + (CLKDIVH + 1)) - SCL = PCLK / (CLK Divsor)

Re: [PATCH -v2] f2fs: Remove lock from check_valid_map

2014-09-07 Thread Jaegeuk Kim
Hi, On Sun, Sep 07, 2014 at 11:38:30AM +0800, Huang Ying wrote: > Only one bit is read in check_valid_map, holding a lock to do that > doesn't help anything except decreasing performance. > > Signed-off-by: Huang, Ying > --- > > v2: Fixed a build warning. > > --- > fs/f2fs/gc.c |3 --- >

Re: 3.17-rc4 fails compile (arch/x86/pci/irq.c)

2014-09-07 Thread Randy Dunlap
On 09/07/14 20:12, Mike Galbraith wrote: > (adds CC) > > On Sun, 2014-09-07 at 22:25 -0400, Pete Clements wrote: >> Fyi: i386 up -- problem introduced in rc3. >> >> CC arch/x86/pci/irq.o >> arch/x86/pci/irq.c: In function 'pirq_disable_irq': >> arch/x86/pci/irq.c:1259:2: error: implicit

Re: [PATCH 6/6] ntb: workaround for high traffic hardware hang

2014-09-07 Thread Jon Mason
On Thu, Aug 28, 2014 at 01:53:29PM -0700, Dave Jiang wrote: > A hardware errata causes the NTB to hang when heavy bi-directional traffic > in addition to the usage of BAR0/1 (where the registers reside, including > the doorbell registers to trigger interrupts). > > This workaround is only

Re: [GIT PULL] cgroup fixes for v3.17-rc4

2014-09-07 Thread Linus Torvalds
On Sun, Sep 7, 2014 at 6:20 PM, Tejun Heo wrote: > > While this is a userland visible > behavior change, given the craziness of allowing '\n' and its > implications, I believe the change is justified. Tejun, absolutely nothing "justifies" things if they break. Not "bad design",

Re: 3.17-rc4 fails compile (arch/x86/pci/irq.c)

2014-09-07 Thread Mike Galbraith
(adds CC) On Sun, 2014-09-07 at 22:25 -0400, Pete Clements wrote: > Fyi: i386 up -- problem introduced in rc3. > > CC arch/x86/pci/irq.o > arch/x86/pci/irq.c: In function 'pirq_disable_irq': > arch/x86/pci/irq.c:1259:2: error: implicit declaration of function > 'mp_should_keep_irq'

RE: [PATCH 1/4] net: stmmac: enhance to support multiple device instances

2014-09-07 Thread Kweh, Hock Leong
> -Original Message- > From: Kweh, Hock Leong > Sent: Saturday, August 30, 2014 11:48 AM > To: David Miller > Cc: peppe.cavall...@st.com; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; Ong, Boon Leong; Rayagond K > Subject: RE: [PATCH 1/4] net: stmmac: enhance to support multiple

Re: [PATCH 0/6] NTB bug fixes and hardware workarounds

2014-09-07 Thread Jon Mason
On Thu, Aug 28, 2014 at 01:52:57PM -0700, Dave Jiang wrote: > The following series contains various fixes and cleanup for NTB. It also > adds the split BAR support on Haswell platform and a hardware errata > workaround in order to allow interrupts to function during bi-directional > traffic under

[PATCH] [media] sn9c20x: refactor initialization functions and fix lint warnings/errors

2014-09-07 Thread Morgan Phillips
Centralize redundant print messages and references to sensor names via macros. Change msleep time from 10ms to 20ms in order to address checkpatch.pl warning: "msleep < 20ms can sleep for up to 20ms." Refactor redundant sensor initialization logic into two functions (_init_sensor_w1,

3.17-rc4 fails compile (arch/x86/pci/irq.c)

2014-09-07 Thread Pete Clements
Fyi: i386 up -- problem introduced in rc3. CC arch/x86/pci/irq.o arch/x86/pci/irq.c: In function 'pirq_disable_irq': arch/x86/pci/irq.c:1259:2: error: implicit declaration of function 'mp_should_keep_irq' [-Werror=implicit-function-declaration] cc1: some warnings being treated as errors

Re : Re: [PATCH ] ARM: pxa: fix section mismatch warning for pxa_timer_nodt_init

2014-09-07 Thread robert . jarzmik
Yes please Arnd, I'm still out for 1.5 week, so it would be great if you take that through your tree with my ack. Thanks. Robert PS: I know, top-posting is madness, but I'm really limited by technology right now. - Mail d'origine - De: Arnd Bergmann À: Robert Jarzmik Cc: Haojian

[PATCH 2/3] percpu-refcount: implement percpu_ref_set_killed()

2014-09-07 Thread Tejun Heo
With the recent addition of percpu_ref_reinit(), percpu_ref now can be used as a persistent switch which can be turned on and off repeatedly where turning off maps to killing the ref and waiting for it to drain; however, there currently isn't a way to initialize a percpu_ref in its off (killed and

[PATCH 1/3] percpu-refcount: improve WARN messages

2014-09-07 Thread Tejun Heo
percpu_ref's WARN messages can be a lot more helpful by indicating who's the culprit. Make them report the release function that the offending percpu-refcount is associated with. This should make it a lot easier to track down the reported invalid refcnting operations. Signed-off-by: Tejun Heo

[PATCH 3/3] percpu-refcount: make percpu_ref based on longs instead of ints

2014-09-07 Thread Tejun Heo
percpu_ref is currently based on ints and the number of refs it can cover is (1 << 31). This makes it impossible to use a percpu_ref to count memory objects or pages on 64bit machines as it may overflow. This forces those users to somehow aggregate the references before contributing to the

[PATCHSET percpu/for-3.18] percpu-refcount: several improvements

2014-09-07 Thread Tejun Heo
Hello, This patchset includes the following three patches improving percpu-refcount. 0001-percpu-refcount-improve-WARN-messages.patch 0002-percpu-refcount-implement-percpu_ref_set_killed.patch 0003-percpu-refcount-make-percpu_ref-based-on-longs-inste.patch The patchset is on top of

[GIT PULL] cgroup fixes for v3.17-rc4

2014-09-07 Thread Tejun Heo
Hello, Linus. Up until now, cgroup has allowed any character including '\n' in cgroup names; unfortunately, as '\n' is used as the row delimiter in /proc/$PID/cgroup, '\n' in a cgroup name makes the content of the file ambiguous to parse. This pull request includes Alban's patch to disallow '\n'

[GIT PULL] percpu fixes for v3.17-rc4

2014-09-07 Thread Tejun Heo
Hello, One patch to fix a failure path in the alloc path. The bug is dangerous but probably not too likely to actually trigger in the wild given that there hasn't been any report yet. The other two are low impact fixes. Thanks. The following changes since commit

[GIT PULL] libata fixes for v3.17-rc4

2014-09-07 Thread Tejun Heo
Hello, Linus. Two patches are to add PCI IDs for ICH9 and all others are device specific fixes. Nothing too interesting. Thanks. The following changes since commit 2a13772a144d2956a7fedd18685921d0a9b8b783: libata: widen Crucial M550 blacklist matching (2014-08-18 17:40:09 -0400) are

Re: linux-next: jdelvare-hwmon tree unfetchable

2014-09-07 Thread Stephen Rothwell
Hi Jean, On Tue, 2 Sep 2014 09:54:55 +0200 Jean Delvare wrote: > > On Mon, 01 Sep 2014 08:49:36 -0700, Guenter Roeck wrote: > > On 09/01/2014 05:28 AM, Mark Brown wrote: > > > Today's linux-next merge of the jdelvare-hwmon tree got a conflict with > > > Linus' > > > tree which appeared to be

Re: [PATCHSET percpu/for-3.18] add @gfp to init functions of percpu data structures

2014-09-07 Thread Tejun Heo
Applied to percpu/for-3.18. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

[PATCH 0.5/3] percpu_counter: make percpu_counters_lock irq-safe

2014-09-07 Thread Tejun Heo
percpu_counter is scheduled to grow @gfp support to allow atomic initialization. This patch makes percpu_counters_lock irq-safe so that it can be safely used from atomic contexts. Signed-off-by: Tejun Heo --- lib/percpu_counter.c | 16 ++-- 1 file changed, 10 insertions(+), 6

[GIT] Networking

2014-09-07 Thread David Miller
1) Fix skb leak in mac802154, from Martin Townsend. 2) Use select not depends on NF_NAT for NFT_NAT, from Pablo Neira Ayuso. 3) Fix union initializer bogosity in vxlan, from Gerhard Stenzel. 4) Fix RX checksum configuration in stmmac driver, from Giuseppe CAVALLARO. 5) Fix TSO with

Re: bit fields && data tearing

2014-09-07 Thread H. Peter Anvin
How many PARISC systems do we have that actually do real work on Linux? On September 7, 2014 4:36:55 PM PDT, "Paul E. McKenney" wrote: >On Sun, Sep 07, 2014 at 04:17:30PM -0700, H. Peter Anvin wrote: >> I'm confused why storing 0x0102 would be a problem. I think gcc does >that even on other

Linus 3.17-rc4

2014-09-07 Thread Linus Torvalds
For a short while there, this week was really nice and calm, but that was mostly because the "linux-foundation.org" entry fell off the DNS universe, and my mailbox got very quiet for a few hours. The rest of the week looked pretty normal. "Pretty normal" isn't bad, though, and I'm not

Re: bit fields && data tearing

2014-09-07 Thread Paul E. McKenney
On Sun, Sep 07, 2014 at 04:17:30PM -0700, H. Peter Anvin wrote: > I'm confused why storing 0x0102 would be a problem. I think gcc does that > even on other cpus. > > More atomicity can't hurt, can it? I must defer to James for any additional details on why PARISC systems don't provide

Re: bit fields && data tearing

2014-09-07 Thread H. Peter Anvin
I'm confused why storing 0x0102 would be a problem. I think gcc does that even on other cpus. More atomicity can't hurt, can it? On September 7, 2014 4:00:19 PM PDT, "Paul E. McKenney" wrote: >On Sun, Sep 07, 2014 at 12:04:47PM -0700, James Bottomley wrote: >> On Sun, 2014-09-07 at 09:21

[PATCH block/for-3.18] blkcg: remove blkcg->id

2014-09-07 Thread Tejun Heo
blkcg->id is a unique id given to each blkcg; however, the cgroup_subsys_state which each blkcg embeds already has ->serial_nr which can be used for the same purpose. Drop blkcg->id and replace its uses with blkcg->css.serial_nr. Rename cfq_cgroup->blkcg_id to ->blkcg_serial_nr and @id in

Re: [PATCH] net: phy: mdio-sun4i: don't select REGULATOR

2014-09-07 Thread David Miller
From: Beniamino Galvani Date: Sat, 6 Sep 2014 00:28:23 +0200 > The mdio-sun4i driver automatically selects REGULATOR and > REGULATOR_FIXED_VOLTAGE because it uses the regulator API. But a > driver selecting a subsystem increases the chance of generating > circular Kconfig dependencies,

Re: pull request: wireless 2014-09-05

2014-09-07 Thread David Miller
From: "John W. Linville" Date: Fri, 5 Sep 2014 11:04:53 -0400 > Please pull this batch of fixes intended for the 3.17 stream... > > For the mac80211 bits, Johannes says: > > "Here are a few fixes for mac80211. One has been discussed for a while > and adds a terminating NUL-byte to the alpha2

[PATCH 3/6] bdi: remove bdi->wb_lock locking around bdi->dev clearing in bdi_unregister()

2014-09-07 Thread Tejun Heo
The only places where NULL test on bdi->dev is used are bdi_[un]register(). The functions can't be called in parallel anyway and there's no point in protecting bdi->dev clearing with a lock. Remove bdi->wb_lock grabbing around bdi->dev clearing and move it after device_unregister() call so that

[PATCH 2/6] bdi: remove unused stuff

2014-09-07 Thread Tejun Heo
Two flags and one bdi_writeback field are no longer used. Remove them. Signed-off-by: Tejun Heo --- include/linux/backing-dev.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/linux/backing-dev.h b/include/linux/backing-dev.h index e488e94..2103a7f 100644 ---

[PATCH 5/6] bdi: explain the dirty list transferring in bdi_destroy()

2014-09-07 Thread Tejun Heo
bdi_destroy() has code to transfer the remaining dirty inodes to the default_backing_dev_info; however, given the shutdown sequence, it isn't clear how such condition would happen. Also, it isn't a full solution as the transferred inodes stlil point to the bdi which is being destroyed.

[PATCH 1/6] block, bdi: an active gendisk always has a request_queue associated with it

2014-09-07 Thread Tejun Heo
bdev_get_queue() returns the request_queue associated with the specified block_device. blk_get_backing_dev_info() makes use of bdev_get_queue() to determine the associated bdi given a block_device. All the callers of bdev_get_queue() including blk_get_backing_dev_info() assume that

[PATCH 4/6] bdi: make backing_dev_info->wb.dwork canceling stricter

2014-09-07 Thread Tejun Heo
Canceling of bdi->wb.dwork is currently a bit mushy. bdi_wb_shutdown() performs cancel_delayed_work_sync() at the end after shutting down and flushing the delayed_work and bdi_destroy() tries yet again after bdi_unregister(). bdi->wb.dwork is queued only after checking BDI_registered while

[PATCH 6/6] bdi: reimplement bdev_inode_switch_bdi()

2014-09-07 Thread Tejun Heo
A block_device may be attached to different gendisks and thus different bdis over time. bdev_inode_switch_bdi() is used to switch the associated bdi. The function assumes that the inode could be dirty and transfers it between bdis if so. This is a bit nasty in that it reaches into bdi

[PATCHSET block/for-3.18] block, bdi: various cleanups

2014-09-07 Thread Tejun Heo
Hello, This patchset contains the following six bdi related cleanup patches. Each patch should be fairly self-explanatory. 0001-block-bdi-an-active-gendisk-always-has-a-request_que.patch 0002-bdi-remove-unused-stuff.patch 0003-bdi-remove-bdi-wb_lock-locking-around-bdi-dev-cleari.patch

[PATCH v6] mm: softdirty: enable write notifications on VMAs after VM_SOFTDIRTY cleared

2014-09-07 Thread Peter Feiner
For VMAs that don't want write notifications, PTEs created for read faults have their write bit set. If the read fault happens after VM_SOFTDIRTY is cleared, then the PTE's softdirty bit will remain clear after subsequent writes. Here's a simple code snippet to demonstrate the bug: char* m =

Re: bit fields && data tearing

2014-09-07 Thread Paul E. McKenney
On Sun, Sep 07, 2014 at 12:04:47PM -0700, James Bottomley wrote: > On Sun, 2014-09-07 at 09:21 -0700, Paul E. McKenney wrote: > > On Sat, Sep 06, 2014 at 10:07:22PM -0700, James Bottomley wrote: > > > On Thu, 2014-09-04 at 21:06 -0700, Paul E. McKenney wrote: > > > > On Thu, Sep 04, 2014 at

Re: WARNING in block layer triggered in 3.17-rc3

2014-09-07 Thread Shirish Pargaonkar
I think the problem is, when a gendisk is detached, its request queue is not put in bypass mode cause when it is re-attached, code tries to put it out of bypass mode, hence the warning. So either of these should work, I have not tested it, just coded it up. diff --git a/block/blk-sysfs.c

Re: Power Scheduler Design

2014-09-07 Thread Mike Turquette
Quoting Abel Vesa (2014-09-07 04:47:13) > For a while now, I've started studying the power aware scheduling problem. > And like many other rookies out there I took all the lkml mails related > and read them all (well, almost all) and I saw that there are some > debating on the implementation.I

Re: Looks like a broken hub? (was Re: 3.16.2: 2TiB Seagate Expansion Desk apparently still broken with both USB mass storage *and* UAS: some debugging output)

2014-09-07 Thread Nix
[linux-scsi dropped, this is not a scsi or uas problem.] On 7 Sep 2014, n...@esperi.org.uk stated: > And... now it works, at least well enough to get a device file. So it's > not the disk that's at fault: it's the no-name hub! (Which is, I think, > USB ID 2109:0811 -- at least two instances of

Re: [Patch v4 04/16] ACPI: Rename processor_core.c as apic_id.c

2014-09-07 Thread Rafael J. Wysocki
On Thursday, August 28, 2014 10:22:29 AM Jiang Liu wrote: > Now all code in processor_core.c is APIC ID related, so rename it as > apic_id.c. Later IOAPIC ID related code will be added into apic_id.c. Actually, I'm not sure about this one. Renames like this make it difficult to backport things

Looks like a broken hub? (was Re: 3.16.2: 2TiB Seagate Expansion Desk apparently still broken with both USB mass storage *and* UAS: some debugging output)

2014-09-07 Thread Nix
On 7 Sep 2014, Alan Stern spake thusly: > On Sun, 7 Sep 2014, Nix wrote: > >> I have a brand new Seagate Expansion Desk drive attached to my x86-64 >> desktop. (I also have a 4TiB model of the same drive, but I haven't even >> unboxed it: there seems little point as long as the 2TiB version

Re: [Patch v4 00/16] Enable support of IOAPIC hotplug on x86 platforms

2014-09-07 Thread Rafael J. Wysocki
On Thursday, August 28, 2014 10:22:25 AM Jiang Liu wrote: > This patch set enhances IOAPIC core and ACPI drivers to support IOAPIC > hotplug on x86 platforms. It's based on latest mainstream kernel at > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > You may pull it from >

RE: [PATCH V2 1/1] Drivers: hv: vmbus: Enable interrupt driven flow control

2014-09-07 Thread KY Srinivasan
> -Original Message- > From: Amos Kong [mailto:kongjian...@gmail.com] > Sent: Sunday, September 7, 2014 4:44 AM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; open list; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; Jason Wang > Subject: Re: [PATCH V2 1/1]

Re: [PATCH v5] mm: softdirty: enable write notifications on VMAs after VM_SOFTDIRTY cleared

2014-09-07 Thread Peter Feiner
On Thu, Sep 04, 2014 at 09:43:11AM -0700, Peter Feiner wrote: > On Mon, Aug 25, 2014 at 09:45:34PM -0700, Hugh Dickins wrote: > > That sets me wondering: have you placed the VM_SOFTDIRTY check in the > > right place in this series of tests? > > > > I think, once pgprot_modify() is correct on all

Re: [PATCH ] ARM: pxa: fix section mismatch warning for pxa_timer_nodt_init

2014-09-07 Thread Arnd Bergmann
On Saturday 26 July 2014 20:50:36 Arnd Bergmann wrote: > commit a38b1f60b5245a3 ("ARM: pxa: Add non device-tree timer link to > clocksource") introduced a harmless section mismatch warning for > all pxa platforms, by introducing a new pxa_timer_init() function > that is not marked __init but that

Re: read()/readv() only from page cache

2014-09-07 Thread Volker Lendecke
On Fri, Sep 05, 2014 at 12:27:21PM -0400, Milosz Tanski wrote: > In a VLDB like workload this would enable me to lower the latency of > common fast requests and. By fast requests I mean ones that do not > require much data, the data is cached, or there's a predictable read > pattern (read-ahead).

Re: [PATCH 1/4] Documentation: use subdir-y to avoid unnecessary built-in.o files

2014-09-07 Thread Randy Dunlap
On 09/07/14 11:48, Mark Brown wrote: > On Sun, Sep 07, 2014 at 10:59:53AM -0700, Randy Dunlap wrote: > >> The quilt-import.log from 20140829 says: > >> $ git am --patch-format=mbox >> ../quilt/rd-docs/001-docum-use-subdiry-avoid-builtin.patch > >> and in the 20140905 git tree it says: > >> $

[PATCH] rtc: pcf8563: fix uninitialized use warning

2014-09-07 Thread Arnd Bergmann
gcc-4.9 found a potential condition under which the 'pending' variable may be used uninitialized: drivers/rtc/rtc-pcf8563.c: In function 'pcf8563_irq': drivers/rtc/rtc-pcf8563.c:173:5: warning: 'pending' may be used uninitialized in this function [-Wmaybe-uninitialized] This is because in the

Re: bit fields && data tearing

2014-09-07 Thread Peter Hurley
On 09/07/2014 03:04 PM, James Bottomley wrote: > On Sun, 2014-09-07 at 09:21 -0700, Paul E. McKenney wrote: >> On Sat, Sep 06, 2014 at 10:07:22PM -0700, James Bottomley wrote: >>> On Thu, 2014-09-04 at 21:06 -0700, Paul E. McKenney wrote: On Thu, Sep 04, 2014 at 10:47:24PM -0400, Peter Hurley

Re: [PATCH v2] perf tools: Fix build-id matching on vmlinux

2014-09-07 Thread Stephane Eranian
On Fri, Sep 5, 2014 at 5:44 PM, Arnaldo Carvalho de Melo wrote: > Em Sat, Sep 06, 2014 at 12:16:40AM +0900, Namhyung Kim escreveu: >> Hi Adrian and Arnaldo, >> >> 2014-09-05 (금), 11:11 -0300, Arnaldo Carvalho de Melo: >> > Em Fri, Sep 05, 2014 at 10:22:40AM +0300, Adrian Hunter escreveu: >> > >

Re: [PATCH v10 net-next 2/2] net: filter: split filter.h and expose eBPF to user space

2014-09-07 Thread Alexei Starovoitov
On Sun, Sep 7, 2014 at 11:07 AM, Pablo Neira Ayuso wrote: > > If the patches that provide the very first user interface don't get in > time to this merge window round for whatever reason, we'll have the > layout of this exposed to userspace in the next kernel version with no > clients at all,

[PATCH RESEND] x86: Reject x32 executables if x32 ABI not supported

2014-09-07 Thread Ben Hutchings
It is currently possible to execve() an x32 executable on an x86_64 kernel that has only ia32 compat enabled. However all its syscalls will fail, even _exit(). This usually causes it to segfault. Change the ELF compat architecture check so that x32 executables are rejected if we don't support

Re: 3.16.2: 2TiB Seagate Expansion Desk apparently still broken with both USB mass storage *and* UAS: some debugging output

2014-09-07 Thread Alan Stern
On Sun, 7 Sep 2014, Nix wrote: > I have a brand new Seagate Expansion Desk drive attached to my x86-64 > desktop. (I also have a 4TiB model of the same drive, but I haven't even > unboxed it: there seems little point as long as the 2TiB version doesn't > work.) I am seeing apparently the same

Re: ARM kirkwood: 1 Gb memory 25% unusable due to vmalloc region overlap

2014-09-07 Thread jurriaan
From: Russell King - ARM Linux Date: Sat, Sep 06, 2014 at 07:29:16PM +0100 > On Sat, Sep 06, 2014 at 07:32:07PM +0200, Jurriaan wrote: > > I updated my Qnap TS-212 with 256 Mb memory to a new TS-221 with 1 Gb > > memory. > > > > On booting, I see that a large chunk of that new memory is not used

Re: f2fs get_dnode_of_data oops

2014-09-07 Thread Tommi Rantala
2014-09-07 22:14 GMT+03:00 Tommi Rantala : > Hello, > > Hit this oops while fuzzing v3.17-rc3-176-g2b12164 with Trinity. > > Tommi > > > BUG: unable to handle kernel paging request at 8804338717a8 > IP: [] get_dnode_of_data+0x3a9/0x440 > PGD 4594067 PUD 0 > Oops: [#1] SMP DEBUG_PAGEALLOC

f2fs get_dnode_of_data oops

2014-09-07 Thread Tommi Rantala
Hello, Hit this oops while fuzzing v3.17-rc3-176-g2b12164 with Trinity. Tommi BUG: unable to handle kernel paging request at 8804338717a8 IP: [] get_dnode_of_data+0x3a9/0x440 PGD 4594067 PUD 0 Oops: [#1] SMP DEBUG_PAGEALLOC CPU: 0 PID: 4719 Comm: trinity-c3 Not tainted 3.17.0-rc3+ #33

Re: [PATCH 0/2] PCI/MSI: Remove arch_msi_check_device()

2014-09-07 Thread Alexander Gordeev
On Fri, Sep 05, 2014 at 03:27:49PM -0600, Bjorn Helgaas wrote: > > I applied these (with Michael's ack on the first, and v2 of the second) to > > pci/msi for v3.18, thanks! Hi Bjorn, I resent a series with updates that fix kbuild robot errors. Hopefully, the rebase for pci/msi would not cause

Re: bit fields && data tearing

2014-09-07 Thread James Bottomley
On Sun, 2014-09-07 at 09:21 -0700, Paul E. McKenney wrote: > On Sat, Sep 06, 2014 at 10:07:22PM -0700, James Bottomley wrote: > > On Thu, 2014-09-04 at 21:06 -0700, Paul E. McKenney wrote: > > > On Thu, Sep 04, 2014 at 10:47:24PM -0400, Peter Hurley wrote: > > > > Hi James, > > > > > > > > On

[PATCH v2 0/3] PCI/MSI: Remove arch_msi_check_device()

2014-09-07 Thread Alexander Gordeev
Hello, This is a cleanup effort to get rid of arch_msi_check_device() function. I am sending v2 series, since kbuild for v1 reports compile errors on ppc4xx and Armada 370. Still, I have not checked the fixes on these platforms. Changes since v1: - patch 1: 'pdev' undeclared compile error

[PATCH v2 3/3] PCI/MSI: Remove arch_msi_check_device()

2014-09-07 Thread Alexander Gordeev
There are no archs that override arch_msi_check_device() hook. Remove it as it is completely redundant. If an arch would need to check MSI/MSI-X possibility for a device it should make it within arch_setup_msi_irqs() hook. Cc: Michael Ellerman Cc: linuxppc-...@lists.ozlabs.org Cc:

[PATCH v2 1/3] PCI/MSI/PPC: Remove arch_msi_check_device()

2014-09-07 Thread Alexander Gordeev
Moving MSI checks from arch_msi_check_device() function to arch_setup_msi_irqs() function makes code more compact and allows removing unnecessary hook arch_msi_check_device() from generic MSI code. Cc: linuxppc-...@lists.ozlabs.org Cc: linux-...@vger.kernel.org Cc: Michael Ellerman

[PATCH v2 2/3] PCI/MSI/Armada-370-xp: Remove arch_msi_check_device()

2014-09-07 Thread Alexander Gordeev
Moving MSI checks from arch_msi_check_device() function to arch_setup_msi_irqs() function makes code more compact and allows removing unnecessary hook arch_msi_check_device() from generic MSI code. Cc: Thomas Gleixner Cc: Jason Cooper Cc: linux-...@vger.kernel.org Signed-off-by: Alexander

Re: [PATCH 0/3] Yet another stab at a fpga framework

2014-09-07 Thread Pavel Machek
Hi! > [resend with fixed email settings] > > The idea of the framework is to provide consistent ways of > programming raw images into FPGA's. > > Programming from device tree overlays is supported. > > The core (fpga-mgr.c) does not include a userspace interface > and just exports kernel

Re: [PATCH 4/6] virtio-blk: implement init_flush_rq

2014-09-07 Thread Christoph Hellwig
A couple comments not directly related to this patch, it's just a convenient vehicle for my rants :) > @@ -556,6 +555,19 @@ static int virtblk_init_request(void *data, struct > request *rq, > struct virtio_blk *vblk = data; > struct virtblk_req *vbr = blk_mq_rq_to_pdu(rq); > > +

Re: [PATCH 5/6] scsi-lib: implement init_flush_rq and its pair

2014-09-07 Thread Christoph Hellwig
> +static int __scsi_init_request(struct request *rq, int numa_node) Nitpick: Please use a sane name here, e.g. scsi_mq_alloc/free_sense_buffer. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH 0/6] blk-mq: initialize pdu of flush req explicitly

2014-09-07 Thread Christoph Hellwig
This works fine for me, although I still don't really like it very much. If you really want to go down the path of major surgery in this area we should probably allocate a flush request per hw_ctx, and initialize it using the normal init/exit functions. If we want to have proper multiqueue

Re: [PATCH 1/4] Documentation: use subdir-y to avoid unnecessary built-in.o files

2014-09-07 Thread Mark Brown
On Sun, Sep 07, 2014 at 10:59:53AM -0700, Randy Dunlap wrote: > The quilt-import.log from 20140829 says: > $ git am --patch-format=mbox > ../quilt/rd-docs/001-docum-use-subdiry-avoid-builtin.patch > and in the 20140905 git tree it says: > $ git am

Re: [PATCH] virtio_blk: merge S/G list entries by default

2014-09-07 Thread Christoph Hellwig
On Sun, Sep 07, 2014 at 02:41:53PM +0300, Michael S. Tsirkin wrote: > > Signed-off-by: Christoph Hellwig > > OK so this is an optimization patch right? > What kind of performance gain is observed with it? None. I actually wrote it when the block layer had a bug when dm was used on top of the

[PATCH] x86-64, ptdump: Mark espfix area only if existent

2014-09-07 Thread Mathias Krause
We should classify the espfix area as such only if we actually have enabled the corresponding option. Otherwise the page table dump might look confusing. Signed-off-by: Mathias Krause Cc: Arjan van de Ven Cc: H. Peter Anvin --- arch/x86/mm/dump_pagetables.c |4 1 file changed, 4

[PATCH 4/4] staging: ozwpan: use kmalloc_array over kmalloc with multiply

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozhcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c index ba2168f..4cfe56e 100644 ---

[PATCH 3/4] staging: ozwpan: fix redundant return in void function

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozproto.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ozwpan/ozproto.c b/drivers/staging/ozwpan/ozproto.c index cae0e6f..3d3a3a8 100644 ---

[PATCH 2/4] staging: ozwpan: fix redundant else after break or return

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozhcd.c | 8 +++- drivers/staging/ozwpan/ozpd.c | 10 -- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/staging/ozwpan/ozhcd.c

[PATCH 1/4] staging: ozwpan: fix missing blank line after declaration

2014-09-07 Thread anicoara
Signed-off-by: Adrian Nicoara --- Patch submitted as part of the Eudyptula challenge. drivers/staging/ozwpan/ozcdev.c| 5 + drivers/staging/ozwpan/ozeltbuf.c | 1 + drivers/staging/ozwpan/ozpd.c | 6 ++ drivers/staging/ozwpan/ozproto.c | 4

Re: [PATCH v10 net-next 2/2] net: filter: split filter.h and expose eBPF to user space

2014-09-07 Thread Pablo Neira Ayuso
On Sat, Sep 06, 2014 at 09:04:23AM -0700, Alexei Starovoitov wrote: > On Sat, Sep 6, 2014 at 7:10 AM, Pablo Neira Ayuso wrote: > > On Thu, Sep 04, 2014 at 10:17:18PM -0700, Alexei Starovoitov wrote: > >> allow user space to generate eBPF programs > >> > >> uapi/linux/bpf.h: eBPF instruction set

[PATCH v2 4/4] PCI: remove assignement in non straight forward if condition

2014-09-07 Thread Quentin Lambert
The modifications effectively change the value of len_tmp in the case where the first condition is not met. Signed-off-by: Quentin Lambert --- drivers/pci/hotplug/ibmphp_res.c | 13 + drivers/pci/pci.c| 6 +- drivers/pci/slot.c | 12 ++-- 3

[PATCH v2 3/4] PCI: remove assignement from if conditions

2014-09-07 Thread Quentin Lambert
The following Coccinelle semantic patch was used to find and correct cases of assignements in if conditions: @@ expression var, expr; statement S; @@ + var = expr; if( - (var = expr) + var ) S Signed-off-by: Quentin Lambert --- drivers/pci/hotplug/acpi_pcihp.c| 3 ++-

[PATCH v2 2/4] PCI: fix a simple if coding style issue

2014-09-07 Thread Quentin Lambert
Removing curly braces in simple if cases. Signed-off-by: Quentin Lambert --- drivers/pci/hotplug/cpci_hotplug_core.c | 4 ++-- drivers/pci/hotplug/cpcihp_zt5550.c | 11 +-- drivers/pci/hotplug/cpqphp_core.c | 3 +-- drivers/pci/hotplug/cpqphp_ctrl.c | 15

[PATCH v2 1/4] PCI: fix a missing space coding style issue

2014-09-07 Thread Quentin Lambert
Modified the files in the directory so that they respect the coding style with regards to parenthesis being preceded by a space. Signed-off-by: Quentin Lambert --- drivers/pci/hotplug/acpiphp_ibm.c| 2 +- drivers/pci/hotplug/cpcihp_generic.c | 28 ++--

[PATCH v2 0/4] Extending the first version of the patch to the entire directory

2014-09-07 Thread Quentin Lambert
Patch 1 and 2 are straight forward extension from the previous version of the patch. Patch 3 introduces checkpatch warning due to lines surpassing 80 characters. Another warning concerning too many preceding tabs is not fixed either. Patch 4 are less straight forward similar modifications the

Re: [PATCH 1/4] Documentation: use subdir-y to avoid unnecessary built-in.o files

2014-09-07 Thread Randy Dunlap
On 09/05/14 17:07, Randy Dunlap wrote: > On 09/04/14 03:42, Mark Brown wrote: >> On Wed, Sep 03, 2014 at 04:38:55PM -0700, Randy Dunlap wrote: >> >>> Hm, it could be a problem with the Index line. I'll disable that option >>> in quilt and see if that helps for the next time. >> >> Still seems to

[PATCH] dmaengine: qcom_bam_dma: add one more optional clock

2014-09-07 Thread Stanimir Varbanov
The BAM is tightly coupled with the peripheral to which it belongs. In that sprit to access the BAM configuration registers the driver needs to enable some peripheral clocks. Currently the DT node enables bamclk which seems is not enough for some peripherals (for example the crypto engine wants

3.16.2: 2TiB Seagate Expansion Desk apparently still broken with both USB mass storage *and* UAS: some debugging output

2014-09-07 Thread Nix
I have a brand new Seagate Expansion Desk drive attached to my x86-64 desktop. (I also have a 4TiB model of the same drive, but I haven't even unboxed it: there seems little point as long as the 2TiB version doesn't work.) I am seeing apparently the same problem as Alexandre Oliva reported in

Re: [PATCH] perf/x86/intel: Mark initialization code as such

2014-09-07 Thread Mathias Krause
On 26 August 2014 18:49, Mathias Krause wrote: > A few of the initialization functions are missing the __init annotation. > Fix this and thereby allow ~680 additional bytes of code to be released > after initialization. > > Signed-off-by: Mathias Krause > --- >

[GIT PULL] RCU fix

2014-09-07 Thread Ingo Molnar
Linus, Please pull the latest core-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git core-urgent-for-linus # HEAD: 651bc1a474ad5f3a94587117cf509d7fa9247f69 Merge branch 'rcu/urgent' of

  1   2   3   4   >