Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Ivan Voras
On 6 December 2010 19:44, Pawel Jakub Dawidek p...@freebsd.org wrote: On Mon, Dec 06, 2010 at 12:18:03PM +, Ivan Voras wrote: Author: ivoras Date: Mon Dec  6 12:18:02 2010 New Revision: 216230 URL: http://svn.freebsd.org/changeset/base/216230 Log:   Use GEOM stripesize field when

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Ivan Voras
On 6 December 2010 20:22, Pawel Jakub Dawidek p...@freebsd.org wrote: On Mon, Dec 06, 2010 at 07:44:53PM +0100, Pawel Jakub Dawidek wrote: On Mon, Dec 06, 2010 at 12:18:03PM +, Ivan Voras wrote: Author: ivoras Date: Mon Dec  6 12:18:02 2010 New Revision: 216230 URL:

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread John Baldwin
On Monday, December 06, 2010 2:53:27 pm Pawel Jakub Dawidek wrote: On Mon, Dec 06, 2010 at 08:35:36PM +0100, Ivan Voras wrote: Please persuade me on technical grounds why ashift, a property intended for address alignment, should not be set in this way. If your answer is I don't know but you

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Ivan Voras
Firstly, thank you, your explanations and questions are very good and address the problems in a good way to discuss about it. I respect that you took the time to write this answer! On 6 December 2010 20:53, Pawel Jakub Dawidek p...@freebsd.org wrote: On Mon, Dec 06, 2010 at 08:35:36PM +0100,

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Ivan Voras
On 6 December 2010 21:18, John Baldwin j...@freebsd.org wrote: On Monday, December 06, 2010 2:53:27 pm Pawel Jakub Dawidek wrote: On Mon, Dec 06, 2010 at 08:35:36PM +0100, Ivan Voras wrote: Please persuade me on technical grounds why ashift, a property intended for address alignment, should

svn commit: r216241 - head/sys/dev/xen/blkfront

2010-12-06 Thread Colin Percival
Author: cperciva Date: Mon Dec 6 20:40:15 2010 New Revision: 216241 URL: http://svn.freebsd.org/changeset/base/216241 Log: Set correct maximum I/O length. We can only handle I/O of up to max_request_segments * PAGE_SIZE if the I/O is page-aligned; the largest I/O we can guarantee will

svn commit: r216242 - head/sys/compat/ndis

2010-12-06 Thread Bernhard Schmidt
Author: bschmidt Date: Mon Dec 6 20:54:53 2010 New Revision: 216242 URL: http://svn.freebsd.org/changeset/base/216242 Log: Implement NdisGetRoutineAddress and MmGetSystemRoutineAddress used in newer Ralink drivers. Submitted by: Paul B Mahol onemda at gmail.com Modified:

svn commit: r216243 - head/usr.sbin/sysinstall

2010-12-06 Thread Bruce Cran
Author: brucec Date: Mon Dec 6 20:55:14 2010 New Revision: 216243 URL: http://svn.freebsd.org/changeset/base/216243 Log: Ignore any failures with the local distribution since it's not present on release CDs and so will normally fail. installCommit() returns a DITEM_ value, not a Boolean.

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Alexander Motin
On 06.12.2010 22:18, John Baldwin wrote: On Monday, December 06, 2010 2:53:27 pm Pawel Jakub Dawidek wrote: On Mon, Dec 06, 2010 at 08:35:36PM +0100, Ivan Voras wrote: Please persuade me on technical grounds why ashift, a property intended for address alignment, should not be set in this way.

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Bruce Cran
On Mon, Dec 06, 2010 at 09:31:39PM +0100, Ivan Voras wrote: For what it's worth, apparently linux has the concept of physical and logical sector sizes (possibly in addition to stripe size), with physical being 4096 and logical 512, for example: # hdparm -I /dev/sde | grep size Logical

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Ivan Voras
On 6 December 2010 22:16, Bruce Cran br...@cran.org.uk wrote: On Mon, Dec 06, 2010 at 09:31:39PM +0100, Ivan Voras wrote: For what it's worth, apparently linux has the concept of physical and logical sector sizes (possibly in addition to stripe size), with physical being 4096 and logical 512,

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Alexander Motin
On 06.12.2010 23:22, Ivan Voras wrote: On 6 December 2010 22:16, Bruce Cranbr...@cran.org.uk wrote: On Mon, Dec 06, 2010 at 09:31:39PM +0100, Ivan Voras wrote: For what it's worth, apparently linux has the concept of physical and logical sector sizes (possibly in addition to stripe size),

Re: svn commit: r216016 - head/sys/sparc64/include

2010-12-06 Thread Marius Strobl
On Thu, Dec 02, 2010 at 11:27:45AM -0600, Alan Cox wrote: Marius Strobl wrote: On Wed, Dec 01, 2010 at 12:19:17PM -0600, Alan Cox wrote: Marius Strobl wrote: On Mon, Nov 29, 2010 at 08:23:08PM +0100, Marius Strobl wrote: On Tue, Nov 30, 2010 at 12:31:31AM +0600, Max Khon

Re: svn commit: r216016 - head/sys/sparc64/include

2010-12-06 Thread mdf
On Mon, Dec 6, 2010 at 2:07 PM, Marius Strobl mar...@alchemy.franken.de wrote: [lots of snip] With that one the kernel now survies memguard_init() but then panics right afterwards when kmeminit() calls kmem_suballoc(): KDB: debugger backends: ddb KDB: current backend: ddb Copyright (c)

Re: svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2010-12-06 Thread Andriy Gapon
on 07/12/2010 00:00 John Baldwin said the following: It is probably the 4K logical sector size that needs to come up with a new field, not vice versa. Just expressing my overall confusion - 4K would be the physical size and 512 would be the logical one? My thinking: on a platter it's a 4K

svn commit: r216246 - head/bin/sh

2010-12-06 Thread Jilles Tjoelker
Author: jilles Date: Mon Dec 6 23:49:27 2010 New Revision: 216246 URL: http://svn.freebsd.org/changeset/base/216246 Log: sh: Improve internal-representation-to-text code to avoid binary output. The code to translate the internal representation to text did not know about various

svn commit: r216247 - in head/lib/msun: man src

2010-12-06 Thread David Schultz
Author: das Date: Tue Dec 7 02:19:15 2010 New Revision: 216247 URL: http://svn.freebsd.org/changeset/base/216247 Log: Fix various nits in style and comments that were pointed out by bde. Code changes verified with md5. Modified: head/lib/msun/man/math.3 head/lib/msun/src/e_log2.c

svn commit: r216248 - head/lib/msun/src

2010-12-06 Thread David Schultz
Author: das Date: Tue Dec 7 03:29:36 2010 New Revision: 216248 URL: http://svn.freebsd.org/changeset/base/216248 Log: Another minor nit: Make sure the constant here is a float so the compiler doesn't promote the entire expression to double. Modified: head/lib/msun/src/e_log2f.c Modified: