Re: svn commit: r368721 - head/stand/efi/loader

2020-12-17 Thread Warner Losh
On Thu, Dec 17, 2020 at 1:00 PM Andrew Gallatin 
wrote:

> On 12/17/20 2:49 PM, Kyle Evans wrote:
> > On Thu, Dec 17, 2020 at 1:47 PM Andrew Gallatin 
> wrote:
> >>
> >> On 12/17/20 12:02 PM, Warner Losh wrote:
> >>> Author: imp
> >>> Date: Thu Dec 17 17:02:09 2020
> >>> New Revision: 368721
> >>> URL:
> https://urldefense.com/v3/__https://svnweb.freebsd.org/changeset/base/368721__;!!OToaGQ!5c1mLnhtRtEV6Cv_MTWpzXWaGZEYYDp4TJ6wVDzjVZiehAItts7ZWC15uNnQYRa5Fg$
> >>>
> >>> Log:
> >>> Drop EFI_STAGING_SIZE back down to 64M
> >>>
> >>> vmware can't cope with anything larger than 64MB. Drop this back to
> >>> 64MB everywhere but arm.
> >>
> >> There were all kinds of booting problems before this was bumped up.
> >> In fact, I still have EFI_STAGING_SIZE=128 in src.conf because I needed
> >> it be be able to boot when using Nvidia graphics.  By reducing this, I
> >> feel like we're just playing whack-a-mole.
> >>
> >
> > IIRC those have long since become OBE as we'll now grow the staging
> > area to accommodate nvidia.
> >
>
> Ah, OK. cool!
>
> Thanks & sorry for the noise.
>

We added the 100 as a hack and then fixed the underlying problem. If 64
doesn't work for you, please let me know and I'll work with you to make it
right.

Warner
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r368721 - head/stand/efi/loader

2020-12-17 Thread Andrew Gallatin

On 12/17/20 2:49 PM, Kyle Evans wrote:

On Thu, Dec 17, 2020 at 1:47 PM Andrew Gallatin  wrote:


On 12/17/20 12:02 PM, Warner Losh wrote:

Author: imp
Date: Thu Dec 17 17:02:09 2020
New Revision: 368721
URL: 
https://urldefense.com/v3/__https://svnweb.freebsd.org/changeset/base/368721__;!!OToaGQ!5c1mLnhtRtEV6Cv_MTWpzXWaGZEYYDp4TJ6wVDzjVZiehAItts7ZWC15uNnQYRa5Fg$

Log:
Drop EFI_STAGING_SIZE back down to 64M

vmware can't cope with anything larger than 64MB. Drop this back to
64MB everywhere but arm.


There were all kinds of booting problems before this was bumped up.
In fact, I still have EFI_STAGING_SIZE=128 in src.conf because I needed
it be be able to boot when using Nvidia graphics.  By reducing this, I
feel like we're just playing whack-a-mole.



IIRC those have long since become OBE as we'll now grow the staging
area to accommodate nvidia.



Ah, OK. cool!

Thanks & sorry for the noise.

Drew
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r368721 - head/stand/efi/loader

2020-12-17 Thread Kyle Evans
On Thu, Dec 17, 2020 at 1:47 PM Andrew Gallatin  wrote:
>
> On 12/17/20 12:02 PM, Warner Losh wrote:
> > Author: imp
> > Date: Thu Dec 17 17:02:09 2020
> > New Revision: 368721
> > URL: 
> > https://urldefense.com/v3/__https://svnweb.freebsd.org/changeset/base/368721__;!!OToaGQ!5c1mLnhtRtEV6Cv_MTWpzXWaGZEYYDp4TJ6wVDzjVZiehAItts7ZWC15uNnQYRa5Fg$
> >
> > Log:
> >Drop EFI_STAGING_SIZE back down to 64M
> >
> >vmware can't cope with anything larger than 64MB. Drop this back to
> >64MB everywhere but arm.
>
> There were all kinds of booting problems before this was bumped up.
> In fact, I still have EFI_STAGING_SIZE=128 in src.conf because I needed
> it be be able to boot when using Nvidia graphics.  By reducing this, I
> feel like we're just playing whack-a-mole.
>

IIRC those have long since become OBE as we'll now grow the staging
area to accommodate nvidia.
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r368721 - head/stand/efi/loader

2020-12-17 Thread Andrew Gallatin

On 12/17/20 12:02 PM, Warner Losh wrote:

Author: imp
Date: Thu Dec 17 17:02:09 2020
New Revision: 368721
URL: 
https://urldefense.com/v3/__https://svnweb.freebsd.org/changeset/base/368721__;!!OToaGQ!5c1mLnhtRtEV6Cv_MTWpzXWaGZEYYDp4TJ6wVDzjVZiehAItts7ZWC15uNnQYRa5Fg$

Log:
   Drop EFI_STAGING_SIZE back down to 64M
   
   vmware can't cope with anything larger than 64MB. Drop this back to

   64MB everywhere but arm.


There were all kinds of booting problems before this was bumped up.
In fact, I still have EFI_STAGING_SIZE=128 in src.conf because I needed
it be be able to boot when using Nvidia graphics.  By reducing this, I
feel like we're just playing whack-a-mole.

Drew
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


svn commit: r368721 - head/stand/efi/loader

2020-12-17 Thread Warner Losh
Author: imp
Date: Thu Dec 17 17:02:09 2020
New Revision: 368721
URL: https://svnweb.freebsd.org/changeset/base/368721

Log:
  Drop EFI_STAGING_SIZE back down to 64M
  
  vmware can't cope with anything larger than 64MB. Drop this back to
  64MB everywhere but arm.
  
  PR: 251866
  MFC After: 1 week

Modified:
  head/stand/efi/loader/copy.c

Modified: head/stand/efi/loader/copy.c
==
--- head/stand/efi/loader/copy.cThu Dec 17 16:52:40 2020
(r368720)
+++ head/stand/efi/loader/copy.cThu Dec 17 17:02:09 2020
(r368721)
@@ -174,9 +174,7 @@ out:
 #endif /* __i386__ || __amd64__ */
 
 #ifndef EFI_STAGING_SIZE
-#if defined(__amd64__)
-#defineEFI_STAGING_SIZE100
-#elif defined(__arm__)
+#if defined(__arm__)
 #defineEFI_STAGING_SIZE32
 #else
 #defineEFI_STAGING_SIZE64
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"