lkml.org 
[lkml]   [2021]   [Nov]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/5] x86/Swiotlb: Add Swiotlb bounce buffer remap function for HV IVM
The subject is wrong, nothing x86-specific here.  Please use
"swiotlb: " as the prefix

> + * @vaddr: The vaddr of the swiotlb memory pool. The swiotlb
> + * memory pool may be remapped in the memory encrypted case and store

Please avoid the overly long line.

> + /*
> + * With swiotlb_unencrypted_base setting, swiotlb bounce buffer will
> + * be remapped in the swiotlb_update_mem_attributes() and return here
> + * directly.
> + */

I'd word this as:

/*
* If swiotlb_unencrypted_base is set, the bounce buffer memory will
* be remapped and cleared in swiotlb_update_mem_attributes.
*/
> + ret = swiotlb_init_io_tlb_mem(mem, __pa(tlb), nslabs, false);
> + if (ret) {
> + memblock_free(mem->slots, alloc_size);
> + return ret;
> + }

With the latest update swiotlb_init_io_tlb_mem will always return 0,
so no need for the return value change or error handling here.

\
 
 \ /
  Last update: 2021-11-17 11:00    [W:0.080 / U:0.220 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site