[v2,0/3] Fix checkpointing - minors

Message

Mateusz Kusiak Jan. 18, 2024, 10:28 a.m. UTC

This is the first half of splitted patchset "Fix checkpointing" as
asked. It contains minor changes that should be safe to merge prior to
release.

Fixed minor things in "Replace "none" with macro":
- Replaced hardcoded "null" size to "sizeof(STR_COMMON_NONE) - 1"
  in str_is_none().
- Removed is_none() on optarg as it checked only first four chars,
  bad for user input.

I included "Add understanding output section in man" in second patchset
to preserve history (adding it here, prior to release might seem
random).

Mateusz Kusiak (3):
  Define sysfs max buffer size
  Replace "none" with macro
  super-intel: Remove inaccessible code

 Assemble.c    |  3 +--
 Build.c       |  4 ++--
 Create.c      |  2 +-
 Grow.c        | 51 +++++++++++++++++++++++++--------------------------
 Incremental.c |  4 ++--
 Manage.c      | 10 +++++-----
 Monitor.c     |  6 +++---
 config.c      |  2 +-
 managemon.c   |  6 +++---
 maps.c        |  4 ++--
 mdadm.c       |  7 +++----
 mdadm.h       | 18 ++++++++++++++++++
 monitor.c     | 24 ++++++++++++------------
 msg.c         |  4 ++--
 super-intel.c | 35 +++++++++--------------------------
 sysfs.c       | 12 ++++++------
 util.c        |  2 +-
 17 files changed, 96 insertions(+), 98 deletions(-)

Comments

Mariusz Tkaczyk Jan. 24, 2024, 3:19 p.m. UTC | #1

On Thu, 18 Jan 2024 11:28:39 +0100
Mateusz Kusiak <[e-mail]> wrote:

> This is the first half of splitted patchset "Fix checkpointing" as
> asked. It contains minor changes that should be safe to merge prior to
> release.
> 
> Fixed minor things in "Replace "none" with macro":
> - Replaced hardcoded "null" size to "sizeof(STR_COMMON_NONE) - 1"
>   in str_is_none().
> - Removed is_none() on optarg as it checked only first four chars,
>   bad for user input.
> 
> I included "Add understanding output section in man" in second patchset
> to preserve history (adding it here, prior to release might seem
> random).
> 
> Mateusz Kusiak (3):
>   Define sysfs max buffer size
>   Replace "none" with macro
>   super-intel: Remove inaccessible code
> 

All applied! 

Thanks,
Mariusz