Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1565808

Re: KVM: s390: Move two error code assignments in kvm_vm_ioctl_get_dirty_log()

From Paolo Bonzini <pbonzini@redhat.com>
Newsgroups linux.kernel
Subject Re: KVM: s390: Move two error code assignments in kvm_vm_ioctl_get_dirty_log()
Date 2017-01-24 13:20 +0100
Message-ID <t38eR-4bB-19@gated-at.bofh.it> (permalink)
References <t26oN-62y-7@gated-at.bofh.it> <t2IaJ-44k-1@gated-at.bofh.it> <t2KFA-5GZ-19@gated-at.bofh.it> <t2N0J-71T-11@gated-at.bofh.it> <t38eR-4bB-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw



On 24/01/2017 13:10, SF Markus Elfring wrote:
>>> Would you like to check run time consequences
>>> for the shown error code settings once more?
>>
>> Sure, lets for now ignore the fact that the performance of an error path
>> does not matter most of the time.
> 
> I am concerned that extra error code settings within the “success path”
> could influence the run time behaviour in unwanted ways.
> 
> 
>> After tree building and optimization your change should not matter at all
>> regarding performance for a decent compiler.
> 
> I find your optimism interesting.

Mummy says stupid is as stupid does.  Sorry, but I'm putting you on a
kill list.  It's just not possible to discuss things with you. You're
the first person _ever_ to get this treatment in about 20 years of me
using the Internet.

If other people want to merge your patches to KVM, they are free to do so.

Paolo

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-01-21 17:10 +0100
  Re: [PATCH] KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Christian Borntraeger <borntraeger@de.ibm.com> - 2017-01-23 09:30 +0100
    Re: [PATCH] KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Paolo Bonzini <pbonzini@redhat.com> - 2017-01-23 10:30 +0100
    Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-01-23 12:10 +0100
      Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Dan Carpenter <dan.carpenter@oracle.com> - 2017-01-23 13:30 +0100
        Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-01-23 14:30 +0100
          Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Dan Carpenter <dan.carpenter@oracle.com> - 2017-01-23 14:40 +0100
      Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Christian Borntraeger <borntraeger@de.ibm.com> - 2017-01-23 14:40 +0100
        Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Paolo Bonzini <pbonzini@redhat.com> - 2017-01-24 13:20 +0100
          Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() Christian Borntraeger <borntraeger@de.ibm.com> - 2017-01-24 13:40 +0100
            Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-01-24 13:50 +0100
          Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-01-24 13:40 +0100
        Re: KVM: s390: Move two error code assignments in  kvm_vm_ioctl_get_dirty_log() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-01-24 13:20 +0100

csiph-web