Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-18 Thread Masahisa Kojima
Hi Heinrich, On Sun, 17 Dec 2023 at 19:26, Heinrich Schuchardt wrote: > > On 12/14/23 09:23, Masahisa Kojima wrote: > Depending on the number of handles and pointers this will take a > considerable time. A private field for the handle appended to struct > efi_block_io would allow

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-17 Thread Heinrich Schuchardt
On 12/14/23 09:23, Masahisa Kojima wrote: Depending on the number of handles and pointers this will take a considerable time. A private field for the handle appended to struct efi_block_io would allow a fast lookup. EDK II does the same. See the definition of RAM_DISK_PRIVATE_FROM_BLKIO which

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-14 Thread Masahisa Kojima
Hi Heinrich, On Thu, 14 Dec 2023 at 16:31, Heinrich Schuchardt wrote: > > > > Am 14. Dezember 2023 02:55:27 MEZ schrieb Masahisa Kojima > : > >Hi Heinrich, > > > >On Wed, 13 Dec 2023 at 23:22, Heinrich Schuchardt wrote: > >> > >> On 13.12.23 09:57, Masahisa Kojima wrote: > >> > Current code

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-13 Thread Heinrich Schuchardt
Am 14. Dezember 2023 02:55:27 MEZ schrieb Masahisa Kojima : >Hi Heinrich, > >On Wed, 13 Dec 2023 at 23:22, Heinrich Schuchardt wrote: >> >> On 13.12.23 09:57, Masahisa Kojima wrote: >> > Current code uses struct efi_disk_obj to keep information >> > about block devices and partitions. As the

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-13 Thread Masahisa Kojima
Hi Heinrich, On Wed, 13 Dec 2023 at 23:22, Heinrich Schuchardt wrote: > > On 13.12.23 09:57, Masahisa Kojima wrote: > > Current code uses struct efi_disk_obj to keep information > > about block devices and partitions. As the efi handle already > > has a field with the udevice, we should

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-13 Thread AKASHI Takahiro
Hi Kojima-san, On Wed, Dec 13, 2023 at 05:57:37PM +0900, Masahisa Kojima wrote: > Current code uses struct efi_disk_obj to keep information > about block devices and partitions. As the efi handle already > has a field with the udevice, we should eliminate > struct efi_disk_obj and use an pointer

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-13 Thread Simon Glass
Hi Heinrich, On Wed, 13 Dec 2023 at 07:23, Heinrich Schuchardt wrote: > > On 13.12.23 09:57, Masahisa Kojima wrote: > > Current code uses struct efi_disk_obj to keep information > > about block devices and partitions. As the efi handle already > > has a field with the udevice, we should

Re: [PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-13 Thread Heinrich Schuchardt
On 13.12.23 09:57, Masahisa Kojima wrote: Current code uses struct efi_disk_obj to keep information about block devices and partitions. As the efi handle already has a field with the udevice, we should eliminate struct efi_disk_obj and use an pointer to struct efi_object for the handle.

[PATCH] efi_loader: eliminate efi_disk_obj structure

2023-12-13 Thread Masahisa Kojima
Current code uses struct efi_disk_obj to keep information about block devices and partitions. As the efi handle already has a field with the udevice, we should eliminate struct efi_disk_obj and use an pointer to struct efi_object for the handle. efi_link_dev() call is moved inside of