Re: mpt Serious performance issues

2011-01-31 Thread Stephan
Hi all,

some new findings on this issue:

-The OpenBSD driver for this chip is called mpi and performs better
(Write speed ~35MB/s, Read speed ~70 MB/s).

-Speking of write caching, the NetBSD driver does not seem to have
code which could activate it. There is a bit defined in mpt_mpilib.h
which could be set in some kind of settings page
(_RAID_VOL0_SETTINGS):

#define MPI_RAIDVOL0_SETTING_WRITE_CACHING_ENABLE   (0x0001)

-The OpenBSD driver has a function to (de)activate the write caching
called mpi_ioctl_cache() (see:
http://www.openbsd.org/cgi-bin/cvsweb/~checkout~/src/sys/dev/ic/mpi.c?rev=1.165;content-type=text%2Fplain
). How would this be implemented in NetBSD? As a hack for testing,
does anybody see a chance to hard-code this bit-setting into the
driver?

Regards,

Stephan


Re: turning off COMPAT_386BSD_MBRPART in disklabel

2011-01-31 Thread Izumi Tsutsui
 PR 44496 notes that COMPAT_386BSD_MBRPART is still enabled in
 disklabel(8), even though it was turned off by default in the kernel
 early in 4.99.x. The PR also notes that it's not harmless to leave it
 on.
 
 I'm inclined to turn it off in disklabel(8) right now. There's no
 particular reason to have it there if the kernel doesn't support it,
 and since fixing up a really old disk doesn't require disklabel(8),
 only fdisk(8) there doesn't seem to be any upgrade/compatibility
 argument to have it in -6.

I wonder if it's worth to add a sysctl knob that returns
whether the running kernel has options COMPAT_386BSD_MBRPART or not
and make disklabel(8) refer it.

---
Izumi Tsutsui


Re: turning off COMPAT_386BSD_MBRPART in disklabel

2011-01-31 Thread Matthias Drochner

dholland-t...@netbsd.org said:
 PR 44496 notes that COMPAT_386BSD_MBRPART is still enabled in
 disklabel(8), even though it was turned off by default in the kernel
 early in 4.99.x. The PR also notes that it's not harmless to leave it
 on.

The PR rather leads to the conclusion that the support for
old Partition IDs in disklabel(8) is suboptimal.
Originally, the code did only consider a partition with the
old ID if no new one was found. This apparently got broken
when extended partition support was added years later.

best regards
Matthias





Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzender des Aufsichtsrats: MinDirig Dr. Karl Eugen Huthmacher
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt