[PATCH RFC v2 09/29] PCI/MSI: Factor out pci_get_msi_cap() interface

2013-10-19 Thread Alexander Gordeev
Device drivers can use this interface to obtain maximum number of MSI interrupts the device supports and use that number i.e. in a following call to pci_enable_msi_block() interface. Signed-off-by: Alexander Gordeev --- Documentation/PCI/MSI-HOWTO.txt | 15 +++ drivers/pci/msi.c

[PATCH RFC v2 00/29] Introduce pcim_enable_msi*() family helpers

2013-10-19 Thread Alexander Gordeev
This series is against "next" branch in Bjorn's repo: git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git Currently many device drivers need contiguously call functions pci_enable_msix() for MSI-X or pci_enable_msi_block() for MSI in a loop until success or failure. This update generaliz

[PATCH RFC v2 10/29] PCI/MSI: Get rid of pci_enable_msi_block_auto() interface

2013-10-19 Thread Alexander Gordeev
As result of introduction of pci_get_msi_cap() interface pci_enable_msi_block_auto() function became superflous. To enable maximum possible number of MSIs drivers will first obtain that number from pci_get_msi_cap() function and then call pci_enable_msi_block() interface. Signed-off-by: Alexander

[PATCH RFC v2 11/29] PCI/MSI: Convert pci_msix_table_size() to a public interface

2013-10-19 Thread Alexander Gordeev
Make pci_msix_table_size() return error code if the device does not support MSI-X. This update is needed to create a consistent MSI-X counterpart for pci_get_msi_cap() MSI interface. Device drivers can use this interface to obtain maximum number of MSI-X interrupts the device supports and i.e. use

[PATCH RFC v2 12/29] PCI/MSI: Introduce pcim_enable_msi*() family helpers

2013-10-19 Thread Alexander Gordeev
Currently many device drivers need contiguously call functions pci_enable_msix() for MSI-X or pci_enable_msi_block() for MSI in a loop until success or failure. This update generalizes this usage pattern and introduces pcim_enable_msi*() family helpers. As result, device drivers do not have to dea

[PATCH] cpu-idle: cpuidle-ux500: fixed a coding style issue

2013-10-19 Thread Saumya Ranjan Kuanr
Fixed a coding style issue Signed-off-by: Saumya Ranjan Kuanr --- drivers/cpuidle/cpuidle-ux500.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpuidle/cpuidle-ux500.c b/drivers/cpuidle/cpuidle-ux500.c index e056465..746d716 100644 --- a/drivers/cpuidle/cpuidle-ux

!

2013-10-19 Thread Ana Flavia Maria
-- Loan at 3% interest rate Loan offer.. AreYou Interested>? -- 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 P

Re: [PATCH] m68k/atari: Call paging_init() before nf_init()

