>>>>> "Christoph" == Christoph Hellwig <h...@infradead.org> writes:

Christoph> On Wed, May 28, 2014 at 11:28:43PM -0400, Martin K. Petersen wrote:
>> Move flags affecting the integrity code out of the bio bi_flags and
>> into the block integrity payload.

Christoph> It seems like bip is guaranteed to be non-NULL in all callers
Christoph> of the getters and setters.

Yeah, check removed.

Christoph> I'd recommend just dropping them and opencode the flags
Christoph> manipulation.

The reason I don't use test_bit() and set_bit() is that bip_flags is
just a short. And to-shift-or-not-to-shift is a crappy, error prone
interface, IMHO.

-- 
Martin K. Petersen      Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to