[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] sparc64: boot performance improvements
From: |
Vladimir 'φ-coder/phcoder' Serbinenko |
Subject: |
Re: [PATCH] sparc64: boot performance improvements |
Date: |
Fri, 9 Oct 2015 14:54:37 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.2.0 |
On 06.10.2015 19:39, Eric Snowberg wrote:
> Keep of devices open. This can save 6 - 7 seconds per open call and
> can decrease boot times from over 10 minutes to 29 seconds on
> larger SPARC systems. The open/close calls with some vendors'
> SAS controllers cause the entire card to be reinitialized after
> each close.
>
Unfortunately on some systems (AFAIR old sparcs) hard-locks if the same
device is opened twice. We tried to detect when 2 devices are named
differently but it's not possible to do reliably on IEEE1275. Our only
solution was to ensure that only 1 disk is open at a time. Do you use
some kind of RAID? Can you try keeping last_handle open in ofdisk.c
rather than closing it?
> Signed-off-by: Eric Snowberg <address@hidden>
> ---
> grub-core/kern/ieee1275/ieee1275.c | 39
> ++++++++++++++++++++++++++++++++++++
> 1 files changed, 39 insertions(+), 0 deletions(-)
>
> diff --git a/grub-core/kern/ieee1275/ieee1275.c
> b/grub-core/kern/ieee1275/ieee1275.c
> index 9821702..30f973b 100644
> --- a/grub-core/kern/ieee1275/ieee1275.c
> +++ b/grub-core/kern/ieee1275/ieee1275.c
> @@ -19,11 +19,24 @@
>
> #include <grub/ieee1275/ieee1275.h>
> #include <grub/types.h>
> +#include <grub/misc.h>
> +#include <grub/list.h>
> +#include <grub/mm.h>
>
> #define IEEE1275_PHANDLE_INVALID ((grub_ieee1275_cell_t) -1)
> #define IEEE1275_IHANDLE_INVALID ((grub_ieee1275_cell_t) 0)
> #define IEEE1275_CELL_INVALID ((grub_ieee1275_cell_t) -1)
>
> +struct grub_of_opened_device
> +{
> + struct grub_of_opened_device *next;
> + struct grub_of_opened_device **prev;
> + grub_ieee1275_ihandle_t ihandle;
> + char *path;
> +};
> +
> +static struct grub_of_opened_device *grub_of_opened_devices;
> +
>
>
> int
> @@ -452,6 +465,18 @@ grub_ieee1275_open (const char *path,
> grub_ieee1275_ihandle_t *result)
> }
> args;
>
> + struct grub_of_opened_device *dev;
> +
> + FOR_LIST_ELEMENTS(dev, grub_of_opened_devices)
> + if (grub_strcmp(dev->path, path) == 0)
> + break;
> +
> + if (dev)
> + {
> + *result = dev->ihandle;
> + return 0;
> + }
> +
> INIT_IEEE1275_COMMON (&args.common, "open", 1, 1);
> args.path = (grub_ieee1275_cell_t) path;
>
> @@ -460,6 +485,11 @@ grub_ieee1275_open (const char *path,
> grub_ieee1275_ihandle_t *result)
> *result = args.result;
> if (args.result == IEEE1275_IHANDLE_INVALID)
> return -1;
> +
> + dev = grub_zalloc(sizeof(struct grub_of_opened_device));
> + dev->path = grub_strdup(path);
> + dev->ihandle = args.result;
> + grub_list_push(GRUB_AS_LIST_P (&grub_of_opened_devices), GRUB_AS_LIST
> (dev));
> return 0;
> }
>
> @@ -473,6 +503,15 @@ grub_ieee1275_close (grub_ieee1275_ihandle_t ihandle)
> }
> args;
>
> + struct grub_of_opened_device *dev;
> +
> + FOR_LIST_ELEMENTS(dev, grub_of_opened_devices)
> + if (dev->ihandle == ihandle)
> + break;
> +
> + if (dev)
> + return 0;
> +
> INIT_IEEE1275_COMMON (&args.common, "close", 1, 0);
> args.ihandle = ihandle;
>
>
signature.asc
Description: OpenPGP digital signature
- [PATCH] sparc64: boot performance improvements, Eric Snowberg, 2015/10/08
- Re: [PATCH] sparc64: boot performance improvements, Andrei Borzenkov, 2015/10/08
- Re: [PATCH] sparc64: boot performance improvements, Eric Snowberg, 2015/10/08
- Re: [PATCH] sparc64: boot performance improvements, Andrei Borzenkov, 2015/10/09
- Re: [PATCH] sparc64: boot performance improvements, Eric Snowberg, 2015/10/09
- Re: [PATCH] sparc64: boot performance improvements, Vladimir 'phcoder' Serbinenko, 2015/10/10
- Re: [PATCH] sparc64: boot performance improvements, Eric Snowberg, 2015/10/15
- Re: [PATCH] sparc64: boot performance improvements, Vladimir 'φ-coder/phcoder' Serbinenko, 2015/10/25
- Re: [PATCH] sparc64: boot performance improvements, Eric Snowberg, 2015/10/27
- Re: [PATCH] sparc64: boot performance improvements, Vladimir 'phcoder' Serbinenko, 2015/10/28
Re: [PATCH] sparc64: boot performance improvements,
Vladimir 'φ-coder/phcoder' Serbinenko <=