You submit a PR. Get Outlook for iOS<https://aka.ms/o0ukef> ________________________________ From: Philip Robar <[email protected]> Sent: Thursday, July 21, 2022 2:42:28 PM To: illumos-discuss <[email protected]> Subject: Re: [discuss] IPD 34 Rationalize Kernel Architecture Module Paths
I've not learned to use GitHub yet, but I noticed a couple of simple typos in IPD 34. Do I just hit the edit button, review my changes, and then click on Create pull request, or is the process more involved than that? (Apparently, by doing a compare I've already created a commit?) On Thu, Jul 21, 2022 at 2:38 AM Garrett D'Amore <[email protected]<mailto:[email protected]>> wrote: I’ve posted IPD 34 for discussion: https://github.com/illumos/ipd/blob/master/ipd/0034/README.md Personally, I believe this should be about as non-controversial as it is possible for an IPD to be. In a nutshell, this IPD basically prohibits any new architecture ports from using paths like /kernel/drv without including the CPU architecture bit. So in other words, /kerneld/drv/aarch64/mptsas is fine, but /kernel/drv/mptsas is not. This matches the current 64-bit platform we support (and also prior sparcv9), but it specifically precludes new architectures from “assuming” that they can skip this in the same way that the 32-bit legacy architectures did. Note that no code has to change in the current implementation to facilitate this, although it will enable some cleanups and generalizations in man pages, and possibly also in our kernel Makefiles. This should make it easier for the folks working on new platform ports like ARM64. * Garrett illumos<https://illumos.topicbox.com/latest> / illumos-discuss / see discussions<https://illumos.topicbox.com/groups/discuss> + participants<https://illumos.topicbox.com/groups/discuss/members> + delivery options<https://illumos.topicbox.com/groups/discuss/subscription> Permalink<https://illumos.topicbox.com/groups/discuss/T1e2dd14a87abc530-M432845983c9fc2e4102fe765> ------------------------------------------ illumos: illumos-discuss Permalink: https://illumos.topicbox.com/groups/discuss/T1e2dd14a87abc530-M35bc691348f5f85ee4b8ef00 Delivery options: https://illumos.topicbox.com/groups/discuss/subscription
