lkml.org 
[lkml]   [2017]   [Sep]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 1/5] soc: qcom: smem: Use le32_to_cpu for partition size comparison
    On 09/14, Chris Lew wrote:
    > Endianness can vary in the system, add le32_to_cpu when comparing
    > size values from smem.
    >
    > Signed-off-by: Chris Lew <clew@codeaurora.org>
    > ---
    >
    > Changes since v1:
    > - New change
    >
    > drivers/soc/qcom/smem.c | 2 +-
    > 1 file changed, 1 insertion(+), 1 deletion(-)
    >
    > diff --git a/drivers/soc/qcom/smem.c b/drivers/soc/qcom/smem.c
    > index c28275be0038..db04c45d4132 100644
    > --- a/drivers/soc/qcom/smem.c
    > +++ b/drivers/soc/qcom/smem.c
    > @@ -698,7 +698,7 @@ static int qcom_smem_enumerate_partitions(struct qcom_smem *smem,
    > return -EINVAL;
    > }
    >
    > - if (header->size != entry->size) {
    > + if (le32_to_cpu(header->size) != le32_to_cpu(entry->size)) {

    Also, it doesn't really matter. We're comparing two numbers with
    the same endianness, so comparing them for equality before or
    after swapping makes no difference. Sparse also (correctly)
    doesn't complain here, because adding the conversion is not
    necessary. Drop this patch?

    --
    Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
    a Linux Foundation Collaborative Project

    \
     
     \ /
      Last update: 2017-09-15 20:39    [W:5.212 / U:0.204 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site