CVS commit: src/sys/dev/raidframe

2019-10-30 Thread Martin Husemann
Module Name:src Committed By: martin Date: Wed Oct 30 16:00:13 UTC 2019 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Gcc -Os on landisk is not smart enough to follow the conditional initialization and warns, unconditionaly initialize dksc at

CVS commit: src/sys/dev/raidframe

2019-10-30 Thread Martin Husemann
Module Name:src Committed By: martin Date: Wed Oct 30 16:00:13 UTC 2019 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Gcc -Os on landisk is not smart enough to follow the conditional initialization and warns, unconditionaly initialize dksc at

CVS commit: src/sys/dev/raidframe

2019-10-30 Thread Maxime Villard
Module Name:src Committed By: maxv Date: Wed Oct 30 07:59:45 UTC 2019 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Get >sc_dksc only when we know 'rsc' is not NULL. This was actually harmless because we didn't use the pointer then. Reported-by:

CVS commit: src/sys/dev/raidframe

2019-10-30 Thread Maxime Villard
Module Name:src Committed By: maxv Date: Wed Oct 30 07:59:45 UTC 2019 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Get >sc_dksc only when we know 'rsc' is not NULL. This was actually harmless because we didn't use the pointer then. Reported-by:

CVS commit: src/sys/dev/raidframe

2019-10-09 Thread Christos Zoulas
Module Name:src Committed By: christos Date: Thu Oct 10 03:44:00 UTC 2019 Modified Files: src/sys/dev/raidframe: raidframevar.h rf_callback.c rf_callback.h rf_copyback.c rf_dag.h rf_dagdegrd.c rf_dagdegrd.h rf_dagdegwr.c rf_dagdegwr.h rf_dagffrd.c

CVS commit: src/sys/dev/raidframe

2019-10-09 Thread Christos Zoulas
Module Name:src Committed By: christos Date: Thu Oct 10 03:44:00 UTC 2019 Modified Files: src/sys/dev/raidframe: raidframevar.h rf_callback.c rf_callback.h rf_copyback.c rf_dag.h rf_dagdegrd.c rf_dagdegrd.h rf_dagdegwr.c rf_dagdegwr.h rf_dagffrd.c

CVS commit: src/sys/dev/raidframe

2019-09-25 Thread Christos Zoulas
Module Name:src Committed By: christos Date: Thu Sep 26 01:36:11 UTC 2019 Modified Files: src/sys/dev/raidframe: raidframevar.h rf_compat50.c Log Message: make nspares unsigned; it is assigned from numSpares which is unsigned. To generate a diff of this commit: cvs

CVS commit: src/sys/dev/raidframe

2019-09-25 Thread Christos Zoulas
Module Name:src Committed By: christos Date: Thu Sep 26 01:36:11 UTC 2019 Modified Files: src/sys/dev/raidframe: raidframevar.h rf_compat50.c Log Message: make nspares unsigned; it is assigned from numSpares which is unsigned. To generate a diff of this commit: cvs

Re: CVS commit: src/sys/dev/raidframe

2019-02-05 Thread Christos Zoulas
In article , Paul Goyette wrote: >> Module Name:src >> Committed By: mrg >> Date: Tue Feb 5 09:28:00 UTC 2019 >> >> Modified Files: >> src/sys/dev/raidframe: rf_netbsdkintf.c >> >> Log Message: >> fix the previous: >> >> rf_netbsd32 is only relevant on _LP64 as all the

CVS commit: src/sys/dev/raidframe

2019-02-05 Thread Paul Goyette
Module Name:src Committed By: mrg Date: Tue Feb 5 09:28:00 UTC 2019 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: fix the previous: rf_netbsd32 is only relevant on _LP64 as all the structures are the same for arm32 oabi/eabi compat. only do it

Re: CVS commit: src/sys/dev/raidframe

2017-01-13 Thread coypu
It's still a behaviour change.. On Thu, Jan 12, 2017 at 11:22:16PM -0500, Christos Zoulas wrote: > +static void > +rf_handle_hosed(RF_Raid_t *raidPtr, RF_Config_t *cfgPtr, int hosed_column) > +{ > + if (raidPtr->Disks[hosed_column].status == rf_ds_failed) > + return; equivalent

Re: CVS commit: src/sys/dev/raidframe

2017-01-13 Thread coypu
On Thu, Jan 12, 2017 at 11:22:16PM -0500, Christos Zoulas wrote: > +rf_handle_hosed(RF_Raid_t *raidPtr, RF_Config_t *cfgPtr, int hosed_column) > +{ : > + if (raidPtr->Disks[hosed_column].status == rf_ds_failed) > + return; equivalent to if (raidPtr->Disks[hosed_column].status !=

Re: CVS commit: src/sys/dev/raidframe

2016-12-10 Thread coypu
err, this does introduce a functional change... oops it's if panicstr != NULL, but that's not the argument.

Re: CVS commit: src/sys/dev/raidframe

2015-12-26 Thread Robert Elz
Date:Sat, 26 Dec 2015 12:59:01 + From:"Paul Goyette" Message-ID: <20151226125901.230e1f...@cvs.netbsd.org> | if it does, all we really lose is auto-configuration of raid-sets. Auto-config of raid is what makes many of my systems function,

Re: CVS commit: src/sys/dev/raidframe

2015-12-26 Thread Paul Goyette
Module Name:src Committed By: pgoyette Date: Sat Dec 26 00:58:45 UTC 2015 Modified Files: src/sys/dev/raidframe: rf_driver.c rf_driver.h rf_netbsdkintf.c Log Message: Modularize the raidframe driver, including rework of the unit attach code to permit detaching (and

Re: CVS commit: src/sys/dev/raidframe

2015-12-26 Thread Paul Goyette
On Sun, 27 Dec 2015, Robert Elz wrote: Date:Sat, 26 Dec 2015 12:59:01 + From:"Paul Goyette" Message-ID: <20151226125901.230e1f...@cvs.netbsd.org> | if it does, all we really lose is auto-configuration of raid-sets. Auto-config of raid is

Re: CVS commit: src/sys/dev/raidframe

2014-04-03 Thread Christos Zoulas
On Apr 2, 10:33am, g...@ir.bbn.com (Greg Troxel) wrote: -- Subject: Re: CVS commit: src/sys/dev/raidframe | It seems there are 3 behaviors for root | | 1) don't change the root device (old behavior with -A yes) That does autoconfig for raid and does not deal with root at all. | 2) if root

Re: CVS commit: src/sys/dev/raidframe

2014-04-03 Thread Greg Troxel
...@ir.bbn.com (Greg Troxel) wrote: -- Subject: Re: CVS commit: src/sys/dev/raidframe | It seems there are 3 behaviors for root | | 1) don't change the root device (old behavior with -A yes) That does autoconfig for raid and does not deal with root at all. Right, but it is a way that some

re: CVS commit: src/sys/dev/raidframe

2014-04-03 Thread Christos Zoulas
On Apr 3, 7:57am, m...@eterna.com.au (matthew green) wrote: -- Subject: re: CVS commit: src/sys/dev/raidframe | kernel configuration changes are not solutions, so 2 and 3 are out. | | if we do 4, we should instead add an option to mark something as a | 'soft root', and leave the current

re: CVS commit: src/sys/dev/raidframe

2014-04-03 Thread Christos Zoulas
On Apr 3, 8:10am, m...@eterna.com.au (matthew green) wrote: -- Subject: re: CVS commit: src/sys/dev/raidframe | | 4. we can add an option to mark the raid as force root. | | if we do 4, we should instead add an option to mark something as a | 'soft root', and leave the current semantics

Re: CVS commit: src/sys/dev/raidframe

2014-04-03 Thread Greg Troxel
chris...@zoulas.com (Christos Zoulas) writes: On Apr 3, 7:57am, m...@eterna.com.au (matthew green) wrote: -- Subject: re: CVS commit: src/sys/dev/raidframe | kernel configuration changes are not solutions, so 2 and 3 are out. | | if we do 4, we should instead add an option to mark

re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread Christos Zoulas
On Apr 2, 2:10pm, m...@eterna.com.au (matthew green) wrote: -- Subject: re: CVS commit: src/sys/dev/raidframe | | Module Name:src | Committed By: christos | Date: Wed Apr 2 02:17:01 UTC 2014 | | Modified Files: | src/sys/dev/raidframe: rf_netbsdkintf.c

Re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread Martin Husemann
On Wed, Apr 02, 2014 at 06:37:21AM -0400, Christos Zoulas wrote: Well, there are different options here: 1. is there a way to pass the root from ofwboot to netbsd? 2. since netbsd knows it boots normally from raid, you can put a root on raidx statement in your kernel. 3. you can

Re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread Christos Zoulas
On Apr 2, 12:51pm, mar...@duskware.de (Martin Husemann) wrote: -- Subject: Re: CVS commit: src/sys/dev/raidframe | On Wed, Apr 02, 2014 at 06:37:21AM -0400, Christos Zoulas wrote: | Well, there are different options here: | 1. is there a way to pass the root from ofwboot to netbsd? | 2. since

Re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread Greg Troxel
I seemed to have missed the discussion for this change. It seems there are 3 behaviors for root 1) don't change the root device (old behavior with -A yes) 2) if root is on a component, change root to the raid, otherwise don't change. 3) force the root device to be the raid (old

Re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread David Laight
On Wed, Apr 02, 2014 at 06:37:21AM -0400, Christos Zoulas wrote: On Apr 2, 2:10pm, m...@eterna.com.au (matthew green) wrote: | i like this.. i think. | | i wonder if this will break my system that boot from a different | device to the raid root device. I think it will. | eg, ultra10

re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread matthew green
| Log Message: | If we are autoconfiguring root, then only change the booted_device if | we booted from one of the components of the root raid set. This allows | us to boot from other media, without forcing the found raid to always | be root. Allow the old behavior with

re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread matthew green
4. we can add an option to mark the raid as force root. if we do 4, we should instead add an option to mark something as a 'soft root', and leave the current semantics alone. the machines i have that are now not going to reboot properly are both used remotely, so changing semantics about

Re: CVS commit: src/sys/dev/raidframe

2014-04-02 Thread Greg Oster
On Thu, 03 Apr 2014 08:10:01 +1100 matthew green m...@eterna.com.au wrote: 4. we can add an option to mark the raid as force root. if we do 4, we should instead add an option to mark something as a 'soft root', and leave the current semantics alone. the machines i have that are now

re: CVS commit: src/sys/dev/raidframe

2014-04-01 Thread matthew green
Module Name: src Committed By: christos Date: Wed Apr 2 02:17:01 UTC 2014 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: If we are autoconfiguring root, then only change the booted_device if we booted from one of the components of the root raid

CVS commit: src/sys/dev/raidframe

2010-03-03 Thread Greg Oster
Module Name:src Committed By: oster Date: Wed Mar 3 14:23:27 UTC 2010 Modified Files: src/sys/dev/raidframe: rf_paritymap.c Log Message: Don't attempt to read or write component label stuff from/to 'dead disks'. Update used spares with the correct parity map bits too.

CVS commit: src/sys/dev/raidframe

2010-03-03 Thread Greg Oster
Module Name:src Committed By: oster Date: Wed Mar 3 14:23:27 UTC 2010 Modified Files: src/sys/dev/raidframe: rf_paritymap.c Log Message: Don't attempt to read or write component label stuff from/to 'dead disks'. Update used spares with the correct parity map bits too.

CVS commit: src/sys/dev/raidframe

2010-03-01 Thread Greg Oster
Module Name:src Committed By: oster Date: Mon Mar 1 14:51:58 UTC 2010 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Don't merge a good parity map with random stuff in the event that the first component is missing. (Since the merging just OR's the

CVS commit: src/sys/dev/raidframe

2010-03-01 Thread Greg Oster
Module Name:src Committed By: oster Date: Mon Mar 1 14:51:58 UTC 2010 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Don't merge a good parity map with random stuff in the event that the first component is missing. (Since the merging just OR's the

CVS commit: src/sys/dev/raidframe

2010-03-01 Thread Jed Davis
Module Name:src Committed By: jld Date: Mon Mar 1 21:10:27 UTC 2010 Modified Files: src/sys/dev/raidframe: rf_disks.c Log Message: When setting up a non-autoconfigured RAID set, fail a component if the attempt to read its label fails. OKed by os...@. To generate a

CVS commit: src/sys/dev/raidframe

2010-03-01 Thread Jed Davis
Module Name:src Committed By: jld Date: Mon Mar 1 21:10:27 UTC 2010 Modified Files: src/sys/dev/raidframe: rf_disks.c Log Message: When setting up a non-autoconfigured RAID set, fail a component if the attempt to read its label fails. OKed by os...@. To generate a

re: CVS commit: src/sys/dev/raidframe

2009-11-05 Thread matthew green
Module Name: src Committed By:dyoung Date:Thu Nov 5 17:52:32 UTC 2009 Modified Files: src/sys/dev/raidframe: rf_netbsdkintf.c Log Message: Use deviter(9) instead of accessing alldevs directly. Compile-tested, only. this worries me.