All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pasha Tatashin <pasha.tatashin@oracle.com>
To: David Miller <davem@davemloft.net>
Cc: linux-kernel@vger.kernel.org, sparclinux@vger.kernel.org,
	linux-mm@kvack.org, linuxppc-dev@lists.ozlabs.org,
	linux-s390@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	x86@kernel.org, kasan-dev@googlegroups.com,
	borntraeger@de.ibm.com, heiko.carstens@de.ibm.com,
	willy@infradead.org, mhocko@kernel.org,
	ard.biesheuvel@linaro.org, will.deacon@arm.com,
	catalin.marinas@arm.com, sam@ravnborg.org,
	mgorman@techsingularity.net, Steven.Sistare@oracle.com,
	daniel.m.jordan@oracle.com, bob.picco@oracle.com
Subject: Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing
Date: Wed, 30 Aug 2017 09:19:58 -0400	[thread overview]
Message-ID: <e07c05b1-c0be-bf7f-9a29-11dc41b79d10@oracle.com> (raw)
In-Reply-To: <20170829.181208.171985548699678313.davem@davemloft.net>

Hi Dave,

Thank you for acking.

The reason I am not doing initializing stores is because they require a 
membar, even if only regular stores are following (I hoped to do a 
membar before first load). This is something I was thinking was not 
true, but after consulting with colleagues and checking processor 
manual, I verified that it is the case.

Pasha

> 
> You should probably use initializing stores when you are doing 8
> stores and we thus know the page struct is cache line aligned.
> 
> But other than that:
> 
> Acked-by: David S. Miller <davem@davemloft.net>

WARNING: multiple messages have this Message-ID (diff)
From: Pasha Tatashin <pasha.tatashin@oracle.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing
Date: Wed, 30 Aug 2017 13:19:58 +0000	[thread overview]
Message-ID: <e07c05b1-c0be-bf7f-9a29-11dc41b79d10@oracle.com> (raw)
In-Reply-To: <20170829.181208.171985548699678313.davem@davemloft.net>

Hi Dave,

Thank you for acking.

The reason I am not doing initializing stores is because they require a 
membar, even if only regular stores are following (I hoped to do a 
membar before first load). This is something I was thinking was not 
true, but after consulting with colleagues and checking processor 
manual, I verified that it is the case.

Pasha

> 
> You should probably use initializing stores when you are doing 8
> stores and we thus know the page struct is cache line aligned.
> 
> But other than that:
> 
> Acked-by: David S. Miller <davem@davemloft.net>

WARNING: multiple messages have this Message-ID (diff)
From: Pasha Tatashin <pasha.tatashin@oracle.com>
To: David Miller <davem@davemloft.net>
Cc: linux-kernel@vger.kernel.org, sparclinux@vger.kernel.org,
	linux-mm@kvack.org, linuxppc-dev@lists.ozlabs.org,
	linux-s390@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	x86@kernel.org, kasan-dev@googlegroups.com,
	borntraeger@de.ibm.com, heiko.carstens@de.ibm.com,
	willy@infradead.org, mhocko@kernel.org,
	ard.biesheuvel@linaro.org, will.deacon@arm.com,
	catalin.marinas@arm.com, sam@ravnborg.org,
	mgorman@techsingularity.net, Steven.Sistare@oracle.com,
	daniel.m.jordan@oracle.com, bob.picco@oracle.com
Subject: Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing
Date: Wed, 30 Aug 2017 09:19:58 -0400	[thread overview]
Message-ID: <e07c05b1-c0be-bf7f-9a29-11dc41b79d10@oracle.com> (raw)
In-Reply-To: <20170829.181208.171985548699678313.davem@davemloft.net>

Hi Dave,

Thank you for acking.

The reason I am not doing initializing stores is because they require a 
membar, even if only regular stores are following (I hoped to do a 
membar before first load). This is something I was thinking was not 
true, but after consulting with colleagues and checking processor 
manual, I verified that it is the case.

Pasha

