Re: [PATCH] bug fix for efi network

[Vladimir 'φ-coder/phcoder' Serbinenko]

On 29.04.2012 17:05, Bean wrote:
> +  for (i = 0; i < 3; i++)
>      {
> +      grub_uint64_t limit_time;      
> +      
> +      efi_call_3 (net->get_status, net, &int_status, 0);
> +
> +      limit_time = grub_get_time_ms () + 5;
> +      for (;;)
> +     {
> +       st = efi_call_7 (net->transmit, net, 0, (pack->tail - pack->data),
> +                        pack->data, NULL, NULL, NULL);
> +       if (st != GRUB_EFI_NOT_READY)
> +         break;
> +
> +       if (limit_time < grub_get_time_ms ())
> +         {
> +           st = GRUB_EFI_TIMEOUT;
> +           break;
> +         }
> +     }
> +
> +      if (st)
> +     goto quit;
> +
>        void *txbuf = NULL;
> -      st = efi_call_3 (net->get_status, net, 0, &txbuf);
> -      if (st != GRUB_EFI_SUCCESS)
> -     return grub_error (GRUB_ERR_IO, N_("couldn't send network packet"));
> -      if (txbuf)
> -     return GRUB_ERR_NONE;
> -      if (limit_time < grub_get_time_ms ())
> -     return grub_error (GRUB_ERR_TIMEOUT, N_("couldn't send network 
> packet"));
> +      limit_time = grub_get_time_ms () + 5;
> +      for (;;)
> +     {                 
> +       st = efi_call_3 (net->get_status, net, &int_status, &txbuf);
> +
> +       if (txbuf != NULL)
> +         break;
> +
> +       if (limit_time < grub_get_time_ms ())
> +         {
> +           st = GRUB_EFI_TIMEOUT;
> +           break;
> +         }
> +     }
> +
> +    quit:
> +      if (st != GRUB_EFI_TIMEOUT)
> +     break;
>      }
> +
> +  return st;
>  }
I see no justification as to why this part of patch so I can't know if
it's something which is we absolutely need to commit or something that
can wait.

-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko

signature.asc
Description: OpenPGP digital signature