Re: [PATCH] md: do not require mddev_lock() for all options

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mariusz,

Sorry for the late reply.

On Wed, Sep 13, 2023 at 1:55 AM Mariusz Tkaczyk
<mariusz.tkaczyk@xxxxxxxxxxxxxxx> wrote:
>
> We don't need to lock device to reject not supported request
> in array_state_store().
> Main motivation is to make a room for action does not require lock yet,
> like prepare to stop (see md_ioctl()).

I made some changes to the commit log:

    md: do not require mddev_lock() for all options

    We don't need to lock device to reject not supported request
    in array_state_store().
    Main motivation is to make a room for action does not require lock yet,
    like prepare to stop (see md_ioctl()).

But I am not sure what you meant by "make a room for action does not
require lock yet". Could you please explain?

Otherwise, the code looks reasonable to me.

Thanks,
Song




[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux