User Details
- User Since
- Sep 10 2014, 9:01 PM (337 w, 6 d)
Yesterday
Thu, Feb 25
Just committed this. I agree about 13.0 -- I was planning to merge to stable/13 after 13.0 has branched and smoke-test this in HEAD for a bit in the meantime.
Wed, Feb 24
Tue, Feb 23
Now tested and working, with UFS and ZFS, including multi-disk ZFS setups. In principle ready for commit.
Now does ZFS (including redundant setups) as well. Still not fully tested.
@imp any thoughts on the gmirror solution for ZFS? I think it solves 95% of the update issues and shouldn't be too bad to implement.
Sat, Feb 20
Just to be pedantic: We actually *do* have /boot as separate on some systems (e.g. recent powerpc64), but those systems are mutually exclusive with EFI systems. I have a mild preference /boot/efi, but it doesn't really matter much so long as we pick something -- the status quo is it not being mounted *at all*, so reliability of mounting or whatever is obviously a second-order issue.
Dec 17 2020
Dec 13 2020
Oct 29 2020
Oct 15 2020
Jul 6 2020
May 12 2020
May 10 2020
Do we know all supported 32-bit AIM CPUs do the same things with DSISR? Otherwise, it looks good to me.
Dec 1 2019
Nov 30 2019
Sep 7 2019
I have pretty mixed feelings about this approach. It only works for ofwbus children, not simplebus children, and canonicalizes a bunch of behaviors that I don't believe are standards (I'm in an airport departure lounge and don't have the spec handy). I'm also not really sure how it interacts with multipass etc. Could you elaborate a little more on the mechanism?
Aug 27 2019
Aug 26 2019
Fix fat-fginering; this is the right diff. It also sets Python to use Python 2, since a handful of scripts cared.
Updates from review.
Aug 25 2019
Jul 21 2019
OK, sounds good, thanks for testing!
Jul 15 2019
Just make the page table size really small. There's a spill counter in the statistics; when it gets small enough, you should see it moving. If spill-handling doesn't work, and you have spills, everything breaks fast.
Does this break PTE spills? This code path historically was involved in re-adding PVOs to the hashed page table on faults if they got spilled.
Jul 11 2019
That's a fair point. I've used them to assess hash-table spill rates when performance is not great, but not for a long time. We could also just delete them.
Yeah, counter(9) seems like the actual solution here.
Jun 19 2019
Looks good to me; do you know how much this actually helps?
Jun 17 2019
Jun 16 2019
Jun 11 2019
Feb 26 2019
Should we panic if freeze_timebase somehow doesn't get set? Or, alternately, fall back on the old lame thing?
Feb 10 2019
Jan 13 2019
Longer-term, it might be better to avoid a proliferation of special cases like this to set LOADER_MSDOS_SUPPORT for the U-boot loader and make loader flexible enough to find the kernel etc. in the root rather than /boot. But this is fine for now.
Oct 19 2018
Oct 11 2018
Aside from one inline note, this looks OK for non-SPE.
Oct 5 2018
Sep 19 2018
Sep 17 2018
Can we make the check the presence of a PHB4 node in the device tree rather than an #ifdef? It might also be best to do this in platform_powernv.c, but that's a bit larger of a project.
Aug 30 2018
Aug 29 2018
Aug 24 2018
Aug 19 2018
I just looked at the actual patch. The kernel sometimes (almost always on ppc64, less often on ppc32) relocates itself again post-loader to an address that loader does not know, so I think this does not solve the actual problem, or at least solves it in only a subset of cases. Processing these relocations in the kernel would solve the issue in all cases.
Aug 11 2018
Looks OK to me. One question: Do we want the various instances of fdt_check_header() in our code to be spelled FDT_RO_PROBE() now?
Jul 31 2018
Jul 19 2018
Since I'm not familiar with the clock code, where is EXT_RESOURCES defined? Can we just use that code unconditionally?
Jun 29 2018
Assuming you have tested this and it works, looks great.
Jun 21 2018
Looks good to me.
Jun 20 2018
Approved if you fix the #if 0 bit.
Jun 15 2018
Don't you want strcmp() == 0?
It's mildly stupid, since the universe of pci* isn't big, but I think I would prefer a check for "pci" || "pciex" to just checking the first three characters.
Jun 11 2018
Jun 5 2018
It's been another couple weeks. Any objections or approvals?
Jun 4 2018
May 30 2018
May 26 2018
This looks good to me, though I haven't tested on P8.
On POWER9, I think the lock is unnecessary (the ISA spec doesn't mention it).
May 25 2018
Any other thoughts? I'd like to get this in soon since it is blocking other things.
May 20 2018
Fix style and spelling issues in man pages.
Probe for pending devices at the end of each pass and after modules loaded. Add documentation.
See the comment on line 467 about powernv_smp_ap_init(). Otherwise, looks good.
May 19 2018
Approved if you also install it to EXC_HEA.
May 18 2018
This looks good, even if the deice tree situation is a bit lamentable. Two questions:
- Is there a document describing this that we can reference?
- If this is just part of the PowerNV platform, maybe the code belongs in platform_powernv.c instead?
May 14 2018
Thanks -- I'll tweak the implementation to handle KLDs properly and write a man page, then update this diff.