Re: svn commit: r331209 - head

2018-03-27 Thread Alexey Dokuchaev
On Tue, Mar 27, 2018 at 09:48:43AM -0700, John Baldwin wrote:
> On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> > On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > > ...
> > > > Right, but John had mentioned extra reboot *between* installkernel and
> > > > installworld, which I've never heard of before.
> > > 
> > > From the end up of src/UPDATING:
> > > 
> > > 
> > > make buildworld
> > > make kernel KERNCONF=YOUR_KERNEL_HERE
> > > 
> > 
> > If by "reboot" in your original message you meant "drop to single user"
> > then there's no confusion.  It's just that in my understanding "reboot"
> > means, well, reboot.
> 
> No, I meant reboot and it clearly says here "reboot".  Note that
> 'make kernel' does 'make buildkernel' and 'make installkernel', so
> the reboot quoted above is after installkernel but before installworld.

Ah, OK, so "reboot in single user" above means "reboot, then boot -s".
I see now, sorry for the noise. :-(

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


Re: svn commit: r331209 - head

2018-03-27 Thread John Baldwin
On Tuesday, March 27, 2018 11:40:22 AM Ian Lepore wrote:
> On Tue, 2018-03-27 at 09:48 -0700, John Baldwin wrote:
> > On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> > > 
> > > On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > > > 
> > > > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > > 
> > > > > > 
> > > > > > ...
> > > > > Right, but John had mentioned extra reboot *between* installkernel and
> > > > > installworld, which I've never heard of before.
> > > > From the end up of src/UPDATING:
> > > > 
> > > > 
> > > > make buildworld
> > > > make kernel KERNCONF=YOUR_KERNEL_HERE
> > > > 
> > > If by "reboot" in your original message you meant "drop to single user"
> > > then there's no confusion.  It's just that in my understanding "reboot"
> > > means, well, reboot.
> > No, I meant reboot and it clearly says here "reboot".  Note that
> > 'make kernel' does 'make buildkernel' and 'make installkernel', so
> > the reboot quoted above is after installkernel but before installworld.
> > Perhaps 'make kernel' should be expanded into separate lines for
> > 'buildkernel' and 'installkernel' if that is confusing.
> > 
> 
> I've been working with freebsd since 1996, and had no idea there was a
> kernel target that does build+install.  I think even showing it as
> 
>   make buildkernel installkernel KERNCONF=YOUR_KERNEL_HERE
> 
> would be more clear.

Yeah, that's not bad.  I don't ever use 'kernel' either.

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


Re: svn commit: r331209 - head

2018-03-27 Thread John Baldwin
On Tuesday, March 27, 2018 10:48:22 AM Rodney W. Grimes wrote:
> [ Charset ISO-8859-1 unsupported, converting... ]
> > On Tue, 2018-03-27 at 09:48 -0700, John Baldwin wrote:
> > > On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> > > > 
> > > > On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > > > > 
> > > > > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > > > 
> > > > > > > 
> > > > > > > ...
> > > > > > Right, but John had mentioned extra reboot *between* installkernel 
> > > > > > and
> > > > > > installworld, which I've never heard of before.
> > > > > From the end up of src/UPDATING:
> > > > > 
> > > > > 
> > > > > make buildworld
> > > > > make kernel KERNCONF=YOUR_KERNEL_HERE
> > > > > 
> > > > If by "reboot" in your original message you meant "drop to single user"
> > > > then there's no confusion.??It's just that in my understanding "reboot"
> > > > means, well, reboot.
> > > No, I meant reboot and it clearly says here "reboot".??Note that
> > > 'make kernel' does 'make buildkernel' and 'make installkernel', so
> > > the reboot quoted above is after installkernel but before installworld.
> > > Perhaps 'make kernel' should be expanded into separate lines for
> > > 'buildkernel' and 'installkernel' if that is confusing.
> > > 
> > 
> > I've been working with freebsd since 1996, and had no idea there was a
> > kernel target that does build+install. ?I think even showing it as
> 
> I was vegually aware of it, but since I do things between
> buildkernel and installkernel I dont use it, more so cause
> I build kernels on a central box and push them out to
> systems.
> 
> > ? make buildkernel installkernel KERNCONF=YOUR_KERNEL_HERE
> 
> make {kernel|buildkernel installkernel} KERNCONF=YOUR_KERNEL_HERE
> 
> OR
> 
> make kernel KERNCONF=YOUR_KERNEL_HERE
> or
> make buildkernel installkernel KERNCONF=YOUR_KERNEL_HERE

I think just one example is probably fine and explicit buildkernel and
installkernel is probably the more common use.

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


Re: svn commit: r331209 - head

2018-03-27 Thread Rodney W. Grimes
[ Charset ISO-8859-1 unsupported, converting... ]
> On Tue, 2018-03-27 at 09:48 -0700, John Baldwin wrote:
> > On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> > > 
> > > On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > > > 
> > > > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > > 
> > > > > > 
> > > > > > ...
> > > > > Right, but John had mentioned extra reboot *between* installkernel and
> > > > > installworld, which I've never heard of before.
> > > > From the end up of src/UPDATING:
> > > > 
> > > > 
> > > > make buildworld
> > > > make kernel KERNCONF=YOUR_KERNEL_HERE
> > > > 
> > > If by "reboot" in your original message you meant "drop to single user"
> > > then there's no confusion.??It's just that in my understanding "reboot"
> > > means, well, reboot.
> > No, I meant reboot and it clearly says here "reboot".??Note that
> > 'make kernel' does 'make buildkernel' and 'make installkernel', so
> > the reboot quoted above is after installkernel but before installworld.
> > Perhaps 'make kernel' should be expanded into separate lines for
> > 'buildkernel' and 'installkernel' if that is confusing.
> > 
> 
> I've been working with freebsd since 1996, and had no idea there was a
> kernel target that does build+install. ?I think even showing it as

I was vegually aware of it, but since I do things between
buildkernel and installkernel I dont use it, more so cause
I build kernels on a central box and push them out to
systems.

> ? make buildkernel installkernel KERNCONF=YOUR_KERNEL_HERE

make {kernel|buildkernel installkernel} KERNCONF=YOUR_KERNEL_HERE

OR

make kernel KERNCONF=YOUR_KERNEL_HERE
or
make buildkernel installkernel KERNCONF=YOUR_KERNEL_HERE

> would be more clear.

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


Re: svn commit: r331209 - head

2018-03-27 Thread Rodney W. Grimes
> On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> > On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > > ...
> > > > Right, but John had mentioned extra reboot *between* installkernel and
> > > > installworld, which I've never heard of before.
> > > 
> > > From the end up of src/UPDATING:
> > > 
> > > 
> > > make buildworld
> > > make kernel KERNCONF=YOUR_KERNEL_HERE
> > > 

"in" may not be the proper way to express this,
better wording, imho, would be "reboot to single user mode",
Or even more complete to make the reason for the reboot to
single user mode explictly clear:
 *1

*1 This is to facilitate the next step, the single user mode
is required so that binaries are not in use while the installworld
process is done.

> > 
> > If by "reboot" in your original message you meant "drop to single user"
> > then there's no confusion.  It's just that in my understanding "reboot"
> > means, well, reboot.
> 
> No, I meant reboot and it clearly says here "reboot".  Note that
> 'make kernel' does 'make buildkernel' and 'make installkernel', so
> the reboot quoted above is after installkernel but before installworld.
> Perhaps 'make kernel' should be expanded into separate lines for
> 'buildkernel' and 'installkernel' if that is confusing.
> 
> -- 
> John Baldwin
> 
> 

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


Re: svn commit: r331209 - head

2018-03-27 Thread Ian Lepore
On Tue, 2018-03-27 at 09:48 -0700, John Baldwin wrote:
> On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> > 
> > On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > > 
> > > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > 
> > > > > 
> > > > > ...
> > > > Right, but John had mentioned extra reboot *between* installkernel and
> > > > installworld, which I've never heard of before.
> > > From the end up of src/UPDATING:
> > > 
> > > 
> > > make buildworld
> > > make kernel KERNCONF=YOUR_KERNEL_HERE
> > > 
> > If by "reboot" in your original message you meant "drop to single user"
> > then there's no confusion.  It's just that in my understanding "reboot"
> > means, well, reboot.
> No, I meant reboot and it clearly says here "reboot".  Note that
> 'make kernel' does 'make buildkernel' and 'make installkernel', so
> the reboot quoted above is after installkernel but before installworld.
> Perhaps 'make kernel' should be expanded into separate lines for
> 'buildkernel' and 'installkernel' if that is confusing.
> 

I've been working with freebsd since 1996, and had no idea there was a
kernel target that does build+install.  I think even showing it as

  make buildkernel installkernel KERNCONF=YOUR_KERNEL_HERE

would be more clear.

-- Ian

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


Re: svn commit: r331209 - head

2018-03-27 Thread John Baldwin
On Tuesday, March 27, 2018 05:19:12 AM Alexey Dokuchaev wrote:
> On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> > On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > > ...
> > > Right, but John had mentioned extra reboot *between* installkernel and
> > > installworld, which I've never heard of before.
> > 
> > From the end up of src/UPDATING:
> > 
> > 
> > make buildworld
> > make kernel KERNCONF=YOUR_KERNEL_HERE
> > 
> 
> If by "reboot" in your original message you meant "drop to single user"
> then there's no confusion.  It's just that in my understanding "reboot"
> means, well, reboot.

No, I meant reboot and it clearly says here "reboot".  Note that
'make kernel' does 'make buildkernel' and 'make installkernel', so
the reboot quoted above is after installkernel but before installworld.
Perhaps 'make kernel' should be expanded into separate lines for
'buildkernel' and 'installkernel' if that is confusing.

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


Re: svn commit: r331209 - head

2018-03-26 Thread Alexey Dokuchaev
On Thu, Mar 22, 2018 at 08:52:47AM -0700, John Baldwin wrote:
> On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> > > ...
> > Right, but John had mentioned extra reboot *between* installkernel and
> > installworld, which I've never heard of before.
> 
> From the end up of src/UPDATING:
> 
> 
> make buildworld
> make kernel KERNCONF=YOUR_KERNEL_HERE
> 

If by "reboot" in your original message you meant "drop to single user"
then there's no confusion.  It's just that in my understanding "reboot"
means, well, reboot.

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


Re: svn commit: r331209 - head

2018-03-23 Thread John Baldwin
On Thursday, March 22, 2018 04:39:38 AM Alexey Dokuchaev wrote:
> On Wed, Mar 21, 2018 at 10:34:15PM -0600, Warner Losh wrote:
> > On Wed, Mar 21, 2018 at 10:03 PM, Alexey Dokuchaev 
> > wrote:
> > > On Mon, Mar 19, 2018 at 09:20:40AM -0700, John Baldwin wrote:
> > > > ...
> > > > How does this fit with the recommended installation steps of doing
> > > > 'make installkernel' and rebooting before doing a 'make installworld'?
> > >
> > > Rebooting?  I vaguely recall recommended procedure was to switch to single
> > > user mode prior to `installworld', but personally I've never bothered and
> > > installed kernel, [mm -p,] world, mm -Fi, reboot.  Am I missing something?
> > 
> > Old advice was to do the installworld in single user, as that's the safest
> > (and also ensures you have a new kernel and don't install before rebooting).
> > However, we've had enough experience to see that usually an installworld +
> > reboot is sufficient.
> 
> Right, but John had mentioned extra reboot *between* installkernel and
> installworld, which I've never heard of before.

>From the end up of src/UPDATING:

To rebuild everything and install it on the current system.
---
# Note: sometimes if you are running current you gotta do more than
# is listed here if you are upgrading from a really old current.


make buildworld
make kernel KERNCONF=YOUR_KERNEL_HERE
[1]
 [3]
mergemaster -Fp [5]
make installworld
mergemaster -Fi [4]
make delete-old [6]


See also 
https://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/makeworld.html

Often you can take shortcuts, but this is the only process we guarantee will
always work.  (We will also sometimes add additional shims to make some of the
common shortcuts work, but we don't guarantee that.)

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


Re: svn commit: r331209 - head

2018-03-22 Thread Kyle Evans
On Thu, Mar 22, 2018 at 1:13 PM, Steven Hartland
 wrote:
> I think it would be worth specifically detailing the steps to achieve this,
> as its not immediately obvious how this would be done.
>

Hi,

A later commit clarified, but then an even later commit made these
instructions obsolete. =) The problem this entry was made for has been
eradicated, and the entry has since been removed.

Thanks,

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


Re: svn commit: r331209 - head

2018-03-22 Thread Steven Hartland
I think it would be worth specifically detailing the steps to achieve 
this, as its not immediately obvious how this would be done.


On 19/03/2018 15:27, Kyle Evans wrote:

Author: kevans
Date: Mon Mar 19 15:27:53 2018
New Revision: 331209
URL: https://svnweb.freebsd.org/changeset/base/331209

Log:
   Add note to UPDATING about UEFI changes requiring loader(8) update
   
   These problems have only been observed with boards using U-Boot (e.g. ARM)

   where virtual addresses are already set in the memory map by the firmware
   and the firmware is expecting a call to SetVirtualAddressMap to be made.
   I refrain from mentioning this in the note because this could also be the
   case on some not-yet-tested firmware on amd64 and it's not a bad
   recommendation for the general case.

Modified:
   head/UPDATING

Modified: head/UPDATING
==
--- head/UPDATING   Mon Mar 19 15:11:10 2018(r331208)
+++ head/UPDATING   Mon Mar 19 15:27:53 2018(r331209)
@@ -51,6 +51,13 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 12.x IS SLOW:
  
  ** SPECIAL WARNING: **
  
+20180319:

+   For UEFI systems: the UEFI loader(8), loader.efi, should be updated in
+   conjunction with installing a new kernel after r330868. The kernel,
+   after this revision, will be more lenient when mapping addresses for
+   UEFI Runtime Services and this may result in a kernel panic without the
+   corresponding loader(8) update.
+
  20180212:
FreeBSD boot loader enhanced with Lua scripting. It's purely opt-in for
now by building WITH_LOADER_LUA and WITHOUT_FORTH in /etc/src.conf.



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


Re: svn commit: r331209 - head

2018-03-21 Thread Warner Losh
On Wed, Mar 21, 2018 at 10:03 PM, Alexey Dokuchaev 
wrote:

> On Mon, Mar 19, 2018 at 09:20:40AM -0700, John Baldwin wrote:
> > On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
> > > New Revision: 331209
> > > URL: https://svnweb.freebsd.org/changeset/base/331209
> > >
> > > Log:
> > >   Add note to UPDATING about UEFI changes requiring loader(8) update
> > >   ...
> >
> > How does this fit with the recommended installation steps of doing
> > 'make installkernel' and rebooting before doing a 'make installworld'?
>
> Rebooting?  I vaguely recall recommended procedure was to switch to single
> user mode prior to `installworld', but personally I've never bothered and
> installed kernel, [mm -p,] world, mm -Fi, reboot.  Am I missing something?
>

Old advice was to do the installworld in single user, as that's the safest
(and also ensures you have a new kernel and don't install before
rebooting). However, we've had enough experience to see that usually an
installworld + reboot is sufficient.

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


Re: svn commit: r331209 - head

2018-03-21 Thread Alexey Dokuchaev
On Mon, Mar 19, 2018 at 09:20:40AM -0700, John Baldwin wrote:
> On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
> > New Revision: 331209
> > URL: https://svnweb.freebsd.org/changeset/base/331209
> > 
> > Log:
> >   Add note to UPDATING about UEFI changes requiring loader(8) update
> >   ...
> 
> How does this fit with the recommended installation steps of doing
> 'make installkernel' and rebooting before doing a 'make installworld'?

Rebooting?  I vaguely recall recommended procedure was to switch to single
user mode prior to `installworld', but personally I've never bothered and
installed kernel, [mm -p,] world, mm -Fi, reboot.  Am I missing something?

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


Re: svn commit: r331209 - head

2018-03-21 Thread Warner Losh
On Wed, Mar 21, 2018 at 10:39 PM, Alexey Dokuchaev 
wrote:

> On Wed, Mar 21, 2018 at 10:34:15PM -0600, Warner Losh wrote:
> > On Wed, Mar 21, 2018 at 10:03 PM, Alexey Dokuchaev 
> > wrote:
> > > On Mon, Mar 19, 2018 at 09:20:40AM -0700, John Baldwin wrote:
> > > > ...
> > > > How does this fit with the recommended installation steps of doing
> > > > 'make installkernel' and rebooting before doing a 'make
> installworld'?
> > >
> > > Rebooting?  I vaguely recall recommended procedure was to switch to
> single
> > > user mode prior to `installworld', but personally I've never bothered
> and
> > > installed kernel, [mm -p,] world, mm -Fi, reboot.  Am I missing
> something?
> >
> > Old advice was to do the installworld in single user, as that's the
> safest
> > (and also ensures you have a new kernel and don't install before
> rebooting).
> > However, we've had enough experience to see that usually an installworld
> +
> > reboot is sufficient.
>
> Right, but John had mentioned extra reboot *between* installkernel and
> installworld, which I've never heard of before.
>

We've done what we can to make it so you don't brick your system if you
don't do that, but the pedantic way is build everything, install a new
kernel, reboot single user, install world and bring it up multi-user,
though that leaves a copy of the old sh around that doesn't hurt anything.
The alternate way is to install kernel, reboot installworld, reboot, but
usually people skip the reboot after install kernel. You can usually get
away with it, but the ino64 showed that's not always possible (the forward
compat code wasn't committed for weeks and even then there's no guarantee
that it would).

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


Re: svn commit: r331209 - head

2018-03-21 Thread Alexey Dokuchaev
On Wed, Mar 21, 2018 at 10:34:15PM -0600, Warner Losh wrote:
> On Wed, Mar 21, 2018 at 10:03 PM, Alexey Dokuchaev 
> wrote:
> > On Mon, Mar 19, 2018 at 09:20:40AM -0700, John Baldwin wrote:
> > > ...
> > > How does this fit with the recommended installation steps of doing
> > > 'make installkernel' and rebooting before doing a 'make installworld'?
> >
> > Rebooting?  I vaguely recall recommended procedure was to switch to single
> > user mode prior to `installworld', but personally I've never bothered and
> > installed kernel, [mm -p,] world, mm -Fi, reboot.  Am I missing something?
> 
> Old advice was to do the installworld in single user, as that's the safest
> (and also ensures you have a new kernel and don't install before rebooting).
> However, we've had enough experience to see that usually an installworld +
> reboot is sufficient.

Right, but John had mentioned extra reboot *between* installkernel and
installworld, which I've never heard of before.

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


Re: svn commit: r331209 - head

2018-03-19 Thread Warner Losh
On Mon, Mar 19, 2018 at 12:09 PM, Rodney W. Grimes <
free...@pdx.rh.cn85.dnsmgr.net> wrote:

> > On Mon, Mar 19, 2018 at 11:51 AM, Andriy Gapon  wrote:
> >
> > > On 19/03/2018 18:20, John Baldwin wrote:
> > > > On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
> > > >> Author: kevans
> > > >> Date: Mon Mar 19 15:27:53 2018
> > > >> New Revision: 331209
> > > >> URL: https://svnweb.freebsd.org/changeset/base/331209
> > > >>
> > > >> Log:
> > > >>   Add note to UPDATING about UEFI changes requiring loader(8) update
> > > >>
> > > >>   These problems have only been observed with boards using U-Boot
> (e.g.
> > > ARM)
> > > >>   where virtual addresses are already set in the memory map by the
> > > firmware
> > > >>   and the firmware is expecting a call to SetVirtualAddressMap to be
> > > made.
> > > >>   I refrain from mentioning this in the note because this could
> also be
> > > the
> > > >>   case on some not-yet-tested firmware on amd64 and it's not a bad
> > > >>   recommendation for the general case.
> > > >
> > > > How does this fit with the recommended installation steps of doing
> > > > 'make installkernel' and rebooting before doing a 'make
> installworld'?
> > >
> > > Installation of /boot/loader along with likes of cat and cut has always
> > > puzzled
> > > me very much.
> > >
> >
> > Historical reason. We've always installed /boot/loader with installworld.
> > We also install the other boot loaders, but don't deploy them (so
> there's a
> > new /boot/boot, but it isn't installed into /).
>
> More complete historical information.  The "boot" code use to live in
> hidden places in the MBR, and the {disk,bsd}label, so installing it
> into the file system had 0 effect, as the actuall in use running
> code was hidden and had to be replaced using either fdisk or
> {disk,bsd}label.
>

Right. That's still the case, but these bits are considered 'static' and
uninteresting to update.


> When "loader" was added this should of probably changed with
> preseverations of old code and ways to get back to it incase
> things went a foul, and ways to update it in a more controlled
> manner than installworld.  And more approprate timing of
> installing it (should it now go with installkernel?).


No. Where it is fine. The arm64 thing is actually fixed in the kernel so we
don't need to cope with it.

Warner


> > To get around this issue, though, is trivial:
> >
> > make installkenrel
> > cd stand
> > make install
> > reboot
> > make installworld
> >
> > in this weird case.
> >
> > Warner
>
> --
> Rod Grimes
> rgri...@freebsd.org
>
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r331209 - head

2018-03-19 Thread Rodney W. Grimes
> On Mon, Mar 19, 2018 at 11:51 AM, Andriy Gapon  wrote:
> 
> > On 19/03/2018 18:20, John Baldwin wrote:
> > > On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
> > >> Author: kevans
> > >> Date: Mon Mar 19 15:27:53 2018
> > >> New Revision: 331209
> > >> URL: https://svnweb.freebsd.org/changeset/base/331209
> > >>
> > >> Log:
> > >>   Add note to UPDATING about UEFI changes requiring loader(8) update
> > >>
> > >>   These problems have only been observed with boards using U-Boot (e.g.
> > ARM)
> > >>   where virtual addresses are already set in the memory map by the
> > firmware
> > >>   and the firmware is expecting a call to SetVirtualAddressMap to be
> > made.
> > >>   I refrain from mentioning this in the note because this could also be
> > the
> > >>   case on some not-yet-tested firmware on amd64 and it's not a bad
> > >>   recommendation for the general case.
> > >
> > > How does this fit with the recommended installation steps of doing
> > > 'make installkernel' and rebooting before doing a 'make installworld'?
> >
> > Installation of /boot/loader along with likes of cat and cut has always
> > puzzled
> > me very much.
> >
> 
> Historical reason. We've always installed /boot/loader with installworld.
> We also install the other boot loaders, but don't deploy them (so there's a
> new /boot/boot, but it isn't installed into /).

More complete historical information.  The "boot" code use to live in
hidden places in the MBR, and the {disk,bsd}label, so installing it
into the file system had 0 effect, as the actuall in use running
code was hidden and had to be replaced using either fdisk or
{disk,bsd}label.

When "loader" was added this should of probably changed with
preseverations of old code and ways to get back to it incase
things went a foul, and ways to update it in a more controlled
manner than installworld.  And more approprate timing of
installing it (should it now go with installkernel?).
 
> To get around this issue, though, is trivial:
> 
> make installkenrel
> cd stand
> make install
> reboot
> make installworld
> 
> in this weird case.
> 
> Warner

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


Re: svn commit: r331209 - head

2018-03-19 Thread Warner Losh
On Mon, Mar 19, 2018 at 11:51 AM, Andriy Gapon  wrote:

> On 19/03/2018 18:20, John Baldwin wrote:
> > On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
> >> Author: kevans
> >> Date: Mon Mar 19 15:27:53 2018
> >> New Revision: 331209
> >> URL: https://svnweb.freebsd.org/changeset/base/331209
> >>
> >> Log:
> >>   Add note to UPDATING about UEFI changes requiring loader(8) update
> >>
> >>   These problems have only been observed with boards using U-Boot (e.g.
> ARM)
> >>   where virtual addresses are already set in the memory map by the
> firmware
> >>   and the firmware is expecting a call to SetVirtualAddressMap to be
> made.
> >>   I refrain from mentioning this in the note because this could also be
> the
> >>   case on some not-yet-tested firmware on amd64 and it's not a bad
> >>   recommendation for the general case.
> >
> > How does this fit with the recommended installation steps of doing
> > 'make installkernel' and rebooting before doing a 'make installworld'?
>
> Installation of /boot/loader along with likes of cat and cut has always
> puzzled
> me very much.
>

Historical reason. We've always installed /boot/loader with installworld.
We also install the other boot loaders, but don't deploy them (so there's a
new /boot/boot, but it isn't installed into /).

To get around this issue, though, is trivial:

make installkenrel
cd stand
make install
reboot
make installworld

in this weird case.

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


Re: svn commit: r331209 - head

2018-03-19 Thread Andriy Gapon
On 19/03/2018 18:20, John Baldwin wrote:
> On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
>> Author: kevans
>> Date: Mon Mar 19 15:27:53 2018
>> New Revision: 331209
>> URL: https://svnweb.freebsd.org/changeset/base/331209
>>
>> Log:
>>   Add note to UPDATING about UEFI changes requiring loader(8) update
>>   
>>   These problems have only been observed with boards using U-Boot (e.g. ARM)
>>   where virtual addresses are already set in the memory map by the firmware
>>   and the firmware is expecting a call to SetVirtualAddressMap to be made.
>>   I refrain from mentioning this in the note because this could also be the
>>   case on some not-yet-tested firmware on amd64 and it's not a bad
>>   recommendation for the general case.
> 
> How does this fit with the recommended installation steps of doing
> 'make installkernel' and rebooting before doing a 'make installworld'?

Installation of /boot/loader along with likes of cat and cut has always puzzled
me very much.


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


Re: svn commit: r331209 - head

2018-03-19 Thread John Baldwin
On Monday, March 19, 2018 03:27:53 PM Kyle Evans wrote:
> Author: kevans
> Date: Mon Mar 19 15:27:53 2018
> New Revision: 331209
> URL: https://svnweb.freebsd.org/changeset/base/331209
> 
> Log:
>   Add note to UPDATING about UEFI changes requiring loader(8) update
>   
>   These problems have only been observed with boards using U-Boot (e.g. ARM)
>   where virtual addresses are already set in the memory map by the firmware
>   and the firmware is expecting a call to SetVirtualAddressMap to be made.
>   I refrain from mentioning this in the note because this could also be the
>   case on some not-yet-tested firmware on amd64 and it's not a bad
>   recommendation for the general case.

How does this fit with the recommended installation steps of doing
'make installkernel' and rebooting before doing a 'make installworld'?

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


svn commit: r331209 - head

2018-03-19 Thread Kyle Evans
Author: kevans
Date: Mon Mar 19 15:27:53 2018
New Revision: 331209
URL: https://svnweb.freebsd.org/changeset/base/331209

Log:
  Add note to UPDATING about UEFI changes requiring loader(8) update
  
  These problems have only been observed with boards using U-Boot (e.g. ARM)
  where virtual addresses are already set in the memory map by the firmware
  and the firmware is expecting a call to SetVirtualAddressMap to be made.
  I refrain from mentioning this in the note because this could also be the
  case on some not-yet-tested firmware on amd64 and it's not a bad
  recommendation for the general case.

Modified:
  head/UPDATING

Modified: head/UPDATING
==
--- head/UPDATING   Mon Mar 19 15:11:10 2018(r331208)
+++ head/UPDATING   Mon Mar 19 15:27:53 2018(r331209)
@@ -51,6 +51,13 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 12.x IS SLOW:
 
 ** SPECIAL WARNING: **
 
+20180319:
+   For UEFI systems: the UEFI loader(8), loader.efi, should be updated in
+   conjunction with installing a new kernel after r330868. The kernel,
+   after this revision, will be more lenient when mapping addresses for
+   UEFI Runtime Services and this may result in a kernel panic without the
+   corresponding loader(8) update.
+
 20180212:
FreeBSD boot loader enhanced with Lua scripting. It's purely opt-in for
now by building WITH_LOADER_LUA and WITHOUT_FORTH in /etc/src.conf.
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"