Re: [PATCH -perfbook 0/4] cpu/overheads: Make tables more consistent

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

 



On Tue, Feb 14, 2023 at 07:03:12PM +0900, Akira Yokosawa wrote:
> Hi,
> 
> This patch set is a follow-up to commit 34cc066b1d95 ("cpu: Add a QQz
> on table E.1").
> 
> Leo posted a patch to improve Table E.1 at [1].
> 
> I have gone further to improve consistency among tables on overheads
> of atomic operations.
> 
> Patch 1/4 changes the layout of those tables and adjusts upper-/
> lower-casing (with Reported-by: from Leo).
> 
> I found the use of "In-Core" and "Off-Core" in the tables somewhat
> strange. Usually, I find pairs of "On/Off" and "In/Out" more natural.
> In overheads.tex, "On-Core" vs "Off-Core" looks better to me.
> 
> So Patch 2/4 replaces "in-core" with "on-core", and does similar
> replacements.
> 
> Patch 3/4 adds a page reference in QQz 3.8.
> 
> Patch 4/4 is an independent change fixing an issue in -nq builds.
> I noticed an extra "," in the anchor box containing QQz 4.20, 4.21,
> and 4.22 while skimming through QQz's in those builds.
> 
> [1]: https://www.spinics.net/lists/perfbook/msg03827.html

Queued and pushed, thank you!

							Thanx, Paul

>         Thanks, Akira
> --
> Akira Yokosawa (4):
>   cpu: Improve layout and consistency of Tables 3.1, E.1, and E.2
>   cpu: Use 'on-core' rather than 'in-core'
>   cpu: Add page reference to Table E.1 in QQz 3.8
>   toolsoftrade: Fix QQz macro in QQz series
> 
>  cpu/overheads.tex             | 161 +++++++++++++++++++++-------------
>  perfbook-lt.tex               |   6 ++
>  toolsoftrade/toolsoftrade.tex |   2 +-
>  3 files changed, 108 insertions(+), 61 deletions(-)
> 
> 
> base-commit: 50bb4adf51f683ba2c7d90f269632b6f4b4d4704
> -- 
> 2.25.1
> 



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux