| Message ID | 20190718032858.28744-1-bauerman@linux.ibm.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Remove x86-specific code from generic headers | expand |
On 7/17/19 10:28 PM, Thiago Jung Bauermann wrote: > Hello, > > This version is mostly about splitting up patch 2/3 into three separate > patches, as suggested by Christoph Hellwig. Two other changes are a fix in > patch 1 which wasn't selecting ARCH_HAS_MEM_ENCRYPT for s390 spotted by > Janani and removal of sme_active and sev_active symbol exports as suggested > by Christoph Hellwig. > > These patches are applied on top of today's dma-mapping/for-next. > > I don't have a way to test SME, SEV, nor s390's PEF so the patches have only > been build tested. I'll try and get this tested quickly to be sure everything works for SME and SEV. Thanks, Tom > > Changelog > > Since v2: > > - Patch "x86,s390: Move ARCH_HAS_MEM_ENCRYPT definition to arch/Kconfig" > - Added "select ARCH_HAS_MEM_ENCRYPT" to config S390. Suggested by Janani. > > - Patch "DMA mapping: Move SME handling to x86-specific files" > - Split up into 3 new patches. Suggested by Christoph Hellwig. > > - Patch "swiotlb: Remove call to sme_active()" > - New patch. > > - Patch "dma-mapping: Remove dma_check_mask()" > - New patch. > > - Patch "x86,s390/mm: Move sme_active() and sme_me_mask to x86-specific header" > - New patch. > - Removed export of sme_active symbol. Suggested by Christoph Hellwig. > > - Patch "fs/core/vmcore: Move sev_active() reference to x86 arch code" > - Removed export of sev_active symbol. Suggested by Christoph Hellwig. > > - Patch "s390/mm: Remove sev_active() function" > - New patch. > > Since v1: > > - Patch "x86,s390: Move ARCH_HAS_MEM_ENCRYPT definition to arch/Kconfig" > - Remove definition of ARCH_HAS_MEM_ENCRYPT from s390/Kconfig as well. > - Reworded patch title and message a little bit. > > - Patch "DMA mapping: Move SME handling to x86-specific files" > - Adapt s390's <asm/mem_encrypt.h> as well. > - Remove dma_check_mask() from kernel/dma/mapping.c. Suggested by > Christoph Hellwig. > > Thiago Jung Bauermann (6): > x86,s390: Move ARCH_HAS_MEM_ENCRYPT definition to arch/Kconfig > swiotlb: Remove call to sme_active() > dma-mapping: Remove dma_check_mask() > x86,s390/mm: Move sme_active() and sme_me_mask to x86-specific header > fs/core/vmcore: Move sev_active() reference to x86 arch code > s390/mm: Remove sev_active() function > > arch/Kconfig | 3 +++ > arch/s390/Kconfig | 4 +--- > arch/s390/include/asm/mem_encrypt.h | 5 +---- > arch/s390/mm/init.c | 8 +------- > arch/x86/Kconfig | 4 +--- > arch/x86/include/asm/mem_encrypt.h | 10 ++++++++++ > arch/x86/kernel/crash_dump_64.c | 5 +++++ > arch/x86/mm/mem_encrypt.c | 2 -- > fs/proc/vmcore.c | 8 ++++---- > include/linux/crash_dump.h | 14 ++++++++++++++ > include/linux/mem_encrypt.h | 15 +-------------- > kernel/dma/mapping.c | 8 -------- > kernel/dma/swiotlb.c | 3 +-- > 13 files changed, 42 insertions(+), 47 deletions(-) >
Lendacky, Thomas <Thomas.Lendacky@amd.com> writes: > On 7/17/19 10:28 PM, Thiago Jung Bauermann wrote: >> Hello, >> >> This version is mostly about splitting up patch 2/3 into three separate >> patches, as suggested by Christoph Hellwig. Two other changes are a fix in >> patch 1 which wasn't selecting ARCH_HAS_MEM_ENCRYPT for s390 spotted by >> Janani and removal of sme_active and sev_active symbol exports as suggested >> by Christoph Hellwig. >> >> These patches are applied on top of today's dma-mapping/for-next. >> >> I don't have a way to test SME, SEV, nor s390's PEF so the patches have only >> been build tested. > > I'll try and get this tested quickly to be sure everything works for SME > and SEV. Thanks! And thanks for reviewing the patches.
On 7/18/19 2:44 PM, Thiago Jung Bauermann wrote: > > Lendacky, Thomas <Thomas.Lendacky@amd.com> writes: > >> On 7/17/19 10:28 PM, Thiago Jung Bauermann wrote: >>> Hello, >>> >>> This version is mostly about splitting up patch 2/3 into three separate >>> patches, as suggested by Christoph Hellwig. Two other changes are a fix in >>> patch 1 which wasn't selecting ARCH_HAS_MEM_ENCRYPT for s390 spotted by >>> Janani and removal of sme_active and sev_active symbol exports as suggested >>> by Christoph Hellwig. >>> >>> These patches are applied on top of today's dma-mapping/for-next. >>> >>> I don't have a way to test SME, SEV, nor s390's PEF so the patches have only >>> been build tested. >> >> I'll try and get this tested quickly to be sure everything works for SME >> and SEV. Built and tested both SME and SEV and everything appears to be working well (not extensive testing, but should be good enough). Thanks, Tom > > Thanks! And thanks for reviewing the patches. >
Lendacky, Thomas <Thomas.Lendacky@amd.com> writes: > On 7/18/19 2:44 PM, Thiago Jung Bauermann wrote: >> >> Lendacky, Thomas <Thomas.Lendacky@amd.com> writes: >> >>> On 7/17/19 10:28 PM, Thiago Jung Bauermann wrote: >>>> Hello, >>>> >>>> This version is mostly about splitting up patch 2/3 into three separate >>>> patches, as suggested by Christoph Hellwig. Two other changes are a fix in >>>> patch 1 which wasn't selecting ARCH_HAS_MEM_ENCRYPT for s390 spotted by >>>> Janani and removal of sme_active and sev_active symbol exports as suggested >>>> by Christoph Hellwig. >>>> >>>> These patches are applied on top of today's dma-mapping/for-next. >>>> >>>> I don't have a way to test SME, SEV, nor s390's PEF so the patches have only >>>> been build tested. >>> >>> I'll try and get this tested quickly to be sure everything works for SME >>> and SEV. > > Built and tested both SME and SEV and everything appears to be working > well (not extensive testing, but should be good enough). Great news. Thanks for testing!
Hello, This version is mostly about splitting up patch 2/3 into three separate patches, as suggested by Christoph Hellwig. Two other changes are a fix in patch 1 which wasn't selecting ARCH_HAS_MEM_ENCRYPT for s390 spotted by Janani and removal of sme_active and sev_active symbol exports as suggested by Christoph Hellwig. These patches are applied on top of today's dma-mapping/for-next. I don't have a way to test SME, SEV, nor s390's PEF so the patches have only been build tested. Changelog Since v2: - Patch "x86,s390: Move ARCH_HAS_MEM_ENCRYPT definition to arch/Kconfig" - Added "select ARCH_HAS_MEM_ENCRYPT" to config S390. Suggested by Janani. - Patch "DMA mapping: Move SME handling to x86-specific files" - Split up into 3 new patches. Suggested by Christoph Hellwig. - Patch "swiotlb: Remove call to sme_active()" - New patch. - Patch "dma-mapping: Remove dma_check_mask()" - New patch. - Patch "x86,s390/mm: Move sme_active() and sme_me_mask to x86-specific header" - New patch. - Removed export of sme_active symbol. Suggested by Christoph Hellwig. - Patch "fs/core/vmcore: Move sev_active() reference to x86 arch code" - Removed export of sev_active symbol. Suggested by Christoph Hellwig. - Patch "s390/mm: Remove sev_active() function" - New patch. Since v1: - Patch "x86,s390: Move ARCH_HAS_MEM_ENCRYPT definition to arch/Kconfig" - Remove definition of ARCH_HAS_MEM_ENCRYPT from s390/Kconfig as well. - Reworded patch title and message a little bit. - Patch "DMA mapping: Move SME handling to x86-specific files" - Adapt s390's <asm/mem_encrypt.h> as well. - Remove dma_check_mask() from kernel/dma/mapping.c. Suggested by Christoph Hellwig. Thiago Jung Bauermann (6): x86,s390: Move ARCH_HAS_MEM_ENCRYPT definition to arch/Kconfig swiotlb: Remove call to sme_active() dma-mapping: Remove dma_check_mask() x86,s390/mm: Move sme_active() and sme_me_mask to x86-specific header fs/core/vmcore: Move sev_active() reference to x86 arch code s390/mm: Remove sev_active() function arch/Kconfig | 3 +++ arch/s390/Kconfig | 4 +--- arch/s390/include/asm/mem_encrypt.h | 5 +---- arch/s390/mm/init.c | 8 +------- arch/x86/Kconfig | 4 +--- arch/x86/include/asm/mem_encrypt.h | 10 ++++++++++ arch/x86/kernel/crash_dump_64.c | 5 +++++ arch/x86/mm/mem_encrypt.c | 2 -- fs/proc/vmcore.c | 8 ++++---- include/linux/crash_dump.h | 14 ++++++++++++++ include/linux/mem_encrypt.h | 15 +-------------- kernel/dma/mapping.c | 8 -------- kernel/dma/swiotlb.c | 3 +-- 13 files changed, 42 insertions(+), 47 deletions(-)