On Fri, Apr 19, 2024 at 2:07 PM David Matlack wrote:
>
> On 2024-04-19 01:47 PM, James Houghton wrote:
> > On Thu, Apr 11, 2024 at 10:28 AM David Matlack wrote:
> > > On 2024-04-11 10:08 AM, David Matlack wrote:
> > > bool kvm_age_gfn(struct kvm *kv
On Fri, Apr 12, 2024 at 11:41 AM David Matlack wrote:
>
> On 2024-04-01 11:29 PM, James Houghton wrote:
> > This patchset adds a fast path in KVM to test and clear access bits on
> > sptes without taking the mmu_lock. It also adds support for using a
> > bitmap to (1)
On Fri, Apr 12, 2024 at 1:44 PM David Matlack wrote:
>
> On 2024-04-01 11:29 PM, James Houghton wrote:
> > Only handle the TDP MMU case for now. In other cases, if a bitmap was
> > not provided, fallback to the slowpath that takes mmu_lock, or, if a
> > bitmap was provided
On Thu, Apr 11, 2024 at 10:28 AM David Matlack wrote:
>
> On 2024-04-11 10:08 AM, David Matlack wrote:
> > On 2024-04-01 11:29 PM, James Houghton wrote:
> > > Only handle the TDP MMU case for now. In other cases, if a bitmap was
> > > not provided, fallback to the sl
On Fri, Apr 12, 2024 at 1:28 PM David Matlack wrote:
>
> On 2024-04-01 11:29 PM, James Houghton wrote:
> > Add kvm_arch_prepare_bitmap_age() for architectures to indiciate that
> > they support bitmap-based aging in kvm_mmu_notifier_test_clear_young()
> > and that they do
On Fri, Apr 12, 2024 at 11:45 AM David Matlack wrote:
>
> On 2024-04-01 11:29 PM, James Houghton wrote:
> > The bitmap is provided for secondary MMUs to use if they support it. For
> > test_young(), after it returns, the bitmap represents the pages that
> > were young in
On Tue, Apr 9, 2024 at 12:35 PM David Hildenbrand wrote:
>
> On 09.04.24 20:31, James Houghton wrote:
> > Ah, I didn't see this in my inbox, sorry David!
>
> No worries :)
>
> >
> > On Thu, Apr 4, 2024 at 11:52 AM David Hildenbrand wrote:
> >&g
Ah, I didn't see this in my inbox, sorry David!
On Thu, Apr 4, 2024 at 11:52 AM David Hildenbrand wrote:
>
> On 02.04.24 01:29, James Houghton wrote:
> > diff --git a/include/linux/mmu_notifier.h b/include/linux/mmu_notifier.h
> > index f349e08a9dfe..daaa9db625d3 100644
m: rmap: Don't flush TLB after checking PTE young for page
reference" was included so that the comparison is apples to
apples.
https://lore.kernel.org/r/20220706112041.3831-1-21cn...@gmail.com/
Signed-off-by: Yu Zhao
Signed-off-by: James Houghton
---
Documentation/admin-g
ed-in bitmap.
When mkold == true, only age the pages that need aging according to the
passed-in bitmap.
Suggested-by: Yu Zhao
Signed-off-by: James Houghton
---
arch/arm64/include/asm/kvm_host.h| 5 +
arch/arm64/include/asm/kvm_pgtable.h | 4 +++-
arch/arm64/kvm/hyp/pgtable.c
Only handle the TDP MMU case for now. In other cases, if a bitmap was
not provided, fallback to the slowpath that takes mmu_lock, or, if a
bitmap was provided, inform the caller that the bitmap is unreliable.
Suggested-by: Yu Zhao
Signed-off-by: James Houghton
---
arch/x86/include/asm
From: Yu Zhao
tdp_mmu_enabled and shadow_accessed_mask are needed to implement
kvm_arch_prepare_bitmap_age().
Signed-off-by: Yu Zhao
Signed-off-by: James Houghton
---
arch/x86/include/asm/kvm_host.h | 6 ++
arch/x86/kvm/mmu.h | 6 --
arch/x86/kvm/mmu/spte.h | 1
d_age().
Suggested-by: Yu Zhao
Signed-off-by: James Houghton
---
include/linux/kvm_host.h | 60 ++
virt/kvm/kvm_main.c | 92 +---
2 files changed, 127 insertions(+), 25 deletions(-)
diff --git a/include/linux/kvm_host.h b/include/
[1]: https://lore.kernel.org/all/20230609005935.42390-1-yuz...@google.com/
Suggested-by: Yu Zhao
Signed-off-by: James Houghton
---
include/linux/mmu_notifier.h | 93 +---
include/trace/events/kvm.h | 13 +++--
mm/mmu_notifier.c| 20 +---
virt/kvm/
To allow new MMU-notifier-related functions to use gfn_to_hva_memslot(),
move some declarations around.
Also move mmu_notifier_to_kvm() for wider use later.
Signed-off-by: James Houghton
---
include/linux/kvm_host.h | 41 +---
virt/kvm/kvm_main.c | 5
https://lore.kernel.org/kvmarm/20230526234435.662652-1-yuz...@google.com/
[4]: https://lore.kernel.org/all/zitx64bbx5vdj...@google.com/
James Houghton (5):
mm: Add a bitmap into mmu_notifier_{clear,test}_young
KVM: Move MMU notifier function declarations
KVM: Add basic bitmap support into kvm
ob up to passed the consumed tokens, so it actually reads
datablob="keyhandle=0x8101"
However, to do that you'd have to have the updated pointer passed out
of your datablob_parse() above.
There's also a lost !tpm2 in the check for options->keyhandle, but I
suspect Jarkko
On 16/04/2021 18:16, Arnaldo Carvalho de Melo wrote:
> Em Fri, Apr 16, 2021 at 09:07:09AM -0600, Mathieu Poirier escreveu:
>> Hi James,
>>
>> On Fri, Apr 16, 2021 at 01:56:30PM +0300, James Clark wrote:
>>> Changes since v1:
>>> * Improved variable
+++---
3 files changed, 22 insertions(+), 31 deletions(-)
With full diff below.
James
---
diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c
index 7ad11e42306d..bfd2aaa9b66b 100644
--- a/drivers/scsi/libiscsi.c
+++ b/drivers/scsi/libiscsi.c
@@ -3179,9 +3179,10
On 15/04/2021 18:23, Leo Yan wrote:
> On Thu, Apr 15, 2021 at 05:46:31PM +0300, James Clark wrote:
>>
>>
>> On 12/04/2021 12:10, Leo Yan wrote:
>>> In current code, it assigns the arch timer counter to the synthesized
>>> samples Arm SPE trace, thus th
refers to
sample kernel timestamps, and the /timestamp/ event modifier
refers to CS timestamps, so the term is overloaded.
Signed-off-by: James Clark
---
.../perf/util/cs-etm-decoder/cs-etm-decoder.c | 18
tools/perf/util/cs-etm.c | 42 +--
tools/perf
tm__process_queues().
Co-developed-by: Al Grant
Signed-off-by: Al Grant
Signed-off-by: James Clark
---
tools/perf/util/cs-etm.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
index 533f6f2f0685..e5c1a1b22a2a 100
Changes since v1:
* Improved variable name from etm_timestamp -> cs_timestamp
* Fixed ordering of Signed-off-by
James Clark (2):
perf cs-etm: Refactor timestamp variable names
perf cs-etm: Set time on synthesised samples to preserve ordering
.../perf/util/cs-etm-decoder/cs-etm-decode
On 15/04/2021 22:54, Mathieu Poirier wrote:
> On Wed, Apr 14, 2021 at 05:39:19PM +0300, James Clark wrote:
>> The following attribute is set when synthesising samples in
>> timed decoding mode:
>>
>> attr.sample_type |= PERF_SAMPLE_TIME;
>>
>> This res
On 15/04/2021 17:33, Leo Yan wrote:
> Hi James,
>
> On Thu, Apr 15, 2021 at 03:51:46PM +0300, James Clark wrote:
>
> [...]
>
>>> For the orignal perf data file with "--per-thread" option, the decoder
>>> runs into the condition for "etm-
was for cap_task_setnice() instead
>
> Signed-off-by: Randy Dunlap
> Cc: Serge Hallyn
> Cc: James Morris
> Cc: linux-security-mod...@vger.kernel.org
Applied to
git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git
fixes-v5.12
--
James Morris
On 15/04/2021 17:41, Leo Yan wrote:
> Hi James,
>
> On Thu, Apr 15, 2021 at 05:13:36PM +0300, James Clark wrote:
>> On 12/04/2021 12:10, Leo Yan wrote:
>>> The enum value 'ARM_SPE_PER_CPU_MMAPS' is never used so remove it.
>>
>> Hi Leo,
>>
On 12/04/2021 12:10, Leo Yan wrote:
> In current code, it assigns the arch timer counter to the synthesized
> samples Arm SPE trace, thus the samples don't contain the kernel time
> but only contain the raw counter value.
>
> To fix the issue, this patch converts the timer counter to kernel tim
0x8]: failed to process type: 9 [Bad address]
Error:
failed to process sample
Do you have any test suggestions other than looking at the raw data?
Thanks
James
On 12/04/2021 12:10, Leo Yan wrote:
> This patch set is to enable timestamp for Arm SPE trace. It reads out
o process type: 70 [Invalid argument]
Error:
failed to process sample
# To display the perf.data header info, please use
--header/--header-only options.
#
James
>
> Signed-off-by: Leo Yan
> ---
> tools/perf/util/arm-spe.h | 1 -
> 1 file changed, 1 deleti
On 15/04/2021 15:39, Leo Yan wrote:
> On Wed, Apr 14, 2021 at 05:41:46PM +0300, James Clark wrote:
>> Hi,
>>
>> For this change, I also tried removing the setting of PERF_SAMPLE_TIME in
>> cs_etm__synth_events(). In theory, this would remove the sorting when
&
or maybe just checking the options, although that's not how it's done
in cs_etm__is_timeless_decoding() currently).
Or, we could force /time/ and /timestamp/ options to always be enabled together
in the record stage.
Thanks
James
On 14/04/2021 17:39, James Clark wrote:
> The follow
to cs_etm__process_queues().
Signed-off-by: James Clark
Co-developed-by: Al Grant
Signed-off-by: Al Grant
---
tools/perf/util/cs-etm.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
index c25da2ffa8f3..d0fa9dce47f1 100
refers to
sample kernel timestamps, and the /timestamp/ event modifier
refers to etm timestamps, so the term is overloaded.
Signed-off-by: James Clark
---
.../perf/util/cs-etm-decoder/cs-etm-decoder.c | 18
tools/perf/util/cs-etm.c | 42 +--
tools
s the macro
likely needs a new name.
However, perhaps just do
array_size(nseg_new - 1, sizeof(*sge));
And lose the comment?
James
should have been a simple cut and paste ... how did that happen?
And therefore, how safe is the rest of the extraction into
trusted_core.c?
James
8001_hwi.c| 8 ++--
drivers/scsi/scsi_transport_srp.c | 2 +-
drivers/scsi/ufs/ufshcd.c | 31 +++
drivers/target/iscsi/iscsi_target.c | 3 +-
5 files changed, 65 insertions(+), 57 deletions(-)
With full diff below.
James
---
diff --git a/drivers/scsi/hpsa_cmd.h b/dr
I've added this to my tree at:
git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git
landlock_lsm_v33
and merged that into the next-testing branch which is pulled into Linux
next.
--
James Morris
On Thu, 2021-04-08 at 17:41 -0700, Steve Rutherford wrote:
> On Thu, Apr 8, 2021 at 2:15 PM James Bottomley
> wrote:
> > On Thu, 2021-04-08 at 12:48 -0700, Steve Rutherford wrote:
> > > On Thu, Apr 8, 2021 at 10:43 AM James Bottomley <
> > > j...@linux.ibm.com&g
On Thu, 2021-04-08 at 12:48 -0700, Steve Rutherford wrote:
> On Thu, Apr 8, 2021 at 10:43 AM James Bottomley
> wrote:
> > On Fri, 2021-04-02 at 16:20 +0200, Paolo Bonzini wrote:
> > > On 02/04/21 13:58, Ashish Kalra wrote:
> > > > Hi Nathan,
> > > >
ption" with "flag" in hook_sb_mount() comment.
Good to see these changes.
Al: any further comments now on this patch?
--
James Morris
Google is unwilling to publish the
patches, can you at least give some hints about how you did it?
Thanks,
James
Hi Reinette,
On 07/04/2021 00:42, Reinette Chatre wrote:
> On 4/6/2021 10:13 AM, James Morse wrote:
>> On 31/03/2021 22:35, Reinette Chatre wrote:
>>> On 3/12/2021 9:58 AM, James Morse wrote:
>>>> resctrl is the defacto Linux ABI for SoC resource partitioning f
Hi Reinette,
On 31/03/2021 22:36, Reinette Chatre wrote:
> On 3/12/2021 9:58 AM, James Morse wrote:
>> resctrl is the defacto Linux ABI for SoC resource partitioning features.
>> To support it on another architecture, it needs to be abstracted from
>> the features provided
Hi Babu,
On 06/04/2021 22:37, Babu Moger wrote:
> On 4/6/21 12:19 PM, James Morse wrote:
>> On 30/03/2021 21:36, Babu Moger wrote:
>>> On 3/12/21 11:58 AM, James Morse wrote:
>>>> This series re-folds the resctrl code so the CDP resources (L3CODE et al)
>>
Hi Babu,
On 30/03/2021 21:36, Babu Moger wrote:
> On 3/12/21 11:58 AM, James Morse wrote:
>> This series re-folds the resctrl code so the CDP resources (L3CODE et al)
>> behaviour is all contained in the filesystem parts, with a minimum amount
>> of arch specific code.
&g
Hi Reinette,
On 31/03/2021 22:35, Reinette Chatre wrote:
> On 3/12/2021 9:58 AM, James Morse wrote:
>> resctrl is the defacto Linux ABI for SoC resource partitioning features.
>> To support it on another architecture, it needs to be abstracted from
>> the features provided
nt = foo;
> > + struct integrity_iint_cache *iint = (struct integrity_iint_cache
> > *)foo;
> >
> >memset(iint, 0, sizeof(*iint));
> >iint->ima_file_status = INTEGRITY_UNKNOWN;
>
--
James Morris
On Tue, 2021-02-09 at 11:12 -0600, Eddie James wrote:
> In the event that the OCC is not initialized when the driver sends a
> poll
> command, the driver may receive an invalid response. This isn't an
> error
> condition unless there is no valid response before the timeout
&g
the diffstat:
drivers/scsi/scsi_transport_iscsi.c | 14 +-
include/scsi/scsi_transport_iscsi.h | 1 +
2 files changed, 14 insertions(+), 1 deletion(-)
With full diff below.
James
---
diff --git a/drivers/scsi/scsi_transport_iscsi.c
b/drivers/scsi/scsi_transport_iscsi.c
index
this approach is one can then put the build
folder on a ram disk / tmpfs and be able to compile and test much
quicker.
Kind Regards
James
On Thu, 2021-04-01 at 18:50 +0530, Sumit Garg wrote:
> On Thu, 1 Apr 2021 at 15:36, Ahmad Fatoum
> wrote:
> > Hello Richard,
> >
> > On 31.03.21 21:36, Richard Weinberger wrote:
> > > James,
> > >
> > > - Ursprüngliche Mail -
> >
On Wed, 2021-03-31 at 20:36 +0200, Richard Weinberger wrote:
> James,
>
> - Ursprüngliche Mail -
> > Von: "James Bottomley"
> > > On Wed, Mar 17, 2021 at 3:08 PM Ahmad Fatoum <
> > > a.fat...@pengutronix.de wrote:
> > > >
re not they need to be made so before the patches go upstream.
I could possibly see that you might want to be type specific in the
create, but once you're simply loading an already created key, the
trusted key subsystem should be able to figure what to do on its own.
James
x27;t restart, as the locking is broken somehow.
Fixes: 0795fb2021f0 ("net/ncsi: Stop monitor if channel times out or is
inactive")
Signed-off-by: Milton Miller
Signed-off-by: Eddie James
---
net/ncsi/ncsi-manage.c | 20 +---
1 file changed, 13 insertions(+), 7 deletions
The error bits in the FSI2PIB status are only cleared by a reset. So
the driver needs to perform a reset after seeing any of the FSI2PIB
errors, otherwise subsequent operations will also look like failures.
Fixes: 6b293258cded ("fsi: scom: Major overhaul")
Signed-off-by: Eddie James
--
hanged, 74 insertions(+), 28 deletions(-)
With full diff below.
James
---
diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c
index 6a92891ac488..bb64e3247a6c 100644
--- a/drivers/scsi/ibmvscsi/ibmvfc.c
+++ b/drivers/scsi/ibmvscsi/ibmvfc.c
@@ -2371,6 +2371,24 @@
ll of the registers is too much, so we
should continue with the change
to only record the link register. In my opinion a 5% - 10% increase by enabling
LR recording by default wouldn't
be noticeable, so it's worth doing. But obviously we'll need other opinions
On Thu, 2021-03-25 at 13:50 +, Mark Brown wrote:
> On Wed, Mar 24, 2021 at 05:05:16PM -0500, Eddie James wrote:
>
> > Updated restricted chips have trouble processing multiple sequenced
> > operations. So remove the capability to sequence multiple
> > operations an
Updated restricted chips have trouble processing multiple sequenced
operations. So remove the capability to sequence multiple operations and
reduce the maximum transfer size to 8 bytes.
Signed-off-by: Eddie James
---
drivers/spi/spi-fsi.c | 27 +++
1 file changed, 7
On Wed, 2021-03-24 at 16:49 -0400, Mimi Zohar wrote:
> On Wed, 2021-03-24 at 09:14 -0700, James Bottomley wrote:
> > On Tue, 2021-03-23 at 14:07 -0400, Mimi Zohar wrote:
> > > On Tue, 2021-03-23 at 17:35 +0100, Ahmad Fatoum wrote:
> > > > Hello Horia,
> > &
s.
>
> Add a separate NULL check to tell gcc about it as well.
>
> Signed-off-by: Arnd Bergmann
Applied to
git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git
fixes-v5.12
--
James Morris
t? The code is in the repo and discussions can
easily be found by searching for subjects or message IDs.
Is anyone else doing this self linking?
--
James Morris
ost cryptograhpers recommend using a TPM for entropy mixing rather
than directly:
https://blog.cryptographyengineering.com/category/rngs/
The TPMFail paper also shows that in spite of NIST certification
things can go wrong with a TPM:
https://tpm.fail/
James
blic inbox. I assume the search can be
quite wide so we could flag a regression on any list indexed by lore?
James
report" and
the triage team can cc additional lists as appropriate. Then we simply
tell everyone to send kernel bugs to this list and ask maintainers to
cc it if a bug report shows up on their list?
James
ivers/scsi/st.c| 2 +-
drivers/scsi/ufs/ufs-mediatek.c | 2 +-
8 files changed, 20 insertions(+), 15 deletions(-)
With full diff below.
James
---
diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c
index 338e29d3e025..6a92891ac488 100644
--- a/driver
t merge window, but would really want
> distros to confirm that they don't enable it.
FWIW, it's been disabled in Ubuntu for over 10 years now:
https://wiki.ubuntu.com/Security/Features#A.2Fdev.2Fkmem_disabled
--
James
nd testing during that time will be helpful.
--
James Morris
> This commit adds a minimal set of supported filesystem access-control
> which doesn't enable to restrict all file-related actions.
It would be great to get some more review/acks on this patch, particularly
from VFS/FS folk.
--
James Morris
/ufs/ufs-qcom.c| 10 +++
drivers/scsi/ufs/ufshcd.c | 41 +---
drivers/scsi/vmw_pvscsi.c | 2 --
drivers/scsi/vmw_pvscsi.h | 2 --
6 files changed, 60 insertions(+), 59 deletions(-)
With full diff below.
James
---
diff --git a/MAINTAINERS b
iewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* rdt_get_cdp_config() is kept for its comment.
---
arch/x86/kernel/cpu/resctrl/core.c | 174 ++---
arch/x86/kernel/cpu/resctrl/internal.h | 4 -
arch/x86/kernel/cpu/resctrl/rdtgroup.c
configuration type
if cdp is enabled.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
Changes since v1:
* Expanded commit mesasge.
---
arch/x86/kernel/cpu/resctrl/rdtgroup.c | 99 --
1 file changed, 14 insertions(+), 85 deletions(-)
diff --git a/arch/x86/kernel/cpu/resctrl
omplicates
allocating and freeing the ctrl_val arrays. Add a helper to allocate
the ctrl_val array, that returns the value on the L2 or L3 resource
if it already exists. This gets removed once the resources are merged,
and there really is only one ctrl_val array.
Reviewed-by: Jamie Iles
Signed-off
on the staged configuration type. This makes the
static mult and offset parameters redundant.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
arch/x86/kernel/cpu/resctrl/core.c| 12
arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 17 +++--
include/linux
changes it has made
to the ctrl_val array as this allows the hardware to be updated once for
a set of changes.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Removing the patch that moved the closid to the staged config means the
min/max and return from
this is that the caches are reset twice, once
for CODE once for DATA.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
arch/x86/kernel/cpu/resctrl/core.c | 10 +-
arch/x86/kernel/cpu/resctrl/rdtgroup.c | 9 +
2 files changed, 18 insertions(+), 1 deletion(-)
diff --git a
index in the configuration array.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 5 ++--
arch/x86/kernel/cpu/resctrl/monitor.c | 2 +-
arch/x86/kernel/cpu/resctrl/rdtgroup.c| 35 +++
include/linux/resctrl.h
r one resource means it is enable
for all resources, as all resources would need to configure both
interpretations of the closid value.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Added prototype for resctrl_arch_set_cdp_enabled()
* s/Currently/Previ
n the struct need to be
specified per type configuration, and there can be one configuration of
each type staged.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Expanded commit message
* Removed explicit clearing of have_new_ctrl,
* Moved ARRAY_SIZE() trickery to a la
rdt_domain_reconfigure_cdp() infers whether CDP is enabled by
checking the alloc_capable and alloc_enabled flags of the data
resources.
Now that we have an explicit cdp_enabled, use that.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
arch/x86/kernel/cpu/resctrl/rdtgroup.c | 8
staged_configuration array must be
used for each schema. Use the conf_type enum directly as an index.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Renamed max enum value CDP_NUM_TYPES
* Whitespace and parenthesis
* Missing word in the commit message
---
arch/x86
update_domains() merges the staged configuration changes into the
arch codes configuration array. Rename to make it clear its part of the
arch code interface to resctrl.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* The closid is no longer staged as from resctrl
;t take the user-space
control format at all.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 16 ++---
arch/x86/kernel/cpu/resctrl/monitor.c | 6 +++-
arch/x86/kernel/cpu/resctrl/rdtgroup.c| 43 ++-
include/linux/re
es
Signed-off-by: James Morse
---
arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 4 ++--
arch/x86/kernel/cpu/resctrl/internal.h| 6 +++---
arch/x86/kernel/cpu/resctrl/pseudo_lock.c | 8
arch/x86/kernel/cpu/resctrl/rdtgroup.c| 4 ++--
4 files changed, 11 insertions(+), 11 deletion
-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* split from a larger patch
---
arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 23 +--
arch/x86/kernel/cpu/resctrl/internal.h| 6 +++---
arch/x86/kernel/cpu/resctrl/rdtgroup.c| 19
will generate two names
for one resource.
The filesystem code should calculate max_name_width for padding the
schemata file, move this to live with the code that will generate
the names.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1;
* Don't hardcode max_name_
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Rewrote commit message
* Whitespace fixes
* num_closid becomes u32 in all occurences to reduce surprises
---
arch/x86/kernel/cpu/resctrl/core.c | 5 +
arch/x86/kernel/cpu/resctrl/internal.h | 2 +-
arch/x86/kernel/cp
thing' helper.
After the filesystem code is moved, rdtgroup_pseudo_locked_in_hierarchy()
remains part of the x86 specific hooks to support psuedo lock. This code
walks each domain, and still does this after the separate resources are
merged.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
-
d
the name and type of configuration values for resctrl.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Renamed resctrl_all_schema list
* Used schemata_list as a prefix to make these easier to search for
* Added kerneldoc string
* Removed 'pending configuratio
e are changed to use the schema value instead.
schemata_list_create() sets this value, and reaches into the architecture
specific structure to get the value. This will eventually be replaced with
a helper.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Added missing :
to
struct resctrl_schema.
Copying the value ensures there is no mismatch, but allows the filesystem
parts of resctrl to be modified to use the schema. Once the resources are
merged, the filesystem code can assign this value based on the schema
being created.
Reviewed-by: Jamie Iles
Signed-off-by: J
oo.
This difference means the type of the private pointers varies
between control and monitor info dirs.
If the flags are RF_MON_INFO, its a struct rdt_resource. If the
flags are RF_CTRL_INFO, its a struct resctrl_schema. Nothing in
res_common_files[] has both flags.
Reviewed-by: Jamie Iles
Signed
into a new header file. resctrl code paths touching a 'hw'
struct indicates where an abstraction is needed.
Splitting this structure only moves types around, and should not lead
to any change in behaviour.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Tabs
cture only moves types around, and should not lead
to any change in behaviour.
Splitting rdt_domain up in a similar way happens in the next patch.
Reviewed-by: Jamie Iles
Signed-off-by: James Morse
---
Changes since v1:
* Tabs space and other whitespace
* Restored for_each_rdt_resource() calls in
947.39194-1-james.mo...@arm.com/
Thanks,
James Morse (24):
x86/resctrl: Split struct rdt_resource
x86/resctrl: Split struct rdt_domain
x86/resctrl: Add a separate schema list for resctrl
x86/resctrl: Pass the schema in info dir's private pointer
x86/resctrl: Label the resources wit
Hi Reinette,
On 17/11/2020 23:46, Reinette Chatre wrote:
> On 10/30/2020 9:11 AM, James Morse wrote:
>> Once the L2/L2CODE/L2DATA resources are merged, there may be two
>> configurations staged for one resource when CDP is enabled. The
>> closid should always be pa
Hi Reinette,
On 17/11/2020 23:28, Reinette Chatre wrote:
> On 10/30/2020 9:11 AM, James Morse wrote:
>> Arm's MPAM may have surprisingly large bitmaps for its cache
>> portions as the architecture allows up to 4K portions. The size
>> exposed via resctrl may not be
Hi Reinette,
On 17/11/2020 23:11, Reinette Chatre wrote:
> On 10/30/2020 9:11 AM, James Morse wrote:
>> Move the names used for the schemata file out of the resource and
>> into struct resctrl_schema. This allows one resource to have two
>> different names, based on the ot
Hi Reinette,
On 17/11/2020 22:52, Reinette Chatre wrote:
> On 10/30/2020 9:11 AM, James Morse wrote:
>> Now that resctrl has its own list of resources it is using, walk that
>> list instead of the architectures list. This means resctrl has somewhere
>> to keep schema propert
1 - 100 of 5520 matches
Mail list logo