On cleanup we free the crc_eof_page after all skbs are freed. There is no reason why it can't be done earlier. We hold a reference to that page and each skb does. Cc: Vasu Dev <vasu.dev@xxxxxxxxx> Cc: "James E.J. Bottomley" <JBottomley@xxxxxxxx> Cc: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx> Cc: Christoph Hellwig <hch@xxxxxx> Cc: fcoe-devel@xxxxxxxxxxxxx Cc: linux-scsi@xxxxxxxxxxxxxxx Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> --- drivers/scsi/fcoe/fcoe.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c index 15826094cc65..4a877ab95d44 100644 --- a/drivers/scsi/fcoe/fcoe.c +++ b/drivers/scsi/fcoe/fcoe.c @@ -1299,6 +1299,8 @@ static void fcoe_percpu_thread_destroy(unsigned int cpu) p->crc_eof_offset = 0; spin_unlock_bh(&p->fcoe_rx_list.lock); + if (crc_eof) + put_page(crc_eof); /* * Don't bother moving the skb's if this context is running * on the same CPU that is having its thread destroyed. This @@ -1342,9 +1344,6 @@ static void fcoe_percpu_thread_destroy(unsigned int cpu) if (thread) kthread_stop(thread); - - if (crc_eof) - put_page(crc_eof); } /** -- 2.7.0 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
