Re: PATCH: `dkctl dev' shows everything

> >         wedges:
> >         wd0d: 6 wedges:
> >         dk5: wd0h, 314711651 blocks at 173569599,
type: ffs
> >         dk5 at wd0: wd0h
> >         dk5: 314711651 blocks at 173569599, type:
ffs
> >         (...)
>
> This is redundant ... will fix.

        http://w
ww.NetBSD.org/~uebayasi/dkctl2.patch

And
        # dkctl wd0
        strategy:
        wd0d: disksort

        cache:
        wd0d: read cache enabled
        wd0d: write-back cache enabled
        wd0d: read cache enable is not changeable
        wd0d: write cache enable is changeable
        wd0d: cache parameters are not savable

        wedges:
        wd0d: 6 wedges:
        dk5: wd0h, 314711651 blocks at 173569599, type: ffs
        dk4: wd0g, 4128768 blocks at 169440831, type: ffs
        dk3: wd0f, 1032192 blocks at 168408639, type: ffs
        dk2: wd0e, 166860288 blocks at 1548351, type: ffs
        dk1: wd0b, 1032192 blocks at 516159, type: swap
        dk0: wd0a, 516096 blocks at 63, type: ffs

Masao

On Wed, 5 Dec 2007 12:53:35 +0900
"Masao Uebayashi" <uebayasigmail.com> wrote:

> > >         wedges:
> > >         wd0d: 6 wedges:
> > >         dk5: wd0h, 314711651 blocks at
173569599, type: ffs
> > >         dk5 at wd0: wd0h
> > >         dk5: 314711651 blocks at 173569599,
type: ffs
> > >         (...)
> >
> > This is redundant ... will fix.
> 
>         http://w
ww.NetBSD.org/~uebayasi/dkctl2.patch
> 
> And
>         # dkctl wd0
>         strategy:
>         wd0d: disksort
> 
>         cache:
>         wd0d: read cache enabled
>         wd0d: write-back cache enabled
>         wd0d: read cache enable is not changeable
>         wd0d: write cache enable is changeable
>         wd0d: cache parameters are not savable
> 
>         wedges:
>         wd0d: 6 wedges:
>         dk5: wd0h, 314711651 blocks at 173569599, type:
ffs
>         dk4: wd0g, 4128768 blocks at 169440831, type:
ffs
>         dk3: wd0f, 1032192 blocks at 168408639, type:
ffs
>         dk2: wd0e, 166860288 blocks at 1548351, type:
ffs
>         dk1: wd0b, 1032192 blocks at 516159, type:
swap
>         dk0: wd0a, 516096 blocks at 63, type: ffs

I like it, but cannot you simply modify it like:

# dkctl wd0
strategy: disksort
read cache enabled
write-back cache enabled

6 wedges
dk5h: ...
...

to avoid redundancy...

-- 
Juan Romero Pardines	- The NetBSD Project
http://plog.xtrarom.org	-
NetBSD/pkgsrc news in Spanish

Let me rephrase...

My patch is calling existing three dkctl(8) commands
(getcache,
listwedges, strategy) internally, when dkctl(8) is invoked
only with
the device argument.  Currently dkctl(8) *always* wants two
arguments
(device + command), which is a bit long to type.

I think this is a change of command syntax.  So I want to
make sure if
this is OK.  If it's OK we can improve things further. 
(E.g., sort
wedges list.)

Masao