[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/69] Block patches
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PULL 00/69] Block patches |
Date: |
Wed, 04 Mar 2015 09:28:03 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
Am 03.03.2015 um 18:33 schrieb Max Reitz:
> On 2015-03-03 at 12:29, Christian Borntraeger wrote:
>> Am 03.03.2015 um 15:52 schrieb Peter Maydell:
>>> On 28 February 2015 at 03:57, Stefan Hajnoczi <address@hidden> wrote:
>>>> On Fri, Feb 27, 2015 at 6:17 PM, Stefan Hajnoczi <address@hidden> wrote:
>>>>> Ekaterina Tumanova (5):
>>>>> block: add bdrv functions for geometry and blocksize
>>>>> raw-posix: Factor block size detection out of raw_probe_alignment()
>>>>> block: Add driver methods to probe blocksizes and geometry
>>>>> block-backend: Add wrappers for blocksizes and geometry probing
>>>>> BlockConf: Call backend functions to detect geometry and blocksizes
>>>> Max Reitz found an issue with this patch.
>>>>
>>>> Peter: Please squash the following trivial fix into "BlockConf: Call
>>>> backend functions to detect geometry and blocksizes":
>>>> https://lists.nongnu.org/archive/html/qemu-devel/2015-02/msg05512.html
>>> I can't squash fixes into pull requests -- I can only
>>> apply them, or not apply them. You need to respin.
>>>
>>>> I want to avoid spamming the list with another 60 patches.
>>> If it's a trivial change since last time around you can just
>>> send the cover letter to the list with a note in it that there
>>> have only been small changes.
>>>
>>> -- PMM
>>>
>> I think you could just apply this pull request and add the fixup as separate
>> patch.
>> After all it fixes a case were the command line is wrong, so should not
>> on the critical path - I guess.
>
> I agree. It's not nice to have a known break, but we're a bit behind on the
> pull requests already... Also, the segfault is always caused by dereferencing
> a null pointer, so there is no security issue.
Peter,
unless Stefan/Kevin object:
can you pull the original pull request?
Kate,
can you resend patch " scsi-hd: fix property unset case" with an additional
comment
as Markus suggested directly to Peter Maydell, cc
Stefan/Kevin/Max/Markus/qemu-devel?
Christian