lkml.org 
[lkml]   [2021]   [May]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v9 05/12] scsi: ufshpb: Make eviction depends on region's reads
    Date
    In host mode, eviction is considered an extreme measure.
    verify that the entering region has enough reads, and the exiting
    region has much less reads.

    Signed-off-by: Avri Altman <avri.altman@wdc.com>
    Reviewed-by: Daejun Park <daejun7.park@samsung.com>
    ---
    drivers/scsi/ufs/ufshpb.c | 18 +++++++++++++++++-
    1 file changed, 17 insertions(+), 1 deletion(-)

    diff --git a/drivers/scsi/ufs/ufshpb.c b/drivers/scsi/ufs/ufshpb.c
    index e3b1eb857ce3..180e74675912 100644
    --- a/drivers/scsi/ufs/ufshpb.c
    +++ b/drivers/scsi/ufs/ufshpb.c
    @@ -17,6 +17,7 @@
    #include "../sd.h"

    #define ACTIVATION_THRESHOLD 8 /* 8 IOs */
    +#define EVICTION_THRESHOLD (ACTIVATION_THRESHOLD << 5) /* 256 IOs */

    /* memory management */
    static struct kmem_cache *ufshpb_mctx_cache;
    @@ -1055,6 +1056,13 @@ static struct ufshpb_region *ufshpb_victim_lru_info(struct ufshpb_lu *hpb)
    if (ufshpb_check_srgns_issue_state(hpb, rgn))
    continue;

    + /*
    + * in host control mode, verify that the exiting region
    + * has less reads
    + */
    + if (hpb->is_hcm && rgn->reads > (EVICTION_THRESHOLD >> 1))
    + continue;
    +
    victim_rgn = rgn;
    break;
    }
    @@ -1227,7 +1235,7 @@ static int ufshpb_issue_map_req(struct ufshpb_lu *hpb,

    static int ufshpb_add_region(struct ufshpb_lu *hpb, struct ufshpb_region *rgn)
    {
    - struct ufshpb_region *victim_rgn;
    + struct ufshpb_region *victim_rgn = NULL;
    struct victim_select_info *lru_info = &hpb->lru_info;
    unsigned long flags;
    int ret = 0;
    @@ -1254,7 +1262,15 @@ static int ufshpb_add_region(struct ufshpb_lu *hpb, struct ufshpb_region *rgn)
    * It is okay to evict the least recently used region,
    * because the device could detect this region
    * by not issuing HPB_READ
    + *
    + * in host control mode, verify that the entering
    + * region has enough reads
    */
    + if (hpb->is_hcm && rgn->reads < EVICTION_THRESHOLD) {
    + ret = -EACCES;
    + goto out;
    + }
    +
    victim_rgn = ufshpb_victim_lru_info(hpb);
    if (!victim_rgn) {
    dev_warn(&hpb->sdev_ufs_lu->sdev_dev,
    --
    2.25.1
    \
     
     \ /
      Last update: 2021-05-24 13:23    [W:4.189 / U:0.700 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site