> 
> You should probably use initializing stores when you are doing 8
> stores and we thus know the page struct is cache line aligned.
> 
> But other than that:
> 
> Acked-by: David S. Miller <davem@davemloft.net>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: pasha.tatashin@oracle.com (Pasha Tatashin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 07/11] sparc64: optimized struct page zeroing
Date: Wed, 30 Aug 2017 09:19:58 -0400	[thread overview]
Message-ID: <e07c05b1-c0be-bf7f-9a29-11dc41b79d10@oracle.com> (raw)
In-Reply-To: <20170829.181208.171985548699678313.davem@davemloft.net>

Hi Dave,

Thank you for acking.

The reason I am not doing initializing stores is because they require a 
membar, even if only regular stores are following (I hoped to do a 
membar before first load). This is something I was thinking was not 
true, but after consulting with colleagues and checking processor 
manual, I verified that it is the case.

Pasha

> 
> You should probably use initializing stores when you are doing 8
> stores and we thus know the page struct is cache line aligned.
> 
> But other than that:
> 
> Acked-by: David S. Miller <davem@davemloft.net>

  reply	other threads:[~2017-08-30 13:21 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-29  2:02 [PATCH v7 00/11] complete deferred page initialization Pavel Tatashin
2017-08-29  2:02 ` Pavel Tatashin
2017-08-29  2:02 ` Pavel Tatashin
2017-08-29  2:02 ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 01/11] x86/mm: setting fields in deferred pages Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 02/11] sparc64/mm: " Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-30  1:09   ` David Miller
2017-08-30  1:09     ` David Miller
2017-08-30  1:09     ` David Miller
2017-08-30  1:09     ` David Miller
2017-08-29  2:02 ` [PATCH v7 03/11] mm: deferred_init_memmap improvements Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 04/11] sparc64: simplify vmemmap_populate Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-30  1:08   ` David Miller
2017-08-30  1:08     ` David Miller
2017-08-30  1:08     ` David Miller
2017-08-30  1:08     ` David Miller
2017-08-29  2:02 ` [PATCH v7 05/11] mm: defining memblock_virt_alloc_try_nid_raw Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 06/11] mm: zero struct pages during initialization Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 07/11] sparc64: optimized struct page zeroing Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-30  1:12   ` David Miller
2017-08-30  1:12     ` David Miller
2017-08-30  1:12     ` David Miller
2017-08-30  1:12     ` David Miller
2017-08-30 13:19     ` Pasha Tatashin [this message]
2017-08-30 13:19       ` Pasha Tatashin
2017-08-30 13:19       ` Pasha Tatashin
2017-08-30 13:19       ` Pasha Tatashin
2017-08-30 17:46       ` David Miller
2017-08-30 17:46         ` David Miller
2017-08-30 17:46         ` David Miller
2017-08-30 17:46         ` David Miller
2017-08-29  2:02 ` [PATCH v7 08/11] mm: zero reserved and unavailable struct pages Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-30 21:22   ` kbuild test robot
2017-08-30 21:22     ` kbuild test robot
2017-08-30 21:22     ` kbuild test robot
2017-08-30 23:12   ` kbuild test robot
2017-08-30 23:12     ` kbuild test robot
2017-08-30 23:12     ` kbuild test robot
2017-08-30 23:12     ` kbuild test robot
2017-08-29  2:02 ` [PATCH v7 09/11] x86/kasan: explicitly zero kasan shadow memory Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 10/11] arm64/kasan: " Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02 ` [PATCH v7 11/11] mm: stop zeroing memory during allocation in vmemmap Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin
2017-08-29  2:02   ` Pavel Tatashin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e07c05b1-c0be-bf7f-9a29-11dc41b79d10@oracle.com \
    --to=pasha.tatashin@oracle.com \
    --cc=Steven.Sistare@oracle.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=bob.picco@oracle.com \
    --cc=borntraeger@de.ibm.com \
    --cc=catalin.marinas@arm.com \
    --cc=daniel.m.jordan@oracle.com \
    --cc=davem@davemloft.net \
    --cc=heiko.carstens@de.ibm.com \
    --cc=kasan-dev@googlegroups.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=mgorman@techsingularity.net \
    --cc=mhocko@kernel.org \
    --cc=sam@ravnborg.org \
    --cc=sparclinux@vger.kernel.org \
    --cc=will.deacon@arm.com \
    --cc=willy@infradead.org \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.