Re: Panic on current amd64
On 03/31/15 02:15, Konstantin Belousov wrote:
On Tue, Mar 31, 2015 at 02:51:47AM +0300, Konstantin Belousov wrote:
On Tue, Mar 31, 2015 at 01:41:04AM +0300, Gleb Smirnoff wrote:
On Tue, Mar 31, 2015 at 12:30:04AM +0200, Hans Petter Selasky wrote:
H On 03/30/15 23:19, Gleb Smirnoff wrote:
H On Sat, Mar 28, 2015 at 03:02:28PM -0700, Manfred Antar wrote:
H M I get the following panic on current svn ver r280793:
H M
H M Sat Mar 28 14:41:28 PDT 2015
H M
H M FreeBSD/amd64 (pozo.com) (ttyu0)
H M
H M panic: Invalid CPU in callout 16
H
H The same happened to me in the OFED code. After investigation
H it appeared that for some unknown reason, the OFED code used
H /usr/include/sys/callout.h instead of SYSDIR/sys/callout.h,
H that yield in wrong value passing as parameter.
H
H I failed to reproduce the problem. :( So, the fix is a rebuild
H of kernel. But the right fix is to understand what went wrong
H in the previous build.
H
H How did you compile the OFED stuff? Did you set the SYSDIR variable when
H building?
I just have this in my kernel config:
options OFED
device mlxen
Quick grep of the sys/ofed immediately shows the following:
sys/ofed/include/linux/wait.h:#include sys/param.h
sys/ofed/include/linux/wait.h:#include sys/systm.h
sys/ofed/include/linux/wait.h:#include sys/sleepqueue.h
sys/ofed/include/linux/wait.h:#include sys/kernel.h
sys/ofed/include/linux/wait.h:#include sys/proc.h
ys/ofed/include/linux/timer.h:#include sys/callout.h
sys/ofed/include/linux/types.h:#include sys/cdefs.h
sys/ofed/include/linux/types.h:#include sys/types.h
sys/ofed/include/linux/types.h:#include sys/param.h
sys/ofed/include/linux/types.h:#include sys/systm.h
and so on.
Err, I am sorry, scratch this.
Hi,
Could you:
cd sys/amd64/conf
config YOURCONFIG
cd ../compile/YOURCONFIG
less Makefile
and see if it includes something outside /sys ?
OFED uses its own CFLAGS and check if some include directives have space
after the -Ixxxspace, because then this simple flag conversion will fail:
OFEDCFLAGS= ${CFLAGS:N-I*} ${OFEDINCLUDES} ${CFLAGS:M-I*} ${OFEDNOERR}
Thank you!
--HPS
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Re: T40 bootloop on CAM status: Command timeout on both 10.1 and -CURRENT
Currently the cdrom drive and yes it works fine till 10 with the legacy ATA stack. Verbose boot doesn't give out any errors or logs beside what's shown in the picture I have attached with the first email. Regards, Pietro Sammarco On 30/03/2015 21:38, Steven Hartland wrote: Is there anything connected to the second channel, if so what. Was this working in a previous version and if so which one and what was the verbose boot log from it? On 30/03/2015 19:07, bsdml wrote: Hello Kevin, thanks for your clarification. Unfortunately I wasn't aware that the T40 and *4's line used a SATA-PATA convertor and especially that was going to clash with the new ATA stack in FreeBSD. Either OpenBSD and NetBSD do work out of the box without any hassle, however I'd still prefer to use FreeBSD on it as I have been using FreeBSD for about 8 years now and I am very comfortable with it. The question at this point is, is there any hope to see this issue resolved in the future? Or will I have to give up to the second ATA channel in order to use FreeBSD? Regards, Pietro Sammarco On 30/03/2015 06:17, Kevin Oberman wrote: On Sun, Mar 29, 2015 at 2:27 AM, Wolfgang Zenker wolfg...@lyxys.ka.sub.org mailto:wolfg...@lyxys.ka.sub.org wrote: Hi, * bsdml pietro.bs...@gmail.com mailto:pietro.bs...@gmail.com [150329 01:34]: since I tried to install FreeBSD 10.1 on my recently purchased T40 I got stuck at this annoying bootloop that says ATAPY_IDENTIFY. ACB: a1 00 00 00 00 40 00 00 00 etc etc.. CAM status: Command timeout. I have also tried latest 11-CURRENT snapshot and it did not make any difference at all, it is affected from the same exact bootloop. [..] It seems like there might be an issue with the CAM ATA stack that is clashing with the PATA controller on my T40. I had the same problem on an ancient T42p. In my case, disabling the second ata channel allowed me to boot. I added the following line to /boot/device.hints: hint.ata.1.disabled=1 This is an annoying side-effect of the brain-dead SATA-PATA converter in that generation of ThinkPads. The Intel ICH6 chipset is SATA, but, for reasons known ot IBM/Lenovo, the systems used PATA drives! So they has a SATA-PATA converter built in that screwed up a LOT of things, mostly compromising performance and generating assorted log entries. Looks like that also is broken in modern ATA support if a drive is not present. This was always my biggest complaint with this laptop (T42) which I used for several years until I retired and returned to so it could be excessed legally as it was government property (and, I didn't really want it, even if I could have kept it). Not an awful system, but this one issue was really annoying to me. -- Kevin Oberman, Network Engineer, Retired E-mail: rkober...@gmail.com mailto:rkober...@gmail.com ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Build failed in Jenkins: FreeBSD_HEAD #2610
See https://jenkins.freebsd.org/job/FreeBSD_HEAD/2610/changes Changes: [markj] Fix a misparenthesization that could cause a crash if TERM is not set. Reported by:Coverity (internal) MFC after: 3 days Sponsored by: EMC / Isilon Storage Division [cperciva] Partially revert r278118 now that the required logic for deciding whether freebsd-update can be useful has moved into the firstboot_freebsd_update script. [kib] Provide workaround for a performance issue with the popcnt instruction on Intel processors. Clear spurious dependency by explicitely xoring the destination register of popcnt. Use bitcount64() instead of re-implementing SWAR locally, for processors without popcnt instruction. Reviewed by:jhb Discussed with: jilles (previous version) Sponsored by: The FreeBSD Foundation [cperciva] Add bits for building EC2 disk images. Make logic for uploading these to create EC2 AMIs will come in a later commit. [np] cxgbe/tom: return rx credits promptly if the socket buffer's low water mark cannot be reached because the window advertised to the peer isn't wide enough. While here, tweak the normal credit return too. MFC after: 1 month [rrs] Adopt jhb's suggested changes, updated comments and callout_migration() moving to kern/kern_timeout.c This does *not* address his -1 - NOCPU comment. Sponsored by: Netflix Inc. [rrs] Change the c_iflags and c_flags fields to short rather then int. This allows us to keep the KPI the same. Discussed and brain-stormed with imp (thanks for the help Warner!) Sponsored by: Netflix Inc. [bdrewery] Fix --one-file-system to include the directory encountered rather than excluding it. This was broken in 3.0.4 (r238856). Obtained from: https://github.com/libarchive/libarchive/commit/fa9e61 MFC after: 3 days Sponsored by: EMC / Isilon Storage Division [glebius] Catch up on r271387 and remove unused parameter from VOP_GETPAGES_ASYNC(). [andrew] Restore setting cpufuncs on arm1176, it was removed by accident with the arm1136 code. Reviewed by:ian [dim] Add llvm patch corresponding to r280865. [jhb] Wait 100 microseconds for a local APIC to dispatch each startup-related IPI rather than 20. The MP 1.4 specification states in Appendix B.2: A period of 20 microseconds should be sufficient for IPI dispatch to complete under normal operating conditions. (Note that this appears to be separate from the 10 millisecond (INIT) and 200 microsecond (STARTUP) waits after the IPIs are dispatched.) The Intel SDM is silent on this issue as far as I can tell. At least some hardware requires 60 microseconds as noted in the PR, so bump this to 100 to be on the safe side. PR: 197756 Reported by:zap...@berentweb.com MFC after: 1 week [emaste] llvm: Backport upstream r229195 to fix arm64 TLS relocations As is described at http://llvm.org/bugs/show_bug.cgi?id=22408, the GNU linkers ld.bfd and ld.gold currently only support a subset of the whole range of AArch64 ELF TLS relocations. Furthermore, they assume that some of the code sequences to access thread-local variables are produced in a very specific sequence. When the sequence is not as the linker expects, it can silently mis-relaxe/mis-optimize the instructions. Even if that wouldn't be the case, it's good to produce the exact sequence, as that ensures that linkers can perform optimizing relaxations. This patch: * implements support for 16MiB TLS area size instead of 4GiB TLS area size. Ideally clang would grow an -mtls-size option to allow support for both, but that's not part of this patch. * by default doesn't produce local dynamic access patterns, as even modern ld.bfd and ld.gold linkers do not support the associated relocations. An option (-aarch64-elf-ldtls-generation) is added to enable generation of local dynamic code sequence, but is off by default. * makes sure that the exact expected code sequence for local dynamic and general dynamic accesses is produced, by making use of a new pseudo instruction. The patch also removes two (AArch64ISD::TLSDESC_BLR, AArch64ISD::TLSDESC_CALL) pre-existing AArch64-specific pseudo SDNode instructions that are superseded by the new one (TLSDESC_CALLSEQ). Submitted by: Kristof Beyls Differential Revision: https://reviews.freebsd.org/D2175 [dim] Pull in r233552 from upstream libc++ trunk (by Eric Fiselier): [libcxx] Fix PR22771 - Support access control SFINAE in the library version of is_convertible. Summary: Currently the conversion check does not take place in a context where access control SFINAE is applied. This patch changes the context of the test expression so that SFINAE occurs if access control does not permit the conversion. Related bug: https://llvm.org/bugs/show_bug.cgi?id=22771 Reviewers: mclow.lists, rsmith, dim Reviewed By: dim Subscribers: dim, rodrigc, emaste, cfe-commits Differential Revision:
Re: T40 bootloop on CAM status: Command timeout on both 10.1 and -CURRENT
Verbose boot of the working OS version was what I was after, which should provide details of what its detecting ;-) Also a camcontrol identify of the cdrom from the working version may also be useful On 31/03/2015 09:54, Pietro Sammarco wrote: Currently the cdrom drive and yes it works fine till 10 with the legacy ATA stack. Verbose boot doesn't give out any errors or logs beside what's shown in the picture I have attached with the first email. ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Build failed in Jenkins: FreeBSD_HEAD #2611
See https://jenkins.freebsd.org/job/FreeBSD_HEAD/2611/changes Changes: [jhibbits] CCSRBAR_VA is mpc85xx-specific, so add guards, and include the proper header file for it. MFC after: 1 month [jhibbits] machine/fdt.h no longer exists for powerpc. MFC after: 1 month [amdmi3] - Remove more files when MK_ZONEINFO == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 [amdmi3] - Remove more files when MK_TESTS_SUPPORT == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 [amdmi3] - Remove more files when MK_LEGACY_CONSOLE == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 [amdmi3] - Remove more files when MK_KERBEROS_SUPPORT == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 [amdmi3] - Remove more files when MK_KDUMP == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 [amdmi3] - Remove more files when MK_JAIL == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 [cperciva] It would help if I committed the right patch... [amdmi3] - Remove more files when MK_CASPER == no MFC after: 1 week Reviewed by:ngie Approved by:ngie Differential Revision: D1600 -- [...truncated 191296 lines...] # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:379:5: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-err_line_no, ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:380:5: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-err_col_no); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:670:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, statsdir remote configuration ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:724:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen file remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:733:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen type remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:748:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, err); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil)
Build failed in Jenkins: FreeBSD_HEAD #2612
See https://jenkins.freebsd.org/job/FreeBSD_HEAD/2612/changes Changes: [ngie] - Fix -Wsign issue - Bump up to WARNS=6 [ngie] Minor cleanup before converting to ATF testcases - Remove blank (tab-only) lines. - Fix -Wunused warnings. - Bump up to WARNS= 6 [ngie] Cleanup and do minor refactoring before converting testcases to ATF - Convert errx(-1, ..) to errx(1, ..) - Move the aio(4) checks to a single function (aio_available); use modfind(2) instead of depending on SIGSYS (doesn't work when aio(4) support is missing, not documented in the aio syscall manpages). - Use aio_available liberally in the testcase functions - Use mkstemp(3) + unlink(2) instead of mktemp(3) - Fix some -Wunused warnings - Bump WARNS to 6 MFC after: 1 week Submitted by: mjohnston [*] Sponsored by: EMC / Isilon Storage Division -- [...truncated 193766 lines...] ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:670:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, statsdir remote configuration ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:724:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen file remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:733:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen type remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:748:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, err); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:794:9: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-line_no); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:802:9: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-line_no); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:810:9: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-line_no);
Re: [PATCH] Adding backlight support for the i915 driver.
2015-03-27 16:53 GMT+01:00 Hans Petter Selasky h...@selasky.org: On 03/27/15 16:01, Ranjan1018 . wrote: This patch exposes the backlight support via a sysctl: set the backlight to 10%: # sysctl hw.dri.0.i915_backlight=10 hw.dri.0.i915_backlight: 25 - 10 set the backlight to 50%: # sysctl hw.dri.0.i915_backlight=50 hw.dri.0.i915_backlight: 10 - 50 decrease the current backlight value: # sysctl hw.dri.0.i915_backlight=-1000 hw.dri.0.i915_backlight: 50 - 43 increment the current backlight value: # sysctl hw.dri.0.i915_backlight=1000 hw.dri.0.i915_backlight: 43 - 51 # sysctl hw.dri.0.i915_backlight=1000 hw.dri.0.i915_backlight: 51 - 60 I am running this path on for about a week without issue. This path can be found at: https://github.com/maurizio-emmex/i915_backlight_freebsd I thank Elizabeth Myers, elizabeth at interlinked dot me, for the idea of adding the backlight support for the i915 driver and for the original patch. Regards, Maurizio Maybe you want to use CTLFLAG_RWTUN so that it also can be set from /boot/loader.conf ? --HPS The ability to set the backlight at startup may be useful. With this new patch I expose two ways to do this: - with the tunable drm.i915.init_backlight in /boot/loader.conf, just after the driver initialization (eg. drm.i915.init_backlight=20) - with the OID hw.dri.0.i915_backlight in /etc/sysctl.conf as suggested by Hans I don’t know if setting the backlight with the tunable drm.i915.init_backlight is useful, but I have already written the code and is simple to remove it. The patch file is i915_backlight.patch at https://github.com/maurizio-emmex/i915_backlight_freebsd Regards, Maurizio ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Re: Phabricator -- can't update review
On Mar 31, 2015, at 7:56 PM, Devin Teske dte...@freebsd.org wrote: On Mar 31, 2015, at 7:39 PM, Devin Teske dte...@freebsd.org wrote: Hi, I’ve been beating my head on the below issue for an hour now. I’m getting frustrated and don’t understand what it is trying to say. head $ arc diff —update D2105 Exception Field “revisionID” occurs twice in commit message! So I go and look at the commit message. I think it is confused. I am at an impasse. I’m starting to consider closing D2105 and creating a new entry ;( Trace available upon request, but it’s pretty boring stuff. Could use some help here. Full trace here: http://pastebin.com/NzBvDruM http://pastebin.com/NzBvDruM Eitan found it. I needed to remove Differential Review:” line from my commit message. What was non-obvious was that — when using SVN — you can’t see that this line is already being added by the arc template (iirc from Eitan). Removing the Differential Review line from the commit message makes it appear as the line is no longer there (using svn; using hg or git you can see that it is appended to the end of the summary). Just need to take it on Faith that it’s there even if not displayed (you’ll get a warning if it’s not there, so no harm no fowl). All is good now. Thanks again Eitan! — Devin ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Re: Phabricator -- can't update review
On Mar 31, 2015, at 7:39 PM, Devin Teske dte...@freebsd.org wrote: Hi, I’ve been beating my head on the below issue for an hour now. I’m getting frustrated and don’t understand what it is trying to say. head $ arc diff —update D2105 Exception Field “revisionID” occurs twice in commit message! So I go and look at the commit message. I think it is confused. I am at an impasse. I’m starting to consider closing D2105 and creating a new entry ;( Trace available upon request, but it’s pretty boring stuff. Could use some help here. Full trace here: http://pastebin.com/NzBvDruM ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Re: [CFT] Call for testing pkg 1.5.0
Hi, On Tue, Mar 31, 2015 at 9:23 PM, Baptiste Daroussin b...@freebsd.org wrote: Add WITH_PKG=devel in your build make.conf then pkg upgrade will want you to upgrade to 1.4.99.16 (which is pkg 1.5.0 beta1) This does not work for me. # cat /etc/make.conf |grep PKG WITH_PKGNG=yes WITH_PKG=devel # pkg upgrade Updating FreeBSD repository catalogue... FreeBSD repository is up-to-date. All repositories are up-to-date. Checking for upgrades (218 candidates): 100% Processing candidates (218 candidates): 100% Checking integrity... done (0 conflicting) Your packages are up to date. # pkg -v 1.4.12 # pkg info |grep pkg pkg-1.4.12 Package manager Instead, the following succeded: # cd /usr/ports/ports-mgmt/pkg-devel # make reinstall ... # pkg -v 1.4.99.16 All the best, Sergei ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Phabricator -- can't update review
Hi, I’ve been beating my head on the below issue for an hour now. I’m getting frustrated and don’t understand what it is trying to say. head $ arc diff —update D2105 Exception Field “revisionID” occurs twice in commit message! So I go and look at the commit message. I think it is confused. I am at an impasse. I’m starting to consider closing D2105 and creating a new entry ;( Trace available upon request, but it’s pretty boring stuff. Could use some help here. — Thanks in advance, Devin ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Build failed in Jenkins: FreeBSD_HEAD #2614
See https://jenkins.freebsd.org/job/FreeBSD_HEAD/2614/changes Changes: [kevlo] Remove bogus cast. [ae] The offset variable has been cleared all bits except IP6F_OFF_MASK. Use ip6f_mf variable instead of checking its bits. [emaste] unwind-d2 build workaround for arm64 The __builtin_init_dwarf_reg_size_table function is unimplemented in clang 3.6 for AArch64. Comment it out for now and replace it with a message and abort. Tracked in upstream LLVM PR 22997 https://llvm.org/bugs/show_bug.cgi?id=22997 Submitted by: andrew [emaste] Correct copyright typo -- [...truncated 185022 lines...] ip_file-fname, ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:379:5: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-err_line_no, ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:380:5: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-err_col_no); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:670:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, statsdir remote configuration ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:724:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen file remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:733:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen type remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:748:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, err); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:794:9: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-line_no); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil)
Build failed in Jenkins: FreeBSD_HEAD #2613
See https://jenkins.freebsd.org/job/FreeBSD_HEAD/2613/changes Changes: [ganbold] Add kernel config files for Hardkernel Odroid-C1 and Visson ATV-102 devices. Submitted by: John Wehle Approved by: stas (mentor) [ganbold] Add device tree files for Hardkernel Odroid-C1 and Visson ATV-102 devices. Submitted by: John Wehle Approved by:stas (mentor) [ganbold] Add necessary changes to support various Amlogic SoC devices specially aml8726-m6 and aml8726-m8b SoC based devices. aml8726-m6 SoC exist in devices such as Visson ATV-102. Hardkernel ODROID-C1 board has aml8726-m8b SoC. The following support is included: Basic machdep code SMP Interrupt controller Clock control driver (aka gate) Pinctrl Timer Real time clock UART GPIO I2C SD controller SDXC controller USB Watchdog Random number generator PLL / Clock frequency measurement Frame buffer Submitted by: John Wehle Approved by:stas (mentor) [jch] Use appropriate timeout_t* instead of void* in tcp_timer_activate() Suggested by: imp Differential Revision: https://reviews.freebsd.org/D2154 Reviewed by:imp, jhb Approved by:jhb [andrew] Add the arm64 code to the runtime linker. It's not able to be built as we still need libc_pic for a few things, but this is expected to be ready soon. Differential Revision: https://reviews.freebsd.org/D2136 Reviewed by:kib Sponsored by: The FreeBSD Foundation -- [...truncated 188046 lines...] y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:379:5: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-err_line_no, ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:380:5: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? ip_file-err_col_no); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:670:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, statsdir remote configuration ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil) ^ --- all_subdir_tcpdump --- --- nlpid.o --- cc -O2 -pipe -Ihttps://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/tcpdump/tcpdump -Ihttps://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/tcpdump/tcpdump/../../../contrib/tcpdump -DHAVE_CONFIG_H -D_U_=__attribute__((unused)) -DINET6 -DLBL_ALIGN -DHAVE_CAPSICUM -Ihttps://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/obj/builds/FreeBSD_HEAD/tmp/usr/include/openssl -DHAVE_LIBCRYPTO -DHAVE_OPENSSL_EVP_H -DHAVE_NET_PFVAR_H -std=gnu99 -fstack-protector -Wsystem-headers -Werror -Wall -Wno-format-y2k -W -Wno-unused-parameter -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Wno-uninitialized -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable -Wno-tautological-compare -Wno-unused-value -Wno-parentheses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-local-typedef -Qunused-arguments -c https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/tcpdump/tcpdump/../../../contrib/tcpdump/nlpid.c -o nlpid.o --- all_subdir_ntp --- https://jenkins.freebsd.org/job/FreeBSD_HEAD/ws/usr.sbin/ntp/ntpd/../../../contrib/ntp/ntpd/ntp_parser.y:724:13: error: use of undeclared identifier 'ip_file'; did you mean 'ip_fil'? yyerror(ip_file, filegen file remote config ignored); ^~~ ip_fil y.tab.c:1157:1: note: 'ip_fil' declared here YYPARSE_DECL() ^ y.tab.c:89:51: note: expanded from macro 'YYPARSE_DECL' # define YYPARSE_DECL() yyparse(struct FILE_INFO *ip_fil)
[CFT] Call for testing pkg 1.5.0
Hi all, We just released pkg 1.5.0 beta1 (in ports-mgmt/pkg-devel), Here is what happened since pkg 1.4.0: - pkg has grown with an initial support for provides/requires: this is a naive version but good enough to at least make major upgrade of php safer as well as making pear/pecl maintenance saner (note that this will need modifications in the ports tree) - Lots of new regression tests has been added, which allows us not less break your systems an unexpected way (do not worry there are still rooms for breakage) - Initial support for OS X - Initial support for NetBSD/EdjeBSD - Update most of the bundled third party software has been updated to their latest version - Improve the messages reported by pkg (and probably make some other worse) - Properly support file flags - Implement argument support for custom keywords - Extend setting credential via plist to allow to set file flags - Make credential syntax via plist more flexible allow to only defines the first fields and not latest for example @(user,,) can now be written just @(user) - pkg updating now supports case insensitive matching - pkg create now support a verbose mode - Add an option to change the default on question, until now the default answer was No with that option set it would be Yes - lots of fixes to pkg audit -r - Global memory usage reduction and speed up - Improvements and cleanup on pkg alias - pkg annotate --show --all has been fixed - Make pkg.h C++ friendly - Lots of improvements in the solver - Lots of fixes on 32 bits platforms - Add support for: pkg create -M ./plop.ucl -p ./plop.plist - New pkg -r rootdir that will install in the given rootdir without chrooting - Export PKG_ROOTDIR to scripts allow to make them as portable as possible - Stop trying to remove all installed package with the argument of pkg delete is a local file - Be more explicit about why the solver it going to reinstall, remove or upgrade (when possible) - Plenty of bug fixes - Plenty of new bugs Please test and report as much bugs as you can! We could be very grateful if regressions tests could be provided along with the bug reports :) Plan is to release 1.5.0 as soon as possible Best regards, Bapt pgpTrPipIy92F.pgp Description: PGP signature
Re: [CFT] Call for testing pkg 1.5.0
On Tue, Mar 31, 2015 at 03:19:15PM -0400, Shawn Webb wrote: On Tue, 2015-03-31 at 21:03 +0200, Baptiste Daroussin wrote: Hi all, We just released pkg 1.5.0 beta1 (in ports-mgmt/pkg-devel), Hey Baptiste, Great work to you and all those involved in this project! I'm grateful to have such an awesome tool. For those of us who run our own package repos via Poudriere, what kinds of changes should we expect to make once pkg 1.5.0 is released? Do we need to do a full rebuild of our package repo? I'm also assuming that the upgrade path from 1.4.x to 1.5.0 will simply be as easy as running `pkg upgrade`, right? Thanks again for your hard work. Add WITH_PKG=devel in your build make.conf then pkg upgrade will want you to upgrade to 1.4.99.16 (which is pkg 1.5.0 beta1) Best regards, Bapt pgpvO11OJQASF.pgp Description: PGP signature
Re: [CFT] Call for testing pkg 1.5.0
On Tue, 2015-03-31 at 21:03 +0200, Baptiste Daroussin wrote: Hi all, We just released pkg 1.5.0 beta1 (in ports-mgmt/pkg-devel), Hey Baptiste, Great work to you and all those involved in this project! I'm grateful to have such an awesome tool. For those of us who run our own package repos via Poudriere, what kinds of changes should we expect to make once pkg 1.5.0 is released? Do we need to do a full rebuild of our package repo? I'm also assuming that the upgrade path from 1.4.x to 1.5.0 will simply be as easy as running `pkg upgrade`, right? Thanks again for your hard work. Thanks, Shawn signature.asc Description: This is a digitally signed message part
Re: Is a high witness refcount indicative of a missing unlock?
Hi Shane, While our configs shoulds much the same (ignoring 10.1-stable vs current) and I had the nvidia driver loaded, my lockups did not involve the nvidia driver. That of course does not necessarily mean anything if the issue is squarly in zfs somewhere. You can see the reference counts from the ddb kernel debugger (man 8 ddb) using the “show witness” command Lars On Mar 29, 2015, at 19:14, Shane Ambler free...@shaneware.biz wrote: On 30/03/2015 05:59, Lars wrote: Hi, I am poking around for a cause for my repeating deadlock issues on my system based on r 279869. ddb show witness show the “vnode interlock†and the “zfs†locks both with reference counts over 200K. Obviously they are related, and there is a find running (all the filesystems on this machine are zfs ( minus the specialty ones like devfs). I don’t see any other withness entry with reference counts even in the ballpark of these two, so does this indicate that we have a vnode/zfs path were we don’t unlock? I am running 10.1-STABLE and have bad locking issues. Running a witness kernel I got a duplicate lock from nvidia and lock order reversals involving zfs. Any chance your issue is related to mine? What command can give me the witness lock counts? The debug data I have collected so far is at - http://shaneware.biz/freebsddebugdata/ The lock reversal output I had was (after uptime of about 12 mins) - Mar 24 00:24:25 leader kernel: Waiting (max 60 seconds) for system process `vnlru' to stop...done Mar 24 00:24:25 leader kernel: Waiting (max 60 seconds) for system process `bufdaemon' to stop...done Mar 24 00:24:25 leader kernel: Waiting (max 60 seconds) for system process `syncer' to stop... Mar 24 00:24:25 leader kernel: Syncing disks, vnodes remaining...0 0 0 0 0 0 0 0 done Mar 24 00:24:25 leader kernel: All buffers synced. Mar 24 00:24:25 leader kernel: lock order reversal: Mar 24 00:24:25 leader kernel: 1st 0xf800224555f0 zfs (zfs) @ /usr/src/sys/kern/vfs_mount.c:1229 Mar 24 00:24:25 leader kernel: 2nd 0xf800222d67c8 syncer (syncer) @ /usr/src/sys/kern/vfs_subr.c:2268 Mar 24 00:24:25 leader kernel: KDB: stack backtrace: Mar 24 00:24:25 leader kernel: db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfe022df6e4c0 Mar 24 00:24:25 leader kernel: kdb_backtrace() at kdb_backtrace+0x39/frame 0xfe022df6e570 Mar 24 00:24:25 leader kernel: witness_checkorder() at witness_checkorder+0xdc2/frame 0xfe022df6e600 Mar 24 00:24:25 leader kernel: __lockmgr_args() at __lockmgr_args+0x9ea/frame 0xfe022df6e740 Mar 24 00:24:25 leader kernel: vop_stdlock() at vop_stdlock+0x3c/frame 0xfe022df6e760 Mar 24 00:24:25 leader kernel: VOP_LOCK1_APV() at VOP_LOCK1_APV+0xfc/frame 0xfe022df6e790 Mar 24 00:24:25 leader kernel: _vn_lock() at _vn_lock+0xaa/frame 0xfe022df6e800 Mar 24 00:24:25 leader kernel: vputx() at vputx+0x232/frame 0xfe022df6e860 Mar 24 00:24:25 leader kernel: dounmount() at dounmount+0x301/frame 0xfe022df6e8e0 Mar 24 00:24:25 leader kernel: vfs_unmountall() at vfs_unmountall+0x61/frame 0xfe022df6e910 Mar 24 00:24:25 leader kernel: kern_reboot() at kern_reboot+0x540/frame 0xfe022df6e980 Mar 24 00:24:25 leader kernel: sys_reboot() at sys_reboot+0x5a/frame 0xfe022df6e9a0 Mar 24 00:24:25 leader kernel: amd64_syscall() at amd64_syscall+0x25a/frame 0xfe022df6eab0 Mar 24 00:24:25 leader kernel: Xfast_syscall() at Xfast_syscall+0xfb/frame 0xfe022df6eab0 Mar 24 00:24:25 leader kernel: --- syscall (55, FreeBSD ELF64, sys_reboot), rip = 0x40f1bc, rsp = 0x7fffe6d8, rbp = 0x7fffe7d0 --- Mar 24 00:24:25 leader kernel: lock order reversal: Mar 24 00:24:25 leader kernel: 1st 0xf800222d6b78 zfs (zfs) @ /usr/src/sys/modules/zfs/../../cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vfsops.c:1814 Mar 24 00:24:25 leader kernel: 2nd 0x818514a8 allproc (allproc) @ /usr/src/sys/kern/kern_descrip.c:2872 Mar 24 00:24:25 leader kernel: KDB: stack backtrace: Mar 24 00:24:25 leader kernel: db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfe022df6e690 Mar 24 00:24:25 leader kernel: kdb_backtrace() at kdb_backtrace+0x39/frame 0xfe022df6e740 Mar 24 00:24:25 leader kernel: witness_checkorder() at witness_checkorder+0xdc2/frame 0xfe022df6e7d0 Mar 24 00:24:25 leader kernel: _sx_slock() at _sx_slock+0x76/frame 0xfe022df6e810 Mar 24 00:24:25 leader kernel: mountcheckdirs() at mountcheckdirs+0x47/frame 0xfe022df6e860 Mar 24 00:24:25 leader kernel: dounmount() at dounmount+0x36f/frame 0xfe022df6e8e0 Mar 24 00:24:25 leader kernel: vfs_unmountall() at vfs_unmountall+0x61/frame 0xfe022df6e910 Mar 24 00:24:25 leader kernel: kern_reboot() at kern_reboot+0x540/frame 0xfe022df6e980 Mar 24 00:24:25 leader kernel: sys_reboot() at sys_reboot+0x5a/frame 0xfe022df6e9a0 Mar 24 00:24:25 leader kernel: amd64_syscall()
Jenkins build is back to normal : FreeBSD_HEAD #2615
See https://jenkins.freebsd.org/job/FreeBSD_HEAD/2615/changes ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
Re: async pass(4) patches available
On Tue, Mar 31, 2015 at 03:49:12 +0300, Konstantin Belousov wrote: On Mon, Mar 30, 2015 at 04:23:58PM -0600, Kenneth D. Merry wrote: Kernel memory for data transferred via the queued interface is allocated from the zone allocator in MAXPHYS sized chunks, and user data is copied in and out. This is likely faster than the vmapbuf()/vunmapbuf() method used by the CAMIOCOMMAND ioctl in configurations with many processors (there are more TLB shootdowns caused by the mapping/unmapping operation) but may not be as fast as running with unmapped I/O. cam_periph_mapmem() uses vmapbuf() with an indicator to always map the user pages mostly because I do not know CAM code and wanted to make the least intrusive changes there. It is not inherently impossible to pass unmapped pages down from cam_periph_mapmem(), but might require some more plumbing for driver to indicate that it is acceptable. I think that would probably not be too difficult to change. That API isn't one that is exposed, so changing it shouldn't be a problem. The only reason not to do unmapped I/O there is just if the underlying controller doesn't support it. The lower parts of the stack shouldn't be trying to sniff the data that is read or written to the device, although that has happened in the past. We'd have to audit a couple of the drivers to make sure they aren't trying to access the data. The new memory handling model for user requests also allows applications to send CCBs with request sizes that are larger than MAXPHYS. The pass(4) driver now limits queued requests to the I/O size listed by the SIM driver in the maxio field in the Path Inquiry (XPT_PATH_INQ) CCB. There are some things things would be good to add: 1. Come up with a way to do unmapped I/O on multiple buffers. Currently the unmapped I/O interface operates on a struct bio, which includes only one address and length. It would be nice to be able to send an unmapped scatter/gather list down to busdma. This would allow eliminating the copy we currently do for data. Only because nothing more was needed. The struct bio does not use address/length pair when unmapped, it passes the list of physical pages, see bio_ma array pointer. It is indeed taylored to be a pointer to struct buf' b_pages, but it does not have to be. The busdma unmapped non-specific interface is bus_dmamap_load_ma(), which again takes array of pages to load. If you want some additional helper, suitable for your goals, please provide the desired interface definition. What I'd like to be able to do is pass down a CCB with a user virtual S/G list (CAM_DATA_SG, but with user virtual pointers) and have busdma deal with it. The trouble would likely be figuring out a flag to use to indicate that the S/G list in question contains user virtual pointers. (Backwards/binary compatibility is always an issue with CCB flags, since they have all been used.) But that is essentially what is needed. Ken -- Kenneth Merry k...@freebsd.org ___ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org
