Re: mfii(4): add bio(4) support

2018-05-18 Thread Naoki Fukaumi
Hi, From: Jonathan Matthew <jonat...@d14n.org> Subject: Re: mfii(4): add bio(4) support Date: Fri, 18 May 2018 15:30:31 +1000 > On 15/05/18 14:13, Naoki Fukaumi wrote: >> Hi Jonathan Matthew, >> here is updated patch. could you review it? > > I've just committe

Re: mfii(4): add bio(4) support

2018-05-17 Thread Jonathan Matthew
On 15/05/18 14:13, Naoki Fukaumi wrote: Hi Jonathan Matthew, here is updated patch. could you review it? I've just committed a series of changes based on your patch. dlg and I decided that it'd be better if mfii_mgmt() took scsi flags rather than MFII_DATA_IN/OUT, so I reworked all the

Re: mfii(4): add bio(4) support

2018-05-14 Thread Naoki Fukaumi
Hi Jonathan Matthew, here is updated patch. could you review it? This patch adds bio(4) support for mfii(4). from mfi(4) with following addition/fixes, * make "bioctl -R" work after hot swapping Currently "bioctl -R" works only if disk state is "Offline" (set by "bioctl -O") and it

Re: mfii(4): add bio(4) support

2018-03-18 Thread Naoki Fukaumi
Hi, about BBU detection, From: Jonathan Matthew <jonat...@d14n.org> Subject: Re: mfii(4): add bio(4) support Date: Thu, 8 Mar 2018 21:56:53 +1000 >> > I also don't get a sensor for the battery. I haven't looked into this yet. >> > Any idea why that would happen? >&g

Re: mfii(4): add bio(4) support

2018-03-18 Thread Naoki Fukaumi
Hi, thank you for committing (part of) fixes. From: Jonathan Matthew <jonat...@d14n.org> Subject: Re: mfii(4): add bio(4) support Date: Mon, 12 Mar 2018 12:06:31 +1000 > On Mon, Mar 05, 2018 at 09:49:16PM +0900, Naoki Fukaumi wrote: >> Hi tech@, >> >> This patch add

Re: mfii(4): add bio(4) support

2018-03-11 Thread Jonathan Matthew
On Mon, Mar 05, 2018 at 09:49:16PM +0900, Naoki Fukaumi wrote: > Hi tech@, > > This patch adds bio(4) support for mfii(4). > # with "mfii(4): use MFII_FUNCTION_PASSTHRU_IO for MFI commands" > > most parts are taken from mfi(4), plus fix for rebuilding (bioctl -R). I noticed another problem:

Re: mfii(4): add bio(4) support

2018-03-08 Thread Jonathan Matthew
On Thu, Mar 08, 2018 at 06:43:51PM +0900, Naoki Fukaumi wrote: > Hi, > > thank you for your feedback! > > From: Jonathan Matthew <jonat...@d14n.org> > Subject: Re: mfii(4): add bio(4) support > Date: Thu, 8 Mar 2018 19:18:28 +1000 > > > On Mon, Mar 05, 201

Re: mfii(4): add bio(4) support

2018-03-08 Thread Naoki Fukaumi
Hi, thank you for your feedback! From: Jonathan Matthew <jonat...@d14n.org> Subject: Re: mfii(4): add bio(4) support Date: Thu, 8 Mar 2018 19:18:28 +1000 > On Mon, Mar 05, 2018 at 09:49:16PM +0900, Naoki Fukaumi wrote: >> Hi tech@, >> >> This patch add

Re: mfii(4): add bio(4) support

2018-03-08 Thread Jonathan Matthew
On Mon, Mar 05, 2018 at 09:49:16PM +0900, Naoki Fukaumi wrote: > Hi tech@, > > This patch adds bio(4) support for mfii(4). > # with "mfii(4): use MFII_FUNCTION_PASSTHRU_IO for MFI commands" > > most parts are taken from mfi(4), plus fix for rebuilding (bioctl -R). Thanks for working on this,

Re: mfii(4): add bio(4) support

2018-03-05 Thread Naoki Fukaumi
Hi, From: YASUOKA Masahiko <yasu...@openbsd.org> Subject: Re: mfii(4): add bio(4) support Date: Tue, 06 Mar 2018 08:14:44 +0900 (JST) >> most parts are taken from mfi(4), plus fix for rebuilding (bioctl -R). > > Do I understand the rebuilding problem correctly? &g

Re: mfii(4): add bio(4) support

2018-03-05 Thread YASUOKA Masahiko
Hi, > most parts are taken from mfi(4), plus fix for rebuilding (bioctl -R). Do I understand the rebuilding problem correctly? - mfii_makegood() and mfii_makespare() are added before sending the rebuild command to fix the problem - The problem doesn't happen with mfi(4) On Mon, 05 Mar 2018

mfii(4): add bio(4) support

2018-03-05 Thread Naoki Fukaumi
Hi tech@, This patch adds bio(4) support for mfii(4). # with "mfii(4): use MFII_FUNCTION_PASSTHRU_IO for MFI commands" most parts are taken from mfi(4), plus fix for rebuilding (bioctl -R). --- sys/dev/ic/mfireg.h +++ sys/dev/ic/mfireg.h @@ -139,6 +139,9 @@ #define MR_DCMD_CONF_GET