All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rebecca Mckeever <remckee0@gmail.com>
To: outreachy@lists.linux.dev
Cc: Mike Rapoport <rppt@kernel.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	David Hildenbrand <david@redhat.com>,
	Rebecca Mckeever <remckee0@gmail.com>
Subject: [PATCH v4 0/5] memblock tests: update style of comments
Date: Sat, 30 Apr 2022 17:49:35 -0500	[thread overview]
Message-ID: <cover.1651357434.git.remckee0@gmail.com> (raw)

These patches update the style of comments for memblock_add_*(),
memblock_reserve_*(), memblock_remove_*(), and memblock_free_*()
functions to match the style of comments in tests/alloc_*.c and
remove extra spaces in those comments to conform to Linux kernel
coding style. This completes an item from the TODO list.

Rebecca Mckeever (5):
  memblock tests: update style of comments for memblock_add_*()
    functions
  memblock tests: update style of comments for memblock_reserve_*()
    functions
  memblock tests: update style of comments for memblock_remove_*()
    functions
  memblock tests: update style of comments for memblock_free_*()
    functions
  memblock tests: remove completed TODO item

 tools/testing/memblock/TODO              |   3 -
 tools/testing/memblock/tests/basic_api.c | 392 ++++++++++++++++-------
 2 files changed, 278 insertions(+), 117 deletions(-)

---
Changes

v1 -> v2
PATCH 1: 
  Add a comma
  Move separator outside block name
Squash white space edit into PATCH 2
PATCH 1 and PATCH 2: add a colon before memory block visuals
Add PATCH 3: update style of comments for memblock_remove_*() functions
Add PATCH 4: update style of comments for memblock_free_*() functions

v2 -> v3
Add PATCH 5: remove completed TODO item
Add changes summary

v3 -> v4
PATCH 1 and PATCH 2: add Reviewed-by tags
PATCH 1 through PATCH 4:
  Correct grammar
  Replace references to first or second (block/region) with r1 or r2
  Add references to r1 or r2 as needed for clarity
---

-- 
2.32.0


             reply	other threads:[~2022-04-30 22:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-30 22:49 Rebecca Mckeever [this message]
2022-04-30 22:49 ` [PATCH v4 1/5] memblock tests: update style of comments for memblock_add_*() functions Rebecca Mckeever
2022-04-30 22:49 ` [PATCH v4 2/5] memblock tests: update style of comments for memblock_reserve_*() functions Rebecca Mckeever
2022-04-30 22:49 ` [PATCH v4 3/5] memblock tests: update style of comments for memblock_remove_*() functions Rebecca Mckeever
2022-04-30 22:49 ` [PATCH v4 4/5] memblock tests: update style of comments for memblock_free_*() functions Rebecca Mckeever
2022-04-30 22:49 ` [PATCH v4 5/5] memblock tests: remove completed TODO item Rebecca Mckeever
2022-05-08  0:41 ` [PATCH v4 0/5] memblock tests: update style of comments David Hildenbrand
2022-05-09 10:15 ` Mike Rapoport

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=cover.1651357434.git.remckee0@gmail.com \
    --to=remckee0@gmail.com \
    --cc=david@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=outreachy@lists.linux.dev \
    --cc=rppt@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.