All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: Laszlo Ersek <lersek@redhat.com>, Sam Eiderman <sameid@google.com>
Cc: kwolf@redhat.com, qemu-block@nongnu.org, arbel.moshe@oracle.com,
	seabios@seabios.org, qemu-devel@nongnu.org, kraxel@redhat.com,
	John Snow <jsnow@redhat.com>
Subject: Re: [SeaBIOS] Re: [PATCH v7 7/8] bootdevice: FW_CFG interface for LCHS values
Date: Wed, 16 Oct 2019 14:29:48 +0200	[thread overview]
Message-ID: <eb3cc776-b8d9-f8da-269a-2fc0f8c21662@redhat.com> (raw)
In-Reply-To: <bf6ff260-f8ca-a593-dd3e-e78fa0551101@redhat.com>

On 10/16/19 2:14 PM, Laszlo Ersek wrote:
> Hi Sam,
> 
> On 10/16/19 13:02, Sam Eiderman wrote:
>> Gentle Ping,
>>
>> Philippe, John?
>>
>> Just wondering if the series is okay, as Gerd pointed out this series
>> is a blocker for the corresponding changes in SeaBIOS for v 1.13
> 
> The QEMU series is still not merged, due to a bug in the last patch
> (namely, the test case, "hd-geo-test: Add tests for lchs override").
> 
> To my knowledge, SeaBIOS prefers to merge patches with the underlying
> QEMU patches merged first, so you'll likely have to fix that QEMU issue
> first.
> 
> I explained the bug in the QEMU test case here:
> 
>    http://mid.mail-archive.com/6b00dc74-7267-8ce8-3271-5db269edb1b7@redhat.com
>    http://mid.mail-archive.com/700cd594-1446-e478-fb03-d2e6b862dc6c@redhat.com

Yes, I was expecting a respin with find_fw_cfg_file() fixed per Laszlo 
detailed review.

> (Alternative links to the same:
> 
>    https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg01790.html
>    https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg01793.html
> )
> 
> I've never received feedback to those messages, and I think you must
> have missed them.
> 
> FWIW, when I hit "Reply All" in that thread, you were on the "To:" list
> with:
> 
>    Sam Eiderman <shmuel.eiderman@oracle.com>
> 
> but here you are present with
> 
>    Sam Eiderman <sameid@google.com>
> 
> In addition, when I posted those messages, I got the following
> auto-response ("Undelivered Mail Returned to Sender"):
> 
>> This is the mail system at host mx1.redhat.com.
>>
>> I'm sorry to have to inform you that your message could not
>> be delivered to one or more recipients. It's attached below.
>>
>> For further assistance, please send mail to postmaster.
>>
>> If you do so, please include this problem report. You can
>> delete your own text from the attached returned message.
>>
>>                     The mail system
>>
>> <shmuel.eiderman@oracle.com>: host
>> aserp2030.oracle.com[141.146.126.74] said:
>>      550 5.1.1 Unknown recipient address. (in reply to RCPT TO command)

That explains it :)

> 
> I didn't know your new address, so I could only hope you'd find my
> feedback on qemu-devel.
> 
> Thanks
> Laszlo
> 


  reply	other threads:[~2019-10-16 12:31 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-25 11:06 [PATCH v7 0/8] Add Qemu to SeaBIOS LCHS interface Sam Eiderman via
2019-09-25 11:06 ` [PATCH v7 1/8] block: Refactor macros - fix tabbing Sam Eiderman via
2019-09-26  9:38   ` [SeaBIOS] " Philippe Mathieu-Daudé
2019-09-25 11:06 ` [PATCH v7 2/8] block: Support providing LCHS from user Sam Eiderman via
2019-09-25 11:06 ` [PATCH v7 3/8] bootdevice: Add interface to gather LCHS Sam Eiderman via
2019-09-25 11:06 ` [PATCH v7 4/8] scsi: Propagate unrealize() callback to scsi-hd Sam Eiderman via
2019-09-26  9:38   ` [SeaBIOS] " Philippe Mathieu-Daudé
2019-09-25 11:06 ` [PATCH v7 5/8] bootdevice: Gather LCHS from all relevant devices Sam Eiderman via
2019-09-25 11:06 ` [PATCH v7 6/8] bootdevice: Refactor get_boot_devices_list Sam Eiderman via
2019-09-26  9:42   ` [SeaBIOS] " Philippe Mathieu-Daudé
2019-09-25 11:06 ` [PATCH v7 7/8] bootdevice: FW_CFG interface for LCHS values Sam Eiderman via
2019-09-26  9:57   ` [SeaBIOS] " Philippe Mathieu-Daudé
2019-09-26 18:26     ` John Snow
2019-09-26 19:09       ` Philippe Mathieu-Daudé
2019-09-26 19:16         ` Philippe Mathieu-Daudé
2019-09-29 10:13           ` Sam Eiderman
2019-10-08 11:34             ` Philippe Mathieu-Daudé
2019-10-08 11:51               ` Sam Eiderman
2019-10-16 11:02                 ` Sam Eiderman
2019-10-16 12:14                   ` [SeaBIOS] " Laszlo Ersek
2019-10-16 12:29                     ` Philippe Mathieu-Daudé [this message]
2019-10-16 14:55                       ` Sam Eiderman
2019-10-16 15:07                         ` John Snow
2019-10-16 15:19                           ` Sam Eiderman
2019-10-16 16:41                             ` Philippe Mathieu-Daudé
2019-09-25 11:06 ` [PATCH v7 8/8] hd-geo-test: Add tests for lchs override Sam Eiderman via
2019-09-26 10:00   ` [SeaBIOS] " Philippe Mathieu-Daudé
2019-09-26 18:19     ` John Snow
2019-09-25 20:38 ` [PATCH v7 0/8] Add Qemu to SeaBIOS LCHS interface John Snow
2019-09-27 17:02 ` no-reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=eb3cc776-b8d9-f8da-269a-2fc0f8c21662@redhat.com \
    --to=philmd@redhat.com \
    --cc=arbel.moshe@oracle.com \
    --cc=jsnow@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=lersek@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=sameid@google.com \
    --cc=seabios@seabios.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.