This version of the patch addresses the comments from the following review:
https://lore.kernel.org/all/[email protected]/ Previous version: - v1: https://lore.kernel.org/all/[email protected]/ The new version: - fixes the "braod" typo in the comment. - removes the "probe_size" variable in favour of "elems" that is already used for the same purpose. - removes the duplication of the code calculating the page split and adds a separate "addr_i" variable for probing the memory without polluting "addr" used to perform the actual load. - multiplies "elems" by "msize" when calling for the memory probing function so that we pass the number of bytes rather then the number of vector elements as that is what the probe function expects. We also change the heading from RFC to PATCH. I also take the opportunity to thanks Daniel Barboza for the review. Cc: Richard Handerson <[email protected]> Cc: Palmer Dabbelt <[email protected]> Cc: Alistair Francis <[email protected]> Cc: Bin Meng <[email protected]> Cc: Weiwei Li <[email protected]> Cc: Daniel Henrique Barboza <[email protected]> Cc: Liu Zhiwei <[email protected]> Cc: Helene Chelin <[email protected]> Cc: Nathan Egge <[email protected]> Cc: Max Chou <[email protected]> Cc: Jeremy Bennett <[email protected]> Cc: Craig Blackmore <[email protected]> Paolo Savini (1): target/riscv: optimize the memory probing for vector fault-only-first loads. target/riscv/vector_helper.c | 91 ++++++++++++++++++++++-------------- 1 file changed, 56 insertions(+), 35 deletions(-) -- 2.34.1
