Re: [PATCH 1/5] Extend memblock to support memory encryption

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/5/21 2:27 PM, Martin Fernandez wrote:
> Add the capability to mark regions of the memory memory_type able of
> hardware memory encryption.
> 
> Also add the capability to query if all regions of a memory node are
> able to do hardware memory encryption.

This is looking quite a bit nicer than the last post.  One nit on the
changelogs: please make an effort to connect the patches.  If you
introduce a function here but don't use it, tell us where it _will_ be used.

As it stands, this series _looks_ like it is touching 3 disjoint things:
 * pg_data_t
 * e820
 * memblocks

It would be valuable to spend a few sentences to connect the dots.

Subsystem prefixes are also highly preferred.  Perhaps:

	mm: Extend memblock to support memory encryption

or

	mm/memblock: Tag memblocks with crypto capabilities

It also helps make it obvious that this series touches both core mm/ and
arch/x86/ code.



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux