[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities
From: |
Benjamin Herrenschmidt |
Subject: |
Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities |
Date: |
Mon, 26 Aug 2013 20:03:44 +1000 |
On Mon, 2013-08-26 at 10:43 +0200, Alexander Graf wrote:
> Do we ever need to preserve random fields in that struct? Or would it
> be clearer to just set the whole thing to 0 and then go from there?
Yeah sort of, we set/clear bits more/less based on what the guest
put in the first place, it's a bit strange.
> It's definitely a case-by-case decision which way is better, but
> setting all fields by hand gives you potential for missing to set one.
Right. I'll let Nikunj judge here as he looked at that part of the
protocol more than I have.
Cheers,
Ben.
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, (continued)
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Benjamin Herrenschmidt, 2013/08/25
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Alexander Graf, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Benjamin Herrenschmidt, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Alexander Graf, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Alexander Graf, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities,
Benjamin Herrenschmidt <=
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Alexander Graf, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Alexander Graf, 2013/08/26
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/27
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/27
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Alexander Graf, 2013/08/27
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Nikunj A Dadhania, 2013/08/27
- Re: [Qemu-ppc] [PATCH] spapr-vscsi: Adding VSCSI capabilities, Benjamin Herrenschmidt, 2013/08/26