Re: CVS commit: src/distrib/utils/sysinst

2012-11-21 Thread Martin Husemann
On Tue, Nov 20, 2012 at 08:17:43PM +, Jonathan A. Kollasch wrote:
 Module Name:  src
 Committed By: jakllsch
 Date: Tue Nov 20 20:17:43 UTC 2012
 
 Modified Files:
   src/distrib/utils/sysinst: bsddisklabel.c
 
 Log Message:
 Don't enable WAPBL by default. It's far too dangerous.

Please do not make changes like this without discussion.

 (a) I can see no PRs to this effect
 (b) There is no big enough warning in the man pages
 (c) we now have a serious performance degration in the default install
 compared to the softdep world

Martin


Re: CVS commit: src/distrib/utils/sysinst

2012-11-21 Thread Jonathan A. Kollasch
On Wed, Nov 21, 2012 at 12:52:04PM +0100, Martin Husemann wrote:
 On Tue, Nov 20, 2012 at 08:17:43PM +, Jonathan A. Kollasch wrote:
  Module Name:src
  Committed By:   jakllsch
  Date:   Tue Nov 20 20:17:43 UTC 2012
  
  Modified Files:
  src/distrib/utils/sysinst: bsddisklabel.c
  
  Log Message:
  Don't enable WAPBL by default. It's far too dangerous.
 
 Please do not make changes like this without discussion.

I have to admit, I committed this when I was in an unreasonable frame of
mind.

  (a) I can see no PRs to this effect

This is true, I will file one.

  (b) There is no big enough warning in the man pages

True again.

  (c) we now have a serious performance degration in the default install
  compared to the softdep world

AFAIR softdeps were never enabled by default when they were around,
it was always more of a if you want performance, enable them note
somewhere.

Jonathan Kollasch


Re: CVS commit: src/distrib/utils/sysinst

2012-11-21 Thread Martin Husemann
On Wed, Nov 21, 2012 at 11:07:04AM -0600, Jonathan A. Kollasch wrote:
 AFAIR softdeps were never enabled by default when they were around,
 it was always more of a if you want performance, enable them note
 somewhere.

I was suprised, but you are right.

Martin


Re: CVS commit: src/distrib/utils/sysinst

2012-03-01 Thread Marc Balmer
Am 01.03.12 17:31, schrieb Izumi Tsutsui:
 Module Name: src
 Committed By:mbalmer
 Date:Thu Mar  1 16:22:16 UTC 2012

 Modified Files:
  src/distrib/utils/sysinst: msg.mbr.de msg.mbr.en msg.mbr.es msg.mbr.fr
  msg.mbr.pl msg.mi.de msg.mi.en msg.mi.es msg.mi.fr msg.mi.pl
  src/distrib/utils/sysinst/arch/arc: msg.md.en
  src/distrib/utils/sysinst/arch/atari: msg.md.de msg.md.en msg.md.es
  src/distrib/utils/sysinst/arch/cobalt: msg.md.en
  src/distrib/utils/sysinst/arch/evbarm: msg.md.de msg.md.en msg.md.es
  src/distrib/utils/sysinst/arch/hp300: msg.md.de
  src/distrib/utils/sysinst/arch/i386: msg.md.de msg.md.en msg.md.es
  msg.md.pl
  src/distrib/utils/sysinst/arch/mac68k: msg.md.de msg.md.en msg.md.es
  msg.md.pl
  src/distrib/utils/sysinst/arch/ofppc: msg.md.en msg.md.es msg.md.pl
  src/distrib/utils/sysinst/arch/prep: msg.md.en msg.md.es msg.md.pl
  src/distrib/utils/sysinst/arch/sparc: msg.md.de msg.md.en msg.md.es
  src/distrib/utils/sysinst/arch/x68k: msg.md.pl

 Log Message:
 Put back spaces at EOL where needed (in a message file, a space at EOL
 means a line break, a line break means a space...)
 This should fix PR 46042.
 
 Did you check all changes?

I mechanically reverted all whitespace changes so we have the state we
had before the copyright changes.

 
 Trailing spaces in most of MD msg files are not line break,
 but actually extra junk.
 
 It looks only following files needs to be changed:
 src/distrib/utils/sysinst/msg.mbr.de
 src/distrib/utils/sysinst/msg.mbr.en
 src/distrib/utils/sysinst/msg.mbr.es
 src/distrib/utils/sysinst/msg.mbr.fr
 src/distrib/utils/sysinst/msg.mbr.pl
 src/distrib/utils/sysinst/msg.mi.de
 src/distrib/utils/sysinst/msg.mi.en
 src/distrib/utils/sysinst/msg.mi.es
 src/distrib/utils/sysinst/msg.mi.fr
 src/distrib/utils/sysinst/msg.mi.pl
 src/distrib/utils/sysinst/arch/evbarm/msg.md.de
 src/distrib/utils/sysinst/arch/i386/msg.md.en
 
 ---
 Izumi Tsutsui


-- 
  \~.The NetBSD Foundation
   \~'   Marc Balmer, Developer / Marketing
  NetBSD
 \   mbal...@netbsd.org   http://www.NetBSD.org/


Re: CVS commit: src/distrib/utils/sysinst

2012-03-01 Thread Izumi Tsutsui
mbalmer@ wrote:

  Log Message:
  Put back spaces at EOL where needed (in a message file, a space at EOL
  means a line break, a line break means a space...)
  This should fix PR 46042.
  
  Did you check all changes?
 
 I mechanically reverted all whitespace changes so we have the state we
 had before the copyright changes.

Ok, it's fine for pullups.  I'll cleanup actual junk spaces later.

---
Izumi Tsutsui


Re: CVS commit: src/distrib/utils/sysinst

2012-01-11 Thread Joerg Sonnenberger
On Sun, Oct 30, 2011 at 12:30:57AM +, Jonathan A. Kollasch wrote:
 Log Message:
 As both of the ns-wide.wide.ad.jp and light.imasy.or.jp IPv6 resolvers
 now refuse my queries, replace them with google-public-dns-a.google.com
 and google-public-dns-b.google.com.

(a) Why offer two options for the same company? Can we make the
selection of DNS provider just Google with the two addresses in the
backend?

(b) What about offering OpenDNS as alternative?

Joerg


Re: CVS commit: src/distrib/utils/sysinst

2012-01-08 Thread Martin Husemann
On Sun, Jan 08, 2012 at 09:12:31AM +1100, matthew green wrote:
 this breaks the non-X11R7 systems.  please revert it.  this needs
 to be fixed to handle both cases -- they still exist.

Aren't there three cases?

 - arch is using X11R7 on the to-be-installed version and (old) installed
   system is already using X11R7 (easy)
 - same as above, but both systems with X11R6 (easy as well)
 - arch is using X11R7 on the to-be-installed version, but we need to
   upgrade from a previous X11R6 version (basically impossible without
   user interaction, AFAICT - I would go for an empty Xorg install and
   warn the user that all modifications of the old config file, including
   keyboard settings, font pathes etc., have to be merged manually)

Martin


Re: CVS commit: src/distrib/utils/sysinst

2011-11-04 Thread Izumi Tsutsui
 Log Message:
 Add a new hook, md_pre_mount(), that is called after preparation of
 the target filesystem, but before mounting it. Use this on sparc64
 to install the primary boot block, but install the secondary boot
 (/ofwboot) later.
 This allows sysinst to run at securelevel 1 (as long as we don't do an
 upgrade install on the hard disk we booted from).

What won't work on securelevel 1?
installboot(8) against mounted LABELSECTOR?

---
Izumi Tsutsui


Re: CVS commit: src/distrib/utils/sysinst

2011-11-04 Thread Martin Husemann
On Fri, Nov 04, 2011 at 09:02:15PM +0900, Izumi Tsutsui wrote:
 What won't work on securelevel 1?
 installboot(8) against mounted LABELSECTOR?

Yes.

Martin


Re: CVS commit: src/distrib/utils/sysinst

2011-11-04 Thread Izumi Tsutsui
martin@ wrote:

 On Fri, Nov 04, 2011 at 09:02:15PM +0900, Izumi Tsutsui wrote:
  What won't work on securelevel 1?
  installboot(8) against mounted LABELSECTOR?
 
 Yes.

Doesn't DIOCWLABEL work as macppc does?
http://nxr.NetBSD.org/xref/src/usr.sbin/installboot/arch/macppc.c#119

---
Izumi Tsutsui


Re: CVS commit: src/distrib/utils/sysinst

2011-10-31 Thread Greg Troxel

  If TNF can provide a open resolver on a stable IPv6 address for use
  by sysinst, that'd be great.

Do we need only a resolver that answers for things in netbsd.org, to
find mirrors, or really a full resolver?


pgpkxsOKL8CZH.pgp
Description: PGP signature


Re: CVS commit: src/distrib/utils/sysinst

2011-10-31 Thread Jonathan A. Kollasch
On Mon, Oct 31, 2011 at 11:40:25AM -0400, Greg Troxel wrote:
 
   If TNF can provide a open resolver on a stable IPv6 address for use
   by sysinst, that'd be great.
 
 Do we need only a resolver that answers for things in netbsd.org, to
 find mirrors, or really a full resolver?

Depends, I often will install sets I've built myself from a random
server.  But adding a server (preferably further down the
list) that only answers for netbsd.org might be a option some people
would prefer to use.

Jonathan Kollasch


Re: CVS commit: src/distrib/utils/sysinst

2011-10-31 Thread Martin Husemann
On Mon, Oct 31, 2011 at 03:45:05PM +, Jonathan A. Kollasch wrote:
 Depends, I often will install sets I've built myself from a random
 server.  But adding a server (preferably further down the
 list) that only answers for netbsd.org might be a option some people
 would prefer to use.

I wonder if we should add another option to use the ipv6 default gateway
as nameserver and otherwise declare this a convenience option for a
real world corner case most users won't run into (and not realy care
about the actual resolvers used in that case).

Martin
(but hey, I'm living in a mostly v6 free world still, despite realy
hard trying to change that)


Re: CVS commit: src/distrib/utils/sysinst

2011-10-30 Thread Bernd Ernesti
On Sun, Oct 30, 2011 at 12:30:57AM +, Jonathan A. Kollasch wrote:
 Module Name:  src
 Committed By: jakllsch
 Date: Sun Oct 30 00:30:57 UTC 2011
 
 Modified Files:
   src/distrib/utils/sysinst: menus.mi
 
 Log Message:
 As both of the ns-wide.wide.ad.jp and light.imasy.or.jp IPv6 resolvers
 now refuse my queries, replace them with google-public-dns-a.google.com
 and google-public-dns-b.google.com.

IMHO they should be replaced with ?.root-servers.net entries and not a with
an address of a private company.

Bernd



Re: CVS commit: src/distrib/utils/sysinst

2011-10-30 Thread Martin Husemann
On Sun, Oct 30, 2011 at 09:41:09AM +0100, Bernd Ernesti wrote:
  Log Message:
  As both of the ns-wide.wide.ad.jp and light.imasy.or.jp IPv6 resolvers
  now refuse my queries, replace them with google-public-dns-a.google.com
  and google-public-dns-b.google.com.
 
 IMHO they should be replaced with ?.root-servers.net entries and not a with
 an address of a private company.

I don't understand why we offer this menu at all, shouldn't it just go
away?

Martin


Re: CVS commit: src/distrib/utils/sysinst

2011-10-30 Thread Jonathan A. Kollasch
On Sun, Oct 30, 2011 at 11:07:04AM +0100, Martin Husemann wrote:
 On Sun, Oct 30, 2011 at 09:41:09AM +0100, Bernd Ernesti wrote:
   Log Message:
   As both of the ns-wide.wide.ad.jp and light.imasy.or.jp IPv6 resolvers
   now refuse my queries, replace them with google-public-dns-a.google.com
   and google-public-dns-b.google.com.
  
  IMHO they should be replaced with ?.root-servers.net entries and not a with
  an address of a private company.
 
 I don't understand why we offer this menu at all, shouldn't it just go
 away?

Bernd:

Those are not open resolvers, they are authoritative for . only.
I also doubt we want to put bind(8) into the ramdisk.

Martin:

Until RFC 6106 is usable by sysinst, and/or until it is widely deployed,
the menu must stay for people without IPv4 connectivity.  Unless you
want to enter a up-to-39-character address for the resolver or server
the sets are on manually.