2013-10-19 Thread Geert Uytterhoeven
On Sat, Oct 19, 2013 at 2:33 AM, Michael Schmitz wrote: >> With the DISCONTIGMEM memory model, the kernel must be stored in the >> first memory block. As ST-RAM is before FastRAM in memory, you cannot >> have the kernel in FastRAM without losing ST-RAM (as main memory --- >> you can still e.g. ior

Re: [PATCH] x86: Run checksumming in parallel accross multiple alu's

2013-10-19 Thread Ingo Molnar
* Doug Ledford wrote: > >> Based on these, prefetching is obviously a a good improvement, but > >> not as good as parallel execution, and the winner by far is doing > >> both. > > OK, this is where I have to chime in that these tests can *not* be used > to say anything about prefetch, and no

Re: [PATCH] wireless: rt2800lib: Fix typo on checking

2013-10-19 Thread Gertjan van Wingerde
On 10/19/13 02:20, Felipe Pena wrote: > On rt2800_config_channel_rf53xx function the member default_power1 is checked > for bound limit, but default_power2 is used instead. > > Signed-off-by: Felipe Pena Good catch. Acked-by: Gertjan van Wingerde > --- > drivers/net/wireless/rt2x00/rt2800lib

Re: [PATCH 0/3] : KGDB/KDB/UV updates.

2013-10-19 Thread Ingo Molnar
* Mike Travis wrote: > > * Change the fix for KDB not defined build problem by changing > the kgdb_nmicallin() interface to include the KDB specific > reason code. This removes a dependency on KDB in the debug > core. Also requires a change the call in from UV NMI handler. > > * Fix so

[patch] pktcdvd: debugfs functions return NULL on error

2013-10-19 Thread Dan Carpenter
My static checker complains correctly that this is potential NULL dereference because debugfs functions return NULL on error. They return an ERR_PTR if they are configured out. We don't need to check for ERR_PTR because if debugfs is stubbed out the dummy functions won't complain about that. We

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
Hi Ricardo, On 10/14/2013 09:41 AM, Ricardo Ribalda Delgado wrote: vb2_fop_relase does not held the lock although it is modifying the queue->owner field. [...] diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index 9fc4bab..3a961ee 100644 --- a/

Re: [PATCH v3 4/9] ACPI, x86: Extended error log driver for x86 platform

2013-10-19 Thread Borislav Petkov
On Fri, Oct 18, 2013 at 10:22:26PM +, Luck, Tony wrote: > @@ -154,6 +154,10 @@ void mce_log(struct mce *mce) > /* Emit the trace record: */ > trace_mce_record(mce); > > + if (mce_ext_err_print) > + if (mce_ext_err_print(NULL, m.extcpu, i)) > + r

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester On Sat, Oct 19, 2013 at 11:55 AM, Sylwester Nawrocki wrote: > Hi Ricardo, > > > On 10/14/2013 09:41 AM, Ricardo Ribalda Delgado wrote: >> >> vb2_fop_relase does not held the lock although it is modifying the >> queue->owner field. > > [...] > >> diff --git a/drivers/media/v4l2-cor

Re: [PATCH v3] efifb: prevent null-deref when iterating dmi_list

2013-10-19 Thread David Herrmann
ping On Wed, Oct 2, 2013 at 6:43 PM, David Herrmann wrote: > From: James Bates > > The dmi_list array is initialized using gnu designated initializers, and > therefore may contain fewer explicitly defined entries as there are > elements in it. This is because the enum above with M_xyz constants

[no subject]

2013-10-19 Thread Zena, Sinethemba
Your Mailbox have exceeded the storage limit and due for Maintenance. Please CLICK HERE to validate your Mailbox. If it doesn't work please COPY and PASTE it in the Address Url. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
On 10/19/2013 12:22 PM, Ricardo Ribalda Delgado wrote: On Sat, Oct 19, 2013 at 11:55 AM, Sylwester Nawrocki wrote: > On 10/14/2013 09:41 AM, Ricardo Ribalda Delgado wrote: >> >> vb2_fop_relase does not held the lock although it is modifying the >> queue->owner field. > [...] >> diff --

Charity Donation

2013-10-19 Thread Gillian and Adrian Bayford
My wife and i won £148.6 Million Pounds last year, and we have done lot of charity donation, so we decide to give 1.5 Million Pounds each to 5 lucky people, lucky for you, your email, was given to us by Google management as one of our lucky precipitants. For verification process see below Pleas

Re: A review of dm-writeboost

2013-10-19 Thread Akira Hayakawa
Dave, # -EIO retuned corrupts XFS I turned up lockdep, frame pointer, xfs debug and also changed to 3.12.0-rc5 from rc1. What's changed is that the problem we discussed in previous mails *never* reproduce. However, if I turn off the lockdep only it hangs up by setting blockup to 1 and then to 0

RE: [PATCH] usb-storage: scsiglue: Changing the command result

2013-10-19 Thread Vishal Annapurve
Hi, Attaching the new patch which will replace all the occurrences of DID_ABORT with DID_TIMOUT in USB Storage whenever it sees US_FLIDX_TIMED_OUT bit is set. Regards, Vishal -Original Message- From: Alan Stern [mailto:st...@rowland.harvard.edu] Sent: Friday, October 18, 2013 4:10 PM T

net: sctp: possible dereference after freeing

2013-10-19 Thread Geyslan Gregório Bem
Hi maintainers, I would like to know if these are catches: /net/sctp/endpointola.c (281) static void sctp_endpoint_destroy(struct sctp_endpoint *ep) { struct sock *sk; ... kfree(ep); SCTP_DBG_OBJCNT_DEC(ep); } The 'ep' object counter is being decremented?! Is the kfree to be there in

Re: [PATCH v3 8/9] ACPI, APEI, CPER: Cleanup CPER memory error output format

2013-10-19 Thread Chen Gong
On Fri, Oct 18, 2013 at 05:31:21PM +0530, Naveen N. Rao wrote: > Date: Fri, 18 Oct 2013 17:31:21 +0530 > From: "Naveen N. Rao" > To: "Chen, Gong" , tony.l...@intel.com, > b...@alien8.de, j...@perches.com, m.che...@samsung.com > CC: aroza...@redhat.com, linux-a...@vger.kernel.org, > linux-kernel@

Re: [PATCH v3 4/9] ACPI, x86: Extended error log driver for x86 platform

2013-10-19 Thread Chen Gong
On Fri, Oct 18, 2013 at 06:07:56PM +0530, Naveen N. Rao wrote: > Date: Fri, 18 Oct 2013 18:07:56 +0530 > From: "Naveen N. Rao" > To: "Chen, Gong" , tony.l...@intel.com, > b...@alien8.de, j...@perches.com, m.che...@samsung.com > CC: aroza...@redhat.com, linux-a...@vger.kernel.org, > linux-kernel@

Re: net: sctp: possible dereference after freeing

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 Geyslan Gregório Bem : > Hi maintainers, > > I would like to know if these are catches: > > /net/sctp/endpointola.c (281) > static void sctp_endpoint_destroy(struct sctp_endpoint *ep) > { > struct sock *sk; > ... > kfree(ep); > SCTP_DBG_OBJCNT_DEC(ep); > } > > The 'ep' object

linux-next: manual merge of the userns tree

2013-10-19 Thread Mark Brown
Today's linux-next merge of the userns tree got a conflict in fs/fuse/dir.c between 3c70b8eed (fuse: don't check_submounts_and_drop() in RCU walk) in the fuse tree and 40216baa0 (vfs: Lazily remove mounts on unlinked files and directories. v2) in the userns tree. I fixed it up as below and can car

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Oleg Nesterov
On 10/17, Steven Rostedt wrote: > > On Thu, 17 Oct 2013 22:44:56 -0300 > "Geyslan G. Bem" wrote: > > > and fix the possible 'dir' > > assignment after freeing it. This should be safe afaics, nobody will use it anyway. Even subsystem_release() won't be called if .open() fails. But I agree this doe

