The following commit has been merged into the ras/core branch of tip:
Commit-ID: 278b917f8cb9b02923c15249f9d1a5769d2c1976
Gitweb:
https://git.kernel.org/tip/278b917f8cb9b02923c15249f9d1a5769d2c1976
Author:Youquan Song
AuthorDate:Tue, 06 Oct 2020 14:09:07 -07:00
Committer
The following commit has been merged into the ras/core branch of tip:
Commit-ID: 41ce0564bfe2e129d56730418d8c0a9f9f2d31b5
Gitweb:
https://git.kernel.org/tip/41ce0564bfe2e129d56730418d8c0a9f9f2d31b5
Author:Youquan Song
AuthorDate:Tue, 06 Oct 2020 14:09:05 -07:00
Committer
: pbonz...@redhat.com
Cc: rkrc...@redhat.com
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar
Signed-off-by: Youquan Song [v4.4 backport]
---
arch/x86/include/asm/nospec-branch.h | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/arch/x86/include
: pbonz...@redhat.com
Cc: rkrc...@redhat.com
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar
Signed-off-by: Youquan Song [v4.4 backport]
---
arch/x86/include/asm/nospec-branch.h | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/arch/x86/include
: pbonz...@redhat.com
Cc: rkrc...@redhat.com
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ingo Molnar
Signed-off-by: Greg Kroah-Hartman
[Youquan Song: port to 4.4]
Signed-off-by: Youquan Song
---
arch/x86/include/asm/nospec-branch.h | 26 ++
1 file changed, 14 insertions
tlb,
and free/release wrong domain.
Tested-by: Zhiyuan Zhou
Signed-off-by: Youquan Song
---
drivers/iommu/intel-iommu.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index 43b9bfe..9cd522f 100644
--- a/driv
the "found=1" should be "there are other device owned by the domain", the
comments is put at wrong place and make the code reviewing confuse, so move it
to the correct place.
Signed-off-by: Youquan Song
---
drivers/iommu/intel-iommu.c |8
1 files changed
device, so save the loop time and make the code more clear.
Signed-off-by: Youquan Song
---
drivers/iommu/intel-iommu.c | 15 ++-
1 files changed, 6 insertions(+), 9 deletions(-)
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index aa821fc..9f3bf3f 100644
On Sun, Oct 13, 2013 at 08:56:33PM +0530, Vinod Koul wrote:
> On Fri, Oct 11, 2013 at 06:33:43AM -0700, Greg KH wrote:
> > On Fri, Oct 11, 2013 at 05:42:17PM -0400, Youquan Song wrote:
> > > Currently, the DMA channel calculates its data transferred only at network
> > >
When using UART transfers data by DMA mode, but it always shows 0 at
/sys/class/dma/dma0chan*/bytes_transferred.
Call the new function to calculate how many the data has been transferred
after doing it by DMA mode.
Signed-off-by: Youquan Song
---
drivers/tty/serial/8250/8250_dma.c |2
Currently, the DMA channel calculates its data transferred only at network
device driver. When other devices like UART or SPI etc, transfers data by DMA
mode, but it always shows 0 at /sys/class/dma/dma0chan*/bytes_transferred.
It will possibly mislead user that the DMA engine does not work.
This
has been
transferred after doing it by DMA mode. It can be used by other modules and
also simplify current duplicated code.
Signed-off-by: Youquan Song
---
drivers/dma/dmaengine.c | 35 +++
include/linux/dmaengine.h |3 +++
2 files changed, 22 insertions
> Firstly, please use the customary (multi-line) comment
> style:
>
> /*
>* Comment .
>* .. goes here.
>*/
>
> specified in Documentation/CodingStyle.
>
> Secondly, please send a patch against a vanilla (e.g.
> v3.11-rc5) kernel, as I've already zapped your previous
> pa
in the local APIC units.
This reverts commit 3d1acb49d22fbbae96524040e9e2d4cbbb3adbef, do not use
x2apic_pysical mode if interrupt remapping is not enabled even at CPU
number fewer than 256.
Signed-off-by: Youquan Song
---
arch/x86/kernel/apic/apic.c |7 +--
1 files changed, 5 insertions(
> In order to make sure the patch without involving unexpected issues beyond
> I can understand, I will confirm with our expert about it.
>
> so please pend the patch going to mainline. If the patch can move on, I
> think I will also provide other patch changing, like direct EOI.
Hi Yinghai and I
Hi Jeremy,
I try reproduce your result and then fix the issue, but I do not reproduce it
yet.
I run at netperf-2.6.0 at one machine as server: netserver, other
machine: netperf -t TCP_RR -H $SERVER_IP -l 60. The target machine is
used in both client and server. I do not reproduce the performance
> Yes. It would be great, if Youquan can point out where is the intel doc
> about the change.
>
> Also if the patch can move on, hypervisor_x2apic_available() related
> declaration and define
> could be dropped.
Hi Yinghai,
Sorry I do not know the document change but I also do not find the
word
> > Thanks Ingo!
> > The machines will be affected: CPU support x2APIC and CPU number < 256,
> > chipset does not support VT-d2 or VT-d is disabled in BIOS.
>
> I mean, can you guess what rough percentage of new systems
> shipping (or significant number of older systems already
> shipped) will
Commit-ID: 3d1acb49d22fbbae96524040e9e2d4cbbb3adbef
Gitweb: http://git.kernel.org/tip/3d1acb49d22fbbae96524040e9e2d4cbbb3adbef
Author: Youquan Song
AuthorDate: Thu, 11 Jul 2013 21:22:39 -0400
Committer: Ingo Molnar
CommitDate: Tue, 23 Jul 2013 11:15:42 +0200
x86/apic: Enable x2APIC
On Tue, Jul 23, 2013 at 11:17:29AM +0200, Ingo Molnar wrote:
>
> * Youquan Song wrote:
>
> > x2APIC extends APICID from 8 bits to 32 bits, but the device interrupt
> > routed from IOAPIC or delivered in MSI mode will keep 8 bits destination
> > APICID. In order
APIC mmio
- x2APIC only ICR write to deliver interrupt without polling ICR deliver
status bit and xAPIC need poll to read ICR deliver status bit.
- x2APIC 64 bits ICR access instead of xAPIC two 32 bits access.
Signed-off-by: Youquan Song
---
arch/x86/kernel/apic/apic.c |7 ++-
1 file
deliver a similar patch to fix it by disables 32bit PIO in IDE mode.
Signed-off-by: Youquan Song
Cc: sta...@vger.kernel.org
---
drivers/ata/ata_piix.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index 9a8a674..424b
deliver a similar patch to fix it by disables 32bit PIO in IDE mode.
Signed-off-by: Youquan Song
---
drivers/ata/ata_piix.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index 9a8a674..424bcbe 100644
--- a/drivers/ata/
deliver a similar patch to fix it by disables 32bit PIO in IDE mode.
Signed-off-by: Youquan Song
---
drivers/ata/ata_piix.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index 9a8a674..424bcbe 100644
--- a/drivers/ata/
soft limit or initial online nr.
> >
> > we already have nr_cpus= for hard limit.
> >
> > So need to drop
> > commit 3e275a5ba367ab74b3a4e49114307baed989fcac
> > Author: Youquan Song
> > Date: Fri Jun 7 10:07:08 2013 +1000
> >
> > driv
> On 06/12/2013 05:03 AM, Youquan Song wrote:
> > +#ifdef CONFIG_SMP
> > + /* return when cpu number greater than maximum number of
> > CPUs */
> > + if (setup_max_cpus <= num_online_cpus() + 1) {
> > +
t fix
> > in
> > linux-next.
>
> I believe the problem was introduced by the following change. From the
> description, though, this is exactly what this patch was trying to
> change... Adding Youguan to the list.
>
> commit 3e275a5ba367ab74b3a4e49114307baed989fcac
sys/devices/system/cpu/online
Signed-off-by: Youquan Song
---
drivers/base/cpu.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c
index 3d48fc8..c7d603a 100644
--- a/drivers/base/cpu.c
+++ b/drivers/base/cpu.c
@@ -272,6 +272,10 @@
>
> Can you look at the patch which required by some Haswell platforms?
>
Hi Jeff,
What's your opinion about the patch? It block the installation on some
new platforms.
Thanks
-Youquan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord
argument 2 has type â__u64â
make: *** [util/scripting-engines/trace-event-perl.o] Error 1
Signed-off-by: Youquan Song
---
.../perf/util/scripting-engines/trace-event-perl.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/perf/util/scripting-engines/trace-event-perl.c
>
> I found this patch after some googling and for the record, it makes my
> W520 boot with VT-d enabled and the discrete NVidia card.
> Is it still being considered?
>
Yes. I am still in pushing the patch to upstream. The patch is good and
reviewed
by Yinghai but it depends on Yinghai's patch
Hi Maintainer,
Can you look at the patch which required by some Haswell platforms?
Thanks
-Youquan
On Wed, Mar 06, 2013 at 10:49:05AM -0500, Youquan Song wrote:
> There is a quirk patch 5e5a4f5d5a08c9c504fe956391ac3dae2c66556d
> "ata_piix: make DVD Drive recognisable on systems
e patch against 3.8-rc7 kernel.
Tested-by: Lee, Chun-Yi
Signed-off-by: Youquan Song
Cc: sta...@vger.kernel.org
---
drivers/ata/ata_piix.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index 174eca6..4aab5
Hi Maintainer,
Can you take the patch which is needed by some new platforms?
Thanks
-Youquan
On Mon, Feb 18, 2013 at 11:00:55AM -0500, Youquan Song wrote:
> There is a quirk patch 5e5a4f5d5a08c9c504fe956391ac3dae2c66556d
> "ata_piix: make DVD Drive recognisable on systems
>
> As to my understanding Sergei did not suggest citing the whole commit message.
> I also find the numerous references to Sandy Bridge confusing as this is a fix
> for Lynx Point chipset.
>
> How about rephrasing the commit message in a way similar to the following one?
> --8<-
> We've
e patch against 3.8-rc7 kernel.
Tested-by: Lee, Chun-Yi
Signed-off-by: Youquan Song
Cc: sta...@vger.kernel.org
---
drivers/ata/ata_piix.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index 174eca6..4aab5
>> +{ 0x8086, 0x8c08, PCI_ANY_ID, PCI_ANY_ID, 0, 0, ich8_2port_sata_snb },
>> /* SATA Controller IDE (Lynx Point) */
>> { 0x8086, 0x8c09, PCI_ANY_ID, PCI_ANY_ID, 0, 0, ich8_2port_sata },
>
>Also, are you sure this one and the following Lynx Point controllers are
> not affected?
> On 30-01-2013 21:19, Youquan Song wrote:
>
>> There is a quirk patch 5e5a4f5d5a08c9c504fe956391ac3dae2c66556d
>
> Please also specify the summary of that patch in parens.
>
>> fix the 4 ports
>
>s/fix/fixing/
>
>> IDE controller 32bit PIO mode.
&g
by Sergei Shtylyov.
Tested-by: Lee, Chun-Yi
Signed-off-by: Youquan Song
Cc: sta...@vger.kernel.org
---
drivers/ata/ata_piix.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index ef773e1..1993e52 100644
--- a/d
: Youquan Song
---
drivers/ata/ata_piix.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index ef773e1..1993e52 100644
--- a/drivers/ata/ata_piix.c
+++ b/drivers/ata/ata_piix.c
@@ -150,6 +150,7 @@ enum
: Youquan Song
---
drivers/ata/ata_piix.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index ef773e1..1993e52 100644
--- a/drivers/ata/ata_piix.c
+++ b/drivers/ata/ata_piix.c
@@ -150,6 +150,7 @@ enum
Commit-ID: 923d8697e24847000490c187de1aeaca622611a3
Gitweb: http://git.kernel.org/tip/923d8697e24847000490c187de1aeaca622611a3
Author: Youquan Song
AuthorDate: Tue, 18 Dec 2012 12:20:23 -0500
Committer: Ingo Molnar
CommitDate: Thu, 24 Jan 2013 16:14:04 +0100
x86/perf: Add IvyBridge EP
On Tue, Dec 18, 2012 at 09:42:30AM -0800, Yinghai Lu wrote:
> On Tue, Dec 18, 2012 at 9:33 AM, H. Peter Anvin wrote:
> > On 12/18/2012 09:07 AM, Youquan Song wrote:
> >> Blacklist x2apic when Nivida graphics enabled on Lenovo ThinkPad T420.
> >> Also set blacklist x2ap
Would you like to take it? It is needed by Linux OSVs.
Thanks
-Youquan
On Tue, Dec 18, 2012 at 12:20:23PM -0500, Youquan Song wrote:
> Run in perf utility at Ivybridge EP server, encouter "not supported" event
>
> L1-dcache-loads
> L1-dcache-load-mis
When debug kernel, the the below information is found:
intel_idle: unaware of model 0x1a MWAIT 4 please contact lenb@kernel.orgACPI:
Device input0 -> No ACPI support
so this patch separates it.
Signed-off-by: Youquan Song
---
drivers/idle/intel_idle.c |2 +-
1 files changed, 1 inserti
processor.
Reviewed-by: Andi Kleen
Signed-off-by: Youquan Song
---
arch/x86/kernel/cpu/perf_event_intel.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/x86/kernel/cpu/perf_event_intel.c
b/arch/x86/kernel/cpu/perf_event_intel.c
index 324bb52..aea3503 100644
---
/bugs/922037
The patches is based on http://git.kernel.org/?p=linux/kernel/git/yinghai/
linux-yinghai.git;a=patch;h=de38757e964cfee20e6da1977572a2191d7f4aa0
Reviewed-by: Yinghai Lu
Signed-off-by: Youquan Song
---
arch/x86/include/asm/x86_init.h |1 +
arch/x86/kernel/apic/apic.c | 51
t mode
is kept, so it will choose the C1 shallow C-state even though it has chance to
sleep 1 second in deep C-state.
While after patched the kernel, we find that deep C-state will keep >99.6%.
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/governors/
-evaluates deeper C-states possibility.
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/governors/menu.c | 34 +-
1 files changed, 33 insertions(+), 1 deletions(-)
diff --git a/drivers/cpuidle/governors/menu.c b/drivers/cpuidle/governors
such a series, while the
new code will guess a typical sleep interval just shy of 200.
The original patch come from Rik van Riel .
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/governors/menu.c | 69 +
1 files changed, 46
V2: Add menu timer status enums depends on Rafael suggestion.
The prediction for future is difficult and when the cpuidle governor prediction
fails and govenor possibly choose the shallower C-state than it should. How to
quickly notice and find the failure becomes important for power saving.
, it is not reasonable.
So, this patch fix it by set the target C-state residency to 0.
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/cpuidle.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle
-evaluates deeper C-states possibility.
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/governors/menu.c | 22 ++
1 files changed, 22 insertions(+), 0 deletions(-)
diff --git a/drivers/cpuidle/governors/menu.c b/drivers/cpuidle/governors/menu.c
index
such a series, while the
new code will guess a typical sleep interval just shy of 200.
The original patch come from Rik van Riel .
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/governors/menu.c | 69 +
1 files changed, 46
, it is not reasonable.
So, this patch fix it by set the target C-state residency to 0.
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/cpuidle.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle
In general case, the expected residency is much larger than deepest C-state
target residency, but prediction logic still predicts the small predicted
residency, so the prediction history is totally broken. In this situation,
reset the correction factor is the only choice.
Signed-off-by: Youquan
t mode
is kept, so it will choose the C1 shallow C-state even though it has chance to
sleep 1 second in deep C-state.
While after patched the kernel, we find that deep C-state will keep >99.6%.
Signed-off-by: Youquan Song
Signed-off-by: Rik van Riel
---
drivers/cpuidle/governors/
The prediction for future is difficult and when the cpuidle governor prediction
fails and govenor possibly choose the shallower C-state than it should. How to
quickly notice and find the failure becomes important for power saving.
cpuidle menu governor has a method to predict the repeat pa
> > One case is turbostat utility (tools/power/x86/turbostat) at kernel 3.3 or
> > early
> > . turbostat utility will read 10 registers one by one at Sandybridge, so it
> > will
> > generate 10 IPIs to wake up idle CPUs. So cpuidle menu governor will
> > predict it
> > is repeat mode and there
> Could I convince you to try out my variation on
> detect_repeating_intervals? :)
>
> http://people.redhat.com/riel/cstate/cstate-stddev-converge.patch
>
> I suspect that small change might help your code adapt to changed
> conditions even faster.
Yes. of course. your patch of cstate-stddev-conve
, it is not reasonable.
So, this patch fix it by set the target C-state residency to 0.
Signed-off-by: Youquan Song
---
drivers/cpuidle/cpuidle.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c
index 2f0083a
initiatively. When the timer is triggered and menu governor will quickly notice
prediction failure and re-evaluates deeper C-states possibility.
Signed-off-by: Youquan Song
---
drivers/cpuidle/governors/menu.c | 48 ++
1 files changed, 33 insertions(+), 15
t mode
is kept, so it will choose the C1 shallow C-state even though it has chance to
sleep 1 second in deep C-state.
While after patched the kernel, we find that deep C-state will keep >99.6%.
Signed-off-by: Youquan Song
---
diff --git a/drivers/cpuidle/governors/menu.c b/driver
The prediction for future is difficult and when the cpuidle governor prediction
fails and govenor possibly choose the shallower C-state than it should. How to
quickly notice and find the failure becomes important for power saving.
cpuidle menu governor has a method to predict the repeat pat
> Your patches could make a lot of sense when integrated with my
> patches:
>
> http://people.redhat.com/riel/cstate/
> However, we should probably get the tracepoint upstream first,
> so we can know for sure :)
I can not access the patches at this directory. Can you send it to me?
I will look at
> After talking about my RFC patches to the c-state governor with
> Matthew and Arjan, it is clear that the whole concept of how
> things are done could use some more discussion.
>
> Since a good number of us will be in San Diego next week, at
> Kernel Summit / Plumbers / etc, I will organize a c-s
66 matches
Mail list logo
