[gem5-dev] Change in public/gem5[master]: config, arm: Add multi-core KVM support to bL config

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2561 ) Change subject: config, arm: Add multi-core KVM support to bL config .. config, arm: Add multi-core KVM support to bL

[gem5-dev] Change in public/gem5[master]: arm, kvm: Override the kernel's default MPIDR value

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2461 ) Change subject: arm, kvm: Override the kernel's default MPIDR value .. arm, kvm: Override the kernel's default MPIDR

[gem5-dev] Change in public/gem5[master]: config, arm: Unify checkpoint path handling in bL configs

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2560 ) Change subject: config, arm: Unify checkpoint path handling in bL configs .. config, arm: Unify checkpoint path

[gem5-dev] Change in public/gem5[master]: dev, arm: Fix multi-core KVM race in the generic timer

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2460 ) Change subject: dev, arm: Fix multi-core KVM race in the generic timer .. dev, arm: Fix multi-core KVM race

[gem5-dev] Change in public/gem5[master]: dev: Add a dummy VirtIO device

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2325 ) Change subject: dev: Add a dummy VirtIO device .. dev: Add a dummy VirtIO device VirtIO transport interfaces always

[gem5-dev] Change in public/gem5[master]: dev: Align BAR0 size to power of 2 for VirtIO devices

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2580 ) Change subject: dev: Align BAR0 size to power of 2 for VirtIO devices .. dev: Align BAR0 size to power of 2

[gem5-dev] Change in public/gem5[master]: dev: Rename VirtIO PCI debug flag

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2324 ) Change subject: dev: Rename VirtIO PCI debug flag .. dev: Rename VirtIO PCI debug flag Rename VIOPci -> VIOIf

[gem5-dev] Change in public/gem5[master]: dev: Add an InterruptLine class to wrap plain interrupts

). This change introduces an abstract interrupt adaptor class that can be implemented by platforms that need to support such devices. Change-Id: I181d952f17958549f25883e8aed4b00681550158 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.c

[gem5-dev] Change in public/gem5[master]: dev: Add a MMIO transport interface for VirtIO

-coded to VirtQueue::ALIGN_SIZE (4096 bytes). * Only 4096 byte pages are currently supported. Change-Id: Ifd318f5e5bddab0b6a42d8c8af9ff2fbb477f98b Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> Reviewed-by: Re

[gem5-dev] Change in public/gem5[master]: dev, arm: Add a GIC-based interrupt adaptor implementations

ble to use the same device model to generate both PPIs and SPIs (e.g., the PMU). Change-Id: I73d6591c168040faef2443430c4f1da10c387a2a Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> --- M src/dev/arm/Gic.py M src/dev/arm/bas

[gem5-dev] PyBind11 as a SWIG replacement

Hi Everyone, Some time ago, I posted a series of changes [1] to switch from SWIG to PyBind for our Python wrappers. I'd like to know what people think about this move at a high level. Do we want to make this change? These are the main benefits I have identified: * PyBind11 can be

[gem5-dev] Interrupt adaptor for plain devices

Hi Everyone, I have a series of changes that add support for VirtIO over a memory-mapped device [1]. To avoid making the device ARM-specific, I needed to add an interrupt adaptor [2] that interfaces between an interrupt source and an interrupt controller. PCI devices use a platform-specific host

[gem5-dev] Change in public/gem5[master]: sim: Handle cases where Drainable::resume() creates objects

objects have been resumed, the simulator enters the Running state. Change-Id: Ieee8645351ffbdec477e9cd2ff86fc795e459617 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> --- M src/sim/drain.cc M src/sim/drain.hh 2 files

[gem5-dev] Change in public/gem5[master]: dev: Align BAR0 size to power of 2 for VirtIO devices

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2580 Change subject: dev: Align BAR0 size to power of 2 for VirtIO devices .. dev: Align BAR0 size to power of 2 for VirtIO devices

[gem5-dev] Change in public/gem5[master]: config, arm: Add multi-core KVM support to bL config

automatically assigns separate event queues (threads) to each of the simulated CPUs. All simulated devices, including CPU child devices (e.g., interrupt controllers and caches), are assigned to event queue 0. Change-Id: Ic9a3f564db91f5a3d3cb754c5a02fdd5c17d5fdf Signed-off-by: Andreas Sandberg

[gem5-dev] Change in public/gem5[master]: config, arm: Unify checkpoint path handling in bL configs

keyword argument for checkpoint directories (only used by the dist-gem5 script). Change-Id: I9943ec10bd7a256465e29c8de571142ec3fbaa0e Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.b

[gem5-dev] Change in public/gem5[master]: dev, arm: Add a VirtIO MMIO device

.. dev, arm: Add a VirtIO MMIO device Add an ARM-specific VirtIO MMIO device to the VExpress_GEM5_V1 platform. Change-Id: Id1e75398e039aad9d637f46f653cda9084d3d2fe Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Sudhanshu Jha <sudhanshu@arm.com&

[gem5-dev] Change in public/gem5[master]: dev: Add a MMIO transport interface for VirtIO

to VirtQueue::ALIGN_SIZE (4096 bytes). * Only 4096 byte pages are currently supported. Change-Id: Ifd318f5e5bddab0b6a42d8c8af9ff2fbb477f98b Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> Reviewed-by: Rekai Gonzalez

[gem5-dev] Change in public/gem5[master]: dev, arm: Add a GIC-based interrupt adaptor implementations

both PPIs and SPIs (e.g., the PMU). Change-Id: I73d6591c168040faef2443430c4f1da10c387a2a Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> --- M src/dev/arm/Gic.py M src/dev/arm/base_gic.cc M src/dev/arm/base_gic.hh 3 files c

[gem5-dev] Change in public/gem5[master]: dev: Add an InterruptAdaptor class to wrap plain interrupts

be implemented by platforms that need to support such devices. Change-Id: I181d952f17958549f25883e8aed4b00681550158 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> --- M src/dev/Device.py M src/dev/io_device.cc M src/dev/io_devic

[gem5-dev] Change in public/gem5[master]: dev, arm: Add a VirtIO MMIO device

.. dev, arm: Add a VirtIO MMIO device Add an ARM-specific VirtIO MMIO device to the VExpress_GEM5_V1 platform. Change-Id: Id1e75398e039aad9d637f46f653cda9084d3d2fe Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Sudhanshu Jha <sudhanshu@arm.com&

[gem5-dev] Change in public/gem5[master]: arm: correct register read bug in Pl390 GIC

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2440 ) Change subject: arm: correct register read bug in Pl390 GIC .. arm: correct register read bug in Pl390 GIC Change-Id

[gem5-dev] Change in public/gem5[master]: arm, kvm: Override the kernel's default MPIDR value

been instantiated. Change-Id: I916d44978a9c855ab89c80a083af45b0cea6edac Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> --- M src/arch/arm/kvm/armv8_cpu.cc

[gem5-dev] Change in public/gem5[master]: dev, arm: Fix multi-core KVM race in the generic timer

the thread owning the CPU and not the global device thread. Change-Id: Ie3e982258648c8562cce0b30a0c122dfbfaf42cd Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Gabor Dozsa <gabor.do...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-b

[gem5-dev] Change in public/gem5[master]: python: Automatically disable listeners in batch setups

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2322 ) Change subject: python: Automatically disable listeners in batch setups .. python: Automatically disable listeners

[gem5-dev] Change in public/gem5[master]: util: Add a tool to list outgoing/incoming changes

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2329 ) Change subject: util: Add a tool to list outgoing/incoming changes .. util: Add a tool to list outgoing/incoming

Re: [gem5-dev] Unix domain sockets for gdb/m5term connections

Hi Gabe, Using UNIX sockets sounds like a good idea. We should probably use that by default when running in a batch environment (i.e., without a TTY). I have a change on Gerrit (python: Automatically disable listeners in batch setups) [1] that solves the same issue in a slightly different way.

[gem5-dev] Change in public/gem5[master]: tests: Warn not fail when reading invalid pickle status files

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2320 ) Change subject: tests: Warn not fail when reading invalid pickle status files .. tests: Warn not fail when reading

[gem5-dev] Change in public/gem5[master]: cpu: Print progress messages in Trace CPU

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2321 ) Change subject: cpu: Print progress messages in Trace CPU .. cpu: Print progress messages in Trace CPU This change

[gem5-dev] Change in public/gem5[master]: dev, arm: Add draining to the GIC model

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2331 ) Change subject: dev, arm: Add draining to the GIC model .. dev, arm: Add draining to the GIC model The GIC model

[gem5-dev] Change in public/gem5[master]: arm: Clean up the GIC implementation

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2330 ) Change subject: arm: Clean up the GIC implementation .. arm: Clean up the GIC implementation Lots of minor cleaups

[gem5-dev] Change in public/gem5[master]: sim: Implement load_addr_mask auto-calculation

specify the address mask, we use it instead of auto-calculating. Change-Id: I18aabb5d09945c6e3e3819c9c8036ea24b6c35cf Signed-off-by: Geoffrey Blake <geoffrey.bl...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com>

[gem5-dev] Change in public/gem5[master]: dev, arm: Add draining to the GIC model

Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> --- M src/dev/arm/gic_pl390.cc M src/dev/arm/gic_pl390.hh 2 files changed, 49 insertions(+), 21 deletions(-) diff --git a/src/dev/arm/gic_pl390.cc b/src/dev/arm/gic_pl390.cc i

[gem5-dev] Change in public/gem5[master]: arm: Clean up the GIC implementation

o view, visit https://gem5-review.googlesource.com/2330 To unsubscribe, visit https://gem5-review.googlesource.com/settings Gerrit-Project: public/gem5 Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I40a17bc3a38868fb3b8af247790e852cf99ddf1d Gerrit-Change-Number: 2330 Gerrit-PatchSet: 1 Gerrit-Owner: Andreas Sandberg <andreas.sandb...@arm.com> Gerrit-Reviewer: Curtis Dunham <curtis.dun...@arm.com> ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in public/gem5[master]: util: Add a tool to list outgoing/incoming changes

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2329 Change subject: util: Add a tool to list outgoing/incoming changes .. util: Add a tool to list outgoing/incoming changes Add

[gem5-dev] Change in public/gem5[master]: gpu-compute: Fix Python/C++ object hierarchy discrepancies

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2228 ) Change subject: gpu-compute: Fix Python/C++ object hierarchy discrepancies .. gpu-compute: Fix Python/C++ object

[gem5-dev] Change in public/gem5[master]: power: Avoid forward declarations that confuse wrappers

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2227 ) Change subject: power: Avoid forward declarations that confuse wrappers .. power: Avoid forward declarations

[gem5-dev] Change in public/gem5[master]: dev: Add a dummy VirtIO device

-Id: I6cbe12fd403903ef585be40279c3b1321fde48ff Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> Reviewed-by: Sudhanshu Jha <sudhanshu@arm.com> Reviewed-by: Rekai Gonzalez Alberquilla <rekai.gonzalezalberqui.

[gem5-dev] Change in public/gem5[master]: dev, arm: Add a VirtIO MMIO device

a VirtIO MMIO device Add an ARM-specific VirtIO MMIO device to the VExpress_GEM5_V1 platform. Change-Id: Id1e75398e039aad9d637f46f653cda9084d3d2fe Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Sudhanshu Jha <sudhanshu@arm.com> Reviewed-by: Nikos Nikoleris &

[gem5-dev] Change in public/gem5[master]: dev: Rename VirtIO PCI debug flag

debug flag Rename VIOPci -> VIOIface to avoid having a separate flag for the MMIO interface. Change-Id: I99f9210fa36ce33662c48537fd3992cd9a69d349 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Sudhanshu Jha <sudhanshu@arm.com> Reviewed-by:

[gem5-dev] Change in public/gem5[master]: dev: Add a MMIO transport interface for VirtIO

is assumed to be hard-coded to VirtQueue::ALIGN_SIZE (4096 bytes). * Only 4096 byte pages are currently supported. Change-Id: Ifd318f5e5bddab0b6a42d8c8af9ff2fbb477f98b Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com&

[gem5-dev] Change in public/gem5[master]: tests: Warn not fail when reading invalid pickle status files

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2320 Change subject: tests: Warn not fail when reading invalid pickle status files .. tests: Warn not fail when reading invalid

[gem5-dev] Change in public/gem5[master]: cpu: Print progress messages in Trace CPU

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2321 Change subject: cpu: Print progress messages in Trace CPU .. cpu: Print progress messages in Trace CPU This change adds

[gem5-dev] Change in public/gem5[master]: python: Automatically disable listeners in batch setups

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2322 Change subject: python: Automatically disable listeners in batch setups .. python: Automatically disable listeners in batch

[gem5-dev] Change in public/gem5[master]: sim: Implement load_addr_mask auto-calculation

specify the address mask, we use it instead of auto-calculating. Change-Id: I18aabb5d09945c6e3e3819c9c8036ea24b6c35cf Signed-off-by: Geoffrey Blake <geoffrey.bl...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com>

[gem5-dev] Change in public/gem5[master]: arm, kmi: Clear interrupts in KMI devices

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2225 ) Change subject: arm, kmi: Clear interrupts in KMI devices .. arm, kmi: Clear interrupts in KMI devices Added

[gem5-dev] Change in public/gem5[master]: dev, arm: Render HDLCD frames at a fixed rate in KVM

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2242 ) Change subject: dev, arm: Render HDLCD frames at a fixed rate in KVM .. dev, arm: Render HDLCD frames at a fixed rate

[gem5-dev] Change in public/gem5[master]: dev: Add support for single-pass scan out in the PixelPump

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2241 ) Change subject: dev: Add support for single-pass scan out in the PixelPump .. dev: Add support for single-pass scan

[gem5-dev] Change in public/gem5[master]: dev, kvm: Add a fast KVM-aware mode in DmaReadFifo

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/2226 ) Change subject: dev, kvm: Add a fast KVM-aware mode in DmaReadFifo .. dev, kvm: Add a fast KVM-aware mode

Re: [gem5-dev] Best method for adding code to ext/

I agree with Tony. Sub-repos (or submodules as they are called in git) can be quite painful. However, I'm not completely opposed to them. Based on my limited research on the topic, there seems to be two popular alternatives. One is to use the repo tool that Tony already mentioned. I only have

Re: [gem5-dev] Review Request 3814: misc: Add a CONTRIBUTING document

> On March 2, 2017, 10:56 p.m., Brad Beckmann wrote: > > CONTRIBUTING.md, line 55 > > > > > > What happens when a maintainer has not been assigned to a region of > > code? Is there a way even to guarantee that all existing

Re: [gem5-dev] Review Request 3814: misc: Add a CONTRIBUTING document

> On Feb. 28, 2017, 9:48 p.m., Brad Beckmann wrote: > > CONTRIBUTING.md, line 205 > > > > > > Where is the "Generate Password" option? I don't see it after signing > > in. Is this password unique to just pushing patches

Re: [gem5-dev] Review Request 3824: gpu-compute: Fix Python/C++ object hierarchy discrepancies

, 2017, 9:28 a.m., Andreas Sandberg wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3824/ > --- > > (U

[gem5-dev] Change in public/gem5[master]: dev, arm: Render HDLCD frames at a fixed rate in KVM

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2242 Change subject: dev, arm: Render HDLCD frames at a fixed rate in KVM .. dev, arm: Render HDLCD frames at a fixed rate in KVM

[gem5-dev] Change in public/gem5[master]: dev, kvm: Add a fast KVM-aware mode in DmaReadFifo

Andreas Sandberg has uploaded a new patch set (#2). ( https://gem5-review.googlesource.com/2226 ) Change subject: dev, kvm: Add a fast KVM-aware mode in DmaReadFifo .. dev, kvm: Add a fast KVM-aware mode in DmaReadFifo Use

[gem5-dev] Change in public/gem5[master]: dev: Add support for single-pass scan out in the PixelPump

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2241 Change subject: dev: Add support for single-pass scan out in the PixelPump .. dev: Add support for single-pass scan out

[gem5-dev] Change in public/gem5[master]: arm, kmi: Clear interrupts in KMI devices

Andreas Sandberg has uploaded a new patch set (#2). ( https://gem5-review.googlesource.com/2225 ) Change subject: arm, kmi: Clear interrupts in KMI devices .. arm, kmi: Clear interrupts in KMI devices Added functionality

Re: [gem5-dev] Linux Test Project - Testing SE Mode

Hi Brandon, This sounds like a very useful! Thanks for looking into it! On 27/02/2017 17:51, Potter, Brandon wrote: https://github.com/linux-test-project/ltp/tree/master/testcases/kernel/syscalls I was considering writing Linux test cases for SE Mode to work out (potential but more likely

Re: [gem5-dev] Migrating to git and gerrit

] https://github.com/mozilla/moz-git-tools/raw/master/hg-patch-to-git-patch On 16/02/2017 16:54, Jason Lowe-Power wrote: Hi all, We've been talking about this for a while, but now it's time! Special thanks to Andreas Sandberg for all of his hard work for putting this together. We will be migrating

[gem5-dev] Change in public/gem5[master]: power: Avoid forward declarations that confuse wrappers

: Id8c93224f1988edb5fdf9d3abc6237f2f688c02d Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> --- M src/sim/power/SConscript M src/sim/pow

[gem5-dev] Change in public/gem5[master]: power: Avoid forward declarations that confuse wrappers

-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> --- M src/sim/power/SConscript M src/sim/power/Thermal

[gem5-dev] Change in public/gem5[master]: gpu-compute: Fix Python/C++ object hierarchy discrepancies

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2228 Change subject: gpu-compute: Fix Python/C++ object hierarchy discrepancies .. gpu-compute: Fix Python/C++ object hierarchy

[gem5-dev] Change in public/gem5[master]: ext: Add pybind rev f4b81b3

Change-Id: I52e4fc9ebf2f59da57d8cf8f3e37cc79598c2f5f Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> --- M SConstruct A ext/pybind11/.appveyor.yml A ext/pybind1

[gem5-dev] Change in public/gem5[master]: ext: Fix undefined macro in pybind

macro in pybind Change-Id: I63a2506d3c028f78cacce8308e2f0e4880531dec Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> --- M ext/pybind11/include/pybind11/common.h M ex

Re: [gem5-dev] Review Request 3841: gpu-compute: mark functions with override if replacing virtual

. Thanks! - Andreas Sandberg On Feb. 25, 2017, 7:33 p.m., Brandon Potter wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem

[gem5-dev] changeset in gem5: arch: Include generated decoder header after ...

. These fields typically have short names that conflict with names in other header files. Include the generated header after all normal header to avoid this issue. Change-Id: I53d149b75432c20abdbf651e32c3c785d897973b Signed-off-by: Andreas Sandberg <andreas.sa

[gem5-dev] changeset in gem5: tests: Disable descriptions in stat files

reference data. Change-Id: I2a7ead4843586e800ecf83846694b73f0c356373 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> R

[gem5-dev] changeset in gem5: python: Add a generalized mechanism to config...

43061 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> Reviewed-by: Ilias Vougioukas <ilias.vougiou...@arm.com> Review

[gem5-dev] changeset in gem5: base: Refactor logging to make log level sele...

) and higher levels for less critical messages. The highest log level that is printed is controlled by calling Logger:setLevel(). Change-Id: I31e44299d242d953197a8e62679250c91d6ef776 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Re

[gem5-dev] Change in public/gem5[master]: dma, kvm: Bypassing DMA in KVM execution mode

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2226 Change subject: dma, kvm: Bypassing DMA in KVM execution mode .. dma, kvm: Bypassing DMA in KVM execution mode Event-free

[gem5-dev] Change in public/gem5[master]: arm, kmi: Clearing interrupts in KMI devices

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2225 Change subject: arm, kmi: Clearing interrupts in KMI devices .. arm, kmi: Clearing interrupts in KMI devices Added

Re: [gem5-dev] Review Request 3824: gpu-compute: Fix Python/C++ object hierarchy discrepancies

t: http://reviews.gem5.org/r/3824/#review9478 --- On Feb. 22, 2017, 9:28 a.m., Andreas Sandberg wrote: > > --- > This is an automatically generated e-mail. To reply, v

Re: [gem5-dev] Review Request 3817: arm, kvm: enable running 32-bit Guest under ARM KVM64

spotted in the previous version. Don't you need to do the same transformation for integer registers in updateKvmState as you do in updateThreadContext? - Andreas Sandberg On Feb. 25, 2017, 1:24 a.m., Rahul Thakur wrote

Re: [gem5-dev] Review Request 3816: commit 70469eba20cdcf091d66cf2ef463318203c7cc71

ooks good, thanks for refactoring! - Andreas Sandberg On Feb. 23, 2017, 7:17 p.m., Rahul Thakur wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://re

Re: [gem5-dev] Review Request 3827: python: Use PyBind11 instead of SWIG for Python wrappers

gem5.org/r/3827/#review9469 --- On Feb. 24, 2017, 10:20 a.m., Andreas Sandberg wrote: > > --- > This is an automatically generated e-mail. To reply, vis

Re: [gem5-dev] Review Request 3827: python: Use PyBind11 instead of SWIG for Python wrappers

the method's parameters. The new wrappers has the following known issues: * Global events can't be memory managed correctly. This was the case in SWIG as well. Change-Id: I88c5a95b6cf6c32fa9e1ad31dfc08b2e8199a763 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas H

Re: [gem5-dev] Review Request 3814: misc: Add a CONTRIBUTING document

> On Feb. 23, 2017, 11:20 a.m., Andreas Hansson wrote: > > CONTRIBUTING.md, line 69 > > > > > > Is there a step where tests/regressions are added and/or updated, or is > > that perhaps too much detail for this doc? We

Re: [gem5-dev] Review Request 3816: commit 70469eba20cdcf091d66cf2ef463318203c7cc71

R from the miscRegMap and just handle it separately before this loop. If you do this, you'll have to sync it manually early in updateThreadContext() to make sure that gem5 is aware of the CPU state before syncing other registers. - Andreas Sandberg On Feb. 21, 2017, 4:55 p.m., Rahul Thakur

Re: [gem5-dev] Review Request 3819: python: Add a generalized mechanism to configure stats

cifying a plain path). Change-Id: Ic4dce93ab4ead07ffdf71e55a22ba0ae5a143061 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> Reviewed-by: Ilias Vougioukas <ilias.vougiou..

Re: [gem5-dev] Review Request 3819: python: Add a generalized mechanism to configure stats

-- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3819/#review9443 --- On Feb. 21, 2017, 6:53 p.m., Andreas Sandberg wrote: > > --

Re: [gem5-dev] Review Request 3821: base: Refactor logging to make log level selection cleaner

enerated e-mail. To reply, visit: http://reviews.gem5.org/r/3821/#review9446 --- On Feb. 21, 2017, 6:54 p.m., Andreas Sandberg wrote: > > --- > This is an automatically generated

Re: [gem5-dev] Review Request 3824: gpu-compute: Fix Python/C++ object hierarchy discrepancies

system/GPUCoalescer.py ba90ffa751b6 Diff: http://reviews.gem5.org/r/3824/diff/ Testing --- Thanks, Andreas Sandberg ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Review Request 3822: power: Avoid forward declarations that confuse wrappers

: Id8c93224f1988edb5fdf9d3abc6237f2f688c02d Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> Diffs -

[gem5-dev] Review Request 3824: gpu-compute: Fix Python/C++ object hierarchy discrepancies

m5.org/r/3824/diff/ Testing --- Thanks, Andreas Sandberg ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Review Request 3827: python: Use PyBind11 instead of SWIG for Python wrappers

the following known issues: * Global events can't be memory managed correctly. This was the case in SWIG as well. Change-Id: I88c5a95b6cf6c32fa9e1ad31dfc08b2e8199a763 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewe

[gem5-dev] Review Request 3826: ext: Fix undefined macro in pybind

11880:efb01b140aa7 --- ext: Fix undefined macro in pybind Change-Id: I63a2506d3c028f78cacce8308e2f0e4880531dec Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis

[gem5-dev] Review Request 3825: ext: Add pybind rev f4b81b3

11879:4103050978f1 --- ext: Add pybind rev f4b81b3 Change-Id: I52e4fc9ebf2f59da57d8cf8f3e37cc79598c2f5f Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Andreas Hansson <andreas.hans...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@a

