| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/2] xfs: add CRC infrastructure |
| From: | Andi Kleen <andi@xxxxxxxxxxxxxx> |
| Date: | Wed, 7 Nov 2012 22:39:46 +0100 |
| Cc: | Andi Kleen <andi@xxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| In-reply-to: | <20121107203105.GE6434@dastard> |
| References: | <1352295452-4726-1-git-send-email-david@xxxxxxxxxxxxx> <1352295452-4726-2-git-send-email-david@xxxxxxxxxxxxx> <m2ehk5qt88.fsf@xxxxxxxxxxxxxx> <20121107203105.GE6434@dastard> |
| User-agent: | Mutt/1.4.2.2i |
> Why would that be necessary? We never include the checksum field in > the calculation when setting it or verifiying it, and the > verification uses exactly the same method as the original > calculation to check the CRC, so it doesn't matter if the CRC value > is zero or not - if it matches (zero or otherwise), the validation > passes.... I thought zero meant the checksum is not there? You stated that somewhere else. -Andi |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [XFS updates] XFS development tree branch, for-next, updated. v3.7-rc1-26-g69a58a4, xfs |
|---|---|
| Next by Date: | Re: [PATCH 1/2] xfs: add CRC infrastructure, Dave Chinner |
| Previous by Thread: | Re: [PATCH 1/2] xfs: add CRC infrastructure, Dave Chinner |
| Next by Thread: | Re: [PATCH 1/2] xfs: add CRC infrastructure, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |