Update perf-kvm documentation with new live subcommand. Add -p/--pid option
for perf-kvm-stat-report as well.
Signed-off-by: David Ahern
Cc: Arnaldo Carvalho de Melo
Cc: Ingo Molnar
Cc: Frederic Weisbecker
Cc: Peter Zijlstra
Cc: Jiri Olsa
Cc: Namhyung Kim
Cc: Xiao Guangrong
Cc: Runzhen
On 08/08/2013 09:35 AM, Andy Lutomirski wrote:
> On Wed, Aug 7, 2013 at 6:30 PM, Chen Gang wrote:
>> On 08/08/2013 12:58 AM, Andy Lutomirski wrote:
>>> On Wed, Aug 7, 2013 at 9:21 AM, Oleg Nesterov wrote:
On 08/06, Andy Lutomirski wrote:
>
> I assume that what the man page means is
On 08/08/2013 08:36 AM, Paul E. McKenney wrote:
> On Wed, Aug 07, 2013 at 05:38:27AM -0700, Paul E. McKenney wrote:
>> On Wed, Aug 07, 2013 at 06:24:56PM +0800, Lai Jiangshan wrote:
>>> Although all articles declare that rcu read site is deadlock-immunity.
>>> It is not true for rcu-preempt, it
On 08/07/2013 03:55 PM, Eric W. Biederman wrote:
>
> Since this still has not been addressed. I am going to repeat Andrews
> objection again.
>
> Isn't there a better way to get iptables information out than to use
> syslog. I did not have time to follow up on that but it did appear that
>
On Wed, Aug 7, 2013 at 6:30 PM, Chen Gang wrote:
> On 08/08/2013 12:58 AM, Andy Lutomirski wrote:
>> On Wed, Aug 7, 2013 at 9:21 AM, Oleg Nesterov wrote:
>>> On 08/06, Andy Lutomirski wrote:
I assume that what the man page means is that the return value is
whatever fsgid was prior
On Mon, Jul 8, 2013 at 10:35 AM, Zhang Yanfei wrote:
> On 07/09/2013 12:15 AM, Kees Cook wrote:
>> Moves the relocation handling into C, after decompression. This requires
>> that the decompressed size is passed to the decompression routine as
>> well so that relocations can be found. Only
On 08/08/2013 12:58 AM, Andy Lutomirski wrote:
> On Wed, Aug 7, 2013 at 9:21 AM, Oleg Nesterov wrote:
>> On 08/06, Andy Lutomirski wrote:
>>>
>>> I assume that what the man page means is that the return value is
>>> whatever fsgid was prior to the call. On error, fsgid isn't changed, so
>>> the
We offer Business/Company/Project loans at 3%. Reply with amount
needed.Reply Via Email: frankjimmyloancompan...@aol.com
Disclaimer: The information contained in this communication is intended solely
for the use of the individual or entity to whom it is addressed and others
authorized to
On 08/08/2013 03:29 AM, Carsten Emde wrote:
> Hi Paul,
>
>>> Although all articles declare that rcu read site is deadlock-immunity.
>>> It is not true for rcu-preempt, it will be deadlock if rcu read site
>>> overlaps with scheduler lock.
>>
>> The real rule is that if the scheduler does its
On Wed, Aug 7, 2013 at 6:01 PM, Sedat Dilek wrote:
> On Thu, Aug 8, 2013 at 1:34 AM, Colin Cross wrote:
>> On Wed, Aug 7, 2013 at 4:15 PM, Sedat Dilek wrote:
>>> On Thu, Aug 8, 2013 at 12:58 AM, Colin Cross wrote:
Can you try add a call to show_state_filter(TASK_UNINTERRUPTIBLE) in
On Wed, Aug 07, 2013 at 08:50:12PM -0400, Konrad Rzeszutek Wilk wrote:
> On Wed, Aug 07, 2013 at 12:15:21PM +0530, Raghavendra K T wrote:
> > On 08/07/2013 10:18 AM, H. Peter Anvin wrote:
> > >>Please let me know, if I should rebase again.
> > >>
> > >
> > >tip:master is not a stable branch; it is
Hi Andrew,
于 2013/8/8 5:05, a...@linux-foundation.org 写道:
> Subject: + mm-vmalloc-use-well-defined-find_last_bit-func.patch added to -mm
> tree
> To:
> iamjoonsoo@lge.com,han...@cmpxchg.org,liw...@linux.vnet.ibm.com,zhangyan...@cn.fujitsu.con
> From: a...@linux-foundation.org
> Date: Wed,
Hi Andrew,
于 2013/8/8 5:05, a...@linux-foundation.org 写道:
> Subject: + mm-vmalloc-remove-useless-variable-in-vmap_block.patch added to
> -mm tree
> To:
> iamjoonsoo@lge.com,han...@cmpxchg.org,liw...@linux.vnet.ibm.com,zhangyan...@cn.fujitsu.con
> From: a...@linux-foundation.org
> Date: Wed,
On Wed, Aug 07, 2013 at 12:15:21PM +0530, Raghavendra K T wrote:
> On 08/07/2013 10:18 AM, H. Peter Anvin wrote:
> >>Please let me know, if I should rebase again.
> >>
> >
> >tip:master is not a stable branch; it is more like linux-next. We need
> >to figure out which topic branches are
On Aug 7, 2013, at 8:05 PM, Andrew Morton wrote:
> On Wed, 7 Aug 2013 19:54:45 -0400 Ed Cashin wrote:
>
>>> +bio_pageinc(struct bio *bio)
>>> +{
>>> + struct bio_vec *bv;
>>> + struct page *page;
>>> + int i;
>>> +
>>> + bio_for_each_segment(bv, bio, i) {
>>> +
On Thu, 8 Aug 2013 10:22:28 +1000 Stephen Rothwell
wrote:
>
> More quick testing with an empty file: v3.9 is OK, v3.10 gives
> CONFIG_MODULES unset.
Bisecting gives:
cfa98f2e0ae956feca935573e977d7661a9561b9 is the first bad commit
commit cfa98f2e0ae956feca935573e977d7661a9561b9
Author: Yann E.
Hi Kevin,
On Wed, 07 Aug 2013 08:28:35 -0700 Kevin Hilman
wrote:
>
> However, this same commit does introduce a new build failure (not
> present in next-20130806) when ext4 is built as a module:
>
> ERROR: "dirty_expire_interval" [fs/ext4/ext4.ko] undefined!
> make[3]: *** [__modpost] Error 1
>
From: David Miller
Date: Wed, 07 Aug 2013 17:09:26 -0700 (PDT)
> Ok, I'm going to simply revert all of these changes, thanks for testing.
Here is what I just pushed to net-next:
>From 09effa67a18d893fc4e6f81a3659fc9efef1445e Mon Sep 17 00:00:00 2001
From: "David S. Miller"
On Wed, 2013-08-07 at 16:36 -0700, David Miller wrote:
> From: David Miller
> Date: Wed, 07 Aug 2013 16:27:48 -0700 (PDT)
>
> > Look, I'm going to fix this myself, because I'm pretty tired of
> > waiting for the obvious fix.
>
> Someone please test this:
>
> diff --git
On Wed, 7 Aug 2013 19:54:45 -0400 Ed Cashin wrote:
> > +bio_pageinc(struct bio *bio)
> > +{
> > + struct bio_vec *bv;
> > + struct page *page;
> > + int i;
> > +
> > + bio_for_each_segment(bv, bio, i) {
> > + page = bv->bv_page;
> > + /*
gt;>> waiting for the obvious fix.
>>
>> Someone please test this:
>
> Your patch on top of next-20130807 does not fix the issue in my wifi/network.
> No working Internet connection (ping etc.).
Ok, I'm going to simply revert all of these changes, thanks for testing.
--
To unsubsc
The mm-of-the-moment snapshot 2013-08-07-16-55 has been uploaded to
http://www.ozlabs.org/~akpm/mmotm/
mmotm-readme.txt says
README for mm-of-the-moment:
http://www.ozlabs.org/~akpm/mmotm/
This is a snapshot of my -mm patch queue. Uploaded at random hopefully
more than once a week.
You
On Thu, Aug 8, 2013 at 1:36 AM, David Miller wrote:
> From: David Miller
> Date: Wed, 07 Aug 2013 16:27:48 -0700 (PDT)
>
>> Look, I'm going to fix this myself, because I'm pretty tired of
>> waiting for the obvious fix.
>
> Someone please test this:
>
Your patc
On Mon, 5 Aug 2013 15:09:35 -0700 Andi Kleen wrote:
> From: Andi Kleen
>
> Use standard gcc __attribute__((alias(foo))) to define
> the syscall aliases instead of custom assembler macros.
>
> This is far cleaner, and also fixes my LTO kernel build.
>
This makes my x86_64 allmodconfig build
On Aug 7, 2013, at 7:35 PM, Andrew Morton wrote:
> On Wed, 7 Aug 2013 19:27:40 -0400 Ed Cashin wrote:
>
>> On Aug 7, 2013, at 5:26 PM, Andrew Morton wrote:
>>
>>> On Wed, 7 Aug 2013 17:21:32 -0400 Ed Cashin wrote:
>> ...
Sorry. I tried to cover that but maybe should have separated it
On Wed, 7 Aug 2013 19:41:48 -0400 Ed Cashin wrote:
> On Aug 7, 2013, at 5:27 PM, Andrew Morton wrote:
>
> >> elevated refcount, full stop.
> >>
> >
> > err, no. slab.c uses alloc_pages(), so the underlying page indeed has
> > a proper refcount. I'm still not understanding how this situation
On Aug 7, 2013, at 7:41 PM, Ed Cashin wrote:
> It sounds like it's wrong to give block pages with a zero count, so why not
> just have aoe BUG_ON(compound_trans_head(bv->page->_count) == 0) until we're
> sure nobody does that anymore?
Ugh. I goofed the parens and such. I meant,
From: Jason Wang
Date: Tue, 6 Aug 2013 17:45:02 +0800
> This series tries to reduce the code duplication of zerocopy of tun and
> macvtap
> by:
>
> - Move common helpers iov_pages() and zerocopy_sg_from_iovec() to net/core
> - Use exist helpers instead of open coding the new one
>
> With
From: Rafael J. Wysocki
The out_free label in acpi_bind_one() is only jumped to from one
place, so in fact it is not necessary, because the code below it
can be moved to that place directly. Move that code and drop the
label.
Signed-off-by: Rafael J. Wysocki
---
drivers/acpi/glue.c | 19
Hi All,
Two more cleanups for acpi_bind_one() on top of linux-pm.git/linux-next
follow:
[1/2] Drop unnecessary label from acpi_bind_one().
[2/2] Print diagnostic messages if device links cannot be created.
Thanks,
Rafael
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source
From: Rafael J. Wysocki
Although the device links created by acpi_bind_one() are not
essential from the kernel functionality point of view, user space
may be confused when they are missing, so print diagnostic messages
to the kernel log if they can't be created.
Signed-off-by: Rafael J. Wysocki
On Aug 7, 2013, at 5:27 PM, Andrew Morton wrote:
> On Wed, 7 Aug 2013 14:18:35 -0700 Andrew Morton
> wrote:
>
>> On Wed, 7 Aug 2013 17:12:36 -0400 Ed Cashin wrote:
>>
>>>
>>> On Aug 7, 2013, at 4:58 PM, Andrew Morton wrote:
>>>
On Thu, 1 Aug 2013 21:29:59 -0400 Ed Cashin wrote:
On Wednesday, August 07, 2013 06:51:51 PM Tang Chen wrote:
> This patch-set aims to solve some problems at system boot time
> to enhance memory hotplug functionality.
>
> [Background]
>
> The Linux kernel cannot migrate pages used by the kernel because
> of the kernel direct mapping. Since va =
This primitive allows scheduling work to run on a particular set of
cpus described by a "struct cpumask". This can be useful, for example,
if you have a per-cpu variable that requires code execution only if the
per-cpu variable has a certain value (for example, is a non-empty list).
This change makes lru_add_drain_all() only selectively interrupt
the cpus that have per-cpu free pages that can be drained.
This is important in nohz mode where calling mlockall(), for
example, otherwise will interrupt every core unnecessarily.
Signed-off-by: Chris Metcalf
---
v4: don't lose
On Wed, 7 Aug 2013 19:27:40 -0400 Ed Cashin wrote:
> On Aug 7, 2013, at 5:26 PM, Andrew Morton wrote:
>
> > On Wed, 7 Aug 2013 17:21:32 -0400 Ed Cashin wrote:
> ...
> >> Sorry. I tried to cover that but maybe should have separated it more
> >> clearly from the running text.
> >>
> >> The
On Wed, Aug 7, 2013 at 4:15 PM, Sedat Dilek wrote:
> On Thu, Aug 8, 2013 at 12:58 AM, Colin Cross wrote:
>> Can you try add a call to show_state_filter(TASK_UNINTERRUPTIBLE) in
>> the error path of try_to_freeze_tasks(), where it prints the "refusing
>> to freeze" message? It will print the
> From: Yu, Fenghua
> Sent: Wednesday, August 07, 2013 3:46 PM
>
> > From: Borislav Petkov [mailto:b...@alien8.de]
> > Sent: Wednesday, July 24, 2013 6:33 AM
> > On Tue, Jul 23, 2013 at 11:00:26PM +0200, Torsten Kaiser wrote:
> > > Extract common checks and initialisations from load_ucode_ap()
The following changes since commit 3b2f64d00c46e1e4e9bd0bb9bb12619adac27a4b:
Linux 3.11-rc2 (2013-07-21 12:05:29 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux.git
tags/fixes-for-linus
for you to fetch changes up to
From: David Miller
Date: Wed, 07 Aug 2013 16:27:48 -0700 (PDT)
> Look, I'm going to fix this myself, because I'm pretty tired of
> waiting for the obvious fix.
Someone please test this:
diff --git a/include/linux/etherdevice.h b/include/linux/etherdevice.h
index c623861..afc02a6 100644
---
On 08/06/2013 02:11 PM, Kees Cook wrote:
Hi Zach,
I like the idea behind these clean ups. Thanks for working on them!
On Fri, Aug 02, 2013 at 04:21:40PM -0700, Zach Levis wrote:
This v3 is based off Oleg's changes from "exec: more cleanups" and
"exec: minor cleanups + minor fix"
I would
On Wed, 7 Aug 2013 12:39:56 -0400 Ken Steele wrote:
> This change adds TILE-Gx SIMD instructions to the software raid
> (md), modeling the Altivec implementation. This is only for Syndrome
> generation; there is more that could be done to improve recovery,
> as in the recent Intel SSE3 recovery
On Aug 7, 2013, at 5:26 PM, Andrew Morton wrote:
> On Wed, 7 Aug 2013 17:21:32 -0400 Ed Cashin wrote:
...
>> Sorry. I tried to cover that but maybe should have separated it more
>> clearly from the running text.
>>
>> The patch changes the current behavior encountered with direct I/O and aoe:
* Steven Rostedt (rost...@goodmis.org) wrote:
> On Wed, 2013-08-07 at 12:03 -0400, Mathieu Desnoyers wrote:
>
> > You might want to try creating a global array of counters (accessible
> > both from C for printout and assembly for update).
> >
> > Index the array from assembly using: (2f - 1f)
From: Phil Sutter
Date: Wed, 7 Aug 2013 20:37:58 +0200
> One could simply call skb_push(skb, ETH_HLEN) right after calling
> eth_type_trans(skb, dev) in order to undo the 'data' and 'len'
> adjustment. Not sure if this kind of hack is the right way to go here,
> or if the whole af_packet parses
On 08/07/13 15:31, Will Deacon wrote:
> On Wed, Aug 07, 2013 at 09:14:55PM +0100, Vince Weaver wrote:
>
>> [ 388.509063] Unable to handle kernel paging request at virtual address
>> 73fd14cc
>> [ 388.509063] pgd = eca6c000
>> [ 388.519805] [73fd14cc] *pgd=
>> [ 388.523651] Internal
This patch fixes a bug that is triggered when cts(cbc(aes)) is used with
nx-crypto driver on input larger than 32 bytes.
The chaining value from co-processor was not being saved. This value is
needed because it is used as the IV by cts(cbc(aes)).
Signed-off-by: Fionnuala Gunter
Reviewed-by:
On Thu, Aug 8, 2013 at 12:58 AM, Colin Cross wrote:
> Can you try add a call to show_state_filter(TASK_UNINTERRUPTIBLE) in
> the error path of try_to_freeze_tasks(), where it prints the "refusing
> to freeze" message? It will print the stack trace of every thread
> since they are all in the
On Wednesday, August 07, 2013 05:03:59 PM Viresh Kumar wrote:
> On 7 August 2013 17:00, Sudeep KarkadaNagesha
> wrote:
> > Any particular reason we need this check in all drivers after your
> > commit: 5a1c0228 "cpufreq: Avoid calling cpufreq driver's target()
> > routine if target_freq ==
Hi Linus,
I'm stepping in to help Olof in arm-soc land while Arnd is away for a
bit. Please consider the fixes below for -rc5, Thanks.
Kevin
The following changes since commit 5ae90d8e467e625e447000cb4335c4db973b1095:
Linux 3.11-rc3 (2013-07-28 20:53:33 -0700)
are available in the git
The following changes since commit 3b2f64d00c46e1e4e9bd0bb9bb12619adac27a4b:
Linux 3.11-rc2 (2013-07-21 12:05:29 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git
tags/msm-cleanup3-for-3.12
for you to fetch changes up to
Hi Mark,
On Wed, Aug 7, 2013 at 9:45 AM, Mark Brown wrote:
> On Fri, Aug 02, 2013 at 12:59:24AM -0700, Felipe F. Tonello wrote:
>
>> +int jack_ctl_integer_info(struct snd_kcontrol *kcontrol,
>> + struct snd_ctl_elem_info *uinfo)
>> +{
>> + uinfo->type =
Can you try add a call to show_state_filter(TASK_UNINTERRUPTIBLE) in
the error path of try_to_freeze_tasks(), where it prints the "refusing
to freeze" message? It will print the stack trace of every thread
since they are all in the freezer, so the output will be very long.
On Wed, Aug 7, 2013 at
On Wednesday, August 07, 2013 04:25:14 PM Sedat Dilek wrote:
> On Wed, Aug 7, 2013 at 7:54 AM, Stephen Rothwell
> wrote:
> > Hi all,
> >
> > Changes since 20130806:
> >
> > The ext4 tree lost its build failure.
> >
> > The mvebu tree gained a build failure so I used the version from
> >
On 08/06/2013 01:22 PM, Chris Metcalf wrote:
[...]
/**
+ * schedule_on_each_cpu - execute a function synchronously on each online CPU
+ * @func: the function to call
+ *
+ * schedule_on_each_cpu() executes @func on each online CPU using the
+ * system workqueue and blocks until all CPUs have
> From: Borislav Petkov [mailto:b...@alien8.de]
> Sent: Wednesday, July 24, 2013 6:33 AM
> On Tue, Jul 23, 2013 at 11:00:26PM +0200, Torsten Kaiser wrote:
> > Extract common checks and initialisations from load_ucode_ap() and
> > save_microcode_in_initrd_amd() to load_microcode_amd_early().
> >
On Wednesday, August 07, 2013 10:48:54 AM Nishanth Menon wrote:
> Rafael,
> offline question:
Not really offline. :-)
> On 08/06/2013 09:15 AM, Rafael J. Wysocki wrote:
> > On Tuesday, August 06, 2013 08:08:20 AM Nishanth Menon wrote:
> >> change in subject to reflect new discussion.
> >>
> >>
On Wed, Aug 07, 2013 at 04:37:27PM +0200, Michal Hocko wrote:
> On Wed 07-08-13 09:58:18, Tejun Heo wrote:
> > Hello,
> >
> > On Wed, Aug 07, 2013 at 03:46:54PM +0200, Michal Hocko wrote:
> > > OK, I have obviously misunderstood your concern mentioned in the other
> > > email. Could you be more
This change makes the following build warning go away:
[...]
LINKvmlinux
LD vmlinux.o
MODPOST vmlinux.o
WARNING: modpost: Found 2 section mismatch(es).
To see full details build your kernel with:
'make CONFIG_DEBUG_SECTION_MISMATCH=y'
Signed-off-by: Markus Mayer
Reviewed-by:
On Wed, Aug 07, 2013 at 04:30:33PM +0100, Vince Weaver wrote:
> On Wed, 7 Aug 2013, Vince Weaver wrote:
> > On Wed, 7 Aug 2013, Will Deacon wrote:
> > > diff --git a/arch/arm/kernel/perf_event.c b/arch/arm/kernel/perf_event.c
> > > index d9f5cd4..0500f10b 100644
> > > ---
On Wednesday 07 August 2013 15:34:08 Jiri Kosina wrote:
> On Wed, 7 Aug 2013, Peter Wu wrote:
> > [..]
> > I have applied Manoj's patch[1] on top of 3.11-rc4 which seem to fix the
> > issue. One observation is that the new device is named /dev/hidraw1
> > instead of /dev/hidraw0. Example:
> >
> >
On Tue, Aug 6, 2013 at 10:35 PM, Tony Luck wrote:
> ERST is at the whim of the BIOS writer (the ACPI standard doesn't provide any
> suggestions on record sizes). My systems support ~6K record size.
Off by a little - 7896 bytes on my current machine.
> efivars has, IIRC, a 1k limit coded in the
Hi Kent,
On Wed, 2013-08-07 at 14:54 -0700, Kent Overstreet wrote:
> Immutable biovecs are going to require an explicit iterator. To
> implement immutable bvecs, a later patch is going to add a bi_bvec_done
> member to this struct; for now, this patch effectively just renames
> things.
> diff
On Wed, Aug 07, 2013 at 09:14:55PM +0100, Vince Weaver wrote:
> Hello
Hi Vince,
> I don't have time to come up with a test case right now, but I've
> applied the patch to fix the oops from two days ago, and re-ran
> my perf_fuzzer tool and it immediately came up with another issue on ARM.
> This
On Wed, 07 Aug 2013 17:02:23 -0500
Eric Sandeen wrote:
> On 8/7/13 4:43 PM, Sergei Trofimovich wrote:
>
> ...
>
> > Meet uselex.rb: one-file script to parse 'nm' output:
> >
> > https://github.com/trofi/uselex/blob/master/uselex.rb
>
> Nice to meet you! I think I've met your close
On Wed, Aug 07, 2013 at 02:52:59PM -0700, David Brown wrote:
On Wed, Aug 07, 2013 at 01:58:16PM -0700, David Brown wrote:
The following changes since commit 3b2f64d00c46e1e4e9bd0bb9bb12619adac27a4b:
Linux 3.11-rc2 (2013-07-21 12:05:29 -0700)
are available in the git repository at:
Add driver support for the Texas Instruments BQ24190
battery charger. Some of the information provided by
the device is about the charger and other information
is about the battery so create two power_supply objects
(one for each) and provide the appropriate information
for each one.
The device
On 08/07/2013 02:12 PM, Alessandro Rubini wrote:
>
> I understand. My impression of devicetree is exactly like that, I must
> say. What we have here is a very clean PCI enumeration of it all: we
> only need to specify the mapping of GPIO pins (i.e. card-detect for
> mmc) and DMA channels, as all
No need for silly open coding - and struct sg_iovec has exactly the same
layout as struct iovec...
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
---
block/scsi_ioctl.c | 39 ++-
1 file changed, 10 insertions(+), 29 deletions(-)
diff --git
The following changes since commit 48a409644199d5efff6d966cd72ccc7f5a06c2a5:
shell-completion: Make options accept '=' as last char (2013-08-02 12:07:39
-0300)
are available in the git repository at:
git://git.inai.de/kmod master
for you to fetch changes up to
It occurred to an openSUSE user that our mkinitrd would throw a
warning when used with kmod:
libkmod: conf_files_list: unsupported file mode /dev/null: 0x21b6
Grepping for the error message revealed that there might be a missing
"else" keyword here, since it is unusual to put an "if" directly
On Wednesday, August 07, 2013 11:49:26 AM Bjorn Helgaas wrote:
> On Tue, Aug 6, 2013 at 5:00 PM, Rafael J. Wysocki wrote:
> > From: Rafael J. Wysocki
> >
> > In theory, under a given ACPI namespace node there should be only
> > one child device object with _ADR whose value matches a given bus
>
On 08/07/2013 02:12 PM, Alessandro Rubini wrote:
>
> Thank you for your feedback. May I ask three more questions (this is
> the 1st)?
>
> Is the patch-set a viable approach for mainline, modulo serious evaluation
> of the hairy IRQ details and the other bits?
>
Serious evaluation is
Our fancy new bvec iterator makes code like this much easier to write.
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
---
fs/bio.c | 60 +---
1 file changed, 25 insertions(+), 35 deletions(-)
diff --git a/fs/bio.c b/fs/bio.c
index
For immutable biovecs, we'll be introducing a new bio_iovec() that uses
our new bvec iterator to construct a biovec, taking into account
bvec_iter->bi_bvec_done - this patch updates existing users for the new
usage.
Some of the existing users really do need a pointer into the bvec array
- those
Bcache has a hack to avoid cloning the biovec if it's all full pages -
but with immutable biovecs coming this won't be necessary anymore.
For now, we remove the special case and always clone the bvec array so
that the immutable biovec patches are simpler.
Signed-off-by: Kent Overstreet
---
The bio integrity is also stored in a bvec array, so if we use the bvec
iter code we just added, the integrity code won't need to implement its
own iteration stuff (bio_integrity_mark_head(), bio_integrity_mark_tail())
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
Cc: "Martin K. Petersen"
Cc:
Commit b55ae0a9 added code-reading.c which fails to compile on Fedora 16
with compiler version:
$ gcc --version
gcc (GCC) 4.6.3 20120306 (Red Hat 4.6.3-2)
Failure message is:
tests/code-reading.c: In function ‘do_sort_something’:
tests/code-reading.c:305:13: error: stack protector not protecting
Hi Martin,
On Sun, Aug 4, 2013 at 10:32 PM, Martin Nordholts wrote:
> ... I must admit I did not see this. I realize that this makes my
> patch a dead end. I wish I would have seen that earlier, that would
> have saved time for everyone. My apologies.
>
> Regarding the interaction with non-i2c
This adds a mechanism by which we can advance a bio by an arbitrary
number of bytes without modifying the biovec: bio->bi_iter.bi_bvec_done
indicates the number of bytes completed in the current bvec.
Various driver code still needs to be updated to not refer to the bvec
directly before we can
On 8/7/13 4:43 PM, Sergei Trofimovich wrote:
...
> Meet uselex.rb: one-file script to parse 'nm' output:
>
> https://github.com/trofi/uselex/blob/master/uselex.rb
Nice to meet you! I think I've met your close relative,
ftp://ftp.samba.org/pub/unpacked/junkcode/findstatic.pl :)
When we start sharing biovecs, keeping bi_vcnt accurate for splits is
going to be error prone - and unnecessary, if we refactor some code.
So bio_segments() has to go - but most of the existing users just needed
to know if the bio had multiple segments, which is easier - add a
This is prep work for introducing a more general bio_split().
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
Cc: NeilBrown
Cc: Alasdair Kergon
Cc: Lars Ellenberg
Cc: Peter Osterlund
Cc: Sage Weil
---
drivers/block/pktcdvd.c | 2 +-
drivers/md/linear.c | 2 +-
drivers/md/raid0.c |
Now that we've got a mechanism for immutable biovecs -
bi_iter.bi_bvec_done - we need to convert drivers to use primitives that
respect it instead of using the bvec array directly.
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
Cc: NeilBrown
Cc: "Ed L. Cashin"
Cc: Alasdair Kergon
Cc:
We need to convert the dm code to the new bvec_iter primitives which
respect bi_bvec_done; they also allow us to drastically simplify dm's
bio splitting code.
Also kill bio_sector_offset(), dm was the only user and it doesn't make
much sense anymore.
Signed-off-by: Kent Overstreet
Cc: Jens
This adds a generic mechanism for chaining bio completions. This is
going to be used for a bio_split() replacement, and some other things in
the future.
This is implemented with a new bio flag that bio_endio() checks; it
would definitely be cleaner to implement chaining with a bi_end_io
function,
Now that drivers have been converted to the new bvec_iter primitives,
there's no need to trim the bvec before we submit it; and we can't trim
it once we start sharing bvecs.
It used to be that passing a partially completed bio (i.e. one with
nonzero bi_idx) to generic_make_request() was a
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
---
fs/bio-integrity.c | 45 ---
fs/bio.c| 90 -
include/linux/bio.h | 30 --
3 files changed, 165 deletions(-)
diff --git a/fs/bio-integrity.c
Now that we've got a mechanism for immutable biovecs -
bi_iter.bi_bvec_done - we need to convert drivers to use primitives that
respect it instead of using the bvec array directly.
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
Cc: Sage Weil
Cc: ceph-de...@vger.kernel.org
---
The new bio_split() can split arbitrary bios - it's not restricted to
single page bios, like the old bio_split() (previously renamed to
bio_pair_split()). It also has different semantics - it doesn't allocate
a struct bio_pair, leaving it up to the caller to handle completions.
Then convert the
On Wed, Aug 7, 2013 at 2:37 PM, Jason Baron wrote:
>
> ok - I can see 2 variants here as you mentioned:
>
> 1) 'Unbiased' - we want to treat both branches equally but don't want
> the load/test/jmp sequence. For things like the scheduler stats.
>
> 2) 'Biased' - where the unlikely path is moved
On Wed, 07 Aug, at 10:10:54AM, Roy Franz wrote:
> I went this way since the shared code is in a separate file - I really
> didn't like using a global variable as part of the interface to
> the shared code. This has the nice side benefit of allowing the ARM
> stub to not use any global variables,
Now that drivers have been converted to the bvec_iter primitives, they
shouldn't be modifying the biovec anymore and thus saving it is
unnecessary - code that was previously making a backup of the bvec array
can now just save bio->bi_iter.
Also, when cloning bios we can usually just reuse the
Now that we've got drivers converted to the new immutable bvec
primitives, bio splitting becomes much easier. In a few patches,
bio_clone() will be changed to share the old bio's bvec instead of
copying it, and bio_split() will do exactly what's being done here.
Signed-off-by: Kent Overstreet
bio_iovec_idx() and __bio_iovec() don't have any valid uses anymore -
previous users have been converted to bio_iovec_iter() or other methods.
__BVEC_END() has to go too - the bvec array can't be used directly for
the last biovec because we might only be using the first portion of it,
we have to
Someone cut and pasted md's md_trim_bio() into xen-blkfront.c. Come on,
we should know better than this.
Signed-off-by: Kent Overstreet
Cc: Jens Axboe
Cc: Neil Brown
Cc: Konrad Rzeszutek Wilk
Cc: Jeremy Fitzhardinge
---
drivers/block/xen-blkfront.c | 53
This patch doesn't itself have any functional changes, but immutable
biovecs are going to add a bi_bvec_done member to bi_iter, which will
need to be saved too here.
Signed-off-by: Kent Overstreet
Cc: Alasdair Kergon
Cc: dm-de...@redhat.com
---
drivers/md/dm-bio-record.h | 12 +++-
1
Jens - here's the immutable biovec patch series. I'd really like to get
this stuff into 3.12 if at all possible, and I think this series ought
to be ready - and, the dio rewrite and various other fun stuff is gated
on this stuff.
Not much in these patches has changed over the past six months or
Greg Kroah-Hartman wrote:
The proxy device nodes are application specific and need to be
created as needed by applications.
But applications do not have the permissions in a system to create
device nodes. Nor should they need that permission.
Agreed. But you need root permissions to
On Wed, Aug 07, 2013 at 01:58:16PM -0700, David Brown wrote:
The following changes since commit 3b2f64d00c46e1e4e9bd0bb9bb12619adac27a4b:
Linux 3.11-rc2 (2013-07-21 12:05:29 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git
101 - 200 of 1724 matches
Mail list logo