linux.kernel Google Group

Re: [PATCH 1/2] ftrace: Add duration filtering to function graph tracer

tim.b...@am.sony.com (Tim Bird) — Tue, 07 Jul 2009 02:40:07 UT

Doh! I think I saw that, but only had a vague recollection of it.
Yeah, that would be a problem ;-)
I'll give the issues some more thought, and see if I can come
up with something.
Thanks for the feedback.
-- Tim
=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America

Re: [PATCH 4/5] add isolate pages vmstat

minchan....@gmail.com (Minchan Kim) — Tue, 07 Jul 2009 02:40:11 UT

On Tue, 7 Jul 2009 11:12:33 +0900 (JST)
I am okay if it happens without reclaim bomb on server system which
have a lot of memory as you said.

Re: [PATCH 1/2] ftrace: Add duration filtering to function graph tracer

rost...@goodmis.org (Steven Rostedt) — Tue, 07 Jul 2009 02:20:12 UT

There's no guarantee that readers will not be going at the same time
as writers. I tried hard to allow the ring buffer to accept a reader on
another CPU while writers were happening. A writer may even preempt a
reader on the same CPU.
Heh, I doubt that would be acceptable. The problem is that between reserve

Re: [Bug #13318] AGP doesn't work anymore on nforce2

shaohua...@intel.com (Shaohua Li) — Tue, 07 Jul 2009 02:20:10 UT

Can anybody else help look at this issue? I really have no idea why this could
happens.
Thanks,
Shaohua

Re: Possible memory leak via inotify_add_watch

epa...@redhat.com (Eric Paris) — Tue, 07 Jul 2009 02:20:06 UT

Any chance you could give this a shot and see if it fixes it up for you?
-Eric
commit 27d13f5ef42fc4c976eae8318c860e a88c2076aa
Author: Eric Paris <epa...@redhat.com>
inotify: do not leak inode marks on inotify_rm_watch

inotify had a ref cnt problem and inode marks were being leaked when they

Re: [PATCH] drivers/net/cs89x0.c: Avoid using magic number in set_dma_mode

da...@davemloft.net (David Miller) — Tue, 07 Jul 2009 02:20:09 UT

...
Applied to net-next-2.6, thanks!

Re: [PATCH 4/5] add isolate pages vmstat

kosaki.motoh...@jp.fujitsu.com (KOSAKI Motohiro) — Tue, 07 Jul 2009 02:20:10 UT

Hmmm. I disagree.
isolated pages can become more than >1GB on server systems.
Who want >1GB unaccountable memory?

Re: [PATCH][RFC] asm-generic:remove calling flush_write_buffers() in dma_sync_*_for_cpu

tom.leim...@gmail.com (Ming Lei) — Tue, 07 Jul 2009 02:00:18 UT

2009/6/30 Arnd Bergmann <a...@arndb.de>:
It seems that there is only one dma-mmaping implementation on ARM, doesn't it?
Is it necessary that using dma-mapping-common.h on ARM?

Re: PROBLEM: Nvidia OHCI chipset MCP78S (Geforce 8200) hangs on > 1Mbit transfers if noapic/acpi=noirq is missing

st...@rowland.harvard.edu (Alan Stern) — Tue, 07 Jul 2009 02:00:16 UT

The speed of iso operations cannot be changed, because the transactions
have to occur at a specified interval.
OHCI doesn't provide any way to slow down bulk operations either, once
a bulk URB has been submitted. The best you can do is slow down the
rate of submission. But you already tried that and it didn't work.

Re: 2.6.31-rc2 warning in drivers/usb/host/ehci-hcd.c

st...@rowland.harvard.edu (Alan Stern) — Tue, 07 Jul 2009 02:00:16 UT

This is definitely a bug in the option driver. If you remove the call
to usb_clear_halt() in drivers/usb/serial/option.c:op tion_open(), does
the warning go away and does everything continue to work okay?
Alan Stern

Re: cpufreq cleanups - .30 vs .31

da...@redhat.com (Dave Jones) — Tue, 07 Jul 2009 02:00:14 UT

> So if not find too intrusive, I'd say:
> Venkatesh's whole series of:
> [patch 0/4] Take care of cpufreq lockdep issues (take 2)
> should be seen in .31.
> ...
> The one patch from Mathieu:
> [patch 2.6.30 2/4] CPUFREQ: fix (utter) cpufreq_add_dev mess
> is a separate, general cleanup which should show up in .31.

Re: [dm-devel] REQUEST for new 'topology' metrics to be moved out of the 'queue' sysfs directory.

ne...@suse.de (Neil Brown) — Tue, 07 Jul 2009 01:50:10 UT

They might be "clear" but I'm not convinced that they are "correct".
These two paragraphs are contradictory. There is no sense in which a
RAID chunk size is a preferred minimum I/O size.
To some degree it is actually a 'maximum' preferred size for random
IO. If you do random IO is blocks larger than the chunk size then you

Re: [PATCH 1/2] ftrace: Add duration filtering to function graph tracer

tim.b...@am.sony.com (Tim Bird) — Tue, 07 Jul 2009 01:50:09 UT

That's something I'm worried about.
Note that this patch only uses ring_buffer_peek_previous (which doesn't
alter anything in the log), and ring_buffer_event_discard(), which should
be atomic on "blotting out" the entry. Obviously, a change of page
contents between the two would make things interesting, but since

[PATCH] io-controller: Get rid of css id from io cgroup

guijianf...@cn.fujitsu.com (Gui Jianfeng) — Tue, 07 Jul 2009 01:50:07 UT

Get rid of css id from io cgroup since it's nothing
more than keeping track of iocg. An alternative is
caching iocg pointer in io group, just remove the
complexity.
Signed-off-by: Gui Jianfeng <guijianf...@cn.fujitsu.com>
---
block/elevator-fq.c | 36 ++++++++++++------------------ ------
block/elevator-fq.h | 2 +-

Re: [PATCH 4/5] add isolate pages vmstat

minchan....@gmail.com (Minchan Kim) — Tue, 07 Jul 2009 01:50:09 UT

It looks good to me.
Thanks for your effort. I added my review sign. :)
Let remain one side note.
This accounting feature results from direct reclaim bomb.
If we prevent direct reclaim bomb, I think this feature can be removed.
As I know, Rik or Wu is making patch for throttling direct reclaim.