From: Akira Yokosawa <akiyks@gmail.com>
To: "Paul E. McKenney" <paulmck@linux.ibm.com>
Cc: perfbook@vger.kernel.org, Akira Yokosawa <akiyks@gmail.com>
Subject: [PATCH 0/6] Simplify hash_resize.c
Date: Mon, 14 Jan 2019 08:28:27 +0900 [thread overview]
Message-ID: <dff55e7c-6e93-8f4a-79fd-f400ed3b38f8@gmail.com> (raw)
From 7b69a9b37ba9a73a50aad5cbb097235ddfe75870 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Mon, 14 Jan 2019 07:25:14 +0900
Subject: [PATCH 0/6] Simplify hash_resize.c
Hi Paul,
This patch set updates hash_resize.c, which you suggested for me to
take over, and the related text.
I added a couple of Quick Quizzes as well, and in one of them,
I included your version of hash_resize.c.
Patch #1 updates hash_resize.c in the way I suggested. Note that
in this update, "#ifndef FCV_SNIPPET" blocks are used to hide
code for debugging (hash value checks) in code snippets.
This code can actually be compiled with "-DFCV_SNIPPET" to see
the performance without hash value checks.
Patch #2 adds a couple of Quick Quizzes. Your version of hash_resize.c
is added as hash_resize_s.c.
Patch #3 removes unnecessary folding in a code snippet.
Patch #4 adjusts a few sentence to the simpler approach.
Patch #5 adds another Quick Quiz.
Patch #6 adds an "#error" directive for the lack of rcu_barrier().
All of the updates in text would need your native eyes to be polished.
Thanks, Akira
PS:
I couldn't make out your concern of "if we ever want to iterate over
the hash table". Can you elaborate it?
--
Akira Yokosawa (6):
datastruct/hash: Simplify hash_resize.c
datastruct/hash: Add a couple of Quick Quizzes regarding hash_resize.c
datastruct/hash: Fix unnecessary folding in code snippet
datastruct/hash: Adjust context to updated code in hash_resize.c
datastruct/hash: Add Quick Quiz on READ_ONCE/WRITE_ONCE in
hash_resize.c
datastruct/hash: Display error msg if rcu_barrier() is not available
CodeSamples/datastruct/hash/Makefile | 5 +-
CodeSamples/datastruct/hash/hash_resize.c | 62 +++--
CodeSamples/datastruct/hash/hash_resize_s.c | 365 ++++++++++++++++++++++++++++
CodeSamples/datastruct/hash/hashtorture.h | 11 +-
datastruct/datastruct.tex | 146 +++++++----
5 files changed, 513 insertions(+), 76 deletions(-)
create mode 100644 CodeSamples/datastruct/hash/hash_resize_s.c
--
2.7.4
next reply other threads:[~2019-01-13 23:28 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-13 23:28 Akira Yokosawa [this message]
2019-01-13 23:31 ` [PATCH 1/6] datastruct/hash: Simplify hash_resize.c Akira Yokosawa
2019-01-14 10:31 ` Junchang Wang
2019-01-14 13:22 ` Akira Yokosawa
2019-01-14 14:16 ` Junchang Wang
2019-01-13 23:32 ` [PATCH 2/6] datastruct/hash: Add a couple of Quick Quizzes regarding hash_resize.c Akira Yokosawa
2019-01-13 23:33 ` [PATCH 3/6] datastruct/hash: Fix unnecessary folding in code snippet Akira Yokosawa
2019-01-13 23:35 ` [PATCH 4/6] datastruct/hash: Adjust context to updated code in hash_resize.c Akira Yokosawa
2019-01-13 23:36 ` [PATCH 5/6] datastruct/hash: Add Quick Quiz on READ_ONCE/WRITE_ONCE " Akira Yokosawa
2019-01-13 23:38 ` [PATCH 6/6] datastruct/hash: Display error msg if rcu_barrier() is not available Akira Yokosawa
2019-01-14 2:10 ` [PATCH 0/6] Simplify hash_resize.c Paul E. McKenney
2019-01-15 1:33 ` Paul E. McKenney
2019-01-15 15:32 ` Akira Yokosawa
2019-01-15 17:43 ` Paul E. McKenney
2019-01-15 22:15 ` Akira Yokosawa
2019-01-16 0:06 ` Paul E. McKenney
2019-01-15 22:29 ` Akira Yokosawa
2019-01-16 0:07 ` Paul E. McKenney
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=dff55e7c-6e93-8f4a-79fd-f400ed3b38f8@gmail.com \
--to=akiyks@gmail.com \
--cc=paulmck@linux.ibm.com \
--cc=perfbook@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.