From: Alexander Kapshuk <alexander.kapshuk@gmail.com>
To: akpm@linux-foundation.org
Cc: dave.jiang@intel.com, mm-commits@vger.kernel.org,
linux-next@vger.kernel.org
Subject: Re: + mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix.patch added to -mm tree
Date: Tue, 21 Feb 2017 19:12:06 +0200 [thread overview]
Message-ID: <20170221171206.GA4397@box1> (raw)
In-Reply-To: <58a4bc95.wUw3v0FZUCnAXeFL%akpm@linux-foundation.org>
On Wed, Feb 15, 2017 at 12:39:49PM -0800, akpm@linux-foundation.org wrote:
>
> The patch titled
> Subject: mm,x86: fix SMP x86 32bit build for native_pud_clear()
> has been added to the -mm tree. Its filename is
> mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix.patch
>
> This patch should soon appear at
> http://ozlabs.org/~akpm/mmots/broken-out/mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix.patch
> and later at
> http://ozlabs.org/~akpm/mmotm/broken-out/mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix.patch
>
> Before you just go and hit "reply", please:
> a) Consider who else should be cc'ed
> b) Prefer to cc a suitable mailing list as well
> c) Ideally: find the original patch on the mailing list and do a
> reply-to-all to that, adding suitable additional cc's
>
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
>
> The -mm tree is included into linux-next and is updated
> there every 3-4 working days
>
> ------------------------------------------------------
> From: Dave Jiang <dave.jiang@intel.com>
> Subject: mm,x86: fix SMP x86 32bit build for native_pud_clear()
>
> The fix introduced by e4decc90 to fix the UP case for 32bit x86, however
> that broke the SMP case that was working previously. Add ifdef so the dummy
> function only show up for 32bit UP case only.
>
> Fix: e4decc90 mm,x86: native_pud_clear missing on i386 build
>
> Link: http://lkml.kernel.org/r/148719066814.31111.3239231168815337012.stgit@djiang5-desk3.ch.intel.com
> Signed-off-by: Dave Jiang <dave.jiang@intel.com>
> Reported-by: Alexander Kapshuk <alexander.kapshuk@gmail.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
>
> arch/x86/include/asm/pgtable-3level.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff -puN arch/x86/include/asm/pgtable-3level.h~mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix arch/x86/include/asm/pgtable-3level.h
> --- a/arch/x86/include/asm/pgtable-3level.h~mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix
> +++ a/arch/x86/include/asm/pgtable-3level.h
> @@ -121,9 +121,11 @@ static inline void native_pmd_clear(pmd_
> *(tmp + 1) = 0;
> }
>
> +#ifndef CONFIG_SMP
> static inline void native_pud_clear(pud_t *pudp)
> {
> }
> +#endif
>
> static inline void pud_clear(pud_t *pudp)
> {
> _
>
> Patches currently in -mm which might be from dave.jiang@intel.com are
>
> mm-dax-make-pmd_fault-and-friends-to-be-the-same-as-fault.patch
> mm-dax-make-pmd_fault-and-friends-to-be-the-same-as-fault-v7.patch
> mm-dax-move-pmd_fault-to-take-only-vmf-parameter.patch
> mm-fs-reduce-fault-page_mkwrite-and-pfn_mkwrite-to-take-only-vmf.patch
> mmfsdax-change-pmd_fault-to-huge_fault.patch
> mmfsdax-change-pmd_fault-to-huge_fault-fix.patch
> mm-x86-add-support-for-pud-sized-transparent-hugepages-fix.patch
> mm-x86-add-support-for-pud-sized-transparent-hugepages-fix-fix.patch
> dax-support-for-transparent-pud-pages-for-device-dax.patch
> mm-replace-fault_flag_size-with-parameter-to-huge_fault.patch
>
Just a quick note to let you know that the patch included in next-20170221 did fix the problem I reported.
Thanks once again.
parent reply other threads:[~2017-02-21 17:12 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <58a4bc95.wUw3v0FZUCnAXeFL%akpm@linux-foundation.org>]
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=20170221171206.GA4397@box1 \
--to=alexander.kapshuk@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=dave.jiang@intel.com \
--cc=linux-next@vger.kernel.org \
--cc=mm-commits@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox