On Wed, Mar 10, 2021 at 10:59 PM Khem Raj <raj.k...@gmail.com> wrote:
>
>
>
> On 3/10/21 7:16 PM, Jon Mason wrote:
> > On Wed, Mar 10, 2021 at 6:30 PM Andre McCurdy <armccu...@gmail.com> wrote:
> >>
> >> On Wed, Mar 10, 2021 at 3:04 PM Jon Mason <jdma...@kudzu.us> wrote:
> >>>
> >>> Currently seeing the following error when running runqemu on qemuarmv5:
> >>> runqemu - ERROR - Failed to run qemu: qemu-system-arm: versatilepb:
> >>> memory size must not exceed 256MB
> >>>
> >>> This is due to the memory size being set to 512 instead of the max
> >>> allowable of 256. Limit to this and everything is happy.
> >>>
> >>> Signed-off-by: Jon Mason <jon.ma...@arm.com>
> >>> Change-Id: I3339e50a53464b95f52a987ed37868ebf0935b07
> >>> ---
> >>> meta/recipes-sato/images/core-image-sato.bb | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/meta/recipes-sato/images/core-image-sato.bb
> >>> b/meta/recipes-sato/images/core-image-sato.bb
> >>> index e50b24a47691..8ed8f75e9f92 100644
> >>> --- a/meta/recipes-sato/images/core-image-sato.bb
> >>> +++ b/meta/recipes-sato/images/core-image-sato.bb
> >>> @@ -14,3 +14,4 @@ TOOLCHAIN_HOST_TASK_remove_task-populate-sdk-ext = "
> >>> nativesdk-intltool nativesd
> >>>
> >>> QB_MEM = '${@bb.utils.contains("DISTRO_FEATURES", "opengl", "-m 512",
> >>> "-m 256", d)}'
> >>> QB_MEM_qemumips = "-m 256"
> >>> +QB_MEM_qemuarmv5 = "-m 256"
> >>
> >> Grepping for QB_MEM suggests there are other images which need the
> >> same fix. Maybe good to fix all of them at once rather than one at a
> >> time.
> >
> > I do not know if other machines have this same issue with qemu, and
> > this is probably not the best way to resolve this (as I really think
> > this should be part of the machine config file, and have runqemu be
> > smarter logic about setting the memory size). I can open a bug in
> > bugzilla and get back to it later.
>
> perhaps we should define MAX_MEMORY to define this limit for every
> emulated machine and present a diagnostic if QB_MEM exceded MAX value
I'm making an assumption that QB_MEM is the max. If we simply added
sanity checks to the places where it is overridden, there would be no
problems. It shouldn't be too hard to do, but based on other comments
I think it's best to abandon this series and simply remove qemuarmv5.
>
> >
> >>
> >>> 2.20.1
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149282):
https://lists.openembedded.org/g/openembedded-core/message/149282
Mute This Topic: https://lists.openembedded.org/mt/81241074/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