linux-next: Tree for Oct 18

2013-10-19 Thread Mark Brown
Hi all, I've uploaded today's linux-next tree to the master branch of the repository below: git://gitorious.org/thierryreding/linux-next.git A next-20131018 tag is also provided for convenience. A few new conflicts today but otherwise uneventful. x86_64 allmodconfig builds after each m

[GIT PULL] uprobes: fix fork() with the pending ret-probe(s)

2013-10-19 Thread Oleg Nesterov
Ingo, please pull from git://git.kernel.org/pub/scm/linux/kernel/git/oleg/misc uprobes/core This fixes the serious bug, a forked process crashes if it returns from the probed function called by its parent, this cc's stable. Also the minor fix from Ralf. Oleg Nesterov (6): uprobes: Chang

[PATCH] crypto: n2_core: insert '!err' condition in else scope

2013-10-19 Thread Geyslan G. Bem
This patch moves the '!err' condition into the above else scope, what is more obvious and has the secondary goal of avoid false-positives in statical analyze tools. Signed-off-by: Geyslan G. Bem --- drivers/crypto/n2_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/

Re: [GIT PULL] uprobes: fix fork() with the pending ret-probe(s)

2013-10-19 Thread Ingo Molnar
* Oleg Nesterov wrote: > Ingo, please pull from > > git://git.kernel.org/pub/scm/linux/kernel/git/oleg/misc uprobes/core > > This fixes the serious bug, a forked process crashes if it returns > from the probed function called by its parent, this cc's stable. > > Also the minor fix from Ralf

[PATCH] digsig: replace ERR_PTR(PTR_ERR(...)) with ERR_CAST

2013-10-19 Thread Geyslan G. Bem
Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). 'err_cast.cocci' catch. Signed-off-by: Geyslan G. Bem --- lib/digsig.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/digsig.c b/lib/digsig.c index 2f31e6a..8793aed 100644 --- a/lib/digsig.c +++ b/lib/digsi

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Oleg Nesterov
Steven, et all, sorry for off-topic... You probably know that kernel/trace/ is not trivial ;) and the fact that cscope doesn't shows the callers in kernel/trace/trace_events.c doesn't really help. Fixed by the patch below, but I am not sure it is fine to uglify the code to help the buggy tools. I

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 Oleg Nesterov : > On 10/17, Steven Rostedt wrote: >> >> On Thu, 17 Oct 2013 22:44:56 -0300 >> "Geyslan G. Bem" wrote: >> >> > and fix the possible 'dir' >> > assignment after freeing it. > > This should be safe afaics, nobody will use it anyway. Even > subsystem_release() won't be calle

Re: [PATCH linux-next] cifs: Cleanup and clarify CalcNTLMv2_response()

2013-10-19 Thread Shirish Pargaonkar
Tim, Since when a NTLMv2 response is sent, contains a session key, hmac-md5 digest, and the blob, the offset has + 8 to include 16 bytes of hmac-md5 digest. hmac-md5 digest is based on 8 bytes of server challenge and the blob. So hmac-md5 digest (output) overwrites total 16 bytes in overall ntlmv

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Oleg Nesterov
On 10/19, Geyslan Gregório Bem wrote: > > 2013/10/19 Oleg Nesterov : > > On 10/17, Steven Rostedt wrote: > >> > >> I'm thinking of just nuking the tracing_open_generic() here. The only > >> thing it does here is the tracing_disabled check. The assignment of > >> inode->i_private to filp->private_da

[PATCHv4 12/16] staging: usbip: Pass session keys to the kernel

2013-10-19 Thread Dominik Paulus
This extends the userspace code to write the generated session keys to sysfs in hexadecimal encoding after establishing the connection. The kernel code is modified to parse the session keys. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/stub_dev.c

[PATCHv4 03/16] staging: usbip: Add CIDR matching helper functions

2013-10-19 Thread Dominik Paulus
This patch adds a few utility functions to match IP addresses against CIDR masks. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/utils.c | 84 + drivers/staging/usbip/userspace/src/utils.h | 15 ++ 2 files chang

[PATCHv4 10/16] staging: usbip: TLS for all userspace communication

2013-10-19 Thread Dominik Paulus
This patch extends the TLS support to cover all communication in userspace. The TLS connection is released shortly before the socket is passed to the kernel. This requires for additional connection state to be passed between functions. We thus replaced the sockfd by a struct containing the TLS con

[PATCHv4 14/16] staging: usbip: Add encryption support to kernel

2013-10-19 Thread Dominik Paulus
This adds code performing the actual encryption and authentication operations in the usbip kernel code. The whole data stream may now be encrypted and authenticated with AES-GCM and symmetric 128 bit keys. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/Kcon

[PATCHv4 15/16] staging: usbip: Update documentation

2013-10-19 Thread Dominik Paulus
From: Tobias Polzer README was updated and cleaned. It now contains just one example, which was updated to use encryption. Also, the new crypto behaviour is documented. The usbip "port" command has been removed from the README, as it isn't supported by newer userland versions. One dead link was r

[PATCHv4 13/16] staging: usbip: Wrap kernel_sendmsg()/recvmsg()

2013-10-19 Thread Dominik Paulus
This adds two simple wrappers around kernel_sendmsg() and kernel_recvmsg() that can be extended to perform additional cryptographic operations on the data before sending it. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/stub_rx.c | 2 +- drivers/stag

[PATCHv4 11/16] staging: usbip: Exchange session keys in userspace

2013-10-19 Thread Dominik Paulus
In preparation for the kernel crypto support, we exchange two - randomly generated - session keys between usbip and usbipd to be used for encrypting all traffic generated in kernelspace. We use two different 128-bit keys, one for sending and one for receiving. Both are generated by the client (usbi

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 Oleg Nesterov : > On 10/19, Geyslan Gregório Bem wrote: >> >> 2013/10/19 Oleg Nesterov : >> > On 10/17, Steven Rostedt wrote: >> >> >> >> I'm thinking of just nuking the tracing_open_generic() here. The only >> >> thing it does here is the tracing_disabled check. The assignment of >> >>

[PATCHv4 16/16] staging: usbip: Increment version number to 1.2.1

2013-10-19 Thread Dominik Paulus
Also increment the kernel module version number to match the userspace version, as compatibility with old userspace utilities is now at least partially broken. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/usbip_common.h | 2 +- drivers/staging/usb

[PATCHv4 04/16] staging: usbip: Add ACL support to usbip bind

2013-10-19 Thread Dominik Paulus
Add the command line argument -a (--allow) to usbip bind to specify networks allowed to attach to the device and code to store the ACLs in sysfs. Signed-off-by: Kurt Kanzenbach Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/doc/usbip.8 | 8

[PATCHv4 02/16] staging: usbip: Add kernel support for client ACLs

2013-10-19 Thread Dominik Paulus
This patch adds the possibility to stored ACLs for allowed clients for each stub device in sysfs. It adds a new sysfs entry called "usbip_acl" for each stub device, containing a list of CIDR masks of allowed clients. This file will be used by usbip and usbipd to store the ACL. Signed-off-by: Kurt

[PATCHv4 00/16] staging: usbip: Add kernel support for client ACLs

2013-10-19 Thread Dominik Paulus
Hi, sorry for the messed-up error checking. We fixed some of the style nitpicks and hopefully all of the mistakes you pointed out (thank you for that!). Regards, Dominik and Tobias -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majo

[PATCHv4 06/16] staging: usbip: Add proper error reporting

2013-10-19 Thread Dominik Paulus
This patch adds new error codes and features extended error reporting in op_common packets. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/usbip_attach.c | 4 +- drivers/staging/usbip/userspace/src/usbip_list.c | 3 +- .../staging/usbip/us

[PATCHv4 01/16] staging: usbip: Add support for client authentication

2013-10-19 Thread Dominik Paulus
This patch adds support for authenticating both client and server using a pre-shared passphrase using SRP (Secure Remote Password) over TLS (see RFC 5054) using GnuTLS. Both usbip and usbipd now accept a shared secret as a command line argument. Currently, the established TLS connection is only use

[PATCHv4 07/16] staging: usbip: Handle usbip being started as user

2013-10-19 Thread Dominik Paulus
usbip now prints an error message when started as user and requiring root access. Also, some debug messages are changed to error messages so the command line utilities now print less confusing (and more verbose) error messages when not used correctly. Signed-off-by: Dominik Paulus Signed-off-by:

[PATCHv4 09/16] staging: usbip: Separate protocol/program version

2013-10-19 Thread Dominik Paulus
Not all new program versions necessarily introduce non-backwards-compatible protocol changes. We thus move the definition of the protocol version from configure.ac to usbip_network.h, where it logically belongs to. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/u

[PATCHv4 08/16] staging: usbip: Improve debug output

2013-10-19 Thread Dominik Paulus
For IPv6, IP:Port is unreadable. Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/usbipd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/usbip/userspace/src/usbipd.c b/drivers/staging/usbip/userspace/src/us

[PATCHv4 05/16] staging: usbip: Add support for ACLs in usbipd

2013-10-19 Thread Dominik Paulus
Interpret the ACLs stored in sysfs in usbipd and reject clients not matching one of the ACLs. Signed-off-by: Kurt Kanzenbach Signed-off-by: Dominik Paulus Signed-off-by: Tobias Polzer --- drivers/staging/usbip/userspace/src/Makefile.am | 2 +- drivers/staging/usbip/userspace/src/usbipd.c|

Re: [PATCH 1/6] x86: speed up int3-based patching using less paranoid write

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:27), Petr Mladek wrote: > This change is inspired by the int3-based patching code used in > ftrace. See the commit fd4363fff3d9 (x86: Introduce int3 > (breakpoint)-based instruction patching). > > When trying to use text_poke_bp in ftrace, the result was slower > than the original i

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:27), Petr Mladek wrote: > We would like to use text_poke_bp in ftrace. It might be called also during > boot when the interupts are disabled. We need to enable them for syncing > the cores on each CPU. Otherwise, there might be a deadlock, see the > warning in "smp_call_function_many

[RFC PATCH 0/5] nohz: Fix racy sleeptime stats v2

2013-10-19 Thread Frederic Weisbecker
Hi, So this is a new iteration with some more changes on top of the latest discussions we had. Unfortunately it's not yet a real viable solution. Check out patch 4/5 for details, I think that's too much overhead. Also it changes enough the semantics to break the idle sleeptime accounting along

[PATCH 5/5] nohz: Synchronize sleep time stats with seqlock

2013-10-19 Thread Frederic Weisbecker
When some call site uses get_cpu_*_time_us() to read a sleeptime stat, it deduces the total sleeptime by adding the pending time to the last sleeptime snapshot if the CPU target is idle. Namely this sums up to: sleeptime = ts($CPU)->idle_sleeptime; if (ts($CPU)->idle_active)

[PATCH 1/5] nohz: Convert a few places to use local per cpu accesses

2013-10-19 Thread Frederic Weisbecker
A few functions use remote per CPU access APIs when they deal with local values. Just to the right conversion to improve performance, code readability and debug checks. Signed-off-by: Frederic Weisbecker Cc: Fernando Luis Vazquez Cao Cc: Tetsuo Handa Cc: Thomas Gleixner Cc: Ingo Molnar Cc: P

[PATCH 3/5] timer: Change idle/iowait accounting semantics

2013-10-19 Thread Frederic Weisbecker
To prepare for fixing a race between iowait and idle time stats, this patch changes the following semantics: * iowait time is going to be accounted from the scheduler rather than the dynticks idle code, lets remove it from the /proc/timer_list dump. * idle sleeptime now also includes the iowait t

[PATCH 2/5] nohz: Only update sleeptime stats locally

2013-10-19 Thread Frederic Weisbecker
The idle and io sleeptime stats can be updated concurrently from callers of get_cpu_idle_time_us(), get_cpu_iowait_time_us() and tick_nohz_stop_idle(). Updaters can easily race and mess up with internal datas coherency, for example when a governor calls a get_cpu_*_time_us() API and the target CPU

[PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Frederic Weisbecker
The iowait time accounting has awkward semantics. It is performed per CPU. The iowait time of a CPU starts when a task sleeps on IO on this CPU and stops when that task later wakes up on any CPU. This assumes that a sleeping task is still assigned to the CPU it was running on last until it wakes u

Re: [PATCH v3 1/3] SDT markers listing by perf:

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:44), Hemant Kumar wrote: > This patch will enable perf to list all the sdt markers present > in an elf file. The markers are present in the .note.stapsdt section > of the elf. We can traverse through this section and collect the > required info about the markers. > We can use '-M/--m

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Peter Zijlstra
On Sat, Oct 19, 2013 at 05:17:20PM +0200, Frederic Weisbecker wrote: > +u64 get_cpu_iowait_time_us(int cpu, u64 *last_update_time) > +{ > + ktime_t iowait, delta = { .tv64 = 0 }; > + struct rq *rq = cpu_rq(cpu); > + ktime_t now = ktime_get(); > + unsigned int seq; > + > + do { >

Re: [PATCH v3 2/3] Support for perf to probe into SDT markers:

2013-10-19 Thread Masami Hiramatsu
(2013/10/18 23:44), Hemant Kumar wrote: > This allows perf to probe into the sdt markers/notes present in > the libraries and executables. We try to find the associated location > and handle prelinking (since, stapsdt notes section is not allocated > during runtime). Prelinking is handled with the

Re: kprobe pre_handler change return IP

2013-10-19 Thread Masami Hiramatsu
(2013/10/17 21:57), Liuyongan wrote: > I use kprobe to probe a function suppose: > int is_winter(int num) { ... } > int replace_is_winter(int num) { ...} > I want to replace is_winter() with replace_is_winter(), so when we call > is_winter, replace_is_winter will be called. >

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Frederic Weisbecker
On Sat, Oct 19, 2013 at 05:35:35PM +0200, Peter Zijlstra wrote: > On Sat, Oct 19, 2013 at 05:17:20PM +0200, Frederic Weisbecker wrote: > > +u64 get_cpu_iowait_time_us(int cpu, u64 *last_update_time) > > +{ > > + ktime_t iowait, delta = { .tv64 = 0 }; > > + struct rq *rq = cpu_rq(cpu); > > + k

Re: [PATCH v5 0/3] ARM: S5PV210: move to common clk framework

2013-10-19 Thread Tomasz Figa
Hi Mike, Kukjin, Rafael, On Tuesday 24 of September 2013 14:50:06 Mateusz Krawczuk wrote: > This patch series is the new s5pv210 clock implementation > (using common clk framework). > > This implementation is compatible with device tree definition and board > files. > > This patch series is base

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Peter Zijlstra
On Sat, Oct 19, 2013 at 06:02:37PM +0200, Frederic Weisbecker wrote: > > I just had a look at delayacct; wth wrote that crap; that too uses > > gtod. > > I can't find where it does that. kernel/delayacct.c doesn't seem to at least. Look for do_posix_clock_monotonic_gettime() ;-) -- To unsubscribe

[PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda
vb2_fop_relase does not held the lock although it is modifying the queue->owner field. This could lead to race conditions on the vb2_perform_io function when multiple applications are accessing the video device via read/write API: [ 308.297741] BUG: unable to handle kernel NULL pointer dereferenc

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester I have just posted a new version. Please take a look to it, it should fix your issue. I havent tried it in hw because I am out of the office. Regards! On Sat, Oct 19, 2013 at 12:51 PM, Sylwester Nawrocki wrote: > On 10/19/2013 12:22 PM, Ricardo Ribalda Delgado wrote: >> >> On S

Re: [PATCH 4/5] sched: Refactor iowait accounting

2013-10-19 Thread Frederic Weisbecker
On Sat, Oct 19, 2013 at 06:05:17PM +0200, Peter Zijlstra wrote: > On Sat, Oct 19, 2013 at 06:02:37PM +0200, Frederic Weisbecker wrote: > > > I just had a look at delayacct; wth wrote that crap; that too uses > > > gtod. > > > > I can't find where it does that. kernel/delayacct.c doesn't seem to at

[PATCH 0/1] hung_task debugging: Add tracepoint to report the hang

2013-10-19 Thread Oleg Nesterov
Hi, We have a feature request, the customer needs something more hookable than just printk's from check_hung_task() to implement the user-space watchdog which can potentially resolve the problems which caused the hang. The patch simply adds a tracepoint into check_hung_task(), do you think we can

[PATCH] vsprintf: add Bluetooth UUID %pU[rR] format specifier

2013-10-19 Thread Marcel Holtmann
The Bluetooth UUID is used in big endian reversed order. Add new modifier to print a UUID in big endian, but where the input byte stream is actually in reversed order. This is similar to %pMR that allows to print a MAC address in reversed order since that is how the Bluetooth BD_ADDR is actually r

[PATCH 1/1] hung_task debugging: Add tracepoint to report the hang

2013-10-19 Thread Oleg Nesterov
Currently check_hung_task() prints a warning if it detects the problem, but it is not convenient to watch the system logs if user-space wants to be notified about the hang. Add the new trace_sched_process_hang() into check_hung_task(), this way a user-space monitor can easily wait for the hang and

Re: [PATCH] typo fixes (coordiante -> coordinate) in am335x

2013-10-19 Thread Jan Lübbe
On Wed, 2013-08-21 at 00:58 -0700, Tony Lindgren wrote: > * Zubair Lutfullah [130715 08:33]: > > Did a grep for coordiante and replaced them all > > with coordinate. > > > > This applies to the mfd-next tree. > > This should be safe to apply via the MFD tree as a non-critical > fix assuming the

Re: [PATCH v2 03/13] uprobes: allow arch access to xol slot

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > Allow arches to customize how the instruction is filled into the xol > slot. ARM will use this to insert an undefined instruction after the > real instruction in order to simulate a single step of the instruction > without hardware support. OK, but > +void __weak

Re: [PATCH v2 04/13] uprobes: allow arch-specific initialization

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > Add a weak function for any architecture-specific initialization. ARM > will use this to register the handlers for the undefined instructions it > uses to implement uprobes. Could you explain why ARM can't simply do the necessary initialization in arch/arm/kernel/u

Re: [PATCH v2 05/13] uprobes: add arch write opcode hook

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > Allow arches to write the opcode with a custom function. ARM needs to > customize the swbp instruction depending on the condition code of the > instruction it replaces. Well, we already have "__weak set_swbp(auprobe, ...)", can't arm use it? If not, > +void __wea

Re: linux-next: Tree for Oct 15 (asymmetric keys)

2013-10-19 Thread Randy Dunlap
On 10/15/13 15:43, Randy Dunlap wrote: > On 10/15/13 07:02, Thierry Reding wrote: >> Hi all, >> >> I've uploaded today's linux-next tree to the master branch of the >> repository below: >> >> git://gitorious.org/thierryreding/linux-next.git >> >> A next-20131015 tag is also provided for con

Re: [PATCH v2 02/13] uprobes: allow ignoring of probe hits

2013-10-19 Thread Oleg Nesterov
On 10/15, David Long wrote: > > @@ -1732,9 +1732,6 @@ static void handle_swbp(struct pt_regs *regs) > return; > } > > - /* change it in advance for ->handler() and restart */ > - instruction_pointer_set(regs, bp_vaddr); > - Well, this looks obviously wrong. This SET_I

Re: net: sctp: possible dereference after freeing

2013-10-19 Thread Vlad Yasevich
On Oct 19, 2013, at 7:49 AM, Geyslan Gregório Bem wrote: > 2013/10/19 Geyslan Gregório Bem : >> Hi maintainers, >> >> I would like to know if these are catches: >> >> /net/sctp/endpointola.c (281) >> static void sctp_endpoint_destroy(struct sctp_endpoint *ep) >> { >>struct sock *sk; >> ..

Re: int3 doing rcu_read_lock()

2013-10-19 Thread Paul E. McKenney
On Fri, Oct 18, 2013 at 11:13:51PM -0400, Steven Rostedt wrote: > Hey Paul, > > I hit this in my tests: > > [ 1597.688015] === > [ 1597.688015] [ INFO: suspicious RCU usage. ] > [ 1597.688015] 3.12.0-rc4-test+ #48 Not tainted > [ 1597.688015] --

[PATCH 1/2] Fixed style issue. Changed tabs to 8 spaces with expand(1).

2013-10-19 Thread Souvik Banerjee
Fixed a coding style issue. Tabs have to be 8 spaces. Signed-off-by: Souvik Banerjee --- drivers/platform/x86/dell-wmi.c | 396 1 file changed, 198 insertions(+), 198 deletions(-) diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-

[PATCH 2/2] Added support for the monitor hotkey present on laptops like the Dell N4110

2013-10-19 Thread Souvik Banerjee
Adds support for the monitor switching hotkey on laptops such as the N4110. Signed-off-by: Souvik Banerjee --- drivers/platform/x86/dell-wmi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/platform/x86/dell-wmi.c b/drivers/platform/x86/dell-wmi.c index 5adb60a..a880183 100644 ---

Re: [PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
On 10/19/2013 06:07 PM, Ricardo Ribalda wrote: [...] --- drivers/media/platform/exynos4-is/fimc-capture.c | 2 +- drivers/media/platform/exynos4-is/fimc-lite.c| 2 +- drivers/media/usb/em28xx/em28xx-video.c | 2 +- drivers/media/v4l2-core/videobuf2-core.c | 18 +++

Re: 3.12-rcX - NFS regression - kswapd0 / kswapd1 stays using 100% CPU?

2013-10-19 Thread Helge Deller
On 10/18/2013 10:12 PM, Myklebust, Trond wrote: > On Fri, 2013-10-18 at 22:03 퍭, Helge Deller wrote: >> On 10/18/2013 09:36 PM, Myklebust, Trond wrote: >>> Also, could you please try a sysRQ-t the next time it happens, so that >>> we can get a trace of where the mount program is hanging. Knowing th

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Steven Rostedt
On Sat, 19 Oct 2013 11:41:10 -0300 Geyslan Gregório Bem wrote: > Let's wait Steve's reply about further use of tracing_is_disabled(). Might want to ping me later. This weekend I'm taking my daughter to colleges, and early Monday morning I'm leaving to Edinburgh (arriving on Tuesday). -- Steve

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Steven Rostedt
On Sun, 20 Oct 2013 00:02:32 +0900 Masami Hiramatsu wrote: > (2013/10/18 23:27), Petr Mladek wrote: > > We would like to use text_poke_bp in ftrace. It might be called also during > > boot when the interupts are disabled. We need to enable them for syncing > > the cores on each CPU. Otherwise, th

Re: [PATCH 2/6] x86: allow to call text_poke_bp during boot

2013-10-19 Thread Steven Rostedt
[ Added Paul because he'll understand this ] On Sat, 19 Oct 2013 15:16:58 -0400 Steven Rostedt wrote: > On Sun, 20 Oct 2013 00:02:32 +0900 > Masami Hiramatsu wrote: > > > (2013/10/18 23:27), Petr Mladek wrote: > > > We would like to use text_poke_bp in ftrace. It might be called also > > > d

Re: [PATCH] tracing: fix referencing after memory freeing and refactors code

2013-10-19 Thread Geyslan Gregório Bem
2013/10/19 Steven Rostedt : > On Sat, 19 Oct 2013 11:41:10 -0300 > Geyslan Gregório Bem wrote: > >> Let's wait Steve's reply about further use of tracing_is_disabled(). > > Might want to ping me later. This weekend I'm taking my daughter to > colleges, and early Monday morning I'm leaving to Edinb

[PATCH net-next v4 9/9] net: switch net_secret key generation to net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
Cc: Eric Dumazet Cc: "David S. Miller" Signed-off-by: Hannes Frederic Sowa --- net/core/secure_seq.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/net/core/secure_seq.c b/net/core/secure_seq.c index 3f1ec15..b02fd16 100644 --- a/net/core/secure_seq.c +++ b/

[PATCH net-next v4 1/9] ipv4: split inet_ehashfn to hash functions per compilation unit

2013-10-19 Thread Hannes Frederic Sowa
This duplicates a bit of code but let's us easily introduce separate secret keys later. The separate compilation units are ipv4/inet_hashtabbles.o, ipv4/udp.o and rds/connection.o. Cc: Eric Dumazet Cc: "David S. Miller" Signed-off-by: Hannes Frederic Sowa --- include/net/inet_sock.h| 22 ++

[PATCH net-next v4 2/9] ipv6: split inet6_ehashfn to hash functions per compilation unit

2013-10-19 Thread Hannes Frederic Sowa
This patch splits the inet6_ehashfn into separate ones in ipv6/inet6_hashtables.o and ipv6/udp.o to ease the introduction of seperate secrets keys later. Cc: Eric Dumazet Cc: "David S. Miller" Signed-off-by: Hannes Frederic Sowa --- include/net/inet6_hashtables.h | 28 +++--

[PATCH net-next v4 0/9] Introduce support to lazy initialize mostly static keys

2013-10-19 Thread Hannes Frederic Sowa
Hi! This series implements support for delaying the initialization of secret keys, e.g. used for hashing, for as long as possible. This functionality is implemented by a new macro, net_get_random_bytes. I already used it to protect the socket hashes, the syncookie secret (most important) and the

[PATCH net-next v4 8/9] tcp: switch tcp_fastopen key generation to net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
Changed key initialization of tcp_fastopen cookies to net_get_random_once. If the user sets a custom key net_get_random_once must be called at least once to ensure we don't overwrite the user provided key when the first cookie is generated later on. Cc: Yuchung Cheng Cc: Eric Dumazet Cc: "David

Linux 3.12-rc6

2013-10-19 Thread Linus Torvalds
I'm at PDX, about to fly out to the kernel summit, and it has almost become a tradition to do an rc release using the airport wifi. So here it is.. The patch is still busily being pushed out, but the git trees are up-to-date, and the tar-file should already be out. Nothing major happened last week

[PATCH net-next v4 7/9] inet: convert inet_ehash_secret and ipv6_hash_secret to net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
Initialize the ehash and ipv6_hash_secrets with net_get_random_once. Each compilation unit gets its own secret now: ipv4/inet_hashtables.o ipv4/udp.o ipv6/inet6_hashtables.o ipv6/udp.o rds/connection.o The functions still get inlined into the hashing functions. In the fast path we have

[PATCH net-next v4 5/9] net: introduce new macro net_get_random_once

2013-10-19 Thread Hannes Frederic Sowa
net_get_random_once is a new macro which handles the initialization of secret keys. It is possible to call it in the fast path. Only the initialization depends on the spinlock and is rather slow. Otherwise it should get used just before the key is used to delay the entropy extration as late as poss

[PATCH net-next v4 3/9] static_key: WARN on usage before jump_label_init was called

2013-10-19 Thread Hannes Frederic Sowa
Usage of the static key primitives to toggle a branch must not be used before jump_label_init() is called from init/main.c. jump_label_init reorganizes and wires up the jump_entries so usage before that could have unforeseen consequences. Following primitives are now checked for correct use: * sta

  1   2   >