Re: ##freemail## Re: [PATCH v2] mm: hwpoison: disable memory error handling on 1GB hugepage

2019-08-21 Thread Wanpeng Li
On Wed, 21 Aug 2019 at 13:41, Naoya Horiguchi wrote: > > On Tue, Aug 20, 2019 at 03:03:55PM +0800, Wanpeng Li wrote: > > Cc Mel Gorman, Kirill, Dave Hansen, > > On Tue, 11 Jun 2019 at 07:51, Naoya Horiguchi > > wrote: > > > > > > On Wed, May 29, 20

Re: [PATCH v2] mm: hwpoison: disable memory error handling on 1GB hugepage

2019-08-20 Thread Wanpeng Li
Cc Mel Gorman, Kirill, Dave Hansen, On Tue, 11 Jun 2019 at 07:51, Naoya Horiguchi wrote: > > On Wed, May 29, 2019 at 04:31:01PM -0700, Mike Kravetz wrote: > > On 5/28/19 2:49 AM, Wanpeng Li wrote: > > > Cc Paolo, > > > Hi all, > > > On Wed, 14 Feb 20

Re: [PATCH v2] mm: hwpoison: disable memory error handling on 1GB hugepage

2019-06-11 Thread Wanpeng Li
On Tue, 11 Jun 2019 at 07:51, Naoya Horiguchi wrote: > > On Wed, May 29, 2019 at 04:31:01PM -0700, Mike Kravetz wrote: > > On 5/28/19 2:49 AM, Wanpeng Li wrote: > > > Cc Paolo, > > > Hi all, > > > On Wed, 14 Feb 2018 at 06:34, Mike Kravetz > >

Re: [PATCH v2] mm: hwpoison: disable memory error handling on 1GB hugepage

2019-05-28 Thread Wanpeng Li
this 1GB poisoned hugetlbfs page. We have a plan to split this 1GB hugetblfs page by 2MB hugetlbfs pages/4KB pages, maybe file remap to a virtual address range which is 2MB/4KB page granularity, also split the KVM MMU 1GB SPTE into 2MB/4KB and mark the offensive SPTE w/ a hwpoison flag, a sigbus will be delivered to VM at page fault next time for the offensive SPTE. Is this proposal acceptable? Regards, Wanpeng Li

Re: [tip:sched/core] sched/core: Add debugging code to catch missing update_rq_clock() calls

2017-02-22 Thread Wanpeng Li
to cure things in my (very) > limited testing. > Tested-by: Wanpeng Li > --- > diff --git a/kernel/sched/core.c b/kernel/sched/core.c > index 96e4ccc..b773821 100644 > --- a/kernel/sched/core.c > +++ b/kernel/sched/core.c > @@ -5609,7 +5609,7 @@ static void migrate_tas

Re: [PATCH 6/6] doc/kvm: Add halt polling documentation

2016-10-13 Thread Wanpeng Li
be used and the effect of tuning the module parameters. How about replace "halt-polling" by "Adaptive halt-polling"? Btw, thanks for your docs. Regards, Wanpeng Li > > Signed-off-by: Suraj Jitindar Singh > --- > Documentation/virtual/kvm/00-INDEX | 2 + &

Re: [PATCH v2 0/4] implement vcpu preempted check

2016-07-07 Thread Wanpeng Li
2016-07-07 18:12 GMT+08:00 Wanpeng Li : > 2016-07-07 17:42 GMT+08:00 Peter Zijlstra : >> On Thu, Jul 07, 2016 at 04:48:05PM +0800, Wanpeng Li wrote: >>> 2016-07-06 20:28 GMT+08:00 Paolo Bonzini : >>> > Hmm, you're right. We can use bit 0 of struct kvm_steal_time

Re: [PATCH v2 0/4] implement vcpu preempted check

2016-07-07 Thread Wanpeng Li
2016-07-07 17:42 GMT+08:00 Peter Zijlstra : > On Thu, Jul 07, 2016 at 04:48:05PM +0800, Wanpeng Li wrote: >> 2016-07-06 20:28 GMT+08:00 Paolo Bonzini : >> > Hmm, you're right. We can use bit 0 of struct kvm_steal_time's flags to >> > indicate that pad[0] is a

Re: [PATCH v2 0/4] implement vcpu preempted check

2016-07-07 Thread Wanpeng Li
2016-07-06 20:28 GMT+08:00 Paolo Bonzini : > > > On 06/07/2016 14:08, Wanpeng Li wrote: >> 2016-07-06 18:44 GMT+08:00 Paolo Bonzini : >>> >>> >>> On 06/07/2016 08:52, Peter Zijlstra wrote: >>>> On Tue, Jun 28, 2016 at 10:43:07AM -0400, Pan Xin

Re: [PATCH v2 0/4] implement vcpu preempted check

2016-07-06 Thread Wanpeng Li
2016-07-06 20:28 GMT+08:00 Paolo Bonzini : > > > On 06/07/2016 14:08, Wanpeng Li wrote: >> 2016-07-06 18:44 GMT+08:00 Paolo Bonzini : >>> >>> >>> On 06/07/2016 08:52, Peter Zijlstra wrote: >>>> On Tue, Jun 28, 2016 at 10:43:07AM -0400, Pan Xin

Re: [PATCH v2 0/4] implement vcpu preempted check

2016-07-06 Thread Wanpeng Li
er and mutex_spin_on_owner. >>> These spin_on_onwer variant also cause rcu stall before we apply this patch >>> set >>> >> >> Paolo, could you help out with an (x86) KVM interface for this? > > If it's just for spin loops, you can check if the versio

Re: [PATCH v2 2/4] powerpc/spinlock: support vcpu preempted check

2016-07-06 Thread Wanpeng Li
2016-07-06 15:58 GMT+08:00 Peter Zijlstra : > On Wed, Jul 06, 2016 at 02:46:34PM +0800, Wanpeng Li wrote: >> > SO it's easy for ppc to implement such interface. Note that yield_count is >> > set by powerVM/KVM. >> > and only pSeries can run a guest for now. :)

Re: [PATCH v2 2/4] powerpc/spinlock: support vcpu preempted check

2016-07-05 Thread Wanpeng Li
Cc Paolo, kvm ml 2016-07-06 12:58 GMT+08:00 xinhui : > Hi, wanpeng > > On 2016年07月05日 17:57, Wanpeng Li wrote: >> >> Hi Xinhui, >> 2016-06-28 22:43 GMT+08:00 Pan Xinhui : >>> >>> This is to fix some lock holder preemption issues. Some other locks >&

Re: [PATCH v2 2/4] powerpc/spinlock: support vcpu preempted check

