1. 23 Apr, 2021 3 commits
  2. 22 Apr, 2021 1 commit
  3. 21 Apr, 2021 7 commits
  4. 02 Apr, 2021 3 commits
    • Mark Brown's avatar
      Merge series "Support ROHM BD71815 PMIC" from Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>: · aaa8c4e0
      Mark Brown authored
      Patch series introducing support for ROHM BD71815 PMIC
      
      ROHM BD71815 is a power management IC used in some battery powered
      systems. It contains regulators, GPO(s), charger + coulomb counter, RTC
      and a clock gate.
      
      All regulators can be controlled via I2C. LDO4 can additionally be set to
      be enabled/disabled by a GPIO. LDO3 voltage could be selected from two
      voltages written into separate VSEL reisters using GPIO but this mode is
      not supported by driver. On top of that the PMIC has the typical HW
      state machine which is present also on many other ROHM PMICs.
      
      IC contains two GPOs - but one of the GPOs is marked as GND in
      data-sheet. Thus the driver by default only exposes one GPO. The second
      GPO can be enabled by special DT property.
      
      RTC is almost similar to what is on BD71828. For currently used features
      only the register address offset to RTC block differs.
      
      The charger driver is not included in this series. ROHM has a charger
      driver with some fuel-gauging logig written in but this is not included
      here. I am working on separating the logic from HW specific driver and
      supporting both BD71815 and BD71828 chargers in separate patch series.
      
      Changelog v5:
        Regulator:
        - Added regmap helper for regulator ramp-delay and taken it in use
          (patches 13, 14, 16 - they can be just dropped if ramp-delay helper is not
          a good idea. Patch 15 implements old-fashioned ramp-delay)
        GPIO:
        - styling changes to GPIO (Mostly suggested by Andy)
        - implemented init_valid_mask (but can't count on it yet)
      Changelog v4:
        - Sorted ROHM chip ID enum
        - Statcized DVS structures in regulator driver
        - Minor styling for regulator driver
        - rebased on v5.12-rc4
      Changelog v3:
        - GPIO clean-up as suggested by Bartosz
        - MFD clean-up as suggested by Lee
        - clk-mode dt-binding handling in MFD driver corrected to reflect new
          property values.
        - Dropped already applied patches
        - Rebased on v5.12-rc2
      Changelog v2:
        - Rebased on top of v5.11-rc3
        - Added another "preliminary patch" which fixes HW-dvs voltage
          handling (patch 1)
        - split regulator patch to two.
        - changed dt-binding patch ordering.
        regulators:
          - staticized probe
          - removed some unnecessary defines
          - updated comments
          - split rohm-regulator patch adding SNVS and supporting simple
            linear mapping into two - one adding support for mapping, other
            adding SNVS.
        GPIO:
          - removed unnecessary headers
          - clarified dev/parent->dev usage
          - removed forgotten #define DEBUG
        dt-bindings:
          - changed patch order to meet ref-dependencies
          - added missing regulator nodes
          - changed string property for clk mode to tristated
        MFD:
          - header cleanups.
        CLK:
          - fixed commit message
      
      --
      
      Matti Vaittinen (19):
        rtc: bd70528: Do not require parent data
        mfd: bd718x7: simplify by cleaning unnecessary device data
        dt_bindings: bd71828: Add clock output mode
        dt_bindings: regulator: Add ROHM BD71815 PMIC regulators
        dt_bindings: mfd: Add ROHM BD71815 PMIC
        mfd: Add ROHM BD71815 ID
        mfd: Sort ROHM chip ID list for better readability
        mfd: Support for ROHM BD71815 PMIC core
        gpio: support ROHM BD71815 GPOs
        regulator: helpers: Export helper voltage listing
        regulator: rohm-regulator: linear voltage support
        regulator: rohm-regulator: Support SNVS HW state.
        regulator: Add regmap helper for ramp-delay setting
        regulator: bd718x7, bd71828: Use ramp-delay helper
        regulator: Support ROHM BD71815 regulators
        regulator: bd71815: use ramp-delay helper
        clk: bd718x7: Add support for clk gate on ROHM BD71815 PMIC
        rtc: bd70528: Support RTC on ROHM BD71815
        MAINTAINERS: Add ROHM BD71815AGW
      
       .../bindings/mfd/rohm,bd71815-pmic.yaml       | 201 ++++++
       .../bindings/mfd/rohm,bd71828-pmic.yaml       |   6 +
       .../regulator/rohm,bd71815-regulator.yaml     | 116 ++++
       MAINTAINERS                                   |   3 +
       drivers/clk/clk-bd718x7.c                     |   9 +-
       drivers/gpio/Kconfig                          |  10 +
       drivers/gpio/Makefile                         |   1 +
       drivers/gpio/gpio-bd71815.c                   | 193 ++++++
       drivers/mfd/Kconfig                           |  15 +-
       drivers/mfd/rohm-bd71828.c                    | 486 +++++++++----
       drivers/mfd/rohm-bd718x7.c                    |  43 +-
       drivers/regulator/Kconfig                     |  11 +
       drivers/regulator/Makefile                    |   1 +
       drivers/regulator/bd71815-regulator.c         | 651 ++++++++++++++++++
       drivers/regulator/bd71828-regulator.c         |  51 +-
       drivers/regulator/bd718x7-regulator.c         |  60 +-
       drivers/regulator/helpers.c                   | 101 ++-
       drivers/regulator/rohm-regulator.c            |  23 +-
       drivers/rtc/Kconfig                           |   6 +-
       drivers/rtc/rtc-bd70528.c                     | 104 +--
       include/linux/mfd/rohm-bd71815.h              | 562 +++++++++++++++
       include/linux/mfd/rohm-bd71828.h              |   3 +
       include/linux/mfd/rohm-bd718x7.h              |  13 -
       include/linux/mfd/rohm-generic.h              |  15 +-
       include/linux/regulator/driver.h              |   7 +
       25 files changed, 2393 insertions(+), 298 deletions(-)
       create mode 100644 Documentation/devicetree/bindings/mfd/rohm,bd71815-pmic.yaml
       create mode 100644 Documentation/devicetree/bindings/regulator/rohm,bd71815-regulator.yaml
       create mode 100644 drivers/gpio/gpio-bd71815.c
       create mode 100644 drivers/regulator/bd71815-regulator.c
       create mode 100644 include/linux/mfd/rohm-bd71815.h
      
      base-commit: 0d02ec6b
      --
      2.25.4
      
      --
      Matti Vaittinen, Linux device drivers
      ROHM Semiconductors, Finland SWDC
      Kiviharjunlenkki 1E
      90220 OULU
      FINLAND
      
      ~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
      Simon says - in Latin please.
      ~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
      Thanks to Simon Glass for the translation =]
      aaa8c4e0
    • Matti Vaittinen's avatar
      regulator: Add regmap helper for ramp-delay setting · fb8fee9e
      Matti Vaittinen authored
      Quite a few regulator ICs do support setting ramp-delay by writing a value
      matching the delay to a ramp-delay register.
      
      Provide a simple helper for table-based delay setting.
      Signed-off-by: default avatarMatti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
      Link: https://lore.kernel.org/r/f101f1db564cf32cb58719c77af0b00d7236bb89.1617020713.git.matti.vaittinen@fi.rohmeurope.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      fb8fee9e
    • Matti Vaittinen's avatar
      regulator: helpers: Export helper voltage listing · e3baacf5
      Matti Vaittinen authored
      Some drivers need to translate voltage values to selectors prior regulator
      registration. Currently a regulator_desc based list_voltages helper is only
      exported for regulators using the linear_ranges. Export similar helper also
      for regulators using simple linear mapping.
      Signed-off-by: default avatarMatti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
      Link: https://lore.kernel.org/r/1200ef7a50c84327ada019b85f6527b4fc9b5ce1.1617020713.git.matti.vaittinen@fi.rohmeurope.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      e3baacf5
  5. 01 Apr, 2021 4 commits
  6. 24 Mar, 2021 1 commit
  7. 21 Mar, 2021 21 commits
    • Linus Torvalds's avatar
      Linux 5.12-rc4 · 0d02ec6b
      Linus Torvalds authored
      0d02ec6b
    • Linus Torvalds's avatar
      Merge tag 'ext4_for_linus_stable' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4 · d7f5f1bd
      Linus Torvalds authored
      Pull ext4 fixes from Ted Ts'o:
       "Miscellaneous ext4 bug fixes for v5.12"
      
      * tag 'ext4_for_linus_stable' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4:
        ext4: initialize ret to suppress smatch warning
        ext4: stop inode update before return
        ext4: fix rename whiteout with fast commit
        ext4: fix timer use-after-free on failed mount
        ext4: fix potential error in ext4_do_update_inode
        ext4: do not try to set xattr into ea_inode if value is empty
        ext4: do not iput inode under running transaction in ext4_rename()
        ext4: find old entry again if failed to rename whiteout
        ext4: fix error handling in ext4_end_enable_verity()
        ext4: fix bh ref count on error paths
        fs/ext4: fix integer overflow in s_log_groups_per_flex
        ext4: add reclaim checks to xattr code
        ext4: shrink race window in ext4_should_retry_alloc()
      d7f5f1bd
    • Linus Torvalds's avatar
      Merge tag 'io_uring-5.12-2021-03-21' of git://git.kernel.dk/linux-block · 2c41fab1
      Linus Torvalds authored
      Pull io_uring followup fixes from Jens Axboe:
      
       - The SIGSTOP change from Eric, so we properly ignore that for
         PF_IO_WORKER threads.
      
       - Disallow sending signals to PF_IO_WORKER threads in general, we're
         not interested in having them funnel back to the io_uring owning
         task.
      
       - Stable fix from Stefan, ensuring we properly break links for short
         send/sendmsg recv/recvmsg if MSG_WAITALL is set.
      
       - Catch and loop when needing to run task_work before a PF_IO_WORKER
         threads goes to sleep.
      
      * tag 'io_uring-5.12-2021-03-21' of git://git.kernel.dk/linux-block:
        io_uring: call req_set_fail_links() on short send[msg]()/recv[msg]() with MSG_WAITALL
        io-wq: ensure task is running before processing task_work
        signal: don't allow STOP on PF_IO_WORKER threads
        signal: don't allow sending any signals to PF_IO_WORKER threads
      2c41fab1
    • Linus Torvalds's avatar
      Merge tag 'staging-5.12-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging · 1d4345eb
      Linus Torvalds authored
      Pull staging and IIO driver fixes from Greg KH:
       "Some small staging and IIO driver fixes:
      
         - MAINTAINERS changes for the move of the staging mailing list
      
         - comedi driver fixes to get request_irq() to work correctly
      
         - counter driver fixes for reported issues with iio devices
      
         - tiny iio driver fixes for reported issues.
      
        All of these have been in linux-next with no reported problems"
      
      * tag 'staging-5.12-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging:
        staging: vt665x: fix alignment constraints
        staging: comedi: cb_pcidas64: fix request_irq() warn
        staging: comedi: cb_pcidas: fix request_irq() warn
        MAINTAINERS: move the staging subsystem to lists.linux.dev
        MAINTAINERS: move some real subsystems off of the staging mailing list
        iio: gyro: mpu3050: Fix error handling in mpu3050_trigger_handler
        iio: hid-sensor-temperature: Fix issues of timestamp channel
        iio: hid-sensor-humidity: Fix alignment issue of timestamp channel
        counter: stm32-timer-cnt: fix ceiling miss-alignment with reload register
        counter: stm32-timer-cnt: fix ceiling write max value
        counter: stm32-timer-cnt: Report count function when SLAVE_MODE_DISABLED
        iio: adc: ab8500-gpadc: Fix off by 10 to 3
        iio:adc:stm32-adc: Add HAS_IOMEM dependency
        iio: adis16400: Fix an error code in adis16400_initial_setup()
        iio: adc: adi-axi-adc: add proper Kconfig dependencies
        iio: adc: ad7949: fix wrong ADC result due to incorrect bit mask
        iio: hid-sensor-prox: Fix scale not correct issue
        iio:adc:qcom-spmi-vadc: add default scale to LR_MUX2_BAT_ID channel
      1d4345eb
    • Linus Torvalds's avatar
      Merge tag 'usb-5.12-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb · 3001c355
      Linus Torvalds authored
      Pull USB and Thunderbolt driver fixes from Greg KH:
       "Here are some small Thunderbolt and USB driver fixes for some reported
        issues:
      
         - thunderbolt fixes for minor problems
      
         - typec fixes for power issues
      
         - usb-storage quirk addition
      
         - usbip bugfix
      
         - dwc3 bugfix when stopping transfers
      
         - cdnsp bugfix for isoc transfers
      
         - gadget use-after-free fix
      
        All have been in linux-next this week with no reported issues"
      
      * tag 'usb-5.12-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb:
        usb: typec: tcpm: Skip sink_cap query only when VDM sm is busy
        usb: dwc3: gadget: Prevent EP queuing while stopping transfers
        usb: typec: tcpm: Invoke power_supply_changed for tcpm-source-psy-
        usb: typec: Remove vdo[3] part of tps6598x_rx_identity_reg struct
        usb-storage: Add quirk to defeat Kindle's automatic unload
        usb: gadget: configfs: Fix KASAN use-after-free
        usbip: Fix incorrect double assignment to udc->ud.tcp_rx
        usb: cdnsp: Fixes incorrect value in ISOC TRB
        thunderbolt: Increase runtime PM reference count on DP tunnel discovery
        thunderbolt: Initialize HopID IDAs in tb_switch_alloc()
      3001c355
    • Linus Torvalds's avatar
      Merge tag 'irq-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 5ee96fa9
      Linus Torvalds authored
      Pull irq fix from Ingo Molnar:
       "A change to robustify force-threaded IRQ handlers to always disable
        interrupts, plus a DocBook fix.
      
        The force-threaded IRQ handler change has been accelerated from the
        normal schedule of such a change to keep the bad pattern/workaround of
        spin_lock_irqsave() in handlers or IRQF_NOTHREAD as a kludge from
        spreading"
      
      * tag 'irq-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        genirq: Disable interrupts for force threaded handlers
        genirq/irq_sim: Fix typos in kernel doc (fnode -> fwnode)
      5ee96fa9
    • Linus Torvalds's avatar
      Merge tag 'perf-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 1c74516c
      Linus Torvalds authored
      Pull perf fixes from Ingo Molnar:
       "Boundary condition fixes for bugs unearthed by the perf fuzzer"
      
      * tag 'perf-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        perf/x86/intel: Fix unchecked MSR access error caused by VLBR_EVENT
        perf/x86/intel: Fix a crash caused by zero PEBS status
      1c74516c
    • Linus Torvalds's avatar
      Merge tag 'locking-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 5ba33b48
      Linus Torvalds authored
      Pull locking fixes from Ingo Molnar:
      
       - Get static calls & modules right. Hopefully.
      
       - WW mutex fixes
      
      * tag 'locking-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        static_call: Fix static_call_update() sanity check
        static_call: Align static_call_is_init() patching condition
        static_call: Fix static_call_set_init()
        locking/ww_mutex: Fix acquire/release imbalance in ww_acquire_init()/ww_acquire_fini()
        locking/ww_mutex: Simplify use_ww_ctx & ww_ctx handling
      5ba33b48
    • Linus Torvalds's avatar
      Merge tag 'efi-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 92ed88cb
      Linus Torvalds authored
      Pull EFI fixes from Ingo Molnar:
      
       - another missing RT_PROP table related fix, to ensure that the
         efivarfs pseudo filesystem fails gracefully if variable services
         are unsupported
      
       - use the correct alignment for literal EFI GUIDs
      
       - fix a use after unmap issue in the memreserve code
      
      * tag 'efi-urgent-2021-03-21' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        efi: use 32-bit alignment for efi_guid_t literals
        firmware/efi: Fix a use after bug in efi_mem_reserve_persistent
        efivars: respect EFI_UNSUPPORTED return from firmware
      92ed88cb
    • Linus Torvalds's avatar
      Merge tag 'x86_urgent_for_v5.12-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 5e3ddf96
      Linus Torvalds authored
      Pull x86 fixes from Borislav Petkov:
       "The freshest pile of shiny x86 fixes for 5.12:
      
         - Add the arch-specific mapping between physical and logical CPUs to
           fix devicetree-node lookups
      
         - Restore the IRQ2 ignore logic
      
         - Fix get_nr_restart_syscall() to return the correct restart syscall
           number. Split in a 4-patches set to avoid kABI breakage when
           backporting to dead kernels"
      
      * tag 'x86_urgent_for_v5.12-rc4' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        x86/apic/of: Fix CPU devicetree-node lookups
        x86/ioapic: Ignore IRQ2 again
        x86: Introduce restart_block->arch_data to remove TS_COMPAT_RESTART
        x86: Introduce TS_COMPAT_RESTART to fix get_nr_restart_syscall()
        x86: Move TS_COMPAT back to asm/thread_info.h
        kernel, fs: Introduce and use set_restart_fn() and arch_set_restart_data()
      5e3ddf96
    • Linus Torvalds's avatar
      Merge tag 'powerpc-5.12-4' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux · b35660a7
      Linus Torvalds authored
      Pull powerpc fixes from Michael Ellerman:
      
       - Fix a possible stack corruption and subsequent DLPAR failure in the
         rpadlpar_io PCI hotplug driver
      
       - Two build fixes for uncommon configurations
      
      Thanks to Christophe Leroy and Tyrel Datwyler.
      
      * tag 'powerpc-5.12-4' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux:
        PCI: rpadlpar: Fix potential drc_name corruption in store functions
        powerpc: Force inlining of cpu_has_feature() to avoid build failure
        powerpc/vdso32: Add missing _restgpr_31_x to fix build failure
      b35660a7
    • Stefan Metzmacher's avatar
      io_uring: call req_set_fail_links() on short send[msg]()/recv[msg]() with MSG_WAITALL · 0031275d
      Stefan Metzmacher authored
      Without that it's not safe to use them in a linked combination with
      others.
      
      Now combinations like IORING_OP_SENDMSG followed by IORING_OP_SPLICE
      should be possible.
      
      We already handle short reads and writes for the following opcodes:
      
      - IORING_OP_READV
      - IORING_OP_READ_FIXED
      - IORING_OP_READ
      - IORING_OP_WRITEV
      - IORING_OP_WRITE_FIXED
      - IORING_OP_WRITE
      - IORING_OP_SPLICE
      - IORING_OP_TEE
      
      Now we have it for these as well:
      
      - IORING_OP_SENDMSG
      - IORING_OP_SEND
      - IORING_OP_RECVMSG
      - IORING_OP_RECV
      
      For IORING_OP_RECVMSG we also check for the MSG_TRUNC and MSG_CTRUNC
      flags in order to call req_set_fail_links().
      
      There might be applications arround depending on the behavior
      that even short send[msg]()/recv[msg]() retuns continue an
      IOSQE_IO_LINK chain.
      
      It's very unlikely that such applications pass in MSG_WAITALL,
      which is only defined in 'man 2 recvmsg', but not in 'man 2 sendmsg'.
      
      It's expected that the low level sock_sendmsg() call just ignores
      MSG_WAITALL, as MSG_ZEROCOPY is also ignored without explicitly set
      SO_ZEROCOPY.
      
      We also expect the caller to know about the implicit truncation to
      MAX_RW_COUNT, which we don't detect.
      
      cc: netdev@vger.kernel.org
      Link: https://lore.kernel.org/r/c4e1a4cc0d905314f4d5dc567e65a7b09621aab3.1615908477.git.metze@samba.orgSigned-off-by: default avatarStefan Metzmacher <metze@samba.org>
      Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
      0031275d
    • Jens Axboe's avatar
      io-wq: ensure task is running before processing task_work · 00ddff43
      Jens Axboe authored
      Mark the current task as running if we need to run task_work from the
      io-wq threads as part of work handling. If that is the case, then return
      as such so that the caller can appropriately loop back and reset if it
      was part of a going-to-sleep flush.
      
      Fixes: 3bfe6106 ("io-wq: fork worker threads from original task")
      Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
      00ddff43
    • Eric W. Biederman's avatar
      signal: don't allow STOP on PF_IO_WORKER threads · 4db4b1a0
      Eric W. Biederman authored
      Just like we don't allow normal signals to IO threads, don't deliver a
      STOP to a task that has PF_IO_WORKER set. The IO threads don't take
      signals in general, and have no means of flushing out a stop either.
      
      Longer term, we may want to look into allowing stop of these threads,
      as it relates to eg process freezing. For now, this prevents a spin
      issue if a SIGSTOP is delivered to the parent task.
      Reported-by: default avatarStefan Metzmacher <metze@samba.org>
      Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
      Signed-off-by: default avatar"Eric W. Biederman" <ebiederm@xmission.com>
      4db4b1a0
    • Jens Axboe's avatar
      signal: don't allow sending any signals to PF_IO_WORKER threads · 5be28c8f
      Jens Axboe authored
      They don't take signals individually, and even if they share signals with
      the parent task, don't allow them to be delivered through the worker
      thread. Linux does allow this kind of behavior for regular threads, but
      it's really a compatability thing that we need not care about for the IO
      threads.
      Reported-by: default avatarStefan Metzmacher <metze@samba.org>
      Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
      5be28c8f
    • Theodore Ts'o's avatar
      64395d95
    • Pan Bian's avatar
      ext4: stop inode update before return · 512c15ef
      Pan Bian authored
      The inode update should be stopped before returing the error code.
      Signed-off-by: default avatarPan Bian <bianpan2016@163.com>
      Link: https://lore.kernel.org/r/20210117085732.93788-1-bianpan2016@163.com
      Fixes: 8016e29f ("ext4: fast commit recovery path")
      Cc: stable@kernel.org
      Reviewed-by: default avatarHarshad Shirwadkar <harshadshirwadkar@gmail.com>
      Signed-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
      512c15ef
    • Harshad Shirwadkar's avatar
      ext4: fix rename whiteout with fast commit · 8210bb29
      Harshad Shirwadkar authored
      This patch adds rename whiteout support in fast commits. Note that the
      whiteout object that gets created is actually char device. Which
      imples, the function ext4_inode_journal_mode(struct inode *inode)
      would return "JOURNAL_DATA" for this inode. This has a consequence in
      fast commit code that it will make creation of the whiteout object a
      fast-commit ineligible behavior and thus will fall back to full
      commits. With this patch, this can be observed by running fast commits
      with rename whiteout and seeing the stats generated by ext4_fc_stats
      tracepoint as follows:
      
      ext4_fc_stats: dev 254:32 fc ineligible reasons:
      XATTR:0, CROSS_RENAME:0, JOURNAL_FLAG_CHANGE:0, NO_MEM:0, SWAP_BOOT:0,
      RESIZE:0, RENAME_DIR:0, FALLOC_RANGE:0, INODE_JOURNAL_DATA:16;
      num_commits:6, ineligible: 6, numblks: 3
      
      So in short, this patch guarantees that in case of rename whiteout, we
      fall back to full commits.
      
      Amir mentioned that instead of creating a new whiteout object for
      every rename, we can create a static whiteout object with irrelevant
      nlink. That will make fast commits to not fall back to full
      commit. But until this happens, this patch will ensure correctness by
      falling back to full commits.
      
      Fixes: 8016e29f ("ext4: fast commit recovery path")
      Cc: stable@kernel.org
      Signed-off-by: default avatarHarshad Shirwadkar <harshadshirwadkar@gmail.com>
      Link: https://lore.kernel.org/r/20210316221921.1124955-1-harshadshirwadkar@gmail.comSigned-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
      8210bb29
    • Jan Kara's avatar
      ext4: fix timer use-after-free on failed mount · 2a4ae3bc
      Jan Kara authored
      When filesystem mount fails because of corrupted filesystem we first
      cancel the s_err_report timer reminding fs errors every day and only
      then we flush s_error_work. However s_error_work may report another fs
      error and re-arm timer thus resulting in timer use-after-free. Fix the
      problem by first flushing the work and only after that canceling the
      s_err_report timer.
      
      Reported-by: syzbot+628472a2aac693ab0fcd@syzkaller.appspotmail.com
      Fixes: 2d01ddc8 ("ext4: save error info to sb through journal if available")
      CC: stable@vger.kernel.org
      Signed-off-by: default avatarJan Kara <jack@suse.cz>
      Link: https://lore.kernel.org/r/20210315165906.2175-1-jack@suse.czSigned-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
      2a4ae3bc
    • Shijie Luo's avatar
      ext4: fix potential error in ext4_do_update_inode · 7d8bd3c7
      Shijie Luo authored
      If set_large_file = 1 and errors occur in ext4_handle_dirty_metadata(),
      the error code will be overridden, go to out_brelse to avoid this
      situation.
      Signed-off-by: default avatarShijie Luo <luoshijie1@huawei.com>
      Link: https://lore.kernel.org/r/20210312065051.36314-1-luoshijie1@huawei.com
      Cc: stable@kernel.org
      Reviewed-by: default avatarJan Kara <jack@suse.cz>
      Signed-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
      7d8bd3c7
    • zhangyi (F)'s avatar
      ext4: do not try to set xattr into ea_inode if value is empty · 6b224899
      zhangyi (F) authored
      Syzbot report a warning that ext4 may create an empty ea_inode if set
      an empty extent attribute to a file on the file system which is no free
      blocks left.
      
        WARNING: CPU: 6 PID: 10667 at fs/ext4/xattr.c:1640 ext4_xattr_set_entry+0x10f8/0x1114 fs/ext4/xattr.c:1640
        ...
        Call trace:
         ext4_xattr_set_entry+0x10f8/0x1114 fs/ext4/xattr.c:1640
         ext4_xattr_block_set+0x1d0/0x1b1c fs/ext4/xattr.c:1942
         ext4_xattr_set_handle+0x8a0/0xf1c fs/ext4/xattr.c:2390
         ext4_xattr_set+0x120/0x1f0 fs/ext4/xattr.c:2491
         ext4_xattr_trusted_set+0x48/0x5c fs/ext4/xattr_trusted.c:37
         __vfs_setxattr+0x208/0x23c fs/xattr.c:177
        ...
      
      Now, ext4 try to store extent attribute into an external inode if
      ext4_xattr_block_set() return -ENOSPC, but for the case of store an
      empty extent attribute, store the extent entry into the extent
      attribute block is enough. A simple reproduce below.
      
        fallocate test.img -l 1M
        mkfs.ext4 -F -b 2048 -O ea_inode test.img
        mount test.img /mnt
        dd if=/dev/zero of=/mnt/foo bs=2048 count=500
        setfattr -n "user.test" /mnt/foo
      
      Reported-by: syzbot+98b881fdd8ebf45ab4ae@syzkaller.appspotmail.com
      Fixes: 9c6e7853 ("ext4: reserve space for xattr entries/names")
      Cc: stable@kernel.org
      Signed-off-by: default avatarzhangyi (F) <yi.zhang@huawei.com>
      Link: https://lore.kernel.org/r/20210305120508.298465-1-yi.zhang@huawei.comSigned-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
      6b224899