[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [PATCH 1/3] ofpathname: Add partition support to l2of_ide() and l2of_scsi()



On 6/22/20 10:43 PM, John Paul Adrian Glaubitz wrote:
> Hi!
> 
> On 6/23/20 3:32 AM, Tyrel Datwyler wrote:
>> Heads up this may be problematic for pseries lpars. The current behavior is that
>> regardless of the scsi partition the logical device always translates to the
>> full underlying disk. I will need to confirm whether OF is going to complain
>> when the partition number is appended.
> 
> If that should be a problem, we can enable this on the Mac platform only. Although
> I'm surprised this should be a problem since the previously used tools like ofpath
> from Yaboot behaved exactly like that and I'm not aware of any problems with Yaboot
> on these machines.

Ok, so on the bright side I've confirmed that IBM open firmware will boot a
pseries machine when the partition ID is provided and points at the actual PReP
bootstrap partition. I suspect it was just generally omitted because it was
assumed that PReP would always be the first partition historically. I believe
these days that if it isn't the first partition OF is smart enough to locate it
on its own which seems to further the lack of a need to explicitly provide the
partition ID on this platforms.

Now the downside is that ofpathname is not able to translate a OF vscsi device
path to a logical device if that OF path includes the partition ID.

So, I'm still inclined to say that the changes in this patch should be limited
to the Mac platform for the time being.

-Tyrel

> 
> Adrian
> 


Reply to: