The commit is pushed to "work" and will appear at 
https://src.openvz.org/scm/ovz/vzkernel.git
after rh8-4.18.0-193.6.3.vz8.4.16
------>
commit b50d2e2567fbd9f8c8c565fced0e636f2aff73c1
Author: Dmitry Monakhov <dmonak...@openvz.org>
Date:   Fri Nov 6 17:15:49 2020 +0300

    ext4: send abort uevent on ext4 journal abort
    
    Currenlty error from device result in ext4_abort, but uevent not generated 
because
    ext4_abort() caller's context do not allow GFP_KERNEL memory allocation.
    Let's relax submission context requirement and deffer actual uevent 
submission
    to work_queue.  It can be any workqueue I've pick rsv_conversion_wq because 
it is
    already exists.
    
    khorenko@: "system_wq" does not fit here because at the moment of
    work execution sb can be already destroyed.
    "EXT4_SB(sb)->rsv_conversion_wq" is flushed before sb is destroyed.
    
    Signed-off-by: Dmitry Monakhov <dmonak...@openvz.org>
    
    [aryabinin rh8 rebase]
    Signed-off-by: Andrey Ryabinin <aryabi...@virtuozzo.com>
---
 fs/ext4/ext4.h  |  2 ++
 fs/ext4/super.c | 10 ++++++++++
 2 files changed, 12 insertions(+)

diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
index 228492c9518f..bbdd7efc8447 100644
--- a/fs/ext4/ext4.h
+++ b/fs/ext4/ext4.h
@@ -1499,6 +1499,7 @@ struct ext4_sb_info {
        __u32 s_csum_seed;
 
        bool s_err_event_sent;
+       bool s_abrt_event_sent;
 
        /* Reclaim extents from extent status tree */
        struct shrinker s_es_shrinker;
@@ -3126,6 +3127,7 @@ enum ext4_event_type {
      EXT4_UA_UMOUNT,
      EXT4_UA_REMOUNT,
      EXT4_UA_ERROR,
+     EXT4_UA_ABORT,
      EXT4_UA_FREEZE,
      EXT4_UA_UNFREEZE,
 };
diff --git a/fs/ext4/super.c b/fs/ext4/super.c
index 3cc979825ec8..00619f45b1c3 100644
--- a/fs/ext4/super.c
+++ b/fs/ext4/super.c
@@ -420,6 +420,9 @@ static void ext4_send_uevent_work(struct work_struct *w)
        case EXT4_UA_ERROR:
                ret = add_uevent_var(env, "FS_ACTION=%s", "ERROR");
                break;
+       case EXT4_UA_ABORT:
+               ret = add_uevent_var(env, "FS_ACTION=%s", "ABORT");
+               break;
        case EXT4_UA_FREEZE:
                ret = add_uevent_var(env, "FS_ACTION=%s", "FREEZE");
                break;
@@ -576,6 +579,9 @@ static void ext4_handle_error(struct super_block *sb)
        if (!test_opt(sb, ERRORS_CONT)) {
                journal_t *journal = EXT4_SB(sb)->s_journal;
 
+               if (!xchg(&EXT4_SB(sb)->s_abrt_event_sent, 1))
+                       ext4_send_uevent(sb, EXT4_UA_ABORT);
+
                EXT4_SB(sb)->s_mount_flags |= EXT4_MF_FS_ABORTED;
                if (journal)
                        jbd2_journal_abort(journal, -EIO);
@@ -801,6 +807,10 @@ void __ext4_abort(struct super_block *sb, const char 
*function,
 
        if (sb_rdonly(sb) == 0) {
                ext4_msg(sb, KERN_CRIT, "Remounting filesystem read-only");
+
+               if (!xchg(&EXT4_SB(sb)->s_abrt_event_sent, 1))
+                       ext4_send_uevent(sb, EXT4_UA_ABORT);
+
                EXT4_SB(sb)->s_mount_flags |= EXT4_MF_FS_ABORTED;
                /*
                 * Make sure updated value of ->s_mount_flags will be visible
_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to