From owner-freebsd-current@freebsd.org Tue Feb 6 18:50:24 2018 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id DA5E9EF2482 for ; Tue, 6 Feb 2018 18:50:23 +0000 (UTC) (envelope-from ian@freebsd.org) Received: from outbound1a.eu.mailhop.org (outbound1a.eu.mailhop.org [52.58.109.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 63CF2791B8 for ; Tue, 6 Feb 2018 18:50:23 +0000 (UTC) (envelope-from ian@freebsd.org) X-MHO-User: 8bb023b9-0b6e-11e8-91c6-33ffc249f3e8 X-Report-Abuse-To: https://support.duocircle.com/support/solutions/articles/5000540958-duocircle-standard-smtp-abuse-information X-Originating-IP: 67.177.211.60 X-Mail-Handler: DuoCircle Outbound SMTP Received: from ilsoft.org (unknown [67.177.211.60]) by outbound1.eu.mailhop.org (Halon) with ESMTPSA id 8bb023b9-0b6e-11e8-91c6-33ffc249f3e8; Tue, 06 Feb 2018 18:50:11 +0000 (UTC) Received: from rev (rev [172.22.42.240]) by ilsoft.org (8.15.2/8.15.2) with ESMTP id w16Io1YV019128; Tue, 6 Feb 2018 11:50:01 -0700 (MST) (envelope-from ian@freebsd.org) Message-ID: <1517943001.1366.100.camel@freebsd.org> Subject: Re: Fatal trap 12 booting FreeBSD-CURRENT via isboot kernel module. From: Ian Lepore To: John Nielsen , Maurizio Vairani Cc: freebsd-current , Daisuke Aoyama Date: Tue, 06 Feb 2018 11:50:01 -0700 In-Reply-To: <5EDB0747-41DE-4ECF-925E-EF24FAA550F3@jnielsen.net> References: <3E242843-7D43-4A36-A448-E4B0DACB2AB4@jnielsen.net> <5EDB0747-41DE-4ECF-925E-EF24FAA550F3@jnielsen.net> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.18.5.1 FreeBSD GNOME Team Port Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Feb 2018 18:50:24 -0000 On Tue, 2018-02-06 at 11:33 -0700, John Nielsen wrote: > > > > On Feb 6, 2018, at 10:54 AM, John Nielsen > > wrote: > > > > > > > > > > > On Feb 4, 2018, at 2:50 AM, Maurizio Vairani > > com> wrote: > > > > > > 2018-01-29 18:38 GMT+01:00 John Nielsen : > > > [ resending from correct email address ] > > > > > > > > > > > On Jan 29, 2018, at 6:05 AM, Maurizio Vairani > > > il.com> wrote: > > > > > > > > I am running > > > > # uname > > > > -a > > > > > > > > FreeBSD  12.0-CURRENT FreeBSD 12.0-CURRENT #0 r328383: Thu Jan > > > > 25 04:48:52 > > > > UTC 2018     root@releng3.nyi.freebsd.org:/usr/obj/usr/src/amd6 > > > > 4.amd64/sys/GENERIC > > > > amd64 > > > > > > > > After compiling the kernel module as discussed in this thread : > > > > https://lists.freebsd.org/pipermail/freebsd-current/2018-Januar > > > > y/068272.html > > > > > > > > I can boot FreeBSD via iSCSI using iPXE. But when the isboot, > > > > the iSCSI > > > > boot driver version 0.2.13, starts I receive a panic: > > > > https://mega.nz/#!tkVwBBKA!PUj14-Za6KCNaoo9hxuXORRLQoWkb4LMvTdU > > > > A1BorD4 > > > > > > > > Any idea? > > > Bummer!  > > > > > > Aoyama-san- > > > > > > Are you still maintaining isboot? Can you help debug this issue > > > on FreeBSD 12-CURRENT? > > > > > > Once we get it working I will update the port with whatever is > > > needed and send you the patches in case you'd like to cut a new > > > release. > > > > > > Thank you! > > > > > > I have solved the issue changing the function isboot_ifup() in > > > the source file isboot.c. > > Here is a patch with some changes to minimize the diff. Except for > > the printed error messages does that look functionally equivalent? > > > > Now the question is why is this change needed and for what values > > of __FreeBSD_version is it appropriate? > Apparently sending a NULL socket pointer to ifioctl hasn't worked > since this commit in 2011: > https://svnweb.freebsd.org/base?view=revision&revision=218757 > > So I'm going to add this patch to the port unconditionally once it > works. > > Unfortunately, I can't compile the port with either my patch below or > your original replacement version of isboot_ifup(). :( Did you make > other changes? Here's the error I'm getting: > > --- isboot.o --- > isboot.c:425:53: error: incomplete definition of type 'struct thread' >     error = socreate(AF_INET, &so, SOCK_DGRAM, 0, td->td_ucred, td); >                                                   ~~^ > /usr/src/sys/sys/systm.h:185:8: note: forward declaration of 'struct > thread' > struct thread; >        ^ > 1 error generated. > Try adding #include if it's not already in the list.  It may be that that file got included via pollution from some other header file in the past and maybe now that has changed. If you're already including sys/proc.h then I'm clueless. -- Ian