linux-kernel - Re: [RESENT PATCH RFC v3 5/5] scsi: ufs: UFS Host Performance Booster(HPB) driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <ff9de169-be8e-1595-5df0-0d09b1a1ce8e@acm.org>
Date:   Fri, 8 May 2020 13:11:38 -0700
From:   Bart Van Assche <bvanassche@....org>
To:     Bean Huo <huobean@...il.com>, alim.akhtar@...sung.com,
        avri.altman@....com, asutoshd@...eaurora.org, jejb@...ux.ibm.com,
        martin.petersen@...cle.com, stanley.chu@...iatek.com,
        beanhuo@...ron.com, tomas.winkler@...el.com, cang@...eaurora.org,
        rdunlap@...radead.org
Cc:     linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
        hch@...radead.org
Subject: Re: [RESENT PATCH RFC v3 5/5] scsi: ufs: UFS Host Performance
 Booster(HPB) driver

On 2020-05-08 07:44, Bean Huo wrote:
> 1: driver module parameter, while system booting, specify maximum HPB
> cache size in kernel boot Parameters:
> 
>    	static int max_hpb_mem = 128;
>    	module_param(max_hpb_mem,int,0444);
> 
> but this paramter is added in the ufshcd.c since the ufshpb.c is not a 
> independent module. looks not very natural.
> [ ... ]
> If you prefer to convert current HPB driver to be a kernel module
> driver, I prefer first apporach.

Hi Bean,

A kernel module parameter sounds good to me.

Thanks,

Bart.