[gem5-dev] Review Request 3823: arch: Include generated decoder header after normal headers

the generated header after all normal header to avoid this issue. Change-Id: I53d149b75432c20abdbf651e32c3c785d897973b Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Diffs - src/arch/generic/memhelpers.hh ba90ffa751b6

[gem5-dev] Review Request 3821: base: Refactor logging to make log level selection cleaner

-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Gabor Dozsa <gabor.do...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Diffs - src/base/misc.hh ba90ffa751b6 src/base/misc.cc ba90ffa751b6 src/python/swig/core.i ba90ffa751b6 tests/con

[gem5-dev] Review Request 3820: tests: Disable descriptions in stat files

11874:40cd0a543902 --- tests: Disable descriptions in stat files Don't output verbose text descriptions in stat files when running tests. This saves a lot of space when storing reference data. Change-Id: I2a7ead4843586e800ecf83846694b73f0c356373 Signed-off-by: Andreas Sandberg

[gem5-dev] Review Request 3819: python: Add a generalized mechanism to configure stats

in path). Change-Id: Ic4dce93ab4ead07ffdf71e55a22ba0ae5a143061 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Curtis Dunham <curtis.dun...@arm.com> Reviewed-by: Sascha Bischoff <sascha.bisch...@arm.com> Reviewed-by: Ilias Vougioukas <ilias.vougiou...@arm.com> Diffs - src/

Re: [gem5-dev] Review Request 3814: misc: Add a CONTRIBUTING document

--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3814/#review9440 --- Ship it! Ship It! - Andreas Sandberg On Feb. 21, 2017, 5:34 p.m

Re: [gem5-dev] Review Request 3814: misc: Add a CONTRIBUTING document

; We might want to move the second half of this paragraph to the description of signed-off-by above. CONTRIBUTING (line 213) <http://reviews.gem5.org/r/3814/#comment7965> The first slash in /refs/for/... isn't needed. I'm not sure if it breaks anything though. - An

[gem5-dev] Change in public/testing[master]: IGNORE ME: Test commit to check gem5-dev integration

Andreas Sandberg has uploaded this change for review. ( https://gem5-review.googlesource.com/2220 Change subject: IGNORE ME: Test commit to check gem5-dev integration .. IGNORE ME: Test commit to check gem5-dev integration

[gem5-dev] changeset in gem5: arm, kvm: Automatically use the MuxingKvmGic

changeset 16dec978b549 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=16dec978b549 description: arm, kvm: Automatically use the MuxingKvmGic Automatically use the MuxingKvmGic in the VExpress_GEM5_V1 platform. This removes the need to patch the host

Re: [gem5-dev] Review Request 3781: riscv: Remove ECALL tests from insttest

> On Jan. 25, 2017, 11:28 a.m., Andreas Sandberg wrote: > > Ship It! > > Andreas Hansson wrote: > What was the verdict? Is it just a matter of an updated patch > description/summary and then this should be pushed? Yes, the description needs to be updated. I'm n

[gem5-dev] changeset in gem5: style: Force Python.h to be included before m...

/extending.html Change-Id: Id6a4f7fc64a336a8fd26691a0ca682abeb1d1579 Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikole...@arm.com> Reviewed-by: Jason Lowe-Power <ja...@lowepower.com> Reviewed-by:

Re: [gem5-dev] Review Request 3794: style: Force Python.h to be included before main header

.gem5.org/r/3792/diff/1/#33. > > > > If my fix does work, I think it's better to just include Python.h there > > instead of changing the style. > > Andreas Sandberg wrote: > I'd like to avoid including Python.h in any header files for dependency > reasons

Re: [gem5-dev] Review Request 3792: misc: Update #!env calls for python to explicit version

--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3792/#review9339 --- Ship it! Ship It! - Andreas Sandberg On Jan. 27, 2017, 5:32 p.m

Re: [gem5-dev] Review Request 3792: misc: Update #!env calls for python to explicit version

it in a separate commit? - Andreas Sandberg On Jan. 26, 2017, 4:47 p.m., Jason Lowe-Power wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem

<    4   5   6   7   8   9   10   11   12   13   >