Re: [EXT] Re: [PATCH v2] ufs: core: Add WB buffer resize support

[Bean Huo <huobean@gmail.com>]

On Tue, 2024-10-29 at 20:03 +0800, Huan Tang wrote:
> > > > On 10/28/24 1:04 PM, Bean Huo wrote:
> > > > > Even though I don't think it's necessary to enable a Sysfs
> > > > > node 
> > > > > entry for this configuration.
> > > > 
> > > > Right, a motivation of why this functionality should be
> > > > available in 
> > > > sysfs is missing. An explanation should be added in the patch 
> > > > description.
> > > > 
> > > > Thanks,
> > > > 
> > > > Bart.
> > > 
> > > Hi Bean & Bart,
> > > 
> > > Motivation: Through the sysfs upper layer code, the WB resize
> > > function 
> > > can be used in some scenarios, or related information can be
> > > obtained 
> > > indirectly to implement different strategies; What is your
> > > suggestion? 
> > > sysfs? exception event? or?
> > > 
> > > Thanks
> > > Huan
> > 
> > hey Huan,
> > 
> > What specific scenarios would require enabling a sysfs node to
> > control this function? Dynamically
> > adjusting the WriteBooster (WB) size on the fly doesn’t seem ideal
> > to me. From my perspective, the main
> > case for this feature is if the OEM didn’t correctly define or set
> > the WriteBooster Buffer size during
> > manufacturing. Even then, adjusting the WB buffer size wouldn’t be
> > a frequent need. If JEDEC has
> > found a reason for this feature to be accepted, isn’t there already
> > an interface available to configure it?
> > Why would we need a duplicate interface for the same purpose?
> > 
> > Kind regards,
> > Bean
> 
> Hi Bean,
> 
> Thanks for your reply
> 
> The scenario I'm thinking of right now： when the old phone transfers
> a large
> amount of data to the new phone's APP (vivo calls it easyshare APP),
> we can 
> explicitly resize the WB buffer.
> Could you please explain this sentence(If JEDEC has found a reason
> for this feature
> to be accepted, isn’t there already an interface available to
> configure it? Why
> would we need a duplicate interface for the same purpose?) more
> simply so I 
> can understand it better?


I see, easyshare is a case, but we have interface which allows user to
configure UFS attributes, such as ufs-bsg, you can use this interface
to achieve this in your application easily, right?


Kind regares,
Bean