From: "Guilherme G. Piccoli" <gpiccoli@linux.vnet.ibm.com>
To: Michael Ellerman <mpe@ellerman.id.au>
Cc: linuxppc-dev@ozlabs.org, Brian King <brking@linux.vnet.ibm.com>,
"dougmill@us.ibm.com" <dougmill@us.ibm.com>
Subject: Re: [PATCH] powerpc/powernv: Turn on SCSI_AACRAID in powernv_defconfig
Date: Tue, 29 Aug 2017 10:03:24 -0300 [thread overview]
Message-ID: <58487ce7-aea1-39c1-b9c8-cd8435e493e0@linux.vnet.ibm.com> (raw)
In-Reply-To: <87h8wq1uvv.fsf@concordia.ellerman.id.au>
On 08/29/2017 08:22 AM, Michael Ellerman wrote:
> "Guilherme G. Piccoli" <gpiccoli@linux.vnet.ibm.com> writes:
>
>> On 08/28/2017 02:56 AM, Michael Ellerman wrote:
>>> Some Power9 boxes will have this adapter installed, so add it to the
>>> defconfig so we can boot on those machines without an initrd.
>>
>> Michael, not sure if this affects Petitboot (I know it has its own
>> default config files), but in the past we had some issues with this
>> driver and it's nice to have the possibility to blacklist it.
>
> Why we were black listing it?
Basically for the same reason you want it built-in - debug purposes heheh
In fact, this driver takes some time in its initialization process, so
if my rootfs is not under it, to speedup my boot-time I could blacklist
it; also, to debug the aacraid driver we could blacklist it in Petitboot
and debug on distro (easier to build kernels there), since the issue
could be in first probe only (or be affected by kexec-out from Petitboot).
Anyway, since make it a module will harm your workflow, I agree built-in
is the best option!
Thanks for your detailed explanation.
Cheers,
Guilherme
>
>> What is the rationale for this change? I mean, how being able to boot
>> without initrd is interesting here? Sorry if it's silly question!
>
> It is a silly question! :)
>
> When you boot as many kernels as me it's preferable to be able to build
> a kernel completely remote from the box, throw the kernel on a tftp
> server and have the machine boot from that kernel all the way to
> userspace.
>
> That requires not using an initrd, and therefore having all the drivers
> built-in that are needed to get to the boot disk.
>
> (Yeah it is "possible" to build an initrd remotely but it's a pain).
>
> And the one P9 box I have here uses that driver for its root disk, so I
> do really want it to be =y.
>
> cheers
>
next prev parent reply other threads:[~2017-08-29 13:03 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-28 5:56 [PATCH] powerpc/powernv: Turn on SCSI_AACRAID in powernv_defconfig Michael Ellerman
2017-08-28 13:16 ` Guilherme G. Piccoli
2017-08-29 11:22 ` Michael Ellerman
2017-08-29 13:03 ` Guilherme G. Piccoli [this message]
2017-08-30 11:07 ` Michael Ellerman
2017-08-30 12:47 ` Guilherme G. Piccoli
2017-08-31 9:48 ` Stewart Smith
2017-08-31 12:37 ` Michael Ellerman
2017-08-31 15:31 ` Brian King
2017-09-01 5:23 ` Michael Ellerman
2017-09-03 23:19 ` Stewart Smith
2017-09-06 13:42 ` Brian King
2018-02-22 22:19 ` Michael Ellerman
2018-02-23 0:16 ` Stewart Smith
2018-02-23 19:41 ` Brian King
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=58487ce7-aea1-39c1-b9c8-cd8435e493e0@linux.vnet.ibm.com \
--to=gpiccoli@linux.vnet.ibm.com \
--cc=brking@linux.vnet.ibm.com \
--cc=dougmill@us.ibm.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=mpe@ellerman.id.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).