Re: kern/152926: [ahci] [patch] HiPoint Rocket Raid 622 ahci(4) driver patch

2010-12-15 Thread mav
Synopsis: [ahci] [patch] HiPoint Rocket Raid 622 ahci(4) driver patch

State-Changed-From-To: open->closed
State-Changed-By: mav
State-Changed-When: Wed Dec 15 08:27:42 UTC 2010
State-Changed-Why: 
Improved patch committed and merged to 8-STABLE.

http://www.freebsd.org/cgi/query-pr.cgi?pr=152926
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: kern/152926: commit references a PR

2010-12-15 Thread dfilter service
The following reply was made to PR kern/152926; it has been noted by GNATS.

From: dfil...@freebsd.org (dfilter service)
To: bug-follo...@freebsd.org
Cc:  
Subject: Re: kern/152926: commit references a PR
Date: Wed, 15 Dec 2010 08:27:22 + (UTC)

 Author: mav
 Date: Wed Dec 15 08:27:17 2010
 New Revision: 216452
 URL: http://svn.freebsd.org/changeset/base/216452
 
 Log:
   MFC r216309:
   Add IDs of HighPoint RocketRAID 62x cards (Marvell 88SE9128 chips).
   
   PR: kern/152926
   Submitted by:   Mike Tancsa 
   Approved by: re (kib)
 
 Modified:
   stable/8/sys/dev/ahci/ahci.c
 Directory Properties:
   stable/8/sys/   (props changed)
   stable/8/sys/amd64/include/xen/   (props changed)
   stable/8/sys/cddl/contrib/opensolaris/   (props changed)
   stable/8/sys/contrib/dev/acpica/   (props changed)
   stable/8/sys/contrib/pf/   (props changed)
 
 Modified: stable/8/sys/dev/ahci/ahci.c
 ==
 --- stable/8/sys/dev/ahci/ahci.c   Wed Dec 15 01:19:14 2010
(r216451)
 +++ stable/8/sys/dev/ahci/ahci.c   Wed Dec 15 08:27:17 2010
(r216452)
 @@ -172,6 +172,10 @@ static struct {
{0x614511ab, 0x00, "Marvell 88SX6145",  
AHCI_Q_NOFORCE|AHCI_Q_4CH|AHCI_Q_EDGEIS},
{0x91231b4b, 0x11, "Marvell 88SE912x",  AHCI_Q_NOBSYRES},
{0x91231b4b, 0x00, "Marvell 88SE912x",  
AHCI_Q_EDGEIS|AHCI_Q_SATA2|AHCI_Q_NOBSYRES},
 +  {0x06201103, 0x00, "HighPoint RocketRAID 620",  AHCI_Q_NOBSYRES},
 +  {0x06201b4b, 0x00, "HighPoint RocketRAID 620",  AHCI_Q_NOBSYRES},
 +  {0x06221103, 0x00, "HighPoint RocketRAID 622",  AHCI_Q_NOBSYRES},
 +  {0x06221b4b, 0x00, "HighPoint RocketRAID 622",  AHCI_Q_NOBSYRES},
{0x044c10de, 0x00, "NVIDIA MCP65",  AHCI_Q_NOAA},
{0x044d10de, 0x00, "NVIDIA MCP65",  AHCI_Q_NOAA},
{0x044e10de, 0x00, "NVIDIA MCP65",  AHCI_Q_NOAA},
 ___
 svn-src-...@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
 
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: Re: bin/153012: iostat(8) requires an argument to -c option

2010-12-15 Thread Bruce Evans

On Tue, 14 Dec 2010, Alexander Best wrote:


how do you feel about the wording of the vmstat(8) manual? it would be possible
to re-use it for the iostat(8) manual. of course just the -c and -w part.


It's even worse than in iostat(8), mainly due to it being more verbose and
having a similar density of bugs.

%  -c  Repeat the display count times.  The first display is for the
%  time since a reboot and each subsequent report is for the time
%  period since the last display.  If no repeat count is specified,
%  and -w is specified, the default is infinity, otherwise the
%  default is one.

[col -bx stripped the markup, so things like "display count times" are
ungrammatical here.]

"time" should probably be "time interval" or just "interval".

"a reboot" should "the system was booted".  I copied this wording from
systat(1).  The interval between the last reboot and this boot is not
covered as is claimed here, and more so for the interval between any
other reboot than the last one.

"repeat count" gives the same bug as in iostat.  It almost makes sense
in the above (you can read it as "the repeat count specified using -c
or in the normal undocumented way of putting it at the end after the
drives and the undocumented wait count at the end), but before col -bx
strips the markup, `count' is highlighted it can only refer to -c's
count arg.  But as stated in the PR, not specifying thsis count is
a syntax error.  This can be fixed by saying something like "-c option"
instead of "repeat count", but that is insufficient since the repeat
count can also be specified in the normal way without -c.

", otherwise" is a grammar error (comma splice).  It should be something
like "; otherwise," or ".  Otherwise,".

"one" is missing units (seconds) and should be spelled "1", giving
"1 second".

The normal syntax for the wait and count options is undocumented here
too.  It especially needs documentation since it is unusual.  It is
documented as "vmstat ... [drives] [wait [count]] in a comment in
iostat's usage() but not in vmstat's usage().  4.4BSD apparently tried
to kill this traditional syntax by putting it under BACKWARDS_COMPATIBILITY
ifdefs and removing it from man pages, but FreeBSD made its support
unconditional.  4.4BSD vmstat also has less-broken checking than 4.4BSD
iostat for counts and intervals of 0 (these should be allowed), but more-
broken for counts and intervals of < 0 (these should be disallowed).

% 
%  -w  Pause wait seconds between each display.  If no repeat wait

%  interval is specified, the default is 1 second.  The vmstat com-
%  mand will accept and honor a non-integer number of seconds.

"repeat wait interval" is even harder to parse than "repeat count",
especially when the markup of `wait' is restored.  It means the interval
between repetitions, which is the non-marked-up wait interval that may
be specified using -w or in the normal undocumented way.

This spells "1 second" correctly.

This is not as verbose as iostat(1) about the resolution and caveats
for "non-integer" numbers.  Neither says what the format of these
numbers is (it is any floating point format accepted by atof(3)).

This fails to specify the interaction of -w with -c.  This is not needed,
since the description of -c gives it.  However, -w is used more often than
-c, and the interaction starting with -w is simpler, so describing it here
too would be clearer.

Bruce
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: misc/152485: /dev/null seek offset is not reported correctly

2010-12-15 Thread Alexander Best
The following reply was made to PR misc/152485; it has been noted by GNATS.

From: Alexander Best 
To: bug-follo...@freebsd.org
Cc:  
Subject: Re: misc/152485: /dev/null seek offset is not reported correctly
Date: Wed, 15 Dec 2010 16:17:41 +

 the null(4) manual page states that the length of the null device is always
 zero, so there's no need to seek into the file really.
 
 however you might want to post a mail to freebsd-hackers@, if you think this
 should be changed. maybe some developers share your point of view.
 
 cheers.
 alex
 
 -- 
 a13x
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: kern/104624: Sound, mouse and keyboard badly interrupted while I/O (disk, net)

2010-12-15 Thread jh
Synopsis: Sound, mouse and keyboard badly interrupted while I/O (disk, net)

State-Changed-From-To: feedback->closed
State-Changed-By: jh
State-Changed-When: Wed Dec 15 18:21:57 UTC 2010
State-Changed-Why: 
Submitter doesn't experience the problem anymore. Reportedly similar
issues still exist but I think it's better to open new PR(s) with
relevant details for them.

http://www.freebsd.org/cgi/query-pr.cgi?pr=104624
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


misc/153183: New port: graphic/mahotas Efficient Image Processing Library for Python

2010-12-15 Thread Pedro Garcia Freitas

>Number: 153183
>Category:   misc
>Synopsis:   New port: graphic/mahotas Efficient Image Processing Library 
>for Python
>Confidential:   no
>Severity:   non-critical
>Priority:   low
>Responsible:freebsd-bugs
>State:  open
>Quarter:
>Keywords:   
>Date-Required:
>Class:  change-request
>Submitter-Id:   current-users
>Arrival-Date:   Wed Dec 15 18:50:11 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator: Pedro Garcia Freitas
>Release:8.1-RELEASE
>Organization:
>Environment:
FreeBSD freebsd 8.1-RELEASE FreeBSD 8.1-RELEASE #0: Mon Jul 19 02:55:53 UTC 
2010 r...@almeida.cse.buffalo.edu:/usr/obj/usr/src/sys/GENERIC  i386
>Description:
Mahotas is an Image Processing Library for Python.
It includes a couple of algorithms implemented in C++ for speed while operating 
in numpy arrays.

WWW: http://packages.python.org/mahotas/
>How-To-Repeat:

>Fix:
New port. Is installing and uinstalling with this files.
Convert .txt to .shar

Patch attached with submission follows:

# This is a shell archive.  Save it in a file, remove anything before
# this line, and then unpack it by entering "sh file".  Note, it may
# create directories; files and directories will be owned by you and
# have default permissions.
#
# This archive contains:
#
#   Makefile
#   distinfo
#   pkg-descr
#
echo x - Makefile
sed 's/^X//' >Makefile << 'b67911656ef5d18c4ae36cb6741b7965'
X# New ports collection makefile for:   py-mahotas
X# Date created:2010-12-14
X# Whom:Pedro Garcia Freitas 
X#
X# $FreeBSD$
X#
X
XPORTNAME=  mahotas
XPORTVERSION=   0.6.1
XCATEGORIES=graphics python
XMASTER_SITES=  http://pypi.python.org/packages/source/m/mahotas/
XPKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX}
X
XMAINTAINER=s...@sawp.com.br
XCOMMENT=   An Efficient Image Processing Library for Python
X
XBUILD_DEPENDS+=${PYNUMPY}
XPYDISTUTILS_EGGINFO=   
${PYDISTUTILS_PKGNAME:C/[^A-Za-z0-9.]+/_/g}-${PYDISTUTILS_PKGVERSION:C/[^A-Za-z0-9.]+/_/g}-${PYTHON_VERSION:S/thon//}-${OPSYS}-${OSREL}-${ARCH}.egg-info
XPYSETUP=   ${WRKSRC}/setup.py
X
XUSE_GCC=   4.4+
XUSE_PYTHON=2.4+
XUSE_PYDISTUTILS=   yes
X
X.include 
X
X.include 
b67911656ef5d18c4ae36cb6741b7965
echo x - distinfo
sed 's/^X//' >distinfo << '57677d168a5ec21bdf22c9501f075a8f'
XMD5 (mahotas-0.6.1.tar.gz) = 1a605717a60850952bba4895a84b736d
XSHA256 (mahotas-0.6.1.tar.gz) = 
d2eadfa55a60689d7d62c12688ec7acecd7cf10a4e7796a6e24a3457091bf992
XSIZE (x-tile-1.8.1.tar.gz) = 55084
57677d168a5ec21bdf22c9501f075a8f
echo x - pkg-descr
sed 's/^X//' >pkg-descr << '7f9555c3c1089940396b503dd37f3979'
XMahotas is an Image Processing Library for Python.
XIt includes a couple of algorithms implemented in C++ for speed while 
operating in numpy arrays.
X
XNotable algorithms:
X - watershed.
X - thresholding.
X - convex points calculations.
X - hit & miss. thinning.
X - Zernike & Haralick features.
X - freeimage based numpy image loading (requires freeimage libraries to be 
installed).
X - Release the GIL in morphological functions
X - Convolution
X
X
XWWW: http://packages.python.org/mahotas/
7f9555c3c1089940396b503dd37f3979
exit



>Release-Note:
>Audit-Trail:
>Unformatted:
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: ports/153183: New port: graphic/mahotas Efficient Image Processing Library for Python

2010-12-15 Thread linimon
Synopsis: New port: graphic/mahotas Efficient Image Processing Library for 
Python

Responsible-Changed-From-To: freebsd-bugs->freebsd-ports-bugs
Responsible-Changed-By: linimon
Responsible-Changed-When: Wed Dec 15 20:07:38 UTC 2010
Responsible-Changed-Why: 
ports PR.

http://www.freebsd.org/cgi/query-pr.cgi?pr=153183
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: kern/148367: [syscons] [patch] Add loader tunable to override SC_HISTORY_SIZE

2010-12-15 Thread Dmitry Pryanishnikov
The following reply was made to PR kern/148367; it has been noted by GNATS.

From: Dmitry Pryanishnikov 
To: bug-follo...@freebsd.org, lynx.r...@gmail.com
Cc:  
Subject: Re: kern/148367: [syscons] [patch] Add loader tunable to override
 SC_HISTORY_SIZE
Date: Wed, 15 Dec 2010 23:39:21 +0200

 This is a multi-part message in MIME format.
 --00080802010608070103
 Content-Type: text/plain; charset=ISO-8859-1; format=flowed
 Content-Transfer-Encoding: 7bit
 
 
 Hello!
 
The patch has been updated for schistory.c,v 1.21.2.2 (current version in 
 RELENG_8).
 
 Sincerely, Dmitry
 -- 
 nic-hdl: LYNX-RIPE
 
 --00080802010608070103
 Content-Type: text/plain;
  name="syscons-history_size.patch"
 Content-Transfer-Encoding: 7bit
 Content-Disposition: attachment;
  filename="syscons-history_size.patch"
 
 --- sys/dev/syscons/syscons.h.orig 2010-08-23 14:33:35.837815000 +0300
 +++ sys/dev/syscons/syscons.h  2010-12-15 23:16:15.492041513 +0200
 @@ -573,6 +573,7 @@
  
  /* schistory.c */
  #ifndef SC_NO_HISTORY
 +void  sc_init_history(void);
  int   sc_alloc_history_buffer(scr_stat *scp, int lines,
int prev_ysize, int wait);
  void  sc_free_history_buffer(scr_stat *scp, int prev_ysize);
 --- sys/dev/syscons/syscons.c.orig 2010-08-23 14:33:35.827279000 +0300
 +++ sys/dev/syscons/syscons.c  2010-12-15 23:16:15.494051860 +0200
 @@ -2726,8 +2726,12 @@
  int i;
  
  /* one time initialization */
 -if (init_done == COLD)
 +if (init_done == COLD) {
sc_get_bios_values(&bios_value);
 +#ifndef SC_NO_HISTORY
 +  sc_init_history();
 +#endif
 +}
  init_done = WARM;
  
  /*
 --- sys/dev/syscons/schistory.c.orig   2010-12-15 22:44:32.506598000 +0200
 +++ sys/dev/syscons/schistory.c2010-12-15 23:16:15.494051860 +0200
 @@ -41,6 +41,7 @@
  #include 
  #include 
  #include 
 +#include 
  
  #if defined(__sparc64__) || defined(__powerpc__)
  #include 
 @@ -77,17 +78,37 @@
  static intextra_history_size
= SC_MAX_HISTORY_SIZE - SC_HISTORY_SIZE*MAXCONS;
  
 +static intsc_history_size = SC_HISTORY_SIZE;
 +
 +SYSCTL_DECL(_hw_syscons);
 +SYSCTL_INT(_hw_syscons, OID_AUTO, history_size, CTLFLAG_RDTUN, 
&sc_history_size, 0,
 +"Number of history buffer lines");
 +
  /* local functions */
  static void copy_history(sc_vtb_t *from, sc_vtb_t *to);
  static void history_to_screen(scr_stat *scp);
  
 +/* tune history buffer size */
 +void
 +sc_init_history(void)
 +{
 +
 +  TUNABLE_INT_FETCH("hw.syscons.history_size", &sc_history_size);
 +  if (sc_history_size < ROW * 4)
 +  sc_history_size = ROW * 4;
 +  extra_history_size =
 +  ((sc_history_size * MAXCONS * MAXSC) > SC_MAX_HISTORY_SIZE ?
 +  sc_history_size * MAXCONS * MAXSC : SC_MAX_HISTORY_SIZE) -
 +  sc_history_size * MAXCONS;
 +}
 +
  /* allocate a history buffer */
  int
  sc_alloc_history_buffer(scr_stat *scp, int lines, int prev_ysize, int wait)
  {
/*
 * syscons unconditionally allocates buffers up to 
 -   * SC_HISTORY_SIZE lines or scp->ysize lines, whichever 
 +   * sc_history_size lines or scp->ysize lines, whichever 
 * is larger. A value greater than that is allowed, 
 * subject to extra_history_size.
 */
 @@ -98,7 +119,7 @@
int delta;  /* lines to put back */
  
if (lines <= 0)
 -  lines = SC_HISTORY_SIZE;/* use the default value */
 +  lines = sc_history_size;/* use the default value */
  
/* make it at least as large as the screen size */
lines = imax(lines, scp->ysize);
 @@ -111,13 +132,13 @@
delta = 0;
if (prev_history) {
cur_lines = sc_vtb_rows(history);
 -  min_lines = imax(SC_HISTORY_SIZE, prev_ysize);
 +  min_lines = imax(sc_history_size, prev_ysize);
if (cur_lines > min_lines)
delta = cur_lines - min_lines;
}
  
/* lines up to min_lines are always allowed. */
 -  min_lines = imax(SC_HISTORY_SIZE, scp->ysize);
 +  min_lines = imax(sc_history_size, scp->ysize);
if (lines > min_lines) {
if (lines - min_lines > extra_history_size + delta) {
/* too many lines are requested */
 @@ -196,7 +217,7 @@
return;
  
cur_lines = sc_vtb_rows(history);
 -  min_lines = imax(SC_HISTORY_SIZE, prev_ysize);
 +  min_lines = imax(sc_history_size, prev_ysize);
extra_history_size += (cur_lines > min_lines) ? 
  cur_lines - min_lines : 0;
  
 --- share/man/man4/syscons.4.orig  2010-03-02 03:56:55.0 +0200
 +++ share/man/man4/syscons.4   2010-12-15 23:16:15.495058710 +0200
 @@ -442,6 +442,20 @@
  for a keyboard device if it is not currently attached to 

misc/153200: post-boot /etc/rc.d/network_ipv6 start can miss neighbour discovery bootstrap

2010-12-15 Thread George Michaelson

>Number: 153200
>Category:   misc
>Synopsis:   post-boot /etc/rc.d/network_ipv6 start can miss neighbour 
>discovery bootstrap
>Confidential:   no
>Severity:   serious
>Priority:   high
>Responsible:freebsd-bugs
>State:  open
>Quarter:
>Keywords:   
>Date-Required:
>Class:  sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Dec 16 04:50:09 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator: George Michaelson
>Release:8.1-RELEASE
>Organization:
APNIC
>Environment:
FreeBSD amchur.rand.apnic.net 8.1-RELEASE FreeBSD 8.1-RELEASE #0: Mon Jul 19 
02:36:49 UTC 2010 r...@mason.cse.buffalo.edu:/usr/obj/usr/src/sys/GENERIC  
amd64
>Description:
We enabled IPv6 in rc.conf and ran /etc/rc.d/network_ipv6 start.

ifconfig bce0 showed nd6 in an indeterminate state.

Thereafter, while hosts on the local network segment could successfully ping6 
the public IPv6, including the edge Cisco router, the cisco lost sight of the 
host if the packets were NOT emitted from its own binding to the local network 
segment.

If we pinged the router, it came back, but 20-30sec after we stopped outbound 
IPv6, inbound routes dropped. on the Cisco:

gw1#sh ipv6 neighbors | inc 6660
2401:2000:6660::102 0 -   INCMP Gi0/2.113

We rebooted. the system is now showing valid V6 routing all the time, even when 
the ipv6 neighbors list shows cache ageing.
>How-To-Repeat:
bootstrap IPv6 after boot time via /etc/rc.d/network_ipv6 start.

route inbound ping6 not on link-local, with passive interface. you should be 
able to demonstrate this, if your ifconfig shows nd6 has not completed.
>Fix:
add a dependency into network_ipv6 which forces the right kernel sysctl and/or 
other runtime to enable all link-local, multicast bindings, and to set the 
interface nd6 state correctly. This might be as simple as a pre-dependency in 
the startup script.

>Release-Note:
>Audit-Trail:
>Unformatted:
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"


Re: conf/153200: post-boot /etc/rc.d/network_ipv6 start can miss neighbour discovery bootstrap

2010-12-15 Thread linimon
Synopsis: post-boot /etc/rc.d/network_ipv6 start can miss neighbour discovery 
bootstrap

Responsible-Changed-From-To: freebsd-bugs->freebsd-rc
Responsible-Changed-By: linimon
Responsible-Changed-When: Thu Dec 16 05:03:45 UTC 2010
Responsible-Changed-Why: 
Over to maintainer(s).

http://www.freebsd.org/cgi/query-pr.cgi?pr=153200
___
freebsd-bugs@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"