mbox series

[v3,0/3] Incremental mode: remove safety verification

Message ID 20221227055044.18168-1-kinga.tanska@intel.com (mailing list archive)
Headers show
Series Incremental mode: remove safety verification | expand

Message

Kinga Tanska Dec. 27, 2022, 5:50 a.m. UTC 
Changes in incremental mode. Removing verification
if remove is safe, when this mode is triggered. Also
moving commit description to obey kernel coding style.

Kinga Tanska (3):
  Manage: do not check array state when drive is removed
  incremental, manage: do not verify if remove is safe
  manage: move comment with function description

 Incremental.c |  2 +-
 Manage.c      | 82 ++++++++++++++++++++++++++++++---------------------
 2 files changed, 50 insertions(+), 34 deletions(-)

Comments

Xiao Ni Dec. 28, 2022, 8:12 a.m. UTC | #1 
On Tue, Dec 27, 2022 at 8:49 PM Kinga Tanska <kinga.tanska@intel.com> wrote:
>
> Changes in incremental mode. Removing verification
> if remove is safe, when this mode is triggered. Also
> moving commit description to obey kernel coding style.
>
> Kinga Tanska (3):
>   Manage: do not check array state when drive is removed
>   incremental, manage: do not verify if remove is safe
>   manage: move comment with function description
>
>  Incremental.c |  2 +-
>  Manage.c      | 82 ++++++++++++++++++++++++++++++---------------------
>  2 files changed, 50 insertions(+), 34 deletions(-)
>
> --
> 2.26.2
>

Acked-by: Xiao Ni <xni@redhat.com>
Jes Sorensen Jan. 5, 2023, 3:38 a.m. UTC | #2 
On 12/27/22 00:50, Kinga Tanska wrote:
> Changes in incremental mode. Removing verification
> if remove is safe, when this mode is triggered. Also
> moving commit description to obey kernel coding style.
> 
> Kinga Tanska (3):
>   Manage: do not check array state when drive is removed
>   incremental, manage: do not verify if remove is safe
>   manage: move comment with function description
> 
>  Incremental.c |  2 +-
>  Manage.c      | 82 ++++++++++++++++++++++++++++++---------------------
>  2 files changed, 50 insertions(+), 34 deletions(-)
> 

Applied 1+2, 3 fails to apply for me.

Thanks,
Jes