From: Jason Gunthorpe <jgg@ziepe•ca>
To: Sumit Semwal <sumit.semwal@linaro•org>, Jiri Pirko <jiri@resnulli•us>
Cc: Maxime Ripard <mripard@kernel•org>, Jiri Pirko <jiri@resnulli•us>,
Christoph Hellwig <hch@infradead•org>,
"T.J. Mercier" <tjmercier@google•com>,
maddy@linux•ibm.com, mpe@ellerman•id.au, npiggin@gmail•com,
chleroy@kernel•org, linuxppc-dev@lists•ozlabs.org, lkp@intel•com,
linux-kernel@vger•kernel.org, iommu@lists•linux.dev,
linux-mm@kvack•org, agordeev@linux•ibm.com,
gerald.schaefer@linux•ibm.com, linux-s390@vger•kernel.org,
Dan Williams <djbw@kernel•org>,
Tom Lendacky <thomas.lendacky@amd•com>,
x86@kernel•org, Arnd Bergmann <arnd@linaro•org>
Subject: Re: [PATCH] powerpc: Export set_memory_encrypted and set_memory_decrypted
Date: Mon, 8 Jun 2026 12:27:43 -0300 [thread overview]
Message-ID: <20260608152743.GD2764304@ziepe.ca> (raw)
In-Reply-To: <CAO_48GH3NP09U6TdB5drbKY0TpwvtBXwrf=Jajsr5ttNbC_u9g@mail.gmail.com>
On Mon, Jun 08, 2026 at 08:47:15PM +0530, Sumit Semwal wrote:
> Hi Jason,
>
> On Thu, 4 Jun 2026 at 19:27, Jason Gunthorpe <jgg@ziepe•ca> wrote:
> >
> > On Thu, Jun 04, 2026 at 12:51:49PM +0530, Sumit Semwal wrote:
> >
> > > Given that Christoph's objection is not really about the modules part,
> > > but that the set_memory_{encrypted,decrypted} should not be used here,
> > > one option is to revert 78b30c50a7ac until that issue is sorted out?
> >
> > Please no, we have stuff already using this so it would be a
> > functional regression. Revert making heaps into a module since that
> > doesn't have a functional regression.
>
> Thanks for your comments.
>
> To me, it looks like while system and system_cc_shared heaps share a
> lot of code, their user bases have different needs. It's apparent that
> system_cc_heap users don't care about it being a module while system
> heap users would very much like so.
>
> I also discussed this with Arnd, and he suggested we could rearrange
> the code so that system_heap_cc_shared_priv depends on a new Kconfig
> symbol like
>
> config DMABUF_HEAPS_CC_SYSTEM
> bool "DMA-BUF System Heap for memory encryption"
> depends on ARCH_HAS_MEM_ENCRYPT && DMABUF_HEAPS_SYSTEM=y
>
> This allows building both into the kernel or leave encryption choice
> up to the consumers of the system heap.
>
> If this is agreeable to everyone, I can post Arnd's patch.
Yeah, that's fine for me for now
Jason
next prev parent reply other threads:[~2026-06-08 15:27 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-22 22:58 [PATCH] powerpc: Export set_memory_encrypted and set_memory_decrypted T.J. Mercier
2026-05-25 6:21 ` Christoph Hellwig
2026-05-27 16:07 ` Jason Gunthorpe
2026-05-27 18:15 ` Borislav Petkov
2026-05-28 8:53 ` Christoph Hellwig
2026-06-04 5:34 ` Maxime Ripard
2026-06-04 7:21 ` Sumit Semwal
2026-06-04 7:32 ` Sumit Semwal
2026-06-04 13:57 ` Jason Gunthorpe
2026-06-08 15:17 ` Sumit Semwal
2026-06-08 15:27 ` Jason Gunthorpe [this message]
2026-06-08 16:04 ` T.J. Mercier
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=20260608152743.GD2764304@ziepe.ca \
--to=jgg@ziepe$(echo .)ca \
--cc=agordeev@linux$(echo .)ibm.com \
--cc=arnd@linaro$(echo .)org \
--cc=chleroy@kernel$(echo .)org \
--cc=djbw@kernel$(echo .)org \
--cc=gerald.schaefer@linux$(echo .)ibm.com \
--cc=hch@infradead$(echo .)org \
--cc=iommu@lists$(echo .)linux.dev \
--cc=jiri@resnulli$(echo .)us \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-mm@kvack$(echo .)org \
--cc=linux-s390@vger$(echo .)kernel.org \
--cc=linuxppc-dev@lists$(echo .)ozlabs.org \
--cc=lkp@intel$(echo .)com \
--cc=maddy@linux$(echo .)ibm.com \
--cc=mpe@ellerman$(echo .)id.au \
--cc=mripard@kernel$(echo .)org \
--cc=npiggin@gmail$(echo .)com \
--cc=sumit.semwal@linaro$(echo .)org \
--cc=thomas.lendacky@amd$(echo .)com \
--cc=tjmercier@google$(echo .)com \
--cc=x86@kernel$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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