I was somewhat hesitant to choose Google, however I sort of doubt
anyone wants to provide open resolvers these days.  Google has the
resources to prevent abuse of their open resolvers, and they have
motivation to keep them running.  The 'other' option where you
enter your own resolver's address is still there.

I can barely remember the 20 character unique-local address of my
resolvers, and I don't have extreme aversion to letting Google see
what addresses I'm resolving for the duration of sysinst.

If TNF can provide a open resolver on a stable IPv6 address for use
by sysinst, that'd be great.

Jonathan Kollasch


Re: CVS commit: src/distrib/utils/sysinst

2010-02-28 Thread David Holland
On Wed, Feb 17, 2010 at 04:09:52PM +, Martin Husemann wrote:
  Modified Files:
   src/distrib/utils/sysinst: disks.c
  
  Log Message:
  eeh says we should not enable fsck_lfs on lfs partitions

Even assuming this is true in the medium to long term rather than a
temporary problem... this is not the right way to do this unless there
is zero chance there will ever be a working fsck_lfs, which I don't
think we want to admit is the case.

The right thing to do is to make fsck_lfs exit without doing anything
(or otherwise behave innocuously) when run with -p.

That can be adjusted easily in the future by deploying a new fsck_lfs
binary; introducing gunk into users' fstab files requires hand-editing
or expensive/delicate postinstall foo later, or creates a permanent
disability, none of which is desirable.

-- 
David A. Holland
dholl...@netbsd.org


CVS commit: src/distrib/utils/sysinst/arch/sparc64

2010-02-28 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Sun Feb 28 22:42:44 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/sparc64: md.h

Log Message:
Enable dynamic sizing of the default swap partition


To generate a diff of this commit:
cvs rdiff -u -r1.15 -r1.16 src/distrib/utils/sysinst/arch/sparc64/md.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst/arch/sparc64

2010-02-28 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Sun Feb 28 22:42:44 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/sparc64: md.h

Log Message:
Enable dynamic sizing of the default swap partition


To generate a diff of this commit:
cvs rdiff -u -r1.15 -r1.16 src/distrib/utils/sysinst/arch/sparc64/md.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/distrib/utils/sysinst/arch/sparc64/md.h
diff -u src/distrib/utils/sysinst/arch/sparc64/md.h:1.15 src/distrib/utils/sysinst/arch/sparc64/md.h:1.16
--- src/distrib/utils/sysinst/arch/sparc64/md.h:1.15	Wed Feb 17 16:11:00 2010
+++ src/distrib/utils/sysinst/arch/sparc64/md.h	Sun Feb 28 22:42:44 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: md.h,v 1.15 2010/02/17 16:11:00 martin Exp $	*/
+/*	$NetBSD: md.h,v 1.16 2010/02/28 22:42:44 martin Exp $	*/
 
 /*
  * Copyright 1997 Piermont Information Systems Inc.
@@ -40,6 +40,7 @@
 /* md.h -- Machine specific definitions for the sparc */
 
 /* Constants and defines */
+#define	DEFSWAPSIZE	(-1)
 
 /*
  * Symbolic names for disk partitions.



CVS commit: src/distrib/utils/sysinst/arch/cobalt

2010-02-21 Thread Thomas Klausner
Module Name:src
Committed By:   wiz
Date:   Sun Feb 21 13:28:47 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/cobalt: msg.md.en

Log Message:
Consistently call the file system ext2.


To generate a diff of this commit:
cvs rdiff -u -r1.2 -r1.3 src/distrib/utils/sysinst/arch/cobalt/msg.md.en

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst

2010-02-18 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Thu Feb 18 12:21:52 UTC 2010

Modified Files:
src/distrib/utils/sysinst: bsddisklabel.c

Log Message:
Since we call /usr and /var with absolute paths, lets do that for /tmp
as well.


To generate a diff of this commit:
cvs rdiff -u -r1.53 -r1.54 src/distrib/utils/sysinst/bsddisklabel.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst

2010-02-17 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Wed Feb 17 16:09:52 UTC 2010

Modified Files:
src/distrib/utils/sysinst: disks.c

Log Message:
eeh says we should not enable fsck_lfs on lfs partitions


To generate a diff of this commit:
cvs rdiff -u -r1.110 -r1.111 src/distrib/utils/sysinst/disks.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst/arch/sparc64

2010-02-17 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Wed Feb 17 16:11:01 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/sparc64: md.h

Log Message:
We can now boot from FFSv2, so offer it + even use it as a default.


To generate a diff of this commit:
cvs rdiff -u -r1.14 -r1.15 src/distrib/utils/sysinst/arch/sparc64/md.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



Re: CVS commit: src/distrib/utils/sysinst

2010-02-17 Thread Martin Husemann
On Thu, Feb 18, 2010 at 01:15:44AM +0900, Izumi Tsutsui wrote:
 I'd also like to make lfs optional in sysinst per ports,
 but I wonder how we can put #ifdef into menuc(1) sources..

I thought it is (runtime-) optional by checking for mount_lfs and friends,
see check_lfs_progs().

Martin


CVS commit: src/distrib/utils/sysinst

2010-01-27 Thread Julio M. Merino Vidal
Module Name:src
Committed By:   jmmv
Date:   Wed Jan 27 11:02:03 UTC 2010

Modified Files:
src/distrib/utils/sysinst: defs.h util.c

Log Message:
Perform in-place replacement of the cypher type instead of renaming the
passwd.conf file to passwd.conf.pre-sysinst file and creating a new one
from scratch:

- This is consistent with all other edits of configuration files performed
  by sysinst.  (E.g. in-place changes of rc.conf.)

- This eases the upgrade of the system to a newer set of files because the
  diffs presented by etcupdate are easier to read.

No objections in tech-inst...@.  Tested installing NetBSD/amd64.


To generate a diff of this commit:
cvs rdiff -u -r1.147 -r1.148 src/distrib/utils/sysinst/defs.h
cvs rdiff -u -r1.162 -r1.163 src/distrib/utils/sysinst/util.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst/arch/mac68k

2010-01-17 Thread Thomas Klausner
Module Name:src
Committed By:   wiz
Date:   Sun Jan 17 22:47:19 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/mac68k: md.c

Log Message:
Close file when finished with it. Found by cppcheck.


To generate a diff of this commit:
cvs rdiff -u -r1.56 -r1.57 src/distrib/utils/sysinst/arch/mac68k/md.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst

2010-01-17 Thread Thomas Klausner
Module Name:src
Committed By:   wiz
Date:   Sun Jan 17 22:48:50 UTC 2010

Modified Files:
src/distrib/utils/sysinst: wskbd.c

Log Message:
Only a return value below 0 denotes an error in open(2). Found by cppcheck.


To generate a diff of this commit:
cvs rdiff -u -r1.8 -r1.9 src/distrib/utils/sysinst/wskbd.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst

2010-01-12 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Wed Jan 13 07:48:58 UTC 2010

Modified Files:
src/distrib/utils/sysinst: net.c

Log Message:
Use the proper structs for ioctls to fetch network configuration.
Might fix PR 42436.


To generate a diff of this commit:
cvs rdiff -u -r1.123 -r1.124 src/distrib/utils/sysinst/net.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst

2010-01-12 Thread Martin Husemann
Module Name:src
Committed By:   martin
Date:   Wed Jan 13 07:52:38 UTC 2010

Modified Files:
src/distrib/utils/sysinst: Makefile.inc

Log Message:
Avoid adding -Os to CFLAGS if DBG is set and contains -g.
This helps gdb a lot when compiling local debug versions of sysinst.


To generate a diff of this commit:
cvs rdiff -u -r1.49 -r1.50 src/distrib/utils/sysinst/Makefile.inc

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst/arch/mac68k

2010-01-02 Thread David Laight
Module Name:src
Committed By:   dsl
Date:   Sat Jan  2 17:15:07 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/mac68k: menus.md.de menus.md.en
menus.md.es menus.md.pl

Log Message:
Sanitise whitespace to minimise the differences between these files.


To generate a diff of this commit:
cvs rdiff -u -r1.3 -r1.4 src/distrib/utils/sysinst/arch/mac68k/menus.md.de
cvs rdiff -u -r1.21 -r1.22 src/distrib/utils/sysinst/arch/mac68k/menus.md.en
cvs rdiff -u -r1.2 -r1.3 src/distrib/utils/sysinst/arch/mac68k/menus.md.es
cvs rdiff -u -r1.8 -r1.9 src/distrib/utils/sysinst/arch/mac68k/menus.md.pl

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/distrib/utils/sysinst/arch

2010-01-02 Thread David Laight
Module Name:src
Committed By:   dsl
Date:   Sat Jan  2 20:54:46 UTC 2010

Modified Files:
src/distrib/utils/sysinst/arch/mvme68k: md.c
src/distrib/utils/sysinst/arch/x68k: md.c

Log Message:
Fix another couple of signed v unsigned comparisons of disk block numbers.


To generate a diff of this commit:
cvs rdiff -u -r1.22 -r1.23 src/distrib/utils/sysinst/arch/mvme68k/md.c
cvs rdiff -u -r1.37 -r1.38 src/distrib/utils/sysinst/arch/x68k/md.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



Re: CVS commit: src/distrib/utils/sysinst

2009-08-24 Thread Soren Jacobsen
On 08/24 05:13, Matthias Scheler wrote:
 On Sun, Aug 23, 2009 at 09:16:17PM +, Jared D. McNeill wrote:
  Modified Files:
  src/distrib/utils/sysinst: menus.mi msg.mi.de msg.mi.en msg.mi.es
  msg.mi.fr msg.mi.pl util.c
  
  Log Message:
  Get rid of the 'select extraction verbosity' menu.
 
 Which option does it use now? The progress bar?

Yes.


Re: CVS commit: src/distrib/utils/sysinst

2009-08-23 Thread Matthias Scheler
On Sun, Aug 23, 2009 at 09:16:17PM +, Jared D. McNeill wrote:
 Modified Files:
   src/distrib/utils/sysinst: menus.mi msg.mi.de msg.mi.en msg.mi.es
   msg.mi.fr msg.mi.pl util.c
 
 Log Message:
 Get rid of the 'select extraction verbosity' menu.

Which option does it use now? The progress bar?

Kind regards

-- 
Matthias Scheler  http://zhadum.org.uk/


Re: CVS commit: src/distrib/utils/sysinst

2009-04-05 Thread Martin Husemann
On Sun, Apr 05, 2009 at 08:52:39AM +0900, Izumi Tsutsui wrote:
 Then, is it matter for modern GB HDD? ;-p

No, that's why the suggestion was to check the disk size and default to 
UFS2 only on those shiny new big disks.

Martin


Re: CVS commit: src/distrib/utils/sysinst

2009-04-04 Thread Izumi Tsutsui
a...@netbsd.org wrote:

 On Sat, Apr 04, 2009 at 07:47:12PM +0900, Izumi Tsutsui wrote:
  a...@netbsd.org wrote:
   Module Name:  src
   Committed By: ad
   Date: Sat Apr  4 10:38:00 UTC 2009
   
   Modified Files:
 src/distrib/utils/sysinst: bsddisklabel.c
   
   Log Message:
   - Default to UFS2 if the platform can boot from it.
  
  Should we reject UFS2 for root partition
  if MD boot loader doesn't support it (like sparc64), or
 
 If you can fit it into the MD code in sysinst why not.
 Ideally sparc64 would have a workaround.

bootblk for sparc64 (written in forth?) doesn't recognize
UFS2 superblock (yet).

  MD sysinst doesn't handle ffsv2 primary boot (like alpha and pmax) ?
 
 alpha and pmax seem unmaintained and as far as I can tell don't boot/work,
 so I don't care.

They have bootxx_ffsv2, but sysinst implicitly install bootxx_ffs.

In any case, __HAVE_UFS2_BOOT in types.h seems ambiguous for me.
Isn't it better to have some other macro in sysinst/arch/${MACHINE}/md.h ?
---
Izumi Tsutsui


Re: CVS commit: src/distrib/utils/sysinst

2009-04-04 Thread Martin Husemann
On Sat, Apr 04, 2009 at 09:43:46PM +0900, Izumi Tsutsui wrote:
 In any case, __HAVE_UFS2_BOOT in types.h seems ambiguous for me.
 Isn't it better to have some other macro in sysinst/arch/${MACHINE}/md.h ?

I agree, it does not belong in types.h.

Martin