Re: [Qemu-devel] hw/scsi/megasas.c: (sdev->id & 0xFF) >> 8

[Markus Armbruster]

Ping?

Markus Armbruster <arm...@redhat.com> writes:

> There are several instances of
>
>     ((sdev->id & 0xFF) >> 8) | (sdev->lun & 0xFF)
>
> or similar.  The left operand of | is always 0.  Did you intend to shift
> to the other right?