Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pelt branch modifications #2

Merged
merged 29 commits into from
Feb 10, 2025
Merged

Add pelt branch modifications #2

merged 29 commits into from
Feb 10, 2025

Conversation

hxsyzl
Copy link
Owner

@hxsyzl hxsyzl commented Feb 10, 2025

No description provided.

hxsyzl and others added 29 commits February 10, 2025 09:21
The identifier is missing when using Unicode
* Remove non-exist sbi identifier
Drop backported single patches in favor of full update to latest v1.9.4
Update lz4 module using official repository from revision [1].

Keep in mind lz4hc wasn't updated thus it is not used. It may not
compile anymore.

[1]: https://github.com/lz4/lz4/tree/4ebe313e00aa52c837ee029ede39a0503a8a39c9
Signed-off-by: Andrzej Perczak <linux@andrzejperczak.com>
Instead of setting up the queuedata as well just use one private data
field.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Andrzej Perczak <linux@andrzejperczak.com>
[ Upstream commit a88e03cf3d190cf46bc4063a9b7efe87590de5f4 ]

snprintf() returns the number of bytes it would have printed if there
were space.  But it does not count the NUL terminator.  So that means
that if "count == copied" then this has already overflowed by one
character.

This bug likely isn't super harmful in real life.

Link: https://lkml.kernel.org/r/20210916130404.GA25094@kili
Fixes: c0265342bff4 ("zram: introduce zram memory tracking")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Sergey Senozhatsky <senozhatsky@chromium.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Andrzej Perczak <linux@andrzejperczak.com>
When the ->init_lock is released in zram_reset_device(), disksize_store()
can come in and try to allocate meta, but zram_reset_device() is freeing
free meta, so cause races.

Link: https://lore.kernel.org/linux-block/20210927163805.808907-1-mcgrof@kernel.org/T/#mc617f865a3fa2778e40f317ddf48f6447c20c073
Reported-by: Luis Chamberlain <mcgrof@kernel.org>
Reviewed-by: Luis Chamberlain <mcgrof@kernel.org>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Acked-by: Minchan Kim <minchan@kernel.org>
Link: https://lore.kernel.org/r/20211025025426.2815424-2-ming.lei@redhat.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Andrzej Perczak <linux@andrzejperczak.com>
Signed-off-by: Yu Zhao <yuzhao@google.com>
Change-Id: I09999793fd6481bcf2acefb29c57d5d40c0b1427
Some DT devices, mainly smartphones, do need more trip points
to allow more fine grained thermal mitigations, hence allowing
a better user experience (and overall performance), for example,
by lowering the CPU clocks just a little for each temperature
step.

Signed-off-by: Henrique Pereira <hlcpereira@pixelexperience.org>
Signed-off-by: Panchajanya1999 <panchajanya@azure-dev.live>
Signed-off-by: Cyber Knight <cyberknight755@gmail.com>
I get this error when I compile with IPC_LOGGING disabled. I have no clue how to fix
seeing as https://github.com/RealJohnGalt/GaltsGulch-sm8150 has exact same cma.c as I do
and that kernel can be compiled as far as I can tell without IPC_LOGGING.
So just include where it is declared?

Anyone please advise if you know where it went wrong

../mm/cma.c:497:9: error: implicit declaration of function 'fatal_signal_pending' [-Werror,-Wimplicit-function-declaration]
                                if (fatal_signal_pending(current))
                                    ^
1 error generated.
Signed-off-by: Park Ju Hyung <qkrwngud825@gmail.com>
Signed-off-by: celtare21 <celtare21@gmail.com>
Others are unused

Signed-off-by: Park Ju Hyung <qkrwngud825@gmail.com>
Signed-off-by: celtare21 <celtare21@gmail.com>
Signed-off-by: celtare21 <celtare21@gmail.com>
Change-Id: I530538ac90d4e1e2e7e1835b2124a2d541f938f0
Bug: 67506682
Change-Id: Iad7ed493bd29e2da85e2d6bc0f1dc0fbdac8a41d
Signed-off-by: Sami Tolvanen <samitolvanen@google.com>
(cherry picked from commit f4dbb3c698af0bcf83df4f6b55ac714295c572d3)
(cherry picked from commit 6b4c6da82352590f93ab25951e415142cb66ea61)
(cherry picked from commit a853bd62173442182dcc7520186095b1da2989c3)
(cherry picked from commit 09b9474be7b961c95ba107b91bccbaab92939b2d)
Signed-off-by: Panchajanya1999 <panchajanya@azure-dev.live>
Signed-off-by: Danny Lin <danny@kdrag0n.dev>
[@0ctobot: Adapted for 4.14]
Signed-off-by: Adam W. Willis <return.of.octobot@gmail.com>
Signed-off-by: azrim <mirzaspc@gmail.com>
Copying from user to a global variable inside the driver is
unsafe and insecure: allocate and free the memory used for
copying from userspace in the functions doing it.

This also has the good side effect of solving a build failure
when the kernel is built with GCC >=4.9

Change-Id: I04bfb0cd0ffb6870481f639818cf8c51191d927d
The smallest string that this function deals with is "1", which is of
size 1. Correct the if to allow this case.

Change-Id: Iaa68bcc452a3428d4752e5c34c98d0bd2e926c5a
Signed-off-by: MarijnS95 <marijns95@gmail.com>
Becauase of using wrong pair of PM operation device
was not resuming if suspend fails. Also device gives
errors in dmesg like below:

[ 2420.308491] dpm_run_callback(): icnss_pm_suspend_noirq+0x0/0xc0 returns -11
[ 2420.308510] PM: Device 18800000.qcom,icnss failed to suspend async: error -11
[ 2420.312069] PM: noirq suspend of devices failed
[ 2423.384002] dpm_run_callback(): icnss_pm_suspend_noirq+0x0/0xc0 returns -11
[ 2423.384020] PM: Device 18800000.qcom,icnss failed to suspend async: error -11
[ 2423.317523] PM: noirq suspend of devices failed
[ 2426.444164] dpm_run_callback(): icnss_pm_suspend_noirq+0x0/0xc0 returns -11
[ 2426.444181] PM: Device 18800000.qcom,icnss failed to suspend async: error -11
[ 2426.447813] PM: noirq suspend of devices failed
[ 2428.915643] dpm_run_callback(): icnss_pm_suspend_noirq+0x0/0xc0 returns -11
[ 2428.915659] PM: Device 18800000.qcom,icnss failed to suspend async: error -11
[ 2428.919208] PM: noirq suspend of devices failed
[ 2429.529067] dpm_run_callback(): icnss_pm_suspend_noirq+0x0/0xc0 returns -11
[ 2429.529086] PM: Device 18800000.qcom,icnss failed to suspend async: error -11
[ 2423.532786] PM: noirq suspend of devices failed

Adding changes to use correct set of PM operations and
fix log spam.

Signed-off-by: atndko <z1281552865@gmail.com>
Signed-off-by: msdx321 <msdx321@gmail.com>
For easier upstream merges.

Signed-off-by: Andrzej Perczak <linux@andrzejperczak.com>
Signed-off-by: Andrzej Perczak <linux@andrzejperczak.com>
This reverts commit 6994c5044dcadcf33568b37b91b3d81becc0c179.
For portrait mode with dual camera synchronization, enable initial skip
to make sure two sensors both start streaming with synchronization
signal.  This fixed the portrait mode switching stability issue.

Bug: 136756925
Change-Id: I2e6e8c463bffd04da441f3da3033e6596f30e72a
Signed-off-by: Chris Ye <lzye@google.com>
Signed-off-by: Yaroslav Furman <yaro330@gmail.com>
Signed-off-by: Cyber Knight <cyberknight755@gmail.com>
This reverts commit 06f863cfe21d36c70ae2270e1cf151c0055d71f7.

Signed-off-by: Cyber Knight <cyberknight755@gmail.com>
…policy free

detect_share_cap_flag() calls cpufreq_cpu_get() to get cpufreq
policy for each possible cpu, meanwhile, it also increments the
kobject reference count of policy to mark it busy. However, a
corresponding call to cpufreq_cpu_put() is ignored to decrement
the kobject reference count back, which may lead to a potential
stuck risk that percpu cpuhp thread deadly waits for dropping
of kobject refcount when percpu cpufreq policy free.

The call trace of stuck risk could be:
cpufreq_online()  //If cpufreq initialization failed, goto out_free_policy.
    ->cpufreq_policy_free()     //Do cpufreq_policy free.
        ->cpufreq_policy_put_kobj()
            ->kobject_put()       //Skip if policy kfref count is not 1.
                ->cpufreq_sysfs_release()
                    ->complete()  //Complete policy->kobj_unregister.
                ->wait_for_completion() //Wait for policy->kobj_unregister.

With this patch, the cpuhp thread can be easily exercised by
attempting to force an unbind of the CPUfreq driver.

Bug: 120440300
Bug: 147378688
Signed-off-by: chenqiwu <chenqiwu@xiaomi.com>
Signed-off-by: qiwu chen <qiwuchen55@gmail.com>
Signed-off-by: Quentin Perret <qperret@google.com>
Change-Id: I976fc697d4090324877b28fdb5bb5a5b63f6d947
Signed-off-by: Pranav Vashi <neobuddy89@gmail.com>
lybxlpsv said that some unity games also looking for the cpu_capacity
to determine about their decision on affinity setup.

So let's mitigate it too with is_sched_lib_based_app() just like on the
previous attempt.

Fixes: d43b69c ("sched/core: fix
userspace affining threads incorrectly")
Signed-off-by: Nauval Rizky <enuma.alrizky@gmail.com>
Signed-off-by: Pranav Vashi <neobuddy89@gmail.com>
Avoiding inbound entry release during freeze callback
to keep the irq domain as it is.Resetting inbound
last value.

Change-Id: Ic5374fa8094d0f4de914076cef348469158b4012
Signed-off-by: Arun Prakash <app@codeaurora.org>
Signed-off-by: Panchajanya1999 <panchajanya@azure-dev.live>
Signed-off-by: Pranav Vashi <neobuddy89@gmail.com>
@hxsyzl hxsyzl merged commit 070406b into erofs Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.