2016-07-05 Thread Wanpeng Li
ith pSeries. So we need return false if we are runnig as > powerNV. The another fact is that lppaca->yiled_count keeps zero on > powerNV. So we can just skip the machine type. Lock holder vCPU preemption can be detected by hardware pSeries o

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-23 Thread Wanpeng Li
On Fri, Jan 24, 2014 at 11:09:07AM +0800, Wanpeng Li wrote: >Hi Christoph, >On Mon, Jan 20, 2014 at 04:13:30PM -0600, Christoph Lameter wrote: >>On Mon, 20 Jan 2014, Wanpeng Li wrote: >> >>> >+ enum zone_type high_zoneidx = gfp_zone(flags); >>>

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-23 Thread Wanpeng Li
Hi Christoph, On Mon, Jan 20, 2014 at 04:13:30PM -0600, Christoph Lameter wrote: >On Mon, 20 Jan 2014, Wanpeng Li wrote: > >> >+ enum zone_type high_zoneidx = gfp_zone(flags); >> > >> >+ if (!node_present_pages(searchnode)) { >> >+

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-20 Thread Wanpeng Li
On Mon, Jan 20, 2014 at 04:13:30PM -0600, Christoph Lameter wrote: >On Mon, 20 Jan 2014, Wanpeng Li wrote: > >> >+ enum zone_type high_zoneidx = gfp_zone(flags); >> > >> >+ if (!node_present_pages(searchnode)) { >> >+

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-20 Thread Wanpeng Li
Hi Joonsoo, On Tue, Jan 07, 2014 at 04:41:36PM +0900, Joonsoo Kim wrote: [...] > >->8 >diff --git a/mm/slub.c b/mm/slub.c >index c3eb3d3..a1f6dfa 100644 >--- a/mm/slub.c >+++ b/mm/slub.c >@@ -1672,7 +1672,19 @@ static void *get_partial(struct kmem_cache *s, gfp_t >f

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-07 Thread Wanpeng Li
balance situation. In this case, the slub slow path can freeze the partial slab in remote node again. So why the slab cache is fragmented as below? Regards, Wanpeng Li >As such we end up only using 1 entry in each slab: > >slabmem objects >

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-07 Thread Wanpeng Li
cache *s, gfp_t >flags, int node, > { >void *object; >int searchnode = (node == NUMA_NO_NODE) ? numa_node_id() : node; >+ struct zonelist *zonelist; >+ struct zoneref *z; >+ struct zone *zone; >+ enum zone_type high_zoneidx = gfp_

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-07 Thread Wanpeng Li
On Tue, Jan 07, 2014 at 06:31:56PM +0900, Joonsoo Kim wrote: >On Tue, Jan 07, 2014 at 05:21:45PM +0800, Wanpeng Li wrote: >> On Tue, Jan 07, 2014 at 06:10:16PM +0900, Joonsoo Kim wrote: >> >On Tue, Jan 07, 2014 at 04:48:40PM +0800, Wanpeng Li wrote: >> >> Hi Joonsoo,

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-07 Thread Wanpeng Li
On Tue, Jan 07, 2014 at 06:10:16PM +0900, Joonsoo Kim wrote: >On Tue, Jan 07, 2014 at 04:48:40PM +0800, Wanpeng Li wrote: >> Hi Joonsoo, >> On Tue, Jan 07, 2014 at 04:41:36PM +0900, Joonsoo Kim wrote: >> >On Tue, Jan 07, 2014 at 01:21:00PM +1100, Anton Blanchard wrote: &

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-07 Thread Wanpeng Li
llowing patch? > So I think we should fold both of your two patches to one. Regards, Wanpeng Li >Thanks. > >->8 >diff --git a/mm/slub.c b/mm/slub.c >index c3eb3d3..a1f6dfa 100644 >--- a/mm/slub.c >+++ b/mm/slub.c >@@ -1672,7 +

Re: [PATCH] slub: Don't throw away partial remote slabs if there is no local memory

2014-01-06 Thread Wanpeng Li
ive > >kmalloc-16384 92 MB 74.27% >task_struct 23 MB 83.46% >idr_layer_cache 18 MB 100.00% >pgtable-2^12 17 MB 100.00% >kmalloc-65536 15 MB 100.00% >inode_cache 14 MB 100.00% >kmalloc-256

[PATCH 0/7][TRIVIAL][resend] trivial patches

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Since these patches has already send more than one week and doesn't get any response, I collect them and send out a patch set. Wanpeng Li (7) powerpc: cleanup some kernel doc warning x86/kernel: cleanup some kernel doc warning drivers/pci: cleanup some kernel doc wa

[PATCH 7/7] mm/memory.c : cleanup the coding style issue

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Signed-off-by: Wanpeng Li --- mm/memory.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/mm/memory.c b/mm/memory.c index 1b7dc66..195d6e1 100644 --- a/mm/memory.c +++ b/mm/memory.c @@ -2447,7 +2447,8 @@ static inline int pte_unmap_same(struct

[PATCH 6/7][TRIVIAL][resend] mm: cleanup page reclaim comment error

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Since there are five lists in LRU cache, the array nr in get_scan_count should be: nr[0] = anon inactive pages to scan; nr[1] = anon active pages to scan nr[2] = file inactive pages to scan; nr[3] = file active pages to scan Signed-off-by: Wanpeng Li Acked-by: KOSAKI Motohiro

[PATCH 5/7][TRIVIAL][resend] mm: cleanup kernel-doc warnings

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li fix kernel-doc warnings just like this one: Warning(../mm/page_cgroup.c:432): No description found for parameter 'id' Warning(../mm/page_cgroup.c:432): Excess function parameter 'mem' description in 'swap_cgroup_record' Signed-off-by: Wanpeng Li

[PATCH 4/7][TRIVIAL][resend] mm: cleanup on the comments of zone_reclaim_stat

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Signed-off-by: Wanpeng Li Acked-by: Minchan Kim --- include/linux/mmzone.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h index 2427706..d6a5f83 100644 --- a/include/linux/mmzone.h +++ b/include/linux

[PATCH 3/7][TRIVIAL][resend] drivers/pci: cleanup kernel-doc warning

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Warning(drivers/pci/setup-bus.c:277): No description found for parameter 'fail_head' Warning(drivers/pci/setup-bus.c:277): Excess function parameter 'failed_list' description in 'assign_requested_resources_sorted' Signed-off-by: Wanpeng Li ---

[PATCH 2/7][TRIVIAL][resend] x86/kernel: cleanup some kernel-doc warnings

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Warning(arch/x86/kernel/kgdb.c:465): No description found for parameter 'e_vector' Warning(arch/x86/kernel/kgdb.c:465): No description found for parameter 'remcomInBuffer' Warning(arch/x86/kernel/kgdb.c:465): No description found for parameter 'remc

[PATCH 1/7][TRIVIAL][resend] powerpc: cleanup kernel-doc warning

2012-06-15 Thread Wanpeng Li
From: Wanpeng Li Warning(arch/powerpc/kernel/pci_of_scan.c:210): Excess function parameter 'node' description in 'of_scan_pci_bridge' Warning(arch/powerpc/kernel/vio.c:636): No description found for parameter 'desired' Warning(arch/powerpc/kernel/vio.c:6

[PATCH] arch/powerpc/: fix kernel-doc warning

2012-06-11 Thread Wanpeng Li
From: Wanpeng Li Warning(arch/powerpc/kernel/pci_of_scan.c:210): Excess function parameter 'node' description in 'of_scan_pci_bridge' Warning(arch/powerpc/kernel/vio.c:636): No description found for parameter 'desired' Warning(arch/powerpc/kernel/vio.c:6