Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20200526131259.GA97001@jade> <20200821184910.GB3407@lx2k>
 <f7c8bb6e-e732-2565-8b50-20f5504701c9@nvidia.com> <CAOesGMjKq3ECJuaMANq=AyDMLXYvorafMYS3LDr9YMH6M9VdYA@mail.gmail.com>
 <CAHUa44EGf1yayXhHUJLUmbNpueKMZ9jArT4narJLzOVWujHR9A@mail.gmail.com> <d757854b-577c-b8f2-8b67-526385101d68@nvidia.com>
In-Reply-To: <d757854b-577c-b8f2-8b67-526385101d68@nvidia.com>
From:   Jens Wiklander <jens.wiklander@linaro.org>
Date:   Mon, 24 Aug 2020 09:46:21 +0200
Message-ID: <CAHUa44FrxidzSUOM_JchOTa5pF6P+j8uZJA5DpKfGLWaS6tCcw@mail.gmail.com>
Subject: Re: [GIT PULL] tee subsystem pin_user_pages for v5.8
To:     John Hubbard <jhubbard@nvidia.com>
Cc:     Olof Johansson <olof@lixom.net>,
        ARM-SoC Maintainers <arm@kernel.org>,
        SoC Team <soc@kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        "tee-dev @ lists . linaro . org" <tee-dev@lists.linaro.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, Aug 24, 2020 at 9:18 AM John Hubbard <jhubbard@nvidia.com> wrote:
>
> On 8/23/20 11:51 PM, Jens Wiklander wrote:
> > On Fri, Aug 21, 2020 at 11:19 PM Olof Johansson <olof@lixom.net> wrote:
> >>
> >> On Fri, Aug 21, 2020 at 12:58 PM John Hubbard <jhubbard@nvidia.com> wrote:
> >>>
> >>> On 8/21/20 11:49 AM, Olof Johansson wrote:
> >>>> On Tue, May 26, 2020 at 03:12:59PM +0200, Jens Wiklander wrote:
> >>>>> Hello arm-soc maintainers,
> >>>>>
> >>>>> Please pull this small patch converting the tee subsystem to use
> >>>>> pin_user_pages() instead of get_user_pages().
> >>>>>
> >>>>> Thanks,
> >>>>> Jens
> >>>>>
> >>>>> The following changes since commit ae83d0b416db002fe95601e7f97f64b59514d936:
> >>>>>
> >>>>>     Linux 5.7-rc2 (2020-04-19 14:35:30 -0700)
> >>>>>
> >>>>> are available in the Git repository at:
> >>>>>
> >>>>>     git://git.linaro.org:/people/jens.wiklander/linux-tee.git tags/tee-pin-user-pages-for-5.8
> >>>>>
> >>>>> for you to fetch changes up to 37f6b4d5f47b600ec4ab6682c005a44a1bfca530:
> >>>>>
> >>>>>     tee: convert get_user_pages() --> pin_user_pages() (2020-05-26 10:42:41 +0200)
> >>>>
> >>>> Hi, I noticed this never got merged, but I don't see any follow-up here that
> >>>> retracts it. Is it still pending merge such that I should queue it for v5.10?
> >>>>
> >>>
> >>> I think so. I had marked it in my notes as "accepted, and the maintainer will
> >>> eventually merge it", and I left it at that. It's still desirable.
> >>>
> >>
> >> Looks like it conflicts with some of the later work. Jens, given the
> >> timelines here it's probably easiest all around if you rebase/respin
> >> and send a fresh pull request. I could fix it up but you'd still need
> >> to review that so the amount of work is probably less if you do it
> >> directly.
> >
> > Agree, I'll send a fresh pull request once we have this rebased.
> > The conflict is with the recently added call to get_kernel_pages()
> > when kernel memory is shared.
> > The conflict isn't trivial, I guess we need to handle the different
> > types of pages differently when releasing them.
> > John, would you mind rebasing and posting the patch again?
> >
>
> Sure. Should it be against 5.9-rc2, or something else? I can do this in the morning,
> about 10 hrs from now.

5.9-rc2 sounds good.

Thanks,
Jens