Re: [PATCH stable 5.10 0/7] Remove reader optimistic spinning

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

 



On Mon, Feb 13, 2023 at 12:11:55PM +0000, Mohamed Abuelfotoh, Hazem wrote:
> > This is great that you did this work and found this out, but really,
> > shouldn't you have done the less work and just moved to 5.15.y instead?
> > You're going to have to do that anyway, what's preventing that from
> > happening now, with the HUGE justification that you get a big workload
> > increase and power savings (i.e. real money)?
> 
> 
> 
> 
> 
> 
> 
> 
> Hey Greg,
> 
> 
> 
> 
> 
> 
> 
> 

Hi,

Odd whitespace, you should fix your email client :)

> We are actually shipping kernel 5.15 as part of Amazon Linux kernel releases so theoretically moving to 5.15 should be the way to go however usually the relevant teams take some time for workload specific testing and benchmark before they do a major upgrade like moving from 5.10 to 5.15. We usually ask whoever is reporting a regression/bug/kernel enhancement to run with the latest kernel as you said while sometimes we backport fixes if the production migration to the latest kernel is something that will take time for the reasons I mentioned above. We thought that this performance improvement will also be beneficial for Linux 5.10 users hence we preferred these patches to be merged to the stable 5.10 rather than us just consume them as downstream patches. We are currently working with the relevant team on a plan for the possible 5.15 migration as a long term solution.

And use some \n characters :)

But this isn't a regression, it's a speedup.  And a good reason to use a
newer kernel version.  What type of guarantees that these changes work
properly and have all needed future bugfixes applied to them?  How were
they tested?

I am loath to want to replace something as core as this without a lot of
testing and verification and agreement from the subsystem
authors/maintainers that this is ok to make.

Seriously, just move to 5.15.y or 6.1.y, that will be easier for you in
the long run, the amount of testing and validation should have been the
same for this small patchset as it would be for a kernel update, so
there shouldn't be any issues here.

thanks,

greg k-h



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux