Currently, perf-probe --list option ignores given event filter.
# ./perf probe -l vfs\*
probe:vfs_read (on vfs_read@ksrc/linux-3/fs/read_write.c)
probe_libc:malloc(on __libc_malloc@malloc/malloc.c in
/usr/lib64/libc-2.17.so)
This changes --list option to accept t
This makes perf-probe --del option to accept filter rules
not only simple glob pattern. This simplifies the code and
improve the flexibility.
E.g. if we remove 2 different pattern events, we need 2
-d options.
# ./perf probe -d vfs\* -d malloc
Removed event: probe_libc:malloc
Rem
To simplify the perf-probe command code, consolidate some
similar functions and use command short-name for command
classification, instead of separated booleans.
Signed-off-by: Masami Hiramatsu
---
tools/perf/builtin-probe.c | 105
1 file changed, 37
This allows user to pass the filter pattern directly to
--funcs option as below.
# ./perf probe -F *kmalloc
__kmalloc
devm_kmalloc
mempool_kmalloc
sg_kmalloc
sock_kmalloc
We previously need --filter option for that.
Signed-off-by: Masami Hiramatsu
---
tools/perf/Docume
Hi,
Here is a series of patches which improves perf-probe to add
filtering features for --list,--del and --funcs. This also
includes some bugfixes and cleanups.
I've fixed some patches according to Namhyung's review(Thanks!).
E.g. --list/--funcs accepts filter rules.
# ./perf probe -l v
--funcs option is a command which should be given exclusively.
This adds PARSE_OPT_EXCUSIVE flag on --funcs (-F) option.
Without this, 'perf probe --funcs -l' just shows the list of
probes. With this, it shows error message correctly.
This also fixes the help message and the documentation.
Signe
Hello Rob
On Thu, Apr 23, 2015 at 9:28 AM, Ricardo Ribalda Delgado
wrote:
>
> I think it can also use platform_device_add. I will prepare a patch to
> finally remove of_device_add()
I will post right away an updated version of this patchset, and then I
will prepare another one to remove of_devic
This patch removes unwanted semicolon around close braces of code blocks
Signed-off-by: Hari Prasath
---
drivers/staging/i2o/iop.c | 44 ++--
1 file changed, 22 insertions(+), 22 deletions(-)
diff --git a/drivers/staging/i2o/iop.c b/drivers/staging/i2o/io
On Thu, 23 Apr 2015, Gavin Guo wrote:
> - if (KMALLOC_MIN_SIZE <= 64 && !kmalloc_caches[2] && i == 7)
> - kmalloc_caches[2] = create_kmalloc_cache(NULL, 192,
> flags);
> + if (i == 2)
> + i = (KMALLOC_SHIFT_LOW - 1);
> }
Ok th
Since params.filter will be released in cleanup_params,
we don't need to clear it in each command.
Signed-off-by: Masami Hiramatsu
---
tools/perf/builtin-probe.c |6 --
1 file changed, 6 deletions(-)
diff --git a/tools/perf/builtin-probe.c b/tools/perf/builtin-probe.c
index 5a0e8f1..08c
The kerneldoc for the functions doesn't match the code; the
last two parameters (buflen, skip) have been transposed, which
is confusing, especially as they're both integral types and the
compiler won't warn about swapping them.
These functions and the kerneldoc were introduced in commit:
df642
Failure path of platform_device_add was almost the same as
platform_device_del. Refactor same code in a function.
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/base/platform.c | 60 +
1 file changed, 25 insertions(+), 35 deletions(-)
diff --g
of_platform_depopulate can lead to a kernel error when calling
release_resource()
The reason is that it is trying to release a resource that was not allocated
via insert_resource()
of_platform_depopulate()
of_platform_device_destroy()
platform_device_unregister(platform_device *pde
of_platform_device_create_pdata() was using of_device_add() to create
the devices, but of_platform_device_destroy was using
of_platform_device_destroy().
of_device_add(), do not call insert_resource(), which initializes the
parent field of the resource structure, needed by release_resource(),
call
On Thu, 23 Apr 2015, Rasmus Villemoes wrote:
> The first is a keyboard-off-by-one, the other two the ordinary mathy
> kind.
Acked-by: Christoph Lameter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo
insert_resource() can fail when the resource added overlaps
(partially or fully) with another.
Device tree and AMBA devices may contain resources that overlap, so they
could not call platform_device_add (see 02bbde7849e6 ('Revert "of:
use platform_device_add"'))"
On the other hand, device trees
platform_device_del only checks the type of the resource in order to
call release_resource.
On the other hand, platform_device_add calls insert_resource for any
resource that has a parent.
Make both code branches balanced.
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/base/platform.c | 10
On Thu, 9 Apr 2015, Christoph Lameter wrote:
> > I'll submit a new version this week with the securebits. Sorry for the
> > delay.
> Are we going to get a new version?
Replying to my own here. Cant we simply use the SETPCAP approach as per
the patch I posted?
--
To unsubscribe from this list:
- Original Message -
> On Fri, Apr 17 2015, Mathieu Desnoyers
> wrote:
>
> > + */
> > +SYSCALL_DEFINE2(membarrier, int, cmd, int, flags)
> > +{
> > + switch (cmd) {
> > + case MEMBARRIER_CMD_QUERY:
> > + return MEMBARRIER_CMD_BITMASK;
> > + case MEMBARRIER_CMD_SHARED:
> >
On Thu, Apr 23, 2015 at 12:29:16PM +0100, Will Deacon wrote:
> Hi all,
>
> Commit 6428c59a97de ("perf tools: Set JOBS based on CPU or processor")
> causes weird behaviour on arm/arm64 platforms because we use the "CPU"
> prefix for things like:
>
> CPU implementer : 0x41
> CPU architecture: 8
> C
A nit only, I'm afraid: this patch adds a license mismatch.
On Wed, 2015-04-22 at 21:25 +0200, Stephan Mueller wrote:
> --- /dev/null
> +++ b/crypto/jitterentropy.c
> + * License
> + * ===
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are p
Am Donnerstag, 23. April 2015, 16:05:08 schrieb Paul Bolle:
Hi Paul,
> A nit only, I'm afraid: this patch adds a license mismatch.
>
> On Wed, 2015-04-22 at 21:25 +0200, Stephan Mueller wrote:
> > --- /dev/null
> > +++ b/crypto/jitterentropy.c
> >
> > + * License
> > + * ===
> > + *
> > + *
Em Wed, Apr 22, 2015 at 09:23:09AM +0300, Adrian Hunter escreveu:
> On 21/04/15 17:50, Arnaldo Carvalho de Melo wrote:
> > Em Thu, Apr 09, 2015 at 06:53:49PM +0300, Adrian Hunter escreveu:
> >
> >> It is assumed that AUX area decoding will synthesize events for
> >> consumption by other tools. At
Make sure only to copy any actual data rather than the whole buffer,
when releasing the temporary buffer used for unaligned non-isochronous
transfers.
Compile-tested only.
Signed-off-by: Johan Hovold
---
drivers/usb/host/ehci-tegra.c | 12 +---
1 file changed, 9 insertions(+), 3 deletio
Make sure only to copy any actual data rather than the whole buffer,
when releasing the temporary buffer used for unaligned non-isochronous
transfers.
Signed-off-by: Johan Hovold
---
drivers/usb/musb/musb_host.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers
On Thu, 2015-04-23 at 08:30 +, Kweh, Hock Leong wrote:
> > -Original Message-
> > From: James Bottomley [mailto:james.bottom...@hansenpartnership.com]
> > Sent: Wednesday, April 22, 2015 11:19 PM
> >
> >
> > Yes, I think we've all agreed we can do it ... it's now a question of
> > wh
Dear friend,
Please do accept my apologies as I do not wish to invade into your
privacy, I had written an earlier mail to you but without response. My
name is Coffi Kelly, I am a lawyer by profession based in Lome-Togo. I
need your urgent assistance for the claiming of the sum of US$9.2
Million be
On Thu, 23 Apr 2015, Benjamin Herrenschmidt wrote:
> > Anyone
> > wanting performance (and that is the prime reason to use a GPU) would
> > switch this off because the latencies are otherwise not controllable and
> > those may impact performance severely. There are typically multiple
> > parallel
Make sure to copy the whole transfer buffer when releasing the temporary
buffer used for unaligned isochronous transfers as the data is not
necessarily contiguous in that case.
Signed-off-by: Johan Hovold
---
drivers/staging/octeon-usb/octeon-hcd.c | 12 +---
1 file changed, 9 insertions
On Wed, 22 Apr 2015, Paul E. McKenney wrote:
> Agreed, the use case that Jerome is thinking of differs from yours.
> You would not (and should not) tolerate things like page faults because
> it would destroy your worst-case response times. I believe that Jerome
> is more interested in throughput
2015-04-23 22:57 GMT+09:00 Dave Gordon :
> The kerneldoc for the functions doesn't match the code; the
> last two parameters (buflen, skip) have been transposed, which
> is confusing, especially as they're both integral types and the
> compiler won't warn about swapping them.
>
> These functions an
On Wed, Apr 22, 2015 at 07:20:09PM +0900, Chanwoo Choi wrote:
> On 04/22/2015 06:19 PM, Richard Fitzgerald wrote:
> > On Wed, Apr 22, 2015 at 02:53:42PM +0900, Chanwoo Choi wrote:
> >> Hi Richard,
> >>
> >>> @@ -1176,6 +1182,11 @@ static int arizona_extcon_probe(struct
> >>> platform_device *pdev)
Casting spi_driver pointers to "void *" when calling
spi_{,un}register_driver() bypasses all type checking.
Remove the superfluous casts to fix this.
Signed-off-by: Geert Uytterhoeven
---
drivers/tty/serial/ifx6x60.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/driv
On Apr 22, 2015 at 10:53 PM, Vladimir Zapolskiy wrote:
> is it still the case that for shared interrupts a hard IRQ handler is
> mandatory to have?
>
> Here I rely on
> http://lists.kernelnewbies.org/pipermail/kernelnewbies/2011-March/001118.html
>
> With best wishes,
> Vladimir
>
Vladimir,
A har
Em Thu, Apr 09, 2015 at 06:53:58PM +0300, Adrian Hunter escreveu:
> +++ b/tools/perf/Documentation/perf-report.txt
> @@ -323,6 +323,34 @@ OPTIONS
> --header-only::
> Show only perf.data header (forces --stdio).
>
> +-Z::
> +--itrace::
> + Options for decoding AUX area tracing data. The
On 4/23/15 8:04 AM, Will Deacon wrote:
From 28740111e81aa9247bf48e3125dc43cc31d94e6f Mon Sep 17 00:00:00 2001
From: Will Deacon
Date: Thu, 23 Apr 2015 15:00:16 +0100
Subject: [PATCH] tools: perf: use getconf to determine number of online CPUs
Parsing /proc/cpuinfo is a fiddly, arch-dependent b
There's no need to cast the last parameter of {request,free}_irq() to
"void *", as any pointer type is accepted.
Remove the superfluous casts to fix this.
Signed-off-by: Geert Uytterhoeven
---
drivers/tty/serial/ifx6x60.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff
If NO_DMA=y:
drivers/built-in.o: In function `ifx_spi_free_device':
ifx6x60.c:(.text+0x96d9a): undefined reference to `dma_free_coherent'
drivers/built-in.o: In function `ifx_spi_spi_probe':
ifx6x60.c:(.text+0x978a2): undefined reference to `dma_alloc_coherent'
While DMA is option
> Alan, and others, want a tiny, generic, multi-cast IPC method that also
> works across networks. They feel that this is something that D-Bus
I never said - across networks. And locally it has been done, even
microcontrollers have done it.
> Lots of people have said they want something like thi
On Thu, 2015-04-23 at 16:08 +0200, Stephan Mueller wrote:
> Other patches that are in the kernel that I wrote (e.g.
> the crypto/drbg.c) have the same license as above, but use
> MODULE_LICENSE("GPL") -- Thus I would think that leaving it as is should be
> ok.
Why not submit the trivial patches
These patches (for 4.1) make sure that only the received data is copied
from the temporary buffers used for unaligned transfers.
I discovered this when debugging an issue where the Beaglebone Black
would lock up on disconnect.
Turns out it was related to the transfer_buffers not being properly
al
On Thu, 23 Apr 2015, Benjamin Herrenschmidt wrote:
> > There are hooks in glibc where you can replace the memory
> > management of the apps if you want that.
>
> We don't control the app. Let's say we are doing a plugin for libfoo
> which accelerates "foo" using GPUs.
There are numerous examples
On Thu, Apr 23, 2015 at 02:12:44PM +, DHANAPAL, GNANACHANDRAN (G.) wrote:
> This patch removes unused variable in this file.
>
> Signed-off-by: Gnanachandran Dhanapal
>
> Change-Id: I59cc869773d145239d62c3185018b275239fe812
The only thing I can do with a patch that has this line in it is de
On Thu, 23 Apr 2015, Benjamin Herrenschmidt wrote:
> They are via MMIO space. The big differences here are that via CAPI the
> memory can be fully cachable and thus have the same characteristics as
> normal memory from the processor point of view, and the device shares
> the MMU with the host.
>
>
Christoffer Dall writes:
> On Tue, Mar 31, 2015 at 04:08:04PM +0100, Alex Bennée wrote:
>> This adds support for SW breakpoints inserted by userspace.
>>
>> We do this by trapping all BKPT exceptions in the
>> hypervisor (MDCR_EL2_TDE).
>
> you mean trapping all exceptions in the guest to the h
On 4/23/15 5:29 AM, Will Deacon wrote:
Hi all,
Commit 6428c59a97de ("perf tools: Set JOBS based on CPU or processor")
causes weird behaviour on arm/arm64 platforms because we use the "CPU"
prefix for things like:
CPU implementer : 0x41
CPU architecture: 8
CPU variant : 0x0
CPU part:
Em Tue, Mar 31, 2015 at 02:38:29PM +0300, Adrian Hunter escreveu:
> Hi
>
> Here is V7 of some more preparatory patches for Intel PT
> that introduce an abstraction for using the AUX area and
> Instruction tracing.
>
> The master branch of the tree:
>
> git://git.infradead.org/users/ahunter
On 4/23/15 10:51, Steven Miao wrote:
> On Wed, Apr 22, 2015 at 10:32 PM, Chen Gang
> wrote:
>> On 4/22/15 17:00, Steven Miao wrote:
>>> Hi,
>>>
>>> On Thu, Apr 9, 2015 at 5:03 AM, Chen Gang
>>> wrote:
l1_text is at L1_CODE_START (e.g. for bf533, 0xff80). If the kernel
is too big,
On Wed, 22 Apr 2015, Arun Ramamurthy wrote:
> This patch set adds a new API to get phy by index when multiple
> phys are present. This patch is based on discussion with Arnd Bergmann
> about dt bindings for multiple phys.
>
> History:
> v1:
> - Removed null pointers on Dmitry's suggestion
>
On Thu, Apr 23, 2015 at 4:06 PM, Johan Hovold wrote:
> Make sure only to copy any actual data rather than the whole buffer,
> when releasing the temporary buffer used for unaligned non-isochronous
> transfers.
>
> Compile-tested only.
>
> Signed-off-by: Johan Hovold
> ---
> drivers/usb/host/ehci
Am Donnerstag, 23. April 2015, 16:18:31 schrieb Paul Bolle:
Hi Paul,
> On Thu, 2015-04-23 at 16:08 +0200, Stephan Mueller wrote:
> > Other patches that are in the kernel that I wrote (e.g.
> > the crypto/drbg.c) have the same license as above, but use
> > MODULE_LICENSE("GPL") -- Thus I would thi
On Wed, Apr 22, 2015 at 12:00:30PM +0100, Mark Brown wrote:
> On Tue, Apr 21, 2015 at 01:33:55PM +0100, Richard Fitzgerald wrote:
>
> > +static int wm8998_in1mux_ev(struct snd_soc_dapm_widget *w,
> > + struct snd_kcontrol *kcontrol,
> > + int eve
On Thu, 23 Apr 2015, Benjamin Herrenschmidt wrote:
> In fact I'm quite surprised, what we want to achieve is the most natural
> way from an application perspective.
Well the most natural thing would be if the beast would just do what I
tell it in plain english. But then I would not have my job an
Em Wed, Apr 22, 2015 at 09:10:15PM +0200, Jiri Olsa escreveu:
> hi,
> adding support to report error from event string parsing.
>
> v1 changes (from RFC):
> - display list of allowed terms for pmu event error [Ingo]
> - changing 'invalid or unsupported event' string into
> 'event syntax er
On Thu, 23 Apr 2015, Geert Uytterhoeven wrote:
> >> I'm afraid you can't call pm_runtime_get_sync() from these methods, as
> >> they may be called from interrupt context.
> >
> > Ouch. I know the clock framework has prepare/enable separated with
> > context, but with the irqchip callbacks I suppos
On Thu, Apr 23, 2015 at 04:31:51PM +0200, Frans Klaver wrote:
> On Thu, Apr 23, 2015 at 4:06 PM, Johan Hovold wrote:
> > static void free_dma_aligned_buffer(struct urb *urb)
> > {
> > struct dma_aligned_buffer *temp;
> > + size_t length;
> >
> > if (!(urb->transfer_flags &
On Thu, Apr 23, 2015 at 02:05:18AM +, Austin, Brian wrote:
>
>
> > On Apr 21, 2015, at 07:38, Richard Fitzgerald
> > wrote:
> >
> > Signed-off-by: Richard Fitzgerald
> > ---
> > Documentation/devicetree/bindings/mfd/arizona.txt |3 +++
> > 1 files changed, 3 insertions(+), 0 deletions(
On Thu, 23 Apr 2015, Johan Hovold wrote:
> Make sure only to copy any actual data rather than the whole buffer,
> when releasing the temporary buffer used for unaligned non-isochronous
> transfers.
>
> Compile-tested only.
>
> Signed-off-by: Johan Hovold
> ---
> drivers/usb/host/ehci-tegra.c |
Em Thu, Apr 23, 2015 at 10:46:17PM +0900, Masami Hiramatsu escreveu:
> Accept multiple filter options. Each filters are combined
> by logical-or. E.g. --filter abc* --filter *def is same
> as --filter abc*|*def
Please break this patch in two, one introducing the new strfilter
functionality, the ot
On Thu, Apr 23, 2015 at 02:51:26PM +0200, Wolfram Sang wrote:
> On Wed, Mar 11, 2015 at 07:45:31AM -0400, Tejun Heo wrote:
> > On Wed, Mar 11, 2015 at 11:26:02AM +0100, Wolfram Sang wrote:
> > > From: Wolfram Sang
> > >
> > > Grabbing the parent is not happening anymore since 2010 (e72ceb8ccac5f7
Chao Yu writes:
> In elevator_alloc, if we fail to allocate memory for storing elevator
> queue, our eq pointer must be NULL, we do not need to release it in error
> path, so remove it.
>
> Signed-off-by: Chao Yu
> ---
> block/elevator.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/b
Chao Yu writes:
> Our issue is descripted in below call path:
> ->elevator_init
> ->elevator_init_fn
> ->{cfq,deadline,noop}_init_queue
>->elevator_alloc
> ->kzalloc_node
>fail to call kzalloc_node and then put module in elevator_alloc;
> fail to call elevator_init_fn and then put
This patch removes unused return variable in this file.
Signed-off-by: Gnanachandran Dhanapal
---
drivers/staging/rtl8712/os_intfs.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8712/os_intfs.c
b/drivers/staging/rtl8712/os_intfs.c
index 6e776e5
Thanks for the feedback, Greg.
I have sent patch set version 2 for the same changes.
-Original Message-
From: gre...@linuxfoundation.org [mailto:gre...@linuxfoundation.org]
Sent: 23 April 2015 19:51
To: DHANAPAL, GNANACHANDRAN (G.)
Cc: larry.fin...@lwfinger.net; florian.c.schilha...@goog
Using sendfile with below small program to get MD5 sums of some files,
it appear that big files (over 64kbytes with 4k pages system) get a
wrong MD5 sum while small files get the correct sum.
This program uses sendfile() to send a file to an AF_ALG socket
for hashing.
/* md5sum2.c */
#include
#i
The problem with using cycle counter for NMI watchdog is that its
frequency changes with the corresponding core's frequency. This means
that, in particular, if the core frequency scales up, watchdog NMI will
arrive more frequently than what user requested through watchdog_thresh
and also increasing
Hi Mathieu,
On 04/22/2015 06:40 PM, Mathieu Poirier wrote:
> From: Pratik Patel
>
> Adding sysfs entries to access and configure specifics about the
> context ID comparator functions.
>
> Signed-off-by: Pratik Patel
> Signed-off-by: Mathieu Poirier
> ---
> .../ABI/testing/sysfs-bus-coresight
This patch removes unused variable in this file.
Signed-off-by: Gnanachandran Dhanapal
Change-Id: I59cc869773d145239d62c3185018b275239fe812
---
drivers/staging/rtl8712/os_intfs.c |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8712/os_intfs.c
b/d
On Thu, 2015-04-23 at 13:43 +, Gujulan Elango, Hari Prasath (H.)
wrote:
> This patch removes unwanted semicolon around close braces of code blocks
The i2o driver moved into staging ready to be deleted unless someone
steps up with hardware willing to maintain it (which is rather
unlikely).
No
On Thu, Apr 23, 2015 at 9:28 AM, Adam Goode wrote:
>
> On Thu, Apr 23, 2015 at 6:08 AM, Andreas Noever
> wrote:
> > Hi Adam,
> >
> > On my system (MacBookPro10,1 - 4 channel TB1) the bridges and the
> > controller both use 0x1547 and are only differentiated by
> > subvendor/subdevice.
> >
> > 0x1
On Tue, Apr 21, 2015 at 11:08:42AM -0700, Andy Lutomirski wrote:
> I'll take a full implementation of what Intel says over probably
> unmeasurable performance. If anyone in the AMD camp really cared, we
> could add X86_BUG_SYSRET_NEEDS_CANONICAL_RCX and use alternatives to
> patch this out on AMD.
Please pull nfsd changes for 4.0 from:
git://linux-nfs.org/~bfields/linux.git for-4.0
A quiet cycle this time; this is basically entirely bugfixes. The few
that aren't cc'd to stable are cleanup or seemed unlikely to affect
anyone much.
--b.
--
From: Wolfram Sang
Grabbing the parent is not happening anymore since 2010 (e72ceb8ccac5f7
"sysfs: Remove sysfs_get/put_active_two"). Remove this confusing
comment.
Signed-off-by: Wolfram Sang
Acked-by: Tejun Heo
---
fs/kernfs/file.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/fs/kernf
On 23 April 2015 at 09:08, Christopher Covington wrote:
> Hi Mathieu,
>
> On 04/22/2015 06:40 PM, Mathieu Poirier wrote:
>> From: Pratik Patel
>>
>> Adding sysfs entries to access and configure specifics about the
>> context ID comparator functions.
>>
>> Signed-off-by: Pratik Patel
>> Signed-of
On 04/22/2015 06:40 PM, Mathieu Poirier wrote:
> From: Pratik Patel
>
> This driver manages the CoreSight ETMv4 (Embedded Trace Macrocell) IP block
> to support HW assisted tracing on ARMv7 and ARMv8 architectures.
>
> Signed-off-by: Pratik Patel
> Signed-off-by: Kaixu Xia
> Signed-off-by: Mat
On Tue, Apr 21, 2015 at 07:22:34PM +0530, Sudip Mukherjee wrote:
> This is again another WIP for your review.
> almost all the points raised by Greg and Dan has been covered in this
> patch.
>
> apart from those points, I found another problem with my previous code,
> that whenever any driver is t
From: Wolfram Sang
My kernfs patch slipped through because I didn't know which maintainer
to CC. Have been told it's gkh. Add an entry, and sort the file patterns
while we are here.
Signed-off-by: Wolfram Sang
---
MAINTAINERS | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff -
On Thu, Apr 23, 2015 at 5:34 AM, Denys Vlasenko wrote:
>
> It was observed to cause Wine crashes. Conjectured sequence of events
> causing it is as follows:
>
> 1. Wine process enters kernel via syscall insn.
> 2. Context switch to any other task.
> 3. Interrupt or exception happens, CPU loads %ss
Hello,
On Wed, Apr 22, 2015 at 02:55:06PM -0700, Luis R. Rodriguez wrote:
> +int param_set_bool_enable_only(const char *val, const struct kernel_param
> *kp)
> +{
> + int err = 0;
> + bool new_value;
> + bool orig_value = *(bool *)kp->arg;
> + struct kernel_param dummy_kp = *kp;
>
2015-04-23 13:46+0200, Paolo Bonzini:
> From: Radim Krčmář
>
> The kvmclock spec says that the host will increment a version field to
> an odd number, then update stuff, then increment it to an even number.
> The host is buggy and doesn't do this, and the result is observable
> when one vcpu read
On 2015-04-23 10:25, Christoph Lameter wrote:
On Thu, 23 Apr 2015, Benjamin Herrenschmidt wrote:
They are via MMIO space. The big differences here are that via CAPI the
memory can be fully cachable and thus have the same characteristics as
normal memory from the processor point of view, and the
Hi,
This patchset allows vhost to be used with legacy virtio when guest and host
have a different endianness. It is compatible with modern virtio and can be
fully compiled out through kernel config.
FWIW, I could flawlessly kexec/reboot guests from ppc64 to ppc64le and back.
I could also migrate
Signed-off-by: Greg Kurz
---
include/linux/virtio_config.h | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/include/linux/virtio_config.h b/include/linux/virtio_config.h
index ca3ed78..bd1a582 100644
--- a/include/linux/virtio_config.h
+++ b/include/linux/v
Signed-off-by: Greg Kurz
---
drivers/net/macvtap.c |9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/net/macvtap.c b/drivers/net/macvtap.c
index 27ecc5c..a2f2958 100644
--- a/drivers/net/macvtap.c
+++ b/drivers/net/macvtap.c
@@ -49,14 +49,19 @@ struct macvtap
Signed-off-by: Greg Kurz
---
drivers/net/tun.c |9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/net/tun.c b/drivers/net/tun.c
index 857dca4..3c3d6c0 100644
--- a/drivers/net/tun.c
+++ b/drivers/net/tun.c
@@ -206,14 +206,19 @@ struct tun_struct {
u32
Signed-off-by: Greg Kurz
---
drivers/vhost/vhost.h | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h
index 8c1c792..6a49960 100644
--- a/drivers/vhost/vhost.h
+++ b/drivers/vhost/vhost.h
@@ -173,34 +173,39 @@ sta
Signed-off-by: Greg Kurz
---
include/linux/vringh.h | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/include/linux/vringh.h b/include/linux/vringh.h
index a3fa537..3ed62ef 100644
--- a/include/linux/vringh.h
+++ b/include/linux/vringh.h
@@ -226,33 +226,38 @
On 04/23/2015 02:13 PM, Liang Li wrote:
> When compiling kernel on westmere, the performance of eager FPU
> is about 0.4% faster than lazy FPU.
Do you have an theory why this is? What does the regression come from?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
th
Hi Thomas/Rafael,
With latest mainline(commit 27cf3a16b2535a490f8cf1d29a6634f1c70f7831),
and lockdep enabled I see the following inconsistent lock state log.
I am not sure if it's related to recent changes in tick-broadcast or I
might be missing any config ?
=
[ I
On Wed, 2015-04-22 at 16:36 +0100, David Ahern wrote:
> On 4/22/15 8:47 AM, Arnaldo Carvalho de Melo wrote:
> > Em Wed, Apr 22, 2015 at 08:53:14AM -0400, Steven Rostedt escreveu:
> >> >On Tue, 21 Apr 2015 21:24:51 -0500
> >> >Ron Rechenmacher wrote:
> >>> > >I've looked at the above reference brie
On 04/23/2015 10:00 AM, DHANAPAL, GNANACHANDRAN (G.) wrote:
This patch removes unused return variable in this file.
You should hold off with V2 to give others a chance to comment.
Your commit message does not exactly describe what is happening. The return
variable is not unused. More accurate
On 04/23/2015 04:11 PM, Nishanth Menon wrote:
On 04/23/2015 05:17 AM, grygorii.stras...@linaro.org wrote:
On 04/23/2015 03:00 AM, Nishanth Menon wrote:
On 04/22/2015 08:26 AM, grygorii.stras...@linaro.org wrote:
Hi,
On 04/21/2015 03:51 AM, Nishanth Menon wrote:
Alarm interrupt enable registe
The current memory accessors logic is:
- little endian if little_endian
- native endian (i.e. no byteswap) if !little_endian
If we want to fully support cross-endian vhost, we also need to be
able to convert to big endian.
Instead of changing the little_endian argument to some 3-value enum, this
This patch brings cross-endian support to vhost when used to implement
legacy virtio devices. Since it is a relatively rare situation, the
feature availability is controlled by a kernel config option (not set
by default).
The vq->is_le boolean field is added to cache the endianness to be
used for
The VNET_LE flag was introduced to fix accesses to virtio 1.0 headers
that are always little-endian. It can also be used to handle the special
case of a legacy little-endian device implemented by a big-endian host.
Let's add a flag and ioctls for big-endian devices as well. If both flags
are set,
On Thu, 2015-04-23 at 16:28 +0100, Pawel Moll wrote:
> On Wed, 2015-04-22 at 16:36 +0100, David Ahern wrote:
> > On 4/22/15 8:47 AM, Arnaldo Carvalho de Melo wrote:
> > > Em Wed, Apr 22, 2015 at 08:53:14AM -0400, Steven Rostedt escreveu:
> > >> >On Tue, 21 Apr 2015 21:24:51 -0500
> > >> >Ron Rechen
On Tue, Apr 21, 2015 at 11:33 PM, Jan Beulich wrote:
>
> while the description of commit cae2a173fe certainly makes sense, the
> change itself ignores the __probe_kernel_write() code path, for which
> the destination address is expected to be in kernel space but accesses
> may still fault. I.e. th
From: Niklas Cassel
Date: Thu, 23 Apr 2015 15:37:11 +0200
> Signed-off-by: Niklas Cassel
> ---
> drivers/net/phy/micrel.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c
> index 1190fd8..a422036 100644
> --- a/drivers
On Thu, Apr 23, 2015 at 8:10 AM, Borislav Petkov wrote:
> On Tue, Apr 21, 2015 at 11:08:42AM -0700, Andy Lutomirski wrote:
>> I'll take a full implementation of what Intel says over probably
>> unmeasurable performance. If anyone in the AMD camp really cared, we
>> could add X86_BUG_SYSRET_NEEDS_
From: David Gibson
Date: Thu, 23 Apr 2015 14:43:05 +1000
> AFAIK the PAPR document which defines the virtual device interface used by
> the ibmveth driver doesn't specify a specific maximum MTU. So, in the
> ibmveth driver, the maximum allowed MTU is determined by the maximum
> allocated buffer
201 - 300 of 710 matches
Mail list logo