Hi Andrew,
After merging the akpm tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:
net/key/af_key.c: In function 'pfkey_add':
net/key/af_key.c:1476:9: error: implicit declaration of function
'audit_get_sessionid' [-Werror=implicit-function-declaration]
net/core/dev.c:
This is a patch to the 80211mgr.c file that fixes up a tabs
error found by the checkpatch.pl tool
Signed-off-by: Goirskin Ilya reven...@gmail.com
---
drivers/staging/vt6656/80211mgr.c | 628 +++---
1 file changed, 310 insertions(+), 318 deletions(-)
diff --git
On Wed, Sep 26, 2012 at 12:02:34AM -0400, Naoya Horiguchi wrote:
...
+ * page is a thp, not a non-huge compound page.
+ */
+ else if (PageTransCompound(page) !PageSlab(page))
u |= 1 KPF_THP;
Good catch!
Will this report THP for the various drivers that do
Hi Laurent,
Thanks for the review.
On Tue, Sep 25, 2012 at 9:30 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
Hi Prabhakar,
Thank you for the patch.
On Tuesday 25 September 2012 18:29:25 Prabhakar wrote:
From: Lad, Prabhakar prabhakar@ti.com
Signed-off-by: Lad,
Now CMA can't migrate mlocked page so it ends up fail to allocate
contiguous memory space. It's not good for CMA.
This patch makes mlocked page be migrated out.
Of course, it can affect realtime processes but in CMA usecase,
contiguos memory allocation failing is far worse than access latency
to
gas in binutils(2.16.91) could not parse parentheses within macro
parameters, and this is a workaround to make old gas work without
generating below errors:
arch/x86/kernel/entry_64.S: Assembler messages:
arch/x86/kernel/entry_64.S:387: Error: too many positional arguments
From: Lad, Prabhakar prabhakar@ti.com
Signed-off-by: Lad, Prabhakar prabhakar@ti.com
Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Sakari Ailus sakari.ai...@iki.fi
Cc: Paul Gortmaker paul.gortma...@windriver.com
Cc: Jean
On Wednesday 26 September 2012 10:50 AM, Patil, Rachna wrote:
This patch adds support for suspend/resume of
TSC/ADC MFDevice.
this should be merged with the patch adding support else
we may end up in a case where patch a does the runtime calls
and the call back handlers added later.
On 25 September 2012 18:47, Jassi Brar jassisinghb...@gmail.com wrote:
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
Since peripheral channel resources are not being allocated at probe,
no need to flush the channels and free the resources in remove function.
On Tue, 2012-09-25 at 09:57 -0700, Joe Perches wrote:
On Tue, 2012-09-25 at 16:37 +0300, Andy Shevchenko wrote:
On Tue, 2012-09-25 at 06:19 -0700, Joe Perches wrote:
You also added yourself as a maintainer.
Congrats/sympathies, etc...
Actually I prefer to be just a supporter, because
On Wed, Sep 26, 2012 at 9:46 AM, Neil Zhang zhan...@marvell.com wrote:
We need to return 0 or 1 when get gpio value.
Signed-off-by: Neil Zhang zhan...@marvell.com
---
drivers/gpio/gpio-pxa.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/drivers/gpio/gpio-pxa.c
On 09/26/2012 08:02 PM, Chuansheng Liu wrote:
When one CPU is going offline, and fixup_irqs() will re-set the
irq affinity in some cases, we should clean the offlining CPU from
the irq affinity.
The reason is setting offlining CPU as of the affinity is useless.
Moreover, the smp_affinity
The reclaim_clean_pages_from_list reclaims clean pages before
migration so cc.nr_migratepages should be updated.
Currently, there is no problem but it can be wrong if we
try to use the vaule in future.
Cc: Mel Gorman mgor...@suse.de
Cc: Michal Nazarewicz min...@mina86.com
Cc: Bartlomiej
On Wed, 2012-09-26 at 09:20 +0530, viresh kumar wrote:
On Tue, Sep 25, 2012 at 5:43 PM, Andy Shevchenko
andriy.shevche...@linux.intel.com wrote:
From: Heikki Krogerus heikki.kroge...@linux.intel.com
This driver should be usable on all platforms that depend on clk API.
This is not what
On Wed, Sep 26, 2012 at 12:05:11PM +0600, Ilya gorskin wrote:
This is a patch to the 80211mgr.c file that fixes up a tabs
error found by the checkpatch.pl tool
Signed-off-by: Goirskin Ilya reven...@gmail.com
---
drivers/staging/vt6656/80211mgr.c | 628
On Wed, Sep 26, 2012 at 12:17 PM, Andy Shevchenko
andriy.shevche...@linux.intel.com wrote:
This separate driver makes no sense in case it is built properly without
CLK framework. Let me check this and leave comments at patch 1/6.
Following is the commit that introduced this change :)
commit
+ } else if (cpumask_test_cpu(cpu, data-affinity))
+ cpumask_clear_cpu(cpu, data-affinity);
You meant to use 'affinity' (instead of data-affinity) in the above 2
statements
right? Note that we do chip-irq_set_affinity(data, affinity, true); further
down.
Hi,
2012/9/26 Masami Hiramatsu masami.hiramatsu...@hitachi.com:
(2012/09/26 11:52), Hyeoncheol Lee wrote:
Hi,
uprobe_event only supports %REG arguments. I think that memory fetch,
return value fetch, memory dereference functions in
kernel/trace/trace_probe.c are good for uprobe_event. So
On Wed, Sep 26, 2012 at 9:51 AM, viresh kumar viresh.ku...@linaro.org wrote:
On Wed, Sep 26, 2012 at 12:17 PM, Andy Shevchenko
andriy.shevche...@linux.intel.com wrote:
This separate driver makes no sense in case it is built properly without
CLK framework. Let me check this and leave comments
* Arnaldo Carvalho de Melo a...@infradead.org wrote:
Hi Ingo,
Please consider pulling, on top of the previous req,
- Arnaldo
The following changes since commit b1ac754b67b5a875d63bee880f60ccb0c6bd8899:
tools lib traceevent: Handle alloc_arg failure (2012-09-24 12:31:52
On Tue, Sep 25, 2012 at 9:53 AM, Roland Stigge sti...@antcom.de wrote:
For GPIOs of gpio-lpc32xx, gpio_direction_output() ignores the value argument
(initial value of output). This patch fixes this by setting the level
accordingly.
Signed-off-by: Roland Stigge sti...@antcom.de
Acked-by:
2012/9/26 Srikar Dronamraju sri...@linux.vnet.ibm.com:
Perhaps, it is not so small things, but at least, we can try.
In the userspace, memories(pages) can be paged out on swap or
files. In that case, memory dereference function needs to track
down the data on the disk and it causes I/O. This
On Wed, 2012-09-26 at 09:06 +0530, viresh kumar wrote:
On Tue, Sep 25, 2012 at 5:43 PM, Andy Shevchenko
andriy.shevche...@linux.intel.com wrote:
Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com
Cc: Viresh Kumar viresh.li...@gmail.com
---
MAINTAINERS | 16
On 26.09.12 at 08:15, Tao Guo glorious...@gmail.com wrote:
gas in binutils(2.16.91) could not parse parentheses within macro
parameters,
This description of yours contradicts the last hunk of the patch -
iirc the requirement for macro parameters in those old gas
versions is to be fully
Started having audio problems when trying out the latest tree
(v3.6-rc7-10-g56d27ad). When playing any kind of audio, there was
significant distortion, mostly crackling noise. I'm using a Lenovo
ThinkPad X230 (Panther Point).
I did a git-bisect to locate the problem, and it seems this commit is
On Wed, Sep 26, 2012 at 02:06:08AM -0400, Naoya Horiguchi wrote:
On Wed, Sep 26, 2012 at 12:02:34AM -0400, Naoya Horiguchi wrote:
...
+* page is a thp, not a non-huge compound page.
+*/
+ else if (PageTransCompound(page) !PageSlab(page))
u
David Rientjes [Mon, Sep 24, 2012 at 03:43:27PM -0700]:
[...]
And those objects are consuming ~2.3GB of slab on your 4GB machine and
seems to only have occurred between v3.4.2 to v3.5.3.
It would be interesting to see what kmemleak would tell us.
As the Archlinux kernel does not
Good morning,
Dave Kleikamp [Tue, Sep 25, 2012 at 10:07:14AM -0500]:
Active / Total Objects (% used): 1165130 / 1198087 (97.2%)
Active / Total Slabs (% used) : 81027 / 81027 (100.0%)
Active / Total Caches (% used) : 69 / 101 (68.3%)
Active / Total Size (% used) :
On Wed, Sep 26, 2012 at 7:50 AM, David Ahern dsah...@gmail.com wrote:
I like the idea, but can't checkout the patch - does not apply to Arnaldo's
latest perf/core branch. mind rebasing?
It's against tip-master.
David
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
On Wed, 2012-09-26 at 09:30 +0530, viresh kumar wrote:
On Tue, Sep 25, 2012 at 5:43 PM, Andy Shevchenko
andriy.shevche...@linux.intel.com wrote:
diff --git a/drivers/dma/dw_dmac_at32.c b/drivers/dma/dw_dmac_at32.c
index 7bc7ac4..5c9180e 100644
--- a/drivers/dma/dw_dmac_at32.c
+++
This merge reduces code size by unifying the approach for
sending scatter-lists and regular buffers. Any type of
write operation (splice, write, put_chars) will now allocate
a port_buffer and send_buf() and free_buf() can always be used.
Thanks!
This looks much nicer and simpler. I
From: Andy Shevchenko andriy.shevche...@linux.intel.com
Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com
Acked-by: Viresh Kumar viresh.ku...@linaro.org
---
MAINTAINERS | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
From: Namhyung Kim namhyung@lge.com
Now the event grouping viewing requires collapsing all members in a
group to the leader. Thus hists__output_resort should be called after
collapsing all entries in evlist.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
From: Namhyung Kim namhyung@lge.com
The struct he_stat is for separating out statistics data of a hist
entry. It is required for later changes.
It's just a mechanical change and should have no functional
differences.
Cc: Jiri Olsa jo...@redhat.com
Cc: Arun Sharma asha...@fb.com
Cc:
From: Namhyung Kim namhyung@lge.com
Add a few of group-related field in struct perf_{evlist,evsel} so that
the group information in a evlist can be known easily.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
Signed-off-by: Namhyung Kim namhy...@kernel.org
---
* Cyrill Gorcunov gorcu...@openvz.org wrote:
On Tue, Sep 25, 2012 at 12:23:23PM -0400, Vince Weaver wrote:
Hello
This is an updated version of the patch. It uses
ARCH_PERFMON_EVENTSEL_INT for the DATA_READ event, with the assumption
that x86_pmu_hw_config() is going to set that bit
From: Namhyung Kim namhyung@lge.com
Add --group option to enable event grouping. When enabled, all the
group members information will be shown together with the leader.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
Signed-off-by: Namhyung Kim namhy...@kernel.org
---
From: Namhyung Kim namhyung@lge.com
When using event group viewer, it's better to show the group
description rather than the leader information alone.
If a leader did not contain any member, it's a non-group event.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
Cc:
From: Namhyung Kim namhyung@lge.com
Add and use he_stat__add_{period,stat} for calculating hist entry's
stat. It will be used for accumulated stats later as well.
Cc: Jiri Olsa jo...@redhat.com
Cc: Arun Sharma asha...@fb.com
Cc: Stephane Eranian eran...@google.com
Cc: Frederic Weisbecker
From: Namhyung Kim namhyung@lge.com
Since it is set to 1 for a new hist entry, no need to set to
separately. Move it to a template entry.
Cc: Jiri Olsa jo...@redhat.com
Cc: Arun Sharma asha...@fb.com
Cc: Stephane Eranian eran...@google.com
Cc: Frederic Weisbecker fweis...@gmail.com
From: Namhyung Kim namhyung@lge.com
Since we have all necessary information in the leader events and
other members don't, bypass members. Member events will be shown
along with the leaders if event group is enabled.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
Cc:
From: Namhyung Kim namhyung@lge.com
Show group members' overhead also when showing the leader's if event
group is enabled. At this time, only implemented overhead part in
order to ease review and other parts can be added later once this
patch settled down.
Cc: Jiri Olsa jo...@redhat.com
Cc:
From: Namhyung Kim namhyung@lge.com
Show group members' overhead also when showing the leader's if event
group is enabled. At this time, only implemented overhead part in
order to ease review and other parts can be added later once this
patch settled down.
Cc: Jiri Olsa jo...@redhat.com
Cc:
From: Namhyung Kim namhyung@lge.com
Show group members' overhead also when showing the leader's if event
group is enabled. At this time, only implemented overhead part in
order to ease review and other parts can be added later once this
patch settled down.
Cc: Jiri Olsa jo...@redhat.com
Cc:
From: Namhyung Kim namhyung@lge.com
Like group_stats in hist_entry, total periods information also need to
be known to the leader.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
Signed-off-by: Namhyung Kim namhy...@kernel.org
---
tools/perf/util/hist.c | 25
From: Namhyung Kim namhyung@lge.com
Save group relationship information so that it can be restored when
perf report is running.
Cc: Jiri Olsa jo...@redhat.com
Cc: Stephane Eranian eran...@google.com
Signed-off-by: Namhyung Kim namhy...@kernel.org
---
tools/perf/builtin-record.c | 3 +
Hi,
This is my second attempt to support event group on perf report.
For basic idea and usage example, please see my original post [1].
The main difference than v1 is adding HEADER_GROUP_DESC feature and
use it for regenerating group relationship on perf report. It will
save actual (i.e.
From: Namhyung Kim namhyung@lge.com
To support viewing an event group together, collapse all of members in
the group to the leader's tree. The entries in the leaders' tree will
have group_stats to store those information.
This patch introduced an additional field 'event_group' in
From: Namhyung Kim namhyung@lge.com
During the event parsing, perf_evlist can have leader-only groups
which has nr_members as 1. Since they has no difference than a normal
non-group event don't count them as a event group.
Add perf_evlist__recalc_nr_groups to count actual group numbers.
From: Namhyung Kim namhyung@lge.com
So that the perf report won't lost the cpu utilization information.
For example, if there're two process that have same name.
$ perf report --stdio --showcpuutilization -s pid
[SNIP]
# Overhead sysus Command: Pid
#
On 09/26/2012 12:22 PM, Liu, Chuansheng wrote:
+ } else if (cpumask_test_cpu(cpu, data-affinity))
+ cpumask_clear_cpu(cpu, data-affinity);
You meant to use 'affinity' (instead of data-affinity) in the above 2
statements
right? Note that we do
On Tue, Sep 25, 2012 at 08:17:03AM -0300, Arnaldo Carvalho de Melo wrote:
Em Tue, Sep 25, 2012 at 04:20:53PM +0800, Feng Tang escreveu:
On Tue, 25 Sep 2012 11:11:21 +0900
Namhyung Kim namhy...@kernel.org wrote:
Ditto. Plus it might leak previous input_name.
Nice catch, will check the
Btw, on a slightly different note, I'm also rather surprised that the above
code doesn't care about the return value of chip-irq_set_affinity() ..
Shouldn't we warn if that fails?
It seems another case when irq_set_affinity is NULL whenever affinity is
changed or not before that,
For this
On 9/26/12, Jan Beulich jbeul...@suse.com wrote:
On 26.09.12 at 08:15, Tao Guo glorious...@gmail.com wrote:
gas in binutils(2.16.91) could not parse parentheses within macro
parameters,
This description of yours contradicts the last hunk of the patch -
iirc the requirement for macro
Shouldn't we warn if that fails?
printk(Cannot set affinity for irq %i\n, irq);
This is the warning when set affinity failed.
In that case, we would end up with an incorrect data-affinity right?
Moving the clean cpu mask code into if (chip-irq_set_affinity)?
Will resend the patch and will judge the chip-irq_set_affinity(data, affinity,
true)
return value.
On Fri, Sep 21, 2012 at 23:52:11, Porter, Matt wrote:
On Fri, Sep 21, 2012 at 08:27:07AM +, Hebbar, Gururaja wrote:
On Thu, Sep 20, 2012 at 20:13:33, Porter, Matt wrote:
This series adds DMA Engine support for AM33xx, which uses
an EDMA DMAC. The EDMA DMAC has been previously
On 09/26/2012 01:47 PM, Liu, Chuansheng wrote:
Shouldn't we warn if that fails?
printk(Cannot set affinity for irq %i\n, irq);
This is the warning when set affinity failed.
I know.. What I meant is, the code warns only if chip-irq_set_affinity
is NULL and doesn't care if
When one CPU is going offline, and fixup_irqs() will re-set the
irq affinity in some cases, we should clean the offlining CPU from
the irq affinity.
The reason is setting offlining CPU as of the affinity is useless.
Moreover, the smp_affinity value will be confusing when the
offlining CPU come
I know.. What I meant is, the code warns only if chip-irq_set_affinity
is NULL and doesn't care if chip-irq_set_affinity was not NULL and
the function failed to set the affinity (ie., when chip-irq_set_affinity()
returns error). In other words, I meant to say that this is one more
case where
On 09/26/2012 04:46 AM, David Rientjes wrote:
On Tue, 25 Sep 2012, Christoph Lameter wrote:
No cache should ever pass those as a creation flags. We can just ignore
this bit if it happens to be passed (such as when duplicating a cache in
the kmem memcg patches)
Acked-by: Christoph Lameter
/block/zram
which is suggested by Nitin.
I don't know who should merge whose tree.
So I will add both Greg and Jens to To.
This patchset is based on next-20120926.
Minchan Kim (3):
zsmalloc: promote to lib/
zram: promote zram from staging
zram: select ZSMALLOC when ZRAM is configured
drivers
At the monent, we can configure zram in driver/block once zsmalloc
in /lib menu is configured firstly. It's not convenient.
User can configure zram in driver/block regardless of zsmalloc enabling
by this patch.
Signed-off-by: Minchan Kim minc...@kernel.org
---
drivers/block/zram/Kconfig |3
gas in binutils(2.16.91) could not parse parentheses within macro
parameters unless fully parenthesized, and this is a workaround to
make old gas work without generating below errors:
arch/x86/kernel/entry_64.S: Assembler messages:
arch/x86/kernel/entry_64.S:387: Error: too many positional
On 09/26/2012 11:08 PM, Chuansheng Liu wrote:
When one CPU is going offline, and fixup_irqs() will re-set the
irq affinity in some cases, we should clean the offlining CPU from
the irq affinity.
The reason is setting offlining CPU as of the affinity is useless.
Moreover, the smp_affinity
Hi,
We've collected up a goodly number of patches in the -nmw tree now
and we can hold off any further changes until the following merge
window, so here is the current tree content.
The major feature this time is the rbm conversion in the resource
group code. The new struct gfs2_rbm specifies
There were two functions in the xattr code which were nearly
identical, the only difference being that one was copy data into
the unstuffed xattrs and the other was copying data out from it.
This patch merges the two functions such that the code which deal
with iteration over the unstuffed xattrs
Please hold on.. I'm not yet done reviewing, I might have more comments :-)
Sure, welcome, thanks again.
From: Bob Peterson rpete...@redhat.com
This patch changes function gfs2_direct_IO so that it uses a normal
call to gfs2_glock_dq rather than a call to a multiple-dq of one item.
Signed-off-by: Bob Peterson rpete...@redhat.com
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff --git
When we get to the stage of allocating blocks, we know that the
resource group in question must contain enough free blocks, otherwise
gfs2_inplace_reserve() would have failed. So if we are left with only
free blocks which are reserved, then we must use those. This can happen
if another node has
From: Bob Peterson rpete...@redhat.com
This patch stops multiple block allocations if a nonzero
return code is received from gfs2_rbm_from_block. Without
this patch, if enough pressure is put on the file system,
you get a kernel warning quickly followed by:
BUG: unable to handle kernel NULL
From: Bob Peterson rpete...@redhat.com
This patch fixes an infinite loop in gfs2_rbm_find that was introduced
by the previous patch. The problem occurred when the length was less
than 3 but the rbm block was byte-aligned, causing it to improperly
return a extent length of zero, which caused it to
From: Benjamin Marzinski bmarz...@redhat.com
If a dirty GFS2 inode was being deleted but was in use by another node, its
metadata was not getting written out before GFS2 checked for dirty buffers in
gfs2_ail_flush(). GFS2 was relying on inode_go_sync() to write out the
metadata when the other
On 09/26/2012 01:40 PM, Liu, Chuansheng wrote:
Btw, on a slightly different note, I'm also rather surprised that the above
code doesn't care about the return value of chip-irq_set_affinity() ..
Shouldn't we warn if that fails?
It seems another case when irq_set_affinity is NULL whenever
From: Eric Sandeen sand...@redhat.com
gfs2_ail_empty_gl() contains an inline version of gfs2_trans_begin(),
so it needs an explicit sb_start_intwrite() as well, to balance the
sb_end_intwrite() which will be called by gfs2_trans_end().
With this, xfstest 068 passes on lock_nolock local gfs2.
With the recently added block reservation code, an additional function
was added to search for free blocks. This had a restriction of only being
able to search for aligned extents of free blocks. As a result the
allocation patterns when reserving blocks were suboptimal when the
existing allocation
This is part of a series of patches which are introducing the
gfs2_rbm structure throughout the block allocation code. The
main aim of this part is to create a search function which can
deal directly with struct gfs2_rbm. In this case it specifies
the initial position at which to start the search
From: Bob Peterson rpete...@redhat.com
Function gfs2_glock_wait only called function wait_on_holder and
returned its return code, so they were combined for readability.
Signed-off-by: Bob Peterson rpete...@redhat.com
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff --git
From: Bob Peterson rpete...@redhat.com
Function gfs2_bitfit was checking for state 3, but that's
impossible since it is only called from rgblk_search, which receives
only GFS2_BLKST_ constants.
Signed-off-by: Bob Peterson rpete...@redhat.com
Signed-off-by: Steven Whitehouse swhit...@redhat.com
Use the rbm structure for gfs2_setbit() in order to simplify the
arguments to the function. We have to add a bool to control whether
the clone bitmap should be updated (if it exists) but otherwise it
is a more or less direct substitution.
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff
This patch improves the tracing of block reservations by
removing some corner cases and also providing more useful
detail in the traces.
A new field is added to the reservation structure to contain
the inode number. This is used since in certain contexts it is
not possible to access the inode
From: Jan Kara j...@suse.cz
GFS2 uses i_mutex on its system quota inode to synchronize writes to
quota file. Since this is an internal inode to GFS2 (not part of directory
hiearchy or visible by user) we are safe to define locking rules for it. So
let's just get it its own locking class to make
When rgd-rd_free_clone is less than rgd-rd_reserved, the
unclaimed_blocks() calculation would wrap and produce
incorrect results. This patch checks for this condition
when this function is called from gfs2_mblk_search()
In addition, the use of this particular function in other
places in the code
The -show_options() function for GFS2 was not correctly displaying
the value when statfs slow in in use.
Signed-off-by: Steven Whitehouse swhit...@redhat.com
Reported-by: Milos Jakubicek xja...@fi.muni.cz
diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
index 3cbac68..79cac70 100644
---
Change the arguments to gfs2_testbit() so that it now just takes an
rbm specifying the position of the two bit entry to return.
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c
index 47d2346..3a288ce 100644
--- a/fs/gfs2/rgrp.c
+++
From: Bob Peterson rpete...@redhat.com
Function add_to_queue was checking may_grant for the passed-in
holder for every iteration of its gh2 loop. Now it only checks it
once at the beginning to see if a try lock is futile.
Signed-off-by: Bob Peterson rpete...@redhat.com
Signed-off-by: Steven
From: Bob Peterson rpete...@redhat.com
Function gfs2_glock_dq_wait called two-line function wait_on_demote,
so they were combined.
Signed-off-by: Bob Peterson rpete...@redhat.com
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c
index
From: Bob Peterson rpete...@redhat.com
Since function gfs2_glock_schedule_for_reclaim is only two
significant lines, we can eliminate it, simplifying the code
and making it more readable.
Signed-off-by: Bob Peterson rpete...@redhat.com
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff
From: Bob Peterson rpete...@redhat.com
This patch fixes a few small rbm related things. First, it fixes
a corner case where the rbm needs to switch bitmaps and wasn't
adjusting its buffer pointer. Second, there's a white space issue
fixed. Third, the logic in function gfs2_rbm_from_block was
A return value of 0 and 1 are acceptable. So this check isn't correct.
Regards,
Srivatsa S. Bhat
Which case value 1 is acceptable, could you share? Thanks.
OMG, why did you drop the other hunk which cleared the cpu *before*
invoking -irq_set_affinity()? IMO, altering irq affinity
One corner case which the original patch failed to take into
account was when there is a reservation which ended such that
the following block was one beyond the end of the rgrp in
question. This extra test fixes that case.
Signed-off-by: Steven Whitehouse swhit...@redhat.com
Reported-by: Bob
Hi Prabhakar,
Thanks for the patch.
On Wednesday 26 September 2012 12:05:10 Prabhakar wrote:
From: Lad, Prabhakar prabhakar@ti.com
Signed-off-by: Lad, Prabhakar prabhakar@ti.com
Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
Cc: Laurent Pinchart
On 09/26/2012 01:02 AM, Andrew Morton wrote:
nomemcg : memcg compile disabled.
base : memcg enabled, patch not applied.
bypassed : memcg enabled, with patch applied.
basebypassed
User 109.12 105.64
System 1646.84 1597.98
Elapsed
Use the new gfs2_rbm_from_block() function to replace an open
coded version of the same code.
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c
index bd3b926..0c1be38 100644
--- a/fs/gfs2/rgrp.c
+++ b/fs/gfs2/rgrp.c
@@ -1824,27 +1824,14 @@ void
On 25.09.12 at 23:27, Konrad Rzeszutek Wilk konrad.w...@oracle.com wrote:
We call 'pci_disable_device' which sets the bus_master to zero
and it also disables the PCI_COMMAND. There is no need to
do it outside the PCI library.
Not really - pci_disable_device() only does anything if enable_cnt
From: Michel Lespinasse wal...@google.com
gfs2 calls RB_EMPTY_NODE() to check if nodes are not on an rbtree.
The corresponding initialization function is RB_CLEAR_NODE().
rb_init_node() was never clearly defined and is going away.
Signed-off-by: Michel Lespinasse wal...@google.com
Signed-off-by:
Replace open coded version with a call to gfs2_rbm_from_block()
Signed-off-by: Steven Whitehouse swhit...@redhat.com
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c
index 0c1be38..06476b3 100644
--- a/fs/gfs2/rgrp.c
+++ b/fs/gfs2/rgrp.c
@@ -1890,46 +1890,30 @@ static u64 gfs2_alloc_extent(const
This patch introduces a new structure, gfs2_rbm, which is a
tuple of a resource group, a bitmap within the resource group
and an offset within that bitmap. This is designed to make
manipulating these sets of variables easier. There is also a
new helper function which converts this representation
The rs_requested field is left over from the original allocation
code, however this should have been a parameter passed to the
various functions from gfs2_inplace_reserve() and not a member of the
reservation structure as the value is not required after the
initial allocation.
This also helps
On Tue, 25 Sep 2012 10:47:03 +0900
Namhyung Kim namhy...@kernel.org wrote:
Hi Feng,
On Mon, 24 Sep 2012 23:24:05 +0800, Feng Tang wrote:
As suggested by Arnaldo, many scripts have their own usages and need
capture specific events or tracepoints, so only those scripts whose
targe events
1 - 100 of 1162 matches
Mail list logo
