Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Mar 2017 00:37:11 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-mips@FreeBSD.org
Subject:   [Bug 216831] [PATCH] sys/mips/cavium/octeon_machdep.c: add cavium octeon 'bootoctlinux' boot argument support
Message-ID:  <bug-216831-27794-xOcAu6i9JJ@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-216831-27794@https.bugs.freebsd.org/bugzilla/>
References:  <bug-216831-27794@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216831

--- Comment #13 from Alexander Kabaev <kan@FreeBSD.org> ---
(In reply to Alexander Kabaev from comment #12)

Umm, a question - does argv[0] contain useful value? My userland habits protest
against using argv[0] as command line parameter. 

+       for (i = 0; i < app_desc_ptr->argc; i++) {

Anyway, I am committing a slightly different patch that moves parse_boothowto
back into octeon-specific machdep file. I was the one who requested the code to
be shared, but since your change does not update other existing platform to use
common functions, putting it in MI file defeats the purpose. If and when
someone actually makes the function be used by more than one client, then it
can be moved to MI code.

-- 
You are receiving this mail because:
You are the assignee for the bug.


Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-216831-27794-xOcAu6i9JJ>