Re: [PATCH 1/3] mmc: sdio: Add API to manage SDIO IRQs from a workqueue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[...]

>
> With "cap-sdio-irq" commented out but without your 3 patches:
> => Boot up, "ifconfig eth0 down; ping -c 20 8.8.8.8".  Seems OK.
>
> With "cap-sdio-irq" commented out but _with_ your 3 patches:
> => Boot up, "ifconfig eth0 down; ping -c 20 8.8.8.8".  Seems OK.
>
> So I guess the conclusion is that I missed the part about your patch
> only enabling the new features if MMC_CAP_SDIO_IRQ.  Sorry.  :(
> ...and then I must have hit some other unrelated failure that I can't
> reproduce now and assumed it was your patch's fault.
>
> So basically I would say that I've lightly tested your code.  It's not
> code I've stressed a ton, but it survived some basic tests anyway...
> :)  The code also looks pretty sane to me.

Thanks a lot for your feedback a for running a new round of tests.
This seems promising then!

When you have the time, it we awesome if you could run yet another new
round of test with the new version of the series. I posted it
yesterday evening my local time.

I would also be very interested to know if converting to the work
queue approach has any impact on throughput. Maybe you have some
simple test suite to also verify that?

[...]

Kind regards
Uffe
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux