Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread Tom Rini
On Mon, Jun 22, 2020 at 02:03:28PM -0700, H. Peter Anvin wrote: > On 2020-06-22 14:01, Tom Rini wrote: > > > > I'm picky here because, well, there's a whole lot of moving parts in the > > pre-kernel world. In a strict sense, "UEFI" doesn't do anything with > > the kernel but based on hpa's

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread Tom Rini
On Mon, Jun 22, 2020 at 01:48:45PM -0700, H. Peter Anvin wrote: > On 2020-06-22 13:40, Tom Rini wrote: > > On Mon, Jun 22, 2020 at 01:02:16PM -0700, ron minnich wrote: > > > >> The other thing you ought to consider fixing: > >> initrd is documented as follows: > >> > >> initrd=

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread H. Peter Anvin
On 2020-06-22 14:01, Tom Rini wrote: > > I'm picky here because, well, there's a whole lot of moving parts in the > pre-kernel world. In a strict sense, "UEFI" doesn't do anything with > the kernel but based on hpa's comments I assume that at least the > in-kernel UEFI stub does what

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread Tom Rini
On Mon, Jun 22, 2020 at 01:56:24PM -0700, ron minnich wrote: > So, let me first add, the comment can be removed as needed. Comments > offered only for clarification. Noted, thanks. > On Mon, Jun 22, 2020 at 1:40 PM Tom Rini wrote: > > > But what do you mean UEFI "consumes" initrd= ? > >

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread ron minnich
So, let me first add, the comment can be removed as needed. Comments offered only for clarification. On Mon, Jun 22, 2020 at 1:40 PM Tom Rini wrote: > But what do you mean UEFI "consumes" initrd= ? What I mean is, there are bootloaders that will, if they see initrd= in the command line,

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread H. Peter Anvin
On 2020-06-22 13:40, Tom Rini wrote: > On Mon, Jun 22, 2020 at 01:02:16PM -0700, ron minnich wrote: > >> The other thing you ought to consider fixing: >> initrd is documented as follows: >> >> initrd= [BOOT] Specify the location of the initial ramdisk >> >> for bootloaders only.

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread Tom Rini
On Mon, Jun 22, 2020 at 01:02:16PM -0700, ron minnich wrote: > The other thing you ought to consider fixing: > initrd is documented as follows: > > initrd= [BOOT] Specify the location of the initial ramdisk > > for bootloaders only. > > UEFI consumes initrd from the command

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread hpa
On June 19, 2020 5:03:33 PM PDT, ron minnich wrote: >It seems fine to me, but I did not initially object to the use of that >name anyway. hpa, what do you think? > >On Fri, Jun 19, 2020 at 7:31 AM Tom Rini wrote: >> >> Most architectures have been passing the location of an initrd via >the >>

Re: [PATCH] initrd: Remove erroneous comment

2020-06-22 Thread ron minnich
The other thing you ought to consider fixing: initrd is documented as follows: initrd= [BOOT] Specify the location of the initial ramdisk for bootloaders only. UEFI consumes initrd from the command line as well. As ARM servers increasingly use UEFI, there may be situations in

Re: [PATCH] initrd: Remove erroneous comment

2020-06-19 Thread ron minnich
It seems fine to me, but I did not initially object to the use of that name anyway. hpa, what do you think? On Fri, Jun 19, 2020 at 7:31 AM Tom Rini wrote: > > Most architectures have been passing the location of an initrd via the > initrd= option since their inception. Remove the comment as