[PATCH] conf: fix block type CDROM cannot support startupPolicy

Peter Krempa pkrempa at redhat.com
Fri Oct 1 07:59:21 UTC 2021


On Thu, Sep 30, 2021 at 09:58:46 +0800, Han Han wrote:
> On Tue, Sep 28, 2021 at 10:43 AM Jie Wang <wangjie88 at huawei.com> wrote:
> 
> > block type CDROM also support startupPolicy in the past, so
> >
> s/block/Block/
> 
> "in the past" could be more detailed. It's better if you tell from which
> version to which version the
> startupPolicy of block type cdrom is supported

Yes it indeed is, because it makes it easier for the reviewer.
Specifically if it was working before it's a bit simpler since we don't
need to check whether the code paths actually allow it.

In addition the commit message mentions CDROM, but this is really
changing it for all types of disks not only cdroms.

So you need to update the commit message to be more clear what's
happening. If this is fixng a regression, please attempt to find the
commit that caused it.

If not it's necessary to justify in the commit message that just
removing the error is sufficient and that all code paths will be able to
handle block devices just right.

I can do that, but checking that everything works properly is more
expensive for me, so it will take a while before I get back to this
patch.




More information about the libvir-list